Changeset 12096

Timestamp:

30 Nov 2014, 11:53:40 (10 years ago)

Author:

Henrik Bettermann

Message:

Adjust logging formats.

Inform applicants_catalog when last_product_id has been changed.

Location:

main/waeup.ikoba/trunk/src/waeup/ikoba/customers

Files:

• applications.py (modified) (1 diff)
• browser.py (modified) (1 diff)
• tests/test_browser.py (modified) (2 diffs)

main/waeup.ikoba/trunk/src/waeup/ikoba/customers/applications.py

@@ -163 +163 @@
 
 @grok.subscribe(IApplication, grok.IObjectAddedEvent)
-def handle_document_added(application, event):
+def handle_application_added(application, event):
     """If an application is added the transition create is fired.
     The latter produces a logging message.

main/waeup.ikoba/trunk/src/waeup/ikoba/customers/browser.py

@@ -67 +67 @@
     if changed_fields:
         changed_fields = reduce(lambda x,y: x+y, changed_fields.values())
-    if 'product' in changed_fields:
+    if 'product' in changed_fields and data['product'] is not None:
         view.context.last_product_id = data['product'].product_id
+        notify(grok.ObjectModifiedEvent(view.context))
     fields_string = ' + '.join(changed_fields)
     view.flash(_('Form has been saved.'))
     if fields_string:
         view.context.writeLogMessage(
-            view, 'saved %s: %s' % (view.context.__name__, fields_string))
+            view, '%s - saved: %s' % (view.context.__name__, fields_string))
     return
 

main/waeup.ikoba/trunk/src/waeup/ikoba/customers/tests/test_browser.py

@@ -860 +860 @@
         self.assertTrue(
             'INFO - zope.mgr - customers.browser.DocumentManageFormPage '
-            '- K1000000 - saved d102: title'
+            '- K1000000 - d102 - saved: title'
             in logcontent)
         self.assertTrue(
@@ -1101 +1101 @@
         self.assertTrue(
             'INFO - zope.mgr - customers.browser.ApplicationManageFormPage '
-            '- K1000000 - saved a102: title'
+            '- K1000000 - a102 - saved: title'
             in logcontent)
         self.assertTrue(