NPO-Accounting/conservancy_beancount Changeset - 51eee8ec8fd8

Changeset - 51eee8ec8fd8

Parent rev.

Child rev.

[Not reviewed]

0 3 0

Brett Smith - 4 years ago 2020-11-04 18:43:54
brettcsmith@brettcsmith.org

meta_entity: Don't set transaction metadata when payee is None. RT#12913

3 files changed with 12 insertions and 2 deletions:

conservancy_beancount/plugin/meta_entity.py

setup.py

tests/test_meta_entity.py

0 comments (0 inline, 0 general)

conservancy_beancount/plugin/meta_entity.py

➞

Show inline comments

@@ ... / @@ -74,3 +74,3 @@ class MetaEntity(core.TransactionHook): @@
             yield errormod.InvalidMetadataError(txn, self.METADATA_KEY, txn_entity)
         if txn_entity is txn.payee:
+        if txn_entity is txn.payee and txn_entity is not None:
             txn.meta[self.METADATA_KEY] = txn.payee

setup.py

➞

Show inline comments

@@ ... / @@ -7,3 +7,3 @@ setup( @@
     description="Plugin, library, and reports for reading Conservancy's books",
-    version='1.13.0',
+    version='1.13.1',
     author='Software Freedom Conservancy',

tests/test_meta_entity.py

➞

Show inline comments

@@ ... @@
 def test_dont_set_entity_none(hook):
     txn = testutil.Transaction(postings=[
         ('Expenses:Other', 5),
         ('Assets:Cash', -5),
     ])
     assert any(hook.run(txn))
     assert 'entity' not in txn.meta
     for post in txn.postings:
         assert post.meta is None or 'entity' not in post.meta
 def test_not_required_on_opening(hook):

0 comments (0 inline, 0 general)