NPO-Accounting/import2ledger Changeset - f888b13c56b0

Changeset - f888b13c56b0

Parent rev.

Child rev.

[Not reviewed]

0 2 0

Brett Smith - 7 years ago 2017-10-22 18:05:56
brettcsmith@brettcsmith.org

hooks.add_entity: Better handle common name prefix parts.

Keep these with the name they're attached to, rather than breaking the name
in the middle.

2 files changed with 10 insertions and 0 deletions:

import2ledger/hooks/add_entity.py

tests/test_hooks.py

0 comments (0 inline, 0 general)

import2ledger/hooks/add_entity.py

➞

Show inline comments

@@ ... / @@ -2,6 +2,11 @@ import re @@
 import unicodedata
 class AddEntityHook:
     NAME_PREFIXES = frozenset([
         'da',
         'de',
         'van',
     ])
     NONASCII_RE = re.compile(r'[^-A-Za-z0-9]')
     NONALNUM_RE = re.compile(r'[^-\w]')
     OPEN_PARENS = ['\\(', '\\[', '\\{']
@@ ... / @@ -41,6 +46,8 @@ class AddEntityHook: @@
         if not parts:
             return ''
         parts.insert(0, parts.pop())
         if parts[-1].lower() in self.NAME_PREFIXES:
             parts.insert(0, parts.pop())
         return '-'.join(parts)
     def run(self, data):

tests/test_hooks.py

➞

Show inline comments

@@ ... / @@ -19,6 +19,9 @@ def test_load_all(): @@
     ('Alex(Nickname) Smith', 'Smith-Alex'),
     ('稲荷', '稲荷'),
     ('Pøweł', 'Powel'),
     ('Elyse Jan Smith', 'Smith-Elyse-Jan'),
     ('Jan van Smith', 'van-Smith-Jan'),
     ('Francis da Silva', 'da-Silva-Francis'),
 ])
 def test_add_entity(payee, expected):
     data = {'payee': payee}

0 comments (0 inline, 0 general)