[Checkins] SVN: grokui.admin/branches/0.3/src/grokui/admin/objectinfo.py Fix silly bug in object traverser: objects that do evaluate to False

[Uli Fouquet]

Log message for revision 109034:
  Fix silly bug in object traverser: objects that do evaluate to False
  are not neccessarily None.
  

Changed:
  U   grokui.admin/branches/0.3/src/grokui/admin/objectinfo.py

-=-
Modified: grokui.admin/branches/0.3/src/grokui/admin/objectinfo.py
===================================================================
--- grokui.admin/branches/0.3/src/grokui/admin/objectinfo.py	2010-02-14 12:57:13 UTC (rev 109033)
+++ grokui.admin/branches/0.3/src/grokui/admin/objectinfo.py	2010-02-14 12:58:38 UTC (rev 109034)
@@ -304,7 +304,7 @@
                 new_obj = self.obj[name]
 
         # Try to get name as sequence entry...
-        if not new_obj:
+        if new_obj is None:
             # This is not the appropriate way to handle iterators. We
             # must find somehing to handle them too.
             try:
@@ -315,11 +315,11 @@
                 pass
 
         # Get name as obj attribute...
-        if not new_obj and hasattr(self.obj, name):
+        if new_obj is None and hasattr(self.obj, name):
             new_obj = getattr(self.obj, name, None)
 
         # Get name as annotation...
-        if not new_obj:
+        if new_obj is None:
             naked = zope.security.proxy.removeSecurityProxy(self.obj)
             try:
                 annotations = IAnnotations(naked)
@@ -330,7 +330,7 @@
                 pass
 
         # Give obj a location...
-        if new_obj:
+        if new_obj is not None:
             if not IPhysicallyLocatable(new_obj, False):
                 new_obj = location.LocationProxy(
                     new_obj, self.obj, name)