Changeset 10438 for main/waeup.kofa/trunk/src/waeup/kofa/students/browser.py

Timestamp:

29 Jul 2013, 16:25:14 (11 years ago)

Author:

Henrik Bettermann

Message:

Remove redundant code.

File:

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

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

@@ -1260 +1260 @@
 
     @property
-    def content_title_1(self):
-        portal_language = getUtility(IKofaUtils).PORTAL_LANGUAGE
-        return translate(_('1st Semester Courses'), 'waeup.kofa',
-            target_language=portal_language)
-
-    @property
-    def content_title_2(self):
-        portal_language = getUtility(IKofaUtils).PORTAL_LANGUAGE
-        return translate(_('2nd Semester Courses'), 'waeup.kofa',
-            target_language=portal_language)
-
-    @property
-    def content_title_3(self):
-        portal_language = getUtility(IKofaUtils).PORTAL_LANGUAGE
-        return translate(_('Level Courses'), 'waeup.kofa',
-            target_language=portal_language)
-
-    @property
     def label(self):
         portal_language = getUtility(IKofaUtils).PORTAL_LANGUAGE
@@ -1301 +1283 @@
             self.request, self.omit_fields)
         students_utils = getUtility(IStudentsUtils)
-        tabledata_1 = sorted(
-            [value for value in self.context.values() if value.semester == 1],
-            key=lambda value: str(value.semester) + value.code)
-        tabledata_2 = sorted(
-            [value for value in self.context.values() if value.semester == 2],
-            key=lambda value: str(value.semester) + value.code)
-        tabledata_3 = sorted(
-            [value for value in self.context.values() if value.semester == 3],
-            key=lambda value: str(value.semester) + value.code)
-        tableheader = [(Code,'code', 2.5),
-                         (Title,'title', 5),
-                         (Dept,'dcode', 1.5), (Faculty,'fcode', 1.5),
-                         (Cred, 'credits', 1.5),
-                         #(Mand, 'mandatory', 1.5),
-                         (Score, 'score', 1.5),
-                         (Grade, 'grade', 1.5),
-                         #('Auto', 'automatic', 1.5)
-                         ]
+
+        tabledata = []
+        tableheader = []
+        contenttitle = []
+        for i in range(6):
+            tabledata.append(sorted(
+                [value for value in self.context.values() if value.semester == i],
+                key=lambda value: str(value.semester) + value.code))
+            tableheader.append([(Code,'code', 2.5),
+                             (Title,'title', 5),
+                             (Dept,'dcode', 1.5), (Faculty,'fcode', 1.5),
+                             (Cred, 'credits', 1.5),
+                             #(Mand, 'mandatory', 1.5),
+                             (Score, 'score', 1.5),
+                             (Grade, 'grade', 1.5),
+                             #('Auto', 'automatic', 1.5)
+                             ])
+            contenttitle.append(None)
+
+        contenttitle[1] = translate(_('1st Semester Courses'), 'waeup.kofa',
+            target_language=portal_language)
+        contenttitle[2] = translate(_('2nd Semester Courses'), 'waeup.kofa',
+            target_language=portal_language)
+        contenttitle[3] = translate(_('Level Courses'), 'waeup.kofa',
+            target_language=portal_language)
+
         return students_utils.renderPDF(
             self, 'course_registration_slip.pdf',
             self.context.student, studentview,
-            tableheader_1=tableheader,
-            tabledata_1=tabledata_1,
-            tableheader_2=tableheader,
-            tabledata_2=tabledata_2,
-            tableheader_3=tableheader,
-            tabledata_3=tabledata_3,
+            tableheader=tableheader,
+            tabledata=tabledata,
+            contenttitle=contenttitle,
             omit_fields=self.omit_fields
             )