Added the __str property and changed the query_values property from a pseudo property to a real property in the AccountOption data model, so that it does not need to hold the account object forever.

This commit is contained in:
依瑪貓 2023-03-09 22:50:18 +08:00
parent 71af74fc8a
commit c95f4fcc47

View File

@ -126,12 +126,14 @@ class AccountOption:
:param account: The account. :param account: The account.
""" """
self.__account: Account = account
"""The account."""
self.id: str = account.id self.id: str = account.id
"""The account ID.""" """The account ID."""
self.code: str = account.code self.code: str = account.code
"""The account code.""" """The account code."""
self.query_values: list[str] = account.query_values
"""The values to be queried."""
self.__str: str = str(account)
"""The string representation of the account option."""
self.is_in_use: bool = False self.is_in_use: bool = False
"""True if this account is in use, or False otherwise.""" """True if this account is in use, or False otherwise."""
@ -140,15 +142,7 @@ class AccountOption:
:return: The string representation of the account option. :return: The string representation of the account option.
""" """
return str(self.__account) return self.__str
@property
def query_values(self) -> list[str]:
"""Returns the values to be queried.
:return: The values to be queried.
"""
return self.__account.query_values
class JournalEntryForm(FlaskForm): class JournalEntryForm(FlaskForm):