Merge lp:~gs.clearcorp/openerp-costa-rica/7.0_l10n_cr_account_banking_cr_promerica into lp:openerp-costa-rica

Proposed by Glen Sojo on 2014-04-10
Status: Merged
Merged at revision: 282
Proposed branch: lp:~gs.clearcorp/openerp-costa-rica/7.0_l10n_cr_account_banking_cr_promerica
Merge into: lp:openerp-costa-rica
Diff against target: 12 lines (+1/-1)
1 file modified
l10n_cr_account_banking_promerica/ (+1/-1)
To merge this branch: bzr merge lp:~gs.clearcorp/openerp-costa-rica/7.0_l10n_cr_account_banking_cr_promerica
Reviewer Review Type Date Requested Status
ClearCorp drivers 2014-04-10 Pending
Review via email:

Description of the change

[FIX] Added comparison between u'' and empty string values on final row.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'l10n_cr_account_banking_promerica/'
2--- l10n_cr_account_banking_promerica/ 2014-04-07 22:16:25 +0000
3+++ l10n_cr_account_banking_promerica/ 2014-04-10 15:42:30 +0000
4@@ -162,7 +162,7 @@
5 data_rows = worksheet.rows[13:]
6 data = []
7 for row in data_rows:
8- if reduce(lambda result, value: result and (value is None),
9+ if reduce(lambda result, value: result and (value is None or value == ''),
10 [cell.value for cell in row], True):
11 break
12 data.append([cell.value for cell in row])


People subscribed via source and target branches

to all changes: