Merge lp:~therp-nl/banking-addons/7.0_mt940_line_endings into lp:banking-addons

Proposed by Holger Brunn (Therp)
Status: Merged
Merged at revision: 261
Proposed branch: lp:~therp-nl/banking-addons/7.0_mt940_line_endings
Merge into: lp:banking-addons
Diff against target: 12 lines (+1/-1)
1 file modified
account_banking_mt940/mt940.py (+1/-1)
To merge this branch: bzr merge lp:~therp-nl/banking-addons/7.0_mt940_line_endings
Reviewer Review Type Date Requested Status
Stefan Rijnhart (Opener) Approve
Pedro Manuel Baeza code review Approve
Review via email: mp+223754@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

LGTM

review: Approve (code review)
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Tnx!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'account_banking_mt940/mt940.py'
--- account_banking_mt940/mt940.py 2014-03-06 14:05:01 +0000
+++ account_banking_mt940/mt940.py 2014-06-19 14:15:26 +0000
@@ -75,7 +75,7 @@
7575
76 def parse(self, cr, data):76 def parse(self, cr, data):
77 'implements account_banking.parsers.models.parser.parse()'77 'implements account_banking.parsers.models.parser.parse()'
78 iterator = data.split('\r\n').__iter__()78 iterator = data.replace('\r\n', '\n').split('\n').__iter__()
79 line = None79 line = None
80 record_line = ''80 record_line = ''
81 try:81 try:

Subscribers

People subscribed via source and target branches

to status/vote changes: