Changeset 6219 for main/waeup.sirp/trunk/src/waeup/sirp/utils/batching.py

Timestamp:

29 May 2011, 10:19:58 (13 years ago)

Author:

Henrik Bettermann

Message:

Use the message output of CertificateContainer?.addCertificate in CertificateProcessor?.addEntry and finally in BatchProcessor?.doImport to notify that the import failed. This can maybe merged with CertificateProcessor?.entryExists?!

File:

• main/waeup.sirp/trunk/src/waeup/sirp/utils/batching.py (modified) (1 diff)

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

@@ -279 +279 @@
                     self.writeFailedRow(
                         failed_writer, raw_row,
-                        "This object already exists. Skipping.")
+                        "This object already exists in the same container. Skipping.")
                     continue
                 obj = self.callFactory()
+                #import pdb; pdb.set_trace()
                 for key, value in row.items():
                     setattr(obj, key, value)
-                self.addEntry(obj, row, site)
+                if not self.addEntry(obj, row, site):
+                    num_warns += 1
+                    self.writeFailedRow(
+                        failed_writer, raw_row,
+                        "This object code already exists somewhere else. Skipping.")
+                    continue
             elif mode == 'remove':
                 if not self.entryExists(row, site):