[Erp5-report] r35711 fabien - in /erp5/trunk/products/ERP5: interfaces/ mixin/

nobody at svn.erp5.org nobody at svn.erp5.org
Fri May 28 12:13:19 CEST 2010


Author: fabien
Date: Fri May 28 12:13:16 2010
New Revision: 35711

URL: http://svn.erp5.org?rev=35711&view=rev
Log:
remove useless parameter. Thanks jerome for the review

Modified:
    erp5/trunk/products/ERP5/interfaces/encrypted_password.py
    erp5/trunk/products/ERP5/mixin/encrypted_password.py

Modified: erp5/trunk/products/ERP5/interfaces/encrypted_password.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/interfaces/encrypted_password.py?rev=35711&r1=35710&r2=35711&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/interfaces/encrypted_password.py [utf8] (original)
+++ erp5/trunk/products/ERP5/interfaces/encrypted_password.py [utf8] Fri May 28 12:13:16 2010
@@ -56,7 +56,7 @@
     begins with _set*
     """
 
-  def checkUserCanChangePassword(unauthorized_message):
+  def checkUserCanChangePassword():
     """
     check user have permission to change his password. Raise in case he cannot.
     """

Modified: erp5/trunk/products/ERP5/mixin/encrypted_password.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/mixin/encrypted_password.py?rev=35711&r1=35710&r2=35711&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/mixin/encrypted_password.py [utf8] (original)
+++ erp5/trunk/products/ERP5/mixin/encrypted_password.py [utf8] Fri May 28 12:13:16 2010
@@ -63,9 +63,9 @@
       return pw_validate(self.getPassword(), value)
     return False
 
-  def checkUserCanChangePassword(self, unauthorized_message='setPassword'):
+  def checkUserCanChangePassword(self):
     if not _checkPermission(Permissions.SetOwnPassword, self):
-      raise AccessControl_Unauthorized(unauthorized_message)
+      raise AccessControl_Unauthorized('setPassword')
 
   def _setEncodedPassword(self, value, format='default'):
     password = getattr(aq_base(self), 'password', None)
@@ -77,7 +77,7 @@
   def setEncodedPassword(self, value, format='default'):
     """
     """
-    self.checkUserCanChangePassword('setEncodedPassword')
+    self.checkUserCanChangePassword()
     self._setEncodedPassword(value, format=format)
     self.reindexObject()
 
@@ -86,7 +86,7 @@
     self._setEncodedPassword(pw_encrypt(value))
 
   def _setPassword(self, value):
-    self.checkUserCanChangePassword('setPassword')
+    self.checkUserCanChangePassword()
     self._forceSetPassword(value)
 
   security.declarePublic('setPassword')




More information about the Erp5-report mailing list