Changeset 8849 for main/waeup.kofa/trunk/src/waeup

Timestamp:

29 Jun 2012, 14:40:37 (12 years ago)

Author:

Henrik Bettermann

Message:

Some changes according to Uli's suggestions.

Location:

main/waeup.kofa/trunk/src/waeup/kofa/mandates

Files:

• interfaces.py (modified) (2 diffs)
• mandate.py (modified) (4 diffs)
• tests.py (modified) (7 diffs)

main/waeup.kofa/trunk/src/waeup/kofa/mandates/interfaces.py

@@ -34 +34 @@
 
 class IMandate(IKofaObject):
-    """A base representation of mandates.
+    """A representation of all mandates.
 
     """
@@ -40 +40 @@
     expires = Attribute('Expiration Datetime')
     params = Attribute('Dictionary with mandate parameters')
-    category = Attribute('Mandate Category')
+
+    def execute():
+        """Method which is performed by the mandate.
+        """

main/waeup.kofa/trunk/src/waeup/kofa/mandates/mandate.py

@@ -21 +21 @@
 import grok
 import hashlib
+import os
 from datetime import datetime, timedelta
-from time import time
 from grok import index
 from waeup.kofa.interfaces import IUserAccount
@@ -28 +28 @@
 from waeup.kofa.mandates.interfaces import IMandate
 
-class Mandate(grok.Container):
-    """This is a mandate which can set a student password.
+class Mandate(grok.Model):
+    """This is a mandate.
     """
     grok.implements(IMandate)
     grok.provides(IMandate)
+    grok.baseclass()
 
-    def __init__(self, days=1, category=None, mandate_id=None):
+    def __init__(self, days=1, mandate_id=None):
         super(Mandate, self).__init__()
         self.creation_date = datetime.utcnow() # offset-naive datetime
@@ -40 +41 @@
         self.expires = datetime.utcnow() + delta
         if mandate_id is None:
-            timestamp = "%d" % int(time()*1000)
-            mandate_id = hashlib.md5(timestamp).hexdigest()
+            mandate_id = os.urandom(20)
+            mandate_id = hashlib.md5(mandate_id).hexdigest()
         self.mandate_id = mandate_id
-        self.category = category
         self.params = {}
         return
+
+    def execute(self):
+        return _('Nothing to do.')
+
+class StudentPasswordMandate(Mandate):
+    """This is a mandate which can set a student password.
+    """
 
     def _setStudentPassword(self):
@@ -60 +67 @@
         if self.expires < datetime.utcnow():
             msg = _('Mandate expired.')
-        elif self.category == 'student_password':
-            if self._setStudentPassword():
-                msg = _('Password has been successfully set.')
+        if self._setStudentPassword():
+            msg = _('Password has been successfully set.')
         del self.__parent__[self.mandate_id]
         return msg

main/waeup.kofa/trunk/src/waeup/kofa/mandates/tests.py

@@ -31 +31 @@
     IMandatesContainer, IMandate)
 from waeup.kofa.mandates.container import MandatesContainer
-from waeup.kofa.mandates.mandate import Mandate
+from waeup.kofa.mandates.mandate import StudentPasswordMandate
 from waeup.kofa.testing import (FunctionalLayer, FunctionalTestCase)
 
@@ -50 +50 @@
         self.assertTrue(
             verifyClass(
-                IMandate, Mandate)
+                IMandate, StudentPasswordMandate)
             )
         self.assertTrue(
             verifyObject(
-                IMandate, Mandate())
+                IMandate, StudentPasswordMandate())
             )
         return
@@ -88 +88 @@
         self.app['students'].addStudent(student)
         # Add and execute a mandate with missing parameters.
-        mandate = Mandate()
+        mandate = StudentPasswordMandate()
         mandate.category = 'student_password'
         self.app['mandates'].addMandate(mandate)
@@ -94 +94 @@
         self.assertEqual(msg, u'Wrong mandate parameters.')
         # Add and execute an expired mandate.
-        mandate = Mandate(days=0)
+        mandate = StudentPasswordMandate(days=0)
         mandate.category = 'student_password'
         self.app['mandates'].addMandate(mandate)
@@ -100 +100 @@
         self.assertEqual(msg, u'Mandate expired.')
         # Add and execute a perfect mandate
-        mandate = Mandate()
+        mandate = StudentPasswordMandate()
         mandate.category = 'student_password'
         mandate.params['student_id'] = student.student_id
@@ -114 +114 @@
     def test_remove_expired(self):
         # mandate1 is an old mandate which just expired.
-        mandate1 = Mandate(days=0)
+        mandate1 = StudentPasswordMandate(days=0)
         self.app['mandates'].addMandate(mandate1)
         # mandate2 is a new mandate with default time delta.
-        mandate2 = Mandate(mandate_id='23456')
+        mandate2 = StudentPasswordMandate(mandate_id='23456')
         self.app['mandates'].addMandate(mandate2)
         self.assertEqual(len(self.app['mandates'].keys()), 2)
@@ -128 +128 @@
         student = createObject('waeup.Student')
         self.app['students'].addStudent(student)
-        mandate = Mandate()
+        mandate = StudentPasswordMandate()
         mandate.category = 'student_password'
         mandate.params['student_id'] = student.student_id