Merge lp:~aelkner/schooltool/schooltool.zambia into lp:~schooltool-owners/schooltool/schooltool.zambia

Proposed by Alan Elkner
Status: Merged
Merged at revision: 49
Proposed branch: lp:~aelkner/schooltool/schooltool.zambia
Merge into: lp:~schooltool-owners/schooltool/schooltool.zambia
Diff against target: 42 lines (+14/-0)
2 files modified
src/schooltool/zambia/app/browser/fet_import.py (+9/-0)
src/schooltool/zambia/app/browser/templates/fet_import.pt (+5/-0)
To merge this branch: bzr merge lp:~aelkner/schooltool/schooltool.zambia
Reviewer Review Type Date Requested Status
SchoolTool Owners Pending
Review via email: mp+29767@code.launchpad.net

Description of the change

Added consecutive periods handling.

I noticed that the journal change for consecutive periods is in trunk but didn't come down with the egg. Is that a dependency thing?

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/schooltool/zambia/app/browser/fet_import.py'
2--- src/schooltool/zambia/app/browser/fet_import.py 2010-07-12 13:31:51 +0000
3+++ src/schooltool/zambia/app/browser/fet_import.py 2010-07-13 04:46:38 +0000
4@@ -62,6 +62,14 @@
5
6 class FETImportView(BrowserView):
7
8+ @property
9+ def consecutive_label(self):
10+ return _('Show consecutive periods as one period')
11+
12+ @property
13+ def consecutive_value(self):
14+ return self.request.get('consecutive') == 'on'
15+
16 def update(self):
17 self.errors = []
18 if 'form-submitted' not in self.request:
19@@ -209,6 +217,7 @@
20 section.instructors.add(self.getTeacher(teacher))
21
22 timetable = schema.createTimetable(term)
23+ timetable.consecutive_periods_as_one = self.consecutive_value
24 self.addTimetable(section, timetable)
25
26 for day_id, period_id in periods:
27
28=== modified file 'src/schooltool/zambia/app/browser/templates/fet_import.pt'
29--- src/schooltool/zambia/app/browser/templates/fet_import.pt 2010-05-30 23:11:22 +0000
30+++ src/schooltool/zambia/app/browser/templates/fet_import.pt 2010-07-13 04:46:38 +0000
31@@ -22,6 +22,11 @@
32 <div class="controls" style="color:red"
33 tal:repeat="error view/displayErrors"
34 tal:content="error" />
35+ <div class="controls">
36+ <input type="checkbox" id="consecutive" name="consecutive"
37+ tal:attributes="checked view/consecutive_value " />
38+ <span tal:content="view/consecutive_label" />
39+ </div>
40
41 <div class="controls">
42 <input name="UPDATE_SUBMIT" class="button-ok" type="submit"

Subscribers

People subscribed via source and target branches