Changeset 7653

Timestamp:

15 Feb 2012, 11:03:57 (13 years ago)

Author:

Henrik Bettermann

Message:

Log user management. If possible use full sentences in log messages. Use the imperative form when process is going to be started and the passive form when the process is finished.

Location:

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

Files:

• meta.py (modified) (1 diff)
• userscontainer.py (modified) (4 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/meta.py

@@ -99 +99 @@
         for name, index in self.indexes.items():
             try:
-                site.logger.info('Create index `%s` in catalog' % name)
+                site.logger.info('Create index `%s` in catalog.' % name)
                 index.setup(catalog, name, self.context, self.module_info)
-                site.logger.info('index created.')
+                site.logger.info('Index created.')
             except KeyError: #, DuplicationError:
-                site.logger.info('index `%s` already in catalog' % name)
+                site.logger.info('Index `%s` already in catalog.' % name)
                 pass
         site.logger.info('Catalog `%s` ready.' % self.catalog_name)

main/waeup.sirp/trunk/src/waeup/sirp/userscontainer.py

@@ -22 +22 @@
 from waeup.sirp.authentication import Account
 from waeup.sirp.interfaces import IUsersContainer
+from waeup.sirp.utils.logger import Logger
 
-class UsersContainer(grok.Container):
+class UsersContainer(grok.Container, Logger):
     """A container for principals.
 
@@ -42 +43 @@
                              description=description,
                              email=email, phone=phone, roles=roles)
+        self.logger.info('User account %s added.' % name)
 
     def addAccount(self, account):
@@ -47 +49 @@
         """
         self[account.name] = account
+        self.logger.info('User account %s added.' % account.name)
 
     def delUser(self, name):
@@ -55 +58 @@
         if name in self.keys():
             del self[name]
+            self.logger.info('User account %s deleted.' % name)