Commit 367e9ac2 authored by Jérome Perrin's avatar Jérome Perrin

"if sm.getUser() != SUPER_USER" compares an user and an user id, it will always

be false.




git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@15723 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent 7d354c7d
...@@ -179,7 +179,7 @@ class ERP5UserManager(BasePlugin): ...@@ -179,7 +179,7 @@ class ERP5UserManager(BasePlugin):
# because we aren't logged in, we have to create our own # because we aren't logged in, we have to create our own
# SecurityManager to be able to access the Catalog # SecurityManager to be able to access the Catalog
sm = getSecurityManager() sm = getSecurityManager()
if sm.getUser() != SUPER_USER: if sm.getUser().getId() != SUPER_USER:
newSecurityManager(self, self.getUser(SUPER_USER)) newSecurityManager(self, self.getUser(SUPER_USER))
try: try:
...@@ -190,7 +190,7 @@ class ERP5UserManager(BasePlugin): ...@@ -190,7 +190,7 @@ class ERP5UserManager(BasePlugin):
raise raise
except: except:
LOG('ERP5Security', PROBLEM, 'getUserByLogin failed', error=sys.exc_info()) LOG('ERP5Security', PROBLEM, 'getUserByLogin failed', error=sys.exc_info())
# Here we must raise an exception to prevent calers from caching # Here we must raise an exception to prevent callers from caching
# a result of a degraded situation. # a result of a degraded situation.
# The kind of exception does not matter as long as it's catched by # The kind of exception does not matter as long as it's catched by
# PAS and causes a lookup using another plugin or user folder. # PAS and causes a lookup using another plugin or user folder.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment