From: Michael Foord <fuzzyman@voidspace.org.uk>
Date: Sat, 20 Nov 2010 16:58:30 +0000 (+0000)
Subject: Issue 9732: __class__ no longer checked on objects by getattr_static
X-Git-Tag: v3.2b1~301
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=35184edd3d5748db2a0a5bf0d41ffa04b84a7fa3;p=python

Issue 9732: __class__ no longer checked on objects by getattr_static
---

diff --git a/Lib/inspect.py b/Lib/inspect.py
index 2f05829b3c..241cd08bff 100644
--- a/Lib/inspect.py
+++ b/Lib/inspect.py
@@ -1080,6 +1080,13 @@ def _check_class(klass, attr):
             pass
     return _sentinel
 
+def _is_type(obj):
+    try:
+        _static_getmro(obj)
+    except TypeError:
+        return False
+    return True
+
 
 def getattr_static(obj, attr, default=_sentinel):
     """Retrieve attributes without triggering dynamic lookup via the
@@ -1093,7 +1100,7 @@ def getattr_static(obj, attr, default=_sentinel):
        documentation for details.
     """
     instance_result = _sentinel
-    if not isinstance(obj, type):
+    if not _is_type(obj):
         instance_result = _check_instance(obj, attr)
         klass = type(obj)
     else:
diff --git a/Lib/test/test_inspect.py b/Lib/test/test_inspect.py
index b3e131ca10..df480b8b9d 100644
--- a/Lib/test/test_inspect.py
+++ b/Lib/test/test_inspect.py
@@ -860,11 +860,15 @@ class TestGetattrStatic(unittest.TestCase):
             foo = 3
 
         class Something(Base):
+            executed = False
             @property
             def __class__(self):
+                self.executed = True
                 return object
 
-        self.assertEqual(inspect.getattr_static(Something(), 'foo'), 3)
+        instance = Something()
+        self.assertEqual(inspect.getattr_static(instance, 'foo'), 3)
+        self.assertFalse(instance.executed)
         self.assertEqual(inspect.getattr_static(Something, 'foo'), 3)
 
     def test_mro_as_property(self):