changeset:   90214:d08e3586dde3
branch:      2.7
parent:      90212:340a9d2f6bde
user:        Vinay Sajip <vinay_sajip@yahoo.co.uk>
date:        Thu Apr 10 07:07:59 2014 +0100
files:       Lib/logging/__init__.py Misc/NEWS
description:
Issue #21172: isinstance check relaxed from dict to collections.Mapping.


diff -r 340a9d2f6bde -r d08e3586dde3 Lib/logging/__init__.py
--- a/Lib/logging/__init__.py	Thu Apr 10 00:23:18 2014 -0400
+++ b/Lib/logging/__init__.py	Thu Apr 10 07:07:59 2014 +0100
@@ -23,7 +23,7 @@
 To use, simply 'import logging' and log away!
 """
 
-import sys, os, time, cStringIO, traceback, warnings, weakref
+import sys, os, time, cStringIO, traceback, warnings, weakref, collections
 
 __all__ = ['BASIC_FORMAT', 'BufferingFormatter', 'CRITICAL', 'DEBUG', 'ERROR',
            'FATAL', 'FileHandler', 'Filter', 'Formatter', 'Handler', 'INFO',
@@ -261,7 +261,13 @@
         # 'Value is %d' instead of 'Value is 0'.
         # For the use case of passing a dictionary, this should not be a
         # problem.
-        if args and len(args) == 1 and isinstance(args[0], dict) and args[0]:
+        # Issue #21172: a request was made to relax the isinstance check
+        # to hasattr(args[0], '__getitem__'). However, the docs on string
+        # formatting still seem to suggest a mapping object is required.
+        # Thus, while not removing the isinstance check, it does now look
+        # for collections.Mapping rather than, as before, dict.
+        if (args and len(args) == 1 and isinstance(args[0], collections.Mapping)
+            and args[0]):
             args = args[0]
         self.args = args
         self.levelname = getLevelName(level)
diff -r 340a9d2f6bde -r d08e3586dde3 Misc/NEWS
--- a/Misc/NEWS	Thu Apr 10 00:23:18 2014 -0400
+++ b/Misc/NEWS	Thu Apr 10 07:07:59 2014 +0100
@@ -43,6 +43,8 @@
 Library
 -------
 
+- Issue #21172: isinstance check relaxed from dict to collections.Mapping.
+
 - Issue #21191: In os.fdopen, alwyas close the file descriptor when an exception
   happens.