Changeset 6943 for main/waeup.sirp/trunk/src/waeup

Timestamp:

25 Oct 2011, 10:16:01 (13 years ago)

Author:

Henrik Bettermann

Message:

Implement write_log_message function and log removal of payments as well as valid callbacks.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/students/browser.py (modified) (8 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/students/browser.py

@@ -50 +50 @@
 from waeup.sirp.authentication import get_principal_role_manager
 
+def write_log_message(view, message):
+    ob_class = view.__implemented__.__name__.replace('waeup.sirp.','')
+    view.context.getStudent().loggerInfo(ob_class, message)
+    return
+
 # Save function used for save methods in manager pages
 def msave(view, **data):
     form = view.request.form
-    ob_class = view.__implemented__.__name__.replace('waeup.sirp.','')
     changed_fields = view.applyData(view.context, **data)
     # Turn list of lists into single list
@@ -62 +66 @@
     view.flash('Form has been saved.')
     if fields_string:
-        view.context.getStudent().loggerInfo(ob_class, 'saved: % s' % fields_string)
+        write_log_message(view, 'saved: % s' % fields_string)
     return
 
@@ -366 +370 @@
     def save(self, **data):
         form = self.request.form
-        ob_class = self.__implemented__.__name__.replace('waeup.sirp.','')
         password = form.get('password', None)
         password_ctl = form.get('control_password', None)
@@ -376 +379 @@
                 #      are no (other) errors left in form.
                 IUserAccount(self.context).setPassword(password)
-                self.context.loggerInfo(ob_class, 'password changed')
+                write_log_message(self, 'password changed')
 
         #self.reg_number = form.get('form.reg_number', None)
@@ -403 +406 @@
         self.flash('Form has been saved.')
         if fields_string:
-            self.context.loggerInfo(ob_class, 'saved: % s' % fields_string)
+            write_log_message(self, 'saved: % s' % fields_string)
         return
 
@@ -772 +775 @@
 
 # We don't need the display form page yet
-class PaymentsDisplayFormPage(WAeUPDisplayFormPage):
-    """ Page to display the student payments
-    """
-    grok.context(IStudentPaymentsContainer)
-    grok.name('view')
-    grok.require('waeup.viewStudent')
-    form_fields = grok.AutoFields(IStudentPaymentsContainer)
-    grok.template('paymentspage')
-    title = 'Payments'
-    pnav = 4
-
-    def formatDatetime(self,datetimeobj):
-        if isinstance(datetimeobj, datetime):
-            return datetimeobj.strftime("%Y-%m-%d %H:%M:%S")
-        else:
-            return None
-
-    @property
-    def label(self):
-        return '%s: Payments' % self.context.__parent__.fullname
-
-    def update(self):
-        super(PaymentsDisplayFormPage, self).update()
-        datatable.need()
-        return
+#class PaymentsDisplayFormPage(WAeUPDisplayFormPage):
+#    """ Page to display the student payments
+#    """
+#    grok.context(IStudentPaymentsContainer)
+#    grok.name('view')
+#    grok.require('waeup.viewStudent')
+#    form_fields = grok.AutoFields(IStudentPaymentsContainer)
+#    grok.template('paymentspage')
+#    title = 'Payments'
+#    pnav = 4
+
+#    def formatDatetime(self,datetimeobj):
+#        if isinstance(datetimeobj, datetime):
+#            return datetimeobj.strftime("%Y-%m-%d %H:%M:%S")
+#        else:
+#            return None
+
+#    @property
+#    def label(self):
+#        return '%s: Payments' % self.context.__parent__.fullname
+
+#    def update(self):
+#        super(PaymentsDisplayFormPage, self).update()
+#        datatable.need()
+#        return
 
 # This manage form page is for both students and students officers.
@@ -853 +856 @@
         if len(deleted):
             self.flash('Successfully removed: %s' % ', '.join(deleted))
+            write_log_message(self,'removed: % s' % ', '.join(deleted))
         self.redirect(self.url(self.context))
         return
@@ -936 +940 @@
     def update(self):
         student = self.context.getStudent()
+        write_log_message(self,'valid callback: %s' % self.context.p_id)
         self.context.r_amount_approved = self.context.amount_auth
         self.context.r_card_num = u'0000'