Changeset 6622 for main/waeup.sirp/trunk/src/waeup/sirp/students

Timestamp:

24 Aug 2011, 20:25:44 (13 years ago)

Author:

Henrik Bettermann

Message:

Add some basic UI stuff to start and to play with.

Location:

main/waeup.sirp/trunk/src/waeup/sirp/students

Files:

• browser.py (modified) (4 diffs)
• browser_templates/studentaddpage.pt (added)
• browser_templates/studentscontainerpage.pt (modified) (2 diffs)
• interfaces.py (modified) (2 diffs)
• tests/test_browser.py (modified) (2 diffs)

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

@@ -24 +24 @@
 from zope.securitypolicy.interfaces import IPrincipalRoleManager
 from zope.traversing.browser import absoluteURL
+from zope.component import (
+    createObject,)
 
 from hurry.workflow.interfaces import IWorkflowInfo, IWorkflowState
@@ -60 +62 @@
     MultiListWidget, MultiListDisplayWidget)
 from waeup.sirp.students.interfaces import (
-    IStudentsContainer,
+    IStudentsContainer, IStudent,
+    )
+from waeup.sirp.students.students import (
+    Student,
     )
 
@@ -86 +91 @@
     grok.require('waeup.viewStudents')
     grok.template('studentscontainerpage')
-    pnav = 3
 
-    form_fields = grok.AutoFields(IStudentsContainer).omit('title')
+    form_fields = grok.AutoFields(IStudentsContainer)
     #form_fields['description'].custom_widget = ReSTDisplayWidget
 
@@ -98 +102 @@
     def label(self):
         return self.title
+
+class StudentssContainerManageActionButton(ManageActionButton):
+    grok.order(1)
+    grok.context(IStudentsContainer)
+    grok.view(StudentsContainerPage)
+    grok.require('waeup.manageStudents')
+    text = 'Manage students'
+
+class StudentsContainerManagePage(WAeUPDisplayFormPage):
+    """The standard view for regular student containers.
+    """
+    grok.context(IStudentsContainer)
+    grok.name('manage')
+    grok.require('waeup.manageStudents')
+    grok.template('studentscontainerpage')
+
+    form_fields = grok.AutoFields(IStudentsContainer)
+
+    @property
+    def title(self):
+        return "Students"
+
+    @property
+    def label(self):
+        return self.title
+
+    @grok.action('Add student', validator=NullValidator)
+    def addStudent(self, **data):
+        self.redirect(self.url(self.context, 'addstudent'))
+        return
+
+class StudentAddFormPage(WAeUPAddFormPage):
+    """Add-form to add a student.
+    """
+    grok.context(IStudentsContainer)
+    grok.require('waeup.manageStudents')
+    grok.name('addstudent')
+    grok.template('studentaddpage')
+    form_fields = grok.AutoFields(IStudent)
+    title = 'Students'
+    label = 'Add student'
+
+    @grok.action('Create student record')
+    def addStudent(self, **data):
+        student_id = self.request.form.get('form.student_id')
+        student = createObject(u'waeup.Student')
+        self.applyData(student, **data)
+        #import pdb; pdb.set_trace()
+        try:
+            self.context[student_id] = student
+        except KeyError:
+            self.flash('The student id chosen already exists.')
+            return
+        self.redirect(self.url(self.context[student_id], 'index'))
+        return
+
+class DisplayStudent(WAeUPDisplayFormPage):
+    grok.context(IStudent)
+    grok.name('index')
+    grok.require('waeup.viewStudents')
+    form_fields = grok.AutoFields(IStudent)
+
+    @property
+    def title(self):
+        return 'Student: %s' % self.context.name
+
+    @property
+    def label(self):
+        return self.context.name

main/waeup.sirp/trunk/src/waeup/sirp/students/browser_templates/studentscontainerpage.pt

@@ -1 +1 @@
 <h2 tal:content="view/label">Title</h2>
 
+<form method="POST">
 <table class="zebra" tal:condition="layout/isAuthenticated">
   <tbody>
@@ -13 +14 @@
       </tr>
     </tal:block>
+      <tr>
+        <td>
+            <div class="actionButtons" tal:condition="view/availableActions">
+                <span tal:repeat="action view/actions" tal:omit-tag="">
+                    <input tal:replace="structure action/render"/>
+                </span>
+            </div>
+        </td>
+      </tr>
   </tbody>
 </table>
+</form>

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

@@ -8 +8 @@
     """Representation of a student.
     """
+    student_id = schema.TextLine(
+        title = u'Student Id',
+        default = u'None',
+        required = True,
+        )
+
     name = schema.TextLine(
         title = u'Name of student',
@@ -13 +19 @@
         required = True,
         )
+
 
 class IStudentsContainer(IWAeUPObject):

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

@@ -104 +104 @@
         # Anonymous users can't access students containers
         self.assertRaises(
+            Unauthorized, self.browser.open, self.container_path)
+        self.assertRaises(
             Unauthorized, self.browser.open, self.manage_container_path)
         return
@@ -114 +116 @@
         self.assertEqual(self.browser.headers['Status'], '200 Ok')
         self.assertEqual(self.browser.url, self.container_path)
+        self.browser.getLink("Manage students").click()
+        self.assertEqual(self.browser.headers['Status'], '200 Ok')
+        self.assertEqual(self.browser.url, self.manage_container_path)
         return
+
+    def test_add_students(self):
+        # Managers can add students
+        self.browser.addHeader('Authorization', 'Basic mgr:mgrpw')
+        self.add_student_path = self.container_path + '/addstudent'
+        self.container_manage_path = self.container_path + '/@@manage'
+        self.browser.open(self.container_manage_path)
+        self.browser.getControl("Add student").click()
+        self.assertEqual(self.browser.headers['Status'], '200 Ok')
+        self.assertEqual(self.browser.url, self.add_student_path)
+        self.browser.getControl(name="form.student_id").value = 'A123456'
+        self.browser.getControl(name="form.name").value = 'Bob Tester'
+        self.browser.getControl("Create student record").click()