Changeset 7995 for main/waeup.custom/trunk/src/waeup/custom/students

Timestamp:

28 Mar 2012, 06:23:25 (13 years ago)

Author:

Henrik Bettermann

Message:

Adapt changes recently made in the base package.

Location:

main/waeup.custom/trunk/src/waeup/custom/students

Files:

• browser.py (modified) (4 diffs)
• interfaces.py (modified) (3 diffs)
• student.py (modified) (2 diffs)
• tests/sample_student_data.csv (modified) (1 diff)
• tests/test_browser.py (modified) (7 diffs)

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

@@ -24 +24 @@
     StudentPersonalDisplayFormPage, StudentPersonalManageFormPage,
     StudentClearanceManageFormPage, StudentClearanceEditFormPage,
-    StudentClearanceDisplayFormPage)
+    StudentClearanceDisplayFormPage,
+    ExportPDFClearanceSlipPage)
 from waeup.kofa.students.viewlets import RequestCallbackActionButton
 from waeup.custom.students.interfaces import (
-    IStudent, IStudentPersonal, IStudentClearance, IStudentClearanceEdit,
+    IStudent, IStudentPersonal,
+    IUGStudentClearance,IPGStudentClearance,
     )
 from waeup.custom.interfaces import MessageFactory as _
@@ -48 +50 @@
     """
     grok.context(IStudent)
-    form_fields = grok.AutoFields(IStudentClearance).omit('clearance_locked')
-    form_fields['date_of_birth'].custom_widget = FriendlyDateDisplayWidget('le')
+
+    @property
+    def form_fields(self):
+        cm = getattr(self.context,'current_mode', None)
+        if cm is not None and cm.startswith('pg'):
+            form_fields = grok.AutoFields(IPGStudentClearance).omit('clearance_locked')
+        else:
+            form_fields = grok.AutoFields(IUGStudentClearance).omit('clearance_locked')
+        form_fields['date_of_birth'].custom_widget = FriendlyDateDisplayWidget('le')
+        return form_fields
+
+class ExportPDFClearanceSlipPage(ExportPDFClearanceSlipPage):
+    """Deliver a PDF slip of the context.
+    """
+    grok.context(IStudent)
+
+    @property
+    def form_fields(self):
+        cm = getattr(self.context,'current_mode', None)
+        if cm is not None and cm.startswith('pg'):
+            form_fields = grok.AutoFields(IPGStudentClearance).omit('clearance_locked')
+        else:
+            form_fields = grok.AutoFields(IUGStudentClearance).omit('clearance_locked')
+        form_fields['date_of_birth'].custom_widget = FriendlyDateDisplayWidget('le')
+        return form_fields
 
 class StudentClearanceManageFormPage(StudentClearanceManageFormPage):
@@ -55 +80 @@
     """
     grok.context(IStudent)
-    form_fields = grok.AutoFields(IStudentClearance)
-    form_fields['date_of_birth'].custom_widget = FriendlyDateWidget('le-year')
+
+    @property
+    def form_fields(self):
+        cm = getattr(self.context,'current_mode', None)
+        if cm is not None and cm.startswith('pg'):
+            form_fields = grok.AutoFields(IPGStudentClearance)
+        else:
+            form_fields = grok.AutoFields(IUGStudentClearance)
+        form_fields['date_of_birth'].custom_widget = FriendlyDateWidget('le-year')
+        return form_fields
 
 class StudentClearanceEditFormPage(StudentClearanceEditFormPage):
@@ -62 +95 @@
     """
     grok.context(IStudent)
-    form_fields = grok.AutoFields(
-        IStudentClearanceEdit).omit('clearance_locked')
-    form_fields['date_of_birth'].custom_widget = FriendlyDateWidget('le-year')
+
+    @property
+    def form_fields(self):
+        cm = getattr(self.context,'current_mode', None)
+        if cm is not None and cm.startswith('pg'):
+            form_fields = grok.AutoFields(IPGStudentClearance).omit('clearance_locked')
+        else:
+            form_fields = grok.AutoFields(IUGStudentClearance).omit('clearance_locked')
+        form_fields['date_of_birth'].custom_widget = FriendlyDateWidget('le-year')
+        return form_fields
 
 class RequestCallbackActionButton(RequestCallbackActionButton):

main/waeup.custom/trunk/src/waeup/custom/students/interfaces.py

@@ -20 +20 @@
 from waeup.kofa.interfaces import SimpleKofaVocabulary
 from waeup.kofa.students.interfaces import (
-    IStudentBase,IStudentClearance,IStudentPersonal,IStudentNavigation,
+    IStudentBase,IUGStudentClearance,IPGStudentClearance,
+    IStudentPersonal,IStudentNavigation,
     )
 from waeup.kofa.students.vocabularies import (
@@ -51 +52 @@
         )
 
-class IStudentClearance(IStudentClearance):
-    """Representation of student clearance data.
+class IUGStudentClearance(IUGStudentClearance):
+    """Representation of ug student clearance data.
 
     """
@@ -73 +74 @@
         )
 
-class IStudent(IStudentBase,IStudentClearance,IStudentPersonal):
+class IPGStudentClearance(IUGStudentClearance):
+    """Representation of pg student clearance data.
+
+    """
+
+class IStudent(IStudentBase,IUGStudentClearance,IPGStudentClearance,IStudentPersonal):
     """Representation of a student.
 
     """
-
-# Interfaces for students only
-
-class IStudentClearanceEdit(IStudentClearance):
-    """Interface needed for restricted editing of student clearance data.
-
-    """
-
-class IStudentPersonalEdit(IStudentPersonal):
-    """Interface needed for restricted editing of student personal data.
-
-    """

main/waeup.custom/trunk/src/waeup/custom/students/student.py

@@ -23 +23 @@
 from waeup.kofa.utils.helpers import attrs_to_fields
 from waeup.kofa.students.student import Student, StudentFactory
-from waeup.custom.students.interfaces import (
-    IStudent, IStudentNavigation,
-    IStudentPersonalEdit, IStudentClearanceEdit)
+from waeup.custom.students.interfaces import IStudent, IStudentNavigation
 
 
@@ -32 +30 @@
     owned by students.
     """
-    grok.implements(IStudent, IStudentNavigation,
-        IStudentPersonalEdit, IStudentClearanceEdit)
+    grok.implements(IStudent, IStudentNavigation)
     grok.provides(IStudent)
 

main/waeup.custom/trunk/src/waeup/custom/students/tests/sample_student_data.csv

@@ -1 @@
-firstname,lastname,reg_number,date_of_birth,matric_number,email,phone,lga,nationality
-Aaren,Pieri,1,1990-01-02,100000,aa@aa.ng,1234
-Aaren,Finau,2,1990-01-03,100001,aa@aa.ng,1234
-Aaren,Berson,3,1990-01-04,100002,aa@aa.ng,1234
+firstname,lastname,reg_number,date_of_birth,matric_number,email,phone,sex
+Aaren,Pieri,1,,100000,aa@aa.ng,1234,m
+Aaren,Finau,2,,100001,aa@aa.ng,1234,m
+Aaren,Berson,3,,100002,aa@aa.ng,1234,m

main/waeup.custom/trunk/src/waeup/custom/students/tests/test_browser.py

@@ -72 +72 @@
 
     def test_import(self):
-        # We added empty columns 'nationality' and 'lga' to the import file.
-        # The original processor will fail because these fields are required
+        # We have an empty column 'date_of_birth' in  the import file.
+        # The original processor will fail because 'date_of_birth' is required
         # in the base package.
         num, num_warns, fin_file, fail_file = self.processor_base.doImport(
@@ -79 +79 @@
         self.assertEqual(num_warns,3)
         assert len(self.app['students'].keys()) == 0
-        # The customized processor does not complain since both fields are
+        # The customized processor does not complain since 'date_of_birth' is
         # not required in the custom package.
         num, num_warns, fin_file, fail_file = self.processor.doImport(
             self.csv_file, STUDENT_HEADER_FIELDS)
+        #print open(fail_file).read()
         self.assertEqual(num_warns,0)
         assert len(self.app['students'].keys()) == 3
@@ -103 +104 @@
         # Managers can add online payment tickets
         self.browser.addHeader('Authorization', 'Basic mgr:mgrpw')
-        self.browser.open(self.payments_student_path)
+        self.browser.open(self.payments_path)
         self.browser.getControl("Add online payment ticket").click()
         self.browser.getControl("Create ticket").click()
@@ -109 +110 @@
                            self.browser.contents)
         IWorkflowState(self.student).setState('cleared')
-        self.browser.open(self.payments_student_path + '/addop')
+        self.browser.open(self.payments_path + '/addop')
         self.browser.getControl("Create ticket").click()
         self.assertMatches('...ticket created...',
@@ -122 +123 @@
         ticket.p_state = 'paid'
         IWorkflowState(self.student).setState('returning')
-        self.browser.open(self.payments_student_path + '/addop')
+        self.browser.open(self.payments_path + '/addop')
         self.browser.getControl("Create ticket").click()
         self.assertMatches('...This type of payment has already been made...',
@@ -130 +131 @@
         for payment in keys:
             del self.student['payments'][payment]
-        self.browser.open(self.payments_student_path + '/addop')
+        self.browser.open(self.payments_path + '/addop')
         self.browser.getControl("Create ticket").click()
         self.assertMatches('...ticket created...',
                            self.browser.contents)
-        self.browser.open(self.payments_student_path + '/addop')
+        self.browser.open(self.payments_path + '/addop')
         self.browser.getControl(name="form.p_category").value = ['gown']
         self.browser.getControl("Create ticket").click()
-        self.browser.open(self.payments_student_path + '/addop')
+        self.browser.open(self.payments_path + '/addop')
         self.browser.getControl(name="form.p_category").value = ['transfer']
         self.browser.getControl("Create ticket").click()
-        self.browser.open(self.payments_student_path + '/addop')
+        self.browser.open(self.payments_path + '/addop')
         self.browser.getControl(
             name="form.p_category").value = ['bed_allocation']
         self.browser.getControl("Create ticket").click()
-        self.browser.open(self.payments_student_path + '/addop')
+        self.browser.open(self.payments_path + '/addop')
         self.browser.getControl(
             name="form.p_category").value = ['hostel_maintenance']
         self.browser.getControl("Create ticket").click()
-        self.browser.open(self.payments_student_path + '/addop')
+        self.browser.open(self.payments_path + '/addop')
         self.browser.getControl(name="form.p_category").value = ['clearance']
         self.browser.getControl("Create ticket").click()
         self.certificate.study_mode = 'ug_pt'
-        self.browser.open(self.payments_student_path + '/addop')
+        self.browser.open(self.payments_path + '/addop')
         self.browser.getControl(name="form.p_category").value = ['schoolfee']
         self.browser.getControl("Create ticket").click()
@@ -161 +162 @@
         # be shown
         del self.app['configuration']['2004']
-        self.browser.open(self.payments_student_path)
+        self.browser.open(self.payments_path)
         self.browser.getControl("Add online payment ticket").click()
         self.browser.getControl("Create ticket").click()
         self.assertMatches('...Session configuration object is not...',
                            self.browser.contents)
-
-