[Erp5-report] r38725 jm - in /erp5/trunk/products/ERP5Type: Core/Folder.py Utils.py

[nobody at svn.erp5.org]

Author: jm
Date: Tue Sep 28 16:05:08 2010
New Revision: 38725

URL: http://svn.erp5.org?rev=38725&view=rev
Log:
asContext() must not call _setObject on the container

This fix several test failures caused by r38650.

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

Modified: erp5/trunk/products/ERP5Type/Core/Folder.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/Core/Folder.py?rev=38725&r1=38724&r2=38725&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/Core/Folder.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/Core/Folder.py [utf8] Tue Sep 28 16:05:08 2010
@@ -167,7 +167,8 @@ class FolderMixIn(ExtensionClass.Base):
       # TODO :the **kw makes it impossible to create content not based on
       # ERP5TypeInformation, because factory method often do not support
       # keywords arguments.
-
+    if temp_container:
+      container._setObject(new_id, new_instance.aq_base)
     if kw:
       new_instance._edit(force_update=1, **kw)
     return new_instance

Modified: erp5/trunk/products/ERP5Type/Utils.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/Utils.py?rev=38725&r1=38724&r2=38725&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/Utils.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/Utils.py [utf8] Tue Sep 28 16:05:08 2010
@@ -561,8 +561,6 @@ class TempDocumentConstructor(DocumentCo
       # through the factory system.
       if getattr(aq_base(folder), 'Destination', None) is not None:
         folder = folder.Destination()
-      if folder.isTempObject():
-        folder._setObject(id, o)
       o = o.__of__(folder)
       if kw:
         o._edit(force_update=1, **kw)