V0.5.3.1

README.md

@@ -96,6 +96,7 @@ More details available in the [Django Ledger v0.5 Page](https://www.arrobalytics
 
 ### Version 0.6
 
+* IO Digest Context Manager
 * Credit Line Models.
 * Time tracking.
 * Transaction tagging.

django_ledger/__init__.py

@@ -9,7 +9,7 @@
 default_app_config = 'django_ledger.apps.DjangoLedgerConfig'
 
 """Django Ledger"""
-__version__ = '0.5.3.0'
+__version__ = '0.5.3.1'
 __license__ = 'GPLv3 License'
 
 __author__ = 'Miguel Sanda'

django_ledger/io/roles.py

@@ -8,7 +8,7 @@
 
 import sys
 from itertools import chain
-from typing import Set
+from typing import Set, List, Union
 
 from django.utils.translation import gettext as _
 
@@ -609,10 +609,25 @@
     GROUPS_DIRECTORY[group] = getattr(mod, group)
 
 
-def validate_roles(roles) -> Set[str]:
+def validate_roles(roles: Union[str, List[str]], raise_exception: bool = True) -> Set[str]:
+    """
+    Validates a given role identifier against the valid role available.
+    Parameters
+    ----------
+    roles: str or list
+        The role or list of roles to validate.
+    raise_exception: bool
+        Raises InvalidRoleError if any of the roles provided if not valid.
+
+    Returns
+    -------
+    set
+        A set of the valid roles.
+    """
     if isinstance(roles, str):
         roles = set(roles)
     for r in roles:
         if r not in VALID_ROLES:
-            raise InvalidRoleError('{rls}) is invalid. Choices are {ch}'.format(ch=', '.join(VALID_ROLES), rls=r))
+            if raise_exception:
+                raise InvalidRoleError('{rls}) is invalid. Choices are {ch}'.format(ch=', '.join(VALID_ROLES), rls=r))
     return set(roles)

django_ledger/models/accounts.py

@@ -107,6 +107,9 @@ def with_roles(self, roles: Union[List, str]):
         roles = validate_roles(roles)
         return self.filter(role__in=roles)
 
+    def expenses(self):
+        return self.filter(role__in=GROUP_EXPENSES)
+
     def is_coa_root(self):
         return self.filter(role__in=ROOT_GROUP)
 
@@ -139,7 +142,7 @@ def gb_bs_role(self):
         ]
 
     def is_role_default(self):
-        return self.filter(role_default=True)
+        return self.not_coa_root().filter(role_default=True)
 
 
 class AccountModelManager(MP_NodeManager):

django_ledger/models/bill.py

@@ -470,6 +470,7 @@ def configure(self,
             ledger_model.name = ledger_name
 
             ledger_model.clean()
+            ledger_model.clean_fields()
 
             self.ledger = ledger_model
 
@@ -480,6 +481,7 @@ def configure(self,
                 self.generate_bill_number(commit=commit)
 
             self.clean()
+            self.clean_fields()
 
             if commit:
                 self.save()

django_ledger/models/coa_default.py

@@ -309,6 +309,16 @@
 
 ]
 
+
+def get_default_coa() -> List[Dict]:
+    if DJANGO_LEDGER_DEFAULT_COA is not None and isinstance(DJANGO_LEDGER_DEFAULT_COA, list):
+        return DJANGO_LEDGER_DEFAULT_COA
+    return DEFAULT_CHART_OF_ACCOUNTS
+
+
+if DJANGO_LEDGER_DEFAULT_COA:
+    DJANGO_LEDGER_DEFAULT_COA = get_default_coa()
+
 PREFIX_MAP = {
     'in': ROOT_INCOME,
     'ex': ROOT_EXPENSES,
@@ -337,12 +347,6 @@ def verify_unique_code():
         raise DjangoLedgerConfigurationError('Default CoA is not unique.')
 
 
-def get_default_coa() -> List[Dict]:
-    if DJANGO_LEDGER_DEFAULT_COA is not None and isinstance(DJANGO_LEDGER_DEFAULT_COA, list):
-        return DJANGO_LEDGER_DEFAULT_COA
-    return DEFAULT_CHART_OF_ACCOUNTS
-
-
 def get_default_coa_rst(default_coa: Optional[Dict] = None) -> str:
     """
     Converts the provided Chart of Account into restructuredText format.