Changeset 8332

Timestamp:

3 May 2012, 11:41:27 (13 years ago)

Author:

Henrik Bettermann

Message:

ObjectModifiedEvent? must be fired in updateEntry to ensure that catalogs involved are updated after setting attributes.

Location:

main/waeup.kofa/trunk/src/waeup/kofa

Files:

• applicants/tests/test_batching.py (modified) (1 diff)
• utils/batching.py (modified) (2 diffs)

main/waeup.kofa/trunk/src/waeup/kofa/applicants/tests/test_batching.py

@@ -331 +331 @@
         self.assertEqual(container['2345'].reg_number, '6666')
         shutil.rmtree(os.path.dirname(fin_file))
-        # We have to inform the catalog that the reg_number has changed
-        notify(grok.ObjectModifiedEvent(container['2345']))
+
         # Now we import another file which clears all middlename attributes
-        # and uses the new reg_number as locator
+        # and uses the new reg_number as locator. This test also checks
+        # if the catalog has been informed about the reg_no change.
         num, num_warns, fin_file, fail_file = self.processor.doImport(
             self.csv_file_update2, APPLICANT_HEADER_FIELDS_UPDATE2, 'update')

main/waeup.kofa/trunk/src/waeup/kofa/utils/batching.py

@@ -30 +30 @@
 from zope.interface import Interface
 from zope.schema import getFields
+from zope.event import notify
 from waeup.kofa.interfaces import (
     IBatchProcessor, FatalCSVError, IObjectConverter,
@@ -216 +217 @@
             log_value = getattr(value, 'code', value)
             changed.append('%s=%s' % (key, log_value))
+
+        # If any catalog is involved it must be updated
+        notify(grok.ObjectModifiedEvent(obj))
+
         return ', '.join(changed)