Changeset 9836 for main/waeup.kofa/trunk/src/waeup/kofa/browser

Timestamp:

7 Jan 2013, 08:01:24 (12 years ago)

Author:

Henrik Bettermann

Message:

Rename jobconfig to exportconfig.

Improve logging of exporting, downloading and discarding exports.

Location:

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

Files:

• pages.py (modified) (2 diffs)
• templates/datacenterexportpage.pt (modified) (1 diff)
• tests/test_browser.py (modified) (3 diffs)

main/waeup.kofa/trunk/src/waeup/kofa/browser/pages.py

@@ -1556 +1556 @@
             job_id = self.context.start_export_job(
                 exporter, self.request.principal.id)
+            ob_class = self.__implemented__.__name__.replace('waeup.kofa.','')
+            self.context.logger.info(
+                '%s - exported: %s, job_id=%s' % (ob_class, exporter, job_id))
         if DISCARD and job_id:
             entry = self.context.entry_from_job_id(job_id)
             self.context.delete_export_entry(entry)
+            ob_class = self.__implemented__.__name__.replace('waeup.kofa.','')
+            self.context.logger.info(
+                '%s - discarded: job_id=%s' % (ob_class, job_id))
             self.flash(_('Discarded export') + ' %s' % job_id)
         self.entries = doll_up(self, user=None)
@@ -1591 +1597 @@
         #    self.context.entry_from_job_id(job_id))
         ob_class = self.__implemented__.__name__.replace('waeup.kofa.','')
-        self.context.logger.info('%s - exported: %s' % (ob_class, filename))
+        self.context.logger.info(
+            '%s - downloaded: %s, job_id=%s' % (ob_class, filename, job_id))
         return result
 

main/waeup.kofa/trunk/src/waeup/kofa/browser/templates/datacenterexportpage.pt

@@ -34 +34 @@
 <p i18n:translate="">
   Student data exports have to be further configured. You can create CSV files
-  on the corresponding <a href="jobconfig">configuration page</a>.
+  on the corresponding <a href="@@exportconfig">configuration page</a>.
 </p>
 

main/waeup.kofa/trunk/src/waeup/kofa/browser/tests/test_browser.py

@@ -224 +224 @@
             self.app['datacenter'].storage, 'logs', 'datacenter.log')
         logcontent = open(logfile).read()
+        self.assertTrue('zope.mgr - browser.pages.ExportCSVPage - exported: '
+                        'faculties, job_id=' in logcontent)
         self.assertTrue('zope.mgr - browser.pages.ExportCSVView - '
-                        'exported: WAeUP.Kofa_faculties.csv' in logcontent)
+                        'downloaded: WAeUP.Kofa_faculties.csv, job_id=' in logcontent)
         return
 
@@ -308 +310 @@
             'current_session\r\n')
         self.cleanup_run_job() 
+        logfile = os.path.join(
+            self.app['datacenter'].storage, 'logs', 'datacenter.log')
+        logcontent = open(logfile).read()
+        self.assertTrue(
+            'zope.mgr - students.browser.DatacenterExportJobContainerJobConfig '
+            '- exported: students (None, None, None, None), job_id='
+            in logcontent)
+        self.assertTrue(
+            'zope.mgr - browser.pages.ExportCSVView '
+            '- downloaded: WAeUP.Kofa_students.csv, job_id=' in logcontent)
         return
 
@@ -317 +329 @@
         self.browser.getControl("Discard").click()
         self.assertTrue('Discarded export' in self.browser.contents)
+        logfile = os.path.join(
+            self.app['datacenter'].storage, 'logs', 'datacenter.log')
+        logcontent = open(logfile).read()
+        self.assertTrue(
+            'zope.mgr - browser.pages.ExportCSVPage - discarded: job_id='
+            in logcontent)
         return