Changeset 6764

Timestamp:

14 Sep 2011, 08:28:16 (13 years ago)

Author:

Henrik Bettermann

Message:

We don't need the StudentBaseEdit? page any more. Student won't be allowed to edit their base data.

The password shouldn't be a form field on the StudentBaseManageFormPage?. I still prefer a simple attribute declaration. Why do we need a schema field?

Students are not allowed to edit their name.

Location:

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

Files:

• interfaces.py (modified) (3 diffs)
• student.py (modified) (2 diffs)

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

@@ -66 +66 @@
         default = u'Nobody',
         required = True,
+        readonly = True
         )
 
@@ -92 +93 @@
     adm_code = Attribute('Admission checking access code')
 
-    password = schema.Password(
-        title = u'Password',
-        required = False,
-        )
-
     def loggerInfo(ob_class, comment):
         """Adds an INFO message to the log file
@@ -205 +201 @@
 # Interfaces for students only
 
-class IStudentBaseEdit(IStudentBase):
-    """Interface needed for editing of student base data by students.
-    """
-
-    name = schema.TextLine(
-        title = u'Full Name',
-        default = u'Nobody',
-        required = True,
-        readonly = True,
-        )
-
-IStudentBaseEdit['name'].order =  IStudentBase['name'].order
-
 class IStudentClearanceEdit(IStudentClearance):
     """Interface needed for restricted editing of student clearance data.

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

@@ -24 +24 @@
 from waeup.sirp.interfaces import IObjectHistory
 from waeup.sirp.students.interfaces import (
-    IStudent, IStudentNavigation, IStudentBaseEdit, IStudentClearanceEdit,
+    IStudent, IStudentNavigation, IStudentClearanceEdit,
     IStudentPersonalEdit)
 from waeup.sirp.utils.helpers import attrs_to_fields
@@ -34 +34 @@
     """
     grok.implements(IStudent, IStudentNavigation,
-        IStudentBaseEdit, IStudentClearanceEdit, IStudentPersonalEdit)
+        IStudentPersonalEdit)
     grok.provides(IStudent)