Commit 9a2cbdc8 authored by Rafael Monnerat's avatar Rafael Monnerat

Fix Typo, identation (4 spaces to 2 spaces), and remove default arguments...

Fix Typo, identation (4 spaces to 2 spaces), and remove default arguments because according to the code, all parameter should be defined.



git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@36173 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent 4352894f
...@@ -1082,12 +1082,11 @@ class Folder(CopyContainer, CMFBTreeFolder, CMFHBTreeFolder, Base, FolderMixIn, ...@@ -1082,12 +1082,11 @@ class Folder(CopyContainer, CMFBTreeFolder, CMFHBTreeFolder, Base, FolderMixIn,
return update_list return update_list
security.declareProtected( Permissions.ModifyPortalContent, 'upgradeObjectClass' ) security.declareProtected( Permissions.ModifyPortalContent, 'upgradeObjectClass' )
def upgradeObjectClass(self, test_before=None, from_class=None,\ def upgradeObjectClass(self, test_before, from_class, to_class, test_after):
to_class=None, test_after=None):
""" """
upgrade the class of all objects inside this upgrade the class of all objects inside this
particular folder particular folder
test have to be a method with one parameter test_before and test_after have to be a method with one parameter.
migrations is a dictionnary of class, { from_class : to_class } migrations is a dictionnary of class, { from_class : to_class }
""" """
#LOG("upradeObjectClass: folder ",0,self.id) #LOG("upradeObjectClass: folder ",0,self.id)
...@@ -1103,12 +1102,13 @@ class Folder(CopyContainer, CMFBTreeFolder, CMFHBTreeFolder, Base, FolderMixIn, ...@@ -1103,12 +1102,13 @@ class Folder(CopyContainer, CMFBTreeFolder, CMFHBTreeFolder, Base, FolderMixIn,
test_list += o.upgradeObjectClass(test_before=test_before, \ test_list += o.upgradeObjectClass(test_before=test_before, \
from_class=from_class, to_class=to_class, from_class=from_class, to_class=to_class,
test_after=test_after) test_after=test_after)
# Test if we must apply the upgrade # Test if we must apply the upgrade
if test_before(o) is not None: if test_before(o) is not None:
LOG("upradeObjectClass: id ",0,id) LOG("upgradeObjectClass: id ", 0, id)
klass = obase.__class__ klass = obase.__class__
LOG("upradeObjectClass: klass ",0,str(klass)) LOG("upgradeObjectClass: klass ", 0 ,str(klass))
LOG("upradeObjectClass: from_class ",0,str(from_class)) LOG("upgradeObjectClass: from_class ", 0 ,str(from_class))
if klass == from_class: if klass == from_class:
try: try:
newob = to_class(obase.id) newob = to_class(obase.id)
...@@ -1122,9 +1122,9 @@ class Folder(CopyContainer, CMFBTreeFolder, CMFHBTreeFolder, Base, FolderMixIn, ...@@ -1122,9 +1122,9 @@ class Folder(CopyContainer, CMFBTreeFolder, CMFHBTreeFolder, Base, FolderMixIn,
setattr(newob,k,obase.__dict__[k]) setattr(newob,k,obase.__dict__[k])
self.manage_delObjects(id) self.manage_delObjects(id)
LOG("upradeObjectClass: ",0,"add new object: %s" % str(newob.id)) LOG("upgradeObjectClass: ",0,"add new object: %s" % str(newob.id))
transaction.commit() # XXX this commit should be after _setObject transaction.commit() # XXX this commit should be after _setObject
LOG("upradeObjectClass: ",0,"newob.__class__: %s" % str(newob.__class__)) LOG("upgradeObjectClass: ",0,"newob.__class__: %s" % str(newob.__class__))
self._setObject(id, newob) self._setObject(id, newob)
object_to_test = self._getOb(id) object_to_test = self._getOb(id)
test_list += test_after(object_to_test) test_list += test_after(object_to_test)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment