Person title and user title and userprofile are now hybrid properties

OneGov · Jul 16, 2024 · 3beab01 · 3beab01
1 parent b8f3b76
commit 3beab01
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/onegov/people/models/person.py b/src/onegov/people/models/person.py
@@ -1,3 +1,5 @@
+from sqlalchemy.ext.hybrid import hybrid_property
+
 from onegov.core.orm import Base
 from onegov.core.orm.mixins import ContentMixin
 from onegov.core.orm.mixins import TimestampMixin
@@ -55,7 +57,8 @@ class Person(Base, ContentMixin, TimestampMixin, ORMSearchable,
     def es_suggestion(self) -> tuple[str, ...]:
         return (self.title, f'{self.first_name} {self.last_name}')
 
-    @property
+    # @property
+    @hybrid_property
     def title(self) -> str:
         """ Returns the Eastern-ordered name. """
 

diff --git a/src/onegov/user/models/user.py b/src/onegov/user/models/user.py
@@ -59,7 +59,8 @@ class User(Base, TimestampMixin, ORMSearchable):
     def es_suggestion(self) -> tuple[str, str]:
         return (self.realname or self.username, self.username)
 
-    @property
+    # @property
+    @hybrid_property
     def userprofile(self) -> list[str]:
         if not self.data:
             return []