As discussed with seb, it should not be required to add portal_type here,...

As discussed with seb, it should not be required to add portal_type here, because here.getPortalType() must work. Add some comment on the purpose of having some hardcoded items. 

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@20359 20353a03-c40f-0410-a6d1-a30d3c3de9de

product/ZSQLCatalog/SQLCatalog.py

@@ -2354,13 +2354,15 @@ class Catalog(Folder,
             #'request':      getattr( ob, 'REQUEST', None ),
             #'modules':      SecureModuleImporter,
             #'user':         getSecurityManager().getUser(),
-            'isDelivery':   ob.isDelivery, # XXX
-            'isMovement':   ob.isMovement, # XXX
-            'isPredicate':  ob.isPredicate, # XXX
-            'isDocument':   ob.isDocument, # XXX
-            'isInventory':  ob.isInventory, # XXX
-            'portal_type':  ob.portal_type, # XXX
-            'isInventoryMovement': ob.isInventoryMovement, # XXX
+            # XXX these below are defined, because there is no
+            # accessor for some attributes, and restricted environment
+            # may not access them directly.
+            'isDelivery':   ob.isDelivery,
+            'isMovement':   ob.isMovement,
+            'isPredicate':  ob.isPredicate,
+            'isDocument':   ob.isDocument,
+            'isInventory':  ob.isInventory,
+            'isInventoryMovement': ob.isInventoryMovement,
             }
         return getEngine().getContext(data)