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

Proposed by Glen Sojo
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/promerica.py (+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 Pending
Review via email: mp+215220@code.launchpad.net

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
=== modified file 'l10n_cr_account_banking_promerica/promerica.py'
--- l10n_cr_account_banking_promerica/promerica.py 2014-04-07 22:16:25 +0000
+++ l10n_cr_account_banking_promerica/promerica.py 2014-04-10 15:42:30 +0000
@@ -162,7 +162,7 @@
162 data_rows = worksheet.rows[13:]162 data_rows = worksheet.rows[13:]
163 data = []163 data = []
164 for row in data_rows:164 for row in data_rows:
165 if reduce(lambda result, value: result and (value is None),165 if reduce(lambda result, value: result and (value is None or value == ''),
166 [cell.value for cell in row], True):166 [cell.value for cell in row], True):
167 break167 break
168 data.append([cell.value for cell in row])168 data.append([cell.value for cell in row])

Subscribers

People subscribed via source and target branches

to all changes: