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

Timestamp:

16 Dec 2011, 07:32:42 (13 years ago)

Author:

Henrik Bettermann

Message:

Implement fullname constructor as a global utility method which can easily be subject to customization. I think it's better to leave this here rather than selecting the constructor method via the customization object.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/students/student.py (modified) (3 diffs)

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

@@ -21 +21 @@
 import os
 import grok
+from zope.component import getUtility
 from zope.component.interfaces import IFactory
 from zope.interface import implementedBy
@@ -26 +27 @@
 from zope.securitypolicy.interfaces import IPrincipalRoleManager
 from waeup.sirp.interfaces import (
-    IObjectHistory, IUserAccount, IFileStoreNameChooser, IFileStoreHandler)
+    IObjectHistory, IUserAccount, IFileStoreNameChooser, IFileStoreHandler,
+    ISIRPUtils)
 from waeup.sirp.image import SIRPImageFile
 from waeup.sirp.imagestorage import DefaultFileStoreHandler
@@ -63 +65 @@
     @property
     def fullname(self):
-        # We do not necessarily have the middlename attribute
         middlename = getattr(self, 'middlename', None)
-        if middlename:
-            return '%s %s %s' % (self.firstname,
-                middlename, self.lastname)
-        else:
-            return '%s %s' % (self.firstname, self.lastname)
+        sirp_utils = getUtility(ISIRPUtils)
+        return sirp_utils.fullname(self.firstname, self.lastname, middlename)
 
     @property