[Erp5-report] r13867 - /erp5/trunk/products/ERP5Type/Utils.py

[nobody at svn.erp5.org]

Author: yo
Date: Mon Apr  2 23:23:54 2007
New Revision: 13867

URL: http://svn.erp5.org?rev=13867&view=rev
Log:
TempDocumentConstructor now creates a new class at runtime. This was required to set permissions correctly in temporary objects..

Modified:
    erp5/trunk/products/ERP5Type/Utils.py

Modified: erp5/trunk/products/ERP5Type/Utils.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/Utils.py?rev=13867&r1=13866&r2=13867&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/Utils.py (original)
+++ erp5/trunk/products/ERP5Type/Utils.py Mon Apr  2 23:23:54 2007
@@ -347,12 +347,26 @@
 
 class TempDocumentConstructor(DocumentConstructor):
 
+    def __init__(self, klass):
+      # Create a new class to set permissions specific to temporary objects.
+      class TempDocument(klass):
+        pass
+
+      # Replace some attributes.
+      for name in ('isIndexable', 'reindexObject', 'recursiveReindexObject',
+                   'activate', 'setUid', 'setTitle', 'getTitle'):
+        setattr(TempDocument, name, getattr(klass, '_temp_%s' % name))
+
+      # Make some methods public.
+      for method_id in ('reindexObject', 'recursiveReindexObject',
+                        'activate', 'setUid', 'setTitle', 'getTitle',
+                        'edit', 'setProperty'):
+        setattr(TempDocument, '%s__roles__' % method_id, None)
+
+      self.klass = TempDocument
+
     def __call__(self, folder, id, REQUEST=None, **kw):
       o = self.klass(id)
-      # Monkey patch TempBase specific arguments
-      for k in ('isIndexable', 'reindexObject', 'recursiveReindexObject',
-                'activate', 'setUid', 'setTitle', 'getTitle'):
-        setattr(o, k, getattr(o,"_temp_%s" % k))
       if kw:
         o.__of__(folder)._edit(force_update=1, **kw)
       if hasattr(folder, 'isTempObject') and folder.isTempObject():