Changeset 9833 for main/waeup.kofa/trunk

Timestamp:

6 Jan 2013, 08:22:04 (12 years ago)

Author:

Henrik Bettermann

Message:

Let's define DepartmentExportJobContainerJobConfig? as a baseclass which can then also be used for the datacenter.

File:

• main/waeup.kofa/trunk/src/waeup/kofa/students/browser.py (modified) (5 diffs)

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

@@ -46 +46 @@
     IKofaObject, IUserAccount, IExtFileStore, IPasswordValidator, IContactForm,
     IKofaUtils, IUniversity, IObjectHistory, academic_sessions, ICSVExporter,
-    academic_sessions_vocab, IJobManager)
+    academic_sessions_vocab, IJobManager, IDataCenter)
 from waeup.kofa.interfaces import MessageFactory as _
 from waeup.kofa.widgets.datewidget import (
@@ -2673 +2673 @@
         return
 
-class DepartmentExportJobContainerJobConfig(KofaPage):
+class ExportJobContainerJobConfig(KofaPage):
     """Page that configures a students export job.
-    """
-    grok.context(VirtualDepartmentExportJobContainer)
+
+    This is a baseclass.
+    """
+    grok.baseclass()
+    grok.name('jobconfig')
     grok.require('waeup.showStudents')
-    grok.name('jobconfig')
     grok.template('exportjobsjobconfig')
-    label = _('Configure local student data export')
+    label = _('Configure student data export')
     pnav = 1
 
@@ -2712 +2714 @@
         self.exporters = exporters
 
+    @property
+    def depcode(self):
+        return None
+
     def update(self, START=None, session=None, level=None, mode=None,
                exporter=None):
@@ -2726 +2732 @@
         if mode == 'all':
             mode = None
-        code = self.context.__parent__.code
         self.context.start_export_job(exporter, self.request.principal.id,
                                       current_session=session,
                                       current_level=level,
                                       current_mode=mode,
-                                      depcode=code)
-        self.flash(_('Export started for students from') +
-                   ' %s (current_session=%s, current_level=%s, study_mode=%s)' % (
-                       code, session, level, mode))
-        self.redirect(self.url(self.context))
+                                      depcode=self.depcode)
+        self.flash(_('Export started for students with') +
+                   ' current_session=%s, current_level=%s, study_mode=%s' % (
+                   session, level, mode))
         return
 
@@ -2743 +2747 @@
     grok.context(VirtualExportJobContainer)
     grok.require('waeup.showStudents')
+
+class DatacenterExportJobContainerJobConfig(ExportJobContainerJobConfig):
+    """Page that configures a students export job in datacenter.
+
+    """
+    grok.context(IDataCenter)
+
+
+class DepartmentExportJobContainerJobConfig(ExportJobContainerJobConfig):
+    """Page that configures a students export job in departments.
+
+    """
+    grok.context(VirtualDepartmentExportJobContainer)
+
+    @property
+    def depcode(self):
+        return self.context.__parent__.code