banking-addons-team team mailing list archive

["Guewen Baconnier @ Camptocamp" <guewen.baconnier@xxxxxxxxxxxxxx>]

Guewen Baconnier @ Camptocamp has proposed merging lp:~camptocamp/banking-addons/bank-statement-reconcile_vre into lp:banking-addons/bank-statement-reconcile-70.

Requested reviews:
  Banking Addons Core Editors (banking-addons-team)

For more details, see:
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile_vre/+merge/168364

This fix prevent account_advanced_reconcile to crash if ref in the move is not set
-- 
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile_vre/+merge/168364
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~camptocamp/banking-addons/bank-statement-reconcile_vre into lp:banking-addons/bank-statement-reconcile-70.

=== modified file 'account_advanced_reconcile/advanced_reconciliation.py'
--- account_advanced_reconcile/advanced_reconciliation.py	2013-01-03 16:37:46 +0000
+++ account_advanced_reconcile/advanced_reconciliation.py	2013-06-10 07:06:29 +0000
@@ -114,5 +114,5 @@
         :yield: matchers as tuple ('matcher key', value(s))
         """
         yield ('partner_id', move_line['partner_id'])
-        yield ('ref', (move_line['ref'].lower().strip(),
+        yield ('ref', ((move_line['ref'] or '').lower().strip(),
                        move_line['name'].lower().strip()))