ERP5Security: fix getPersonByReference should always return a person

cc3cc6e9 · Cédric Le Ninivin · a63c9cb6 · cc3cc6e9
Commit cc3cc6e9 authored Jan 18, 2016 by Cédric Le Ninivin
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

product/ERP5Security/ERP5UserManager.py product/ERP5Security/ERP5UserManager.py +4 -1

No files found.
--- a/product/ERP5Security/ERP5UserManager.py
+++ b/product/ERP5Security/ERP5UserManager.py
@@ -134,8 +134,11 @@ class ERP5UserManager(BasePlugin):
      cache_factory='erp5_content_short')
    person_relative_url = _getPersonRelativeUrlFromReference(reference)
    if person_relative_url is not None:
-      return self.getPortalObject().unrestrictedTraverse(
+      person = self.getPortalObject().unrestrictedTraverse(
        person_relative_url)
+      if person.getPortalType() == "Person":
+        return person
+      return person.getParentValue()
  def checkPersonValidity(self, person):
    if person.getValidationState() in ('deleted',):