Banking Addons

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

  1. 7.0_mt940_line_endings
  2. Merge into banking-addons-70
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
1=== modified file 'account_banking_mt940/mt940.py'
2--- account_banking_mt940/mt940.py 2014-03-06 14:05:01 +0000
3+++ account_banking_mt940/mt940.py 2014-06-19 14:15:26 +0000
4@@ -75,7 +75,7 @@
5
6 def parse(self, cr, data):
7 'implements account_banking.parsers.models.parser.parse()'
8- iterator = data.split('\r\n').__iter__()
9+ iterator = data.replace('\r\n', '\n').split('\n').__iter__()
10 line = None
11 record_line = ''
12 try:

Subscribers

People subscribed via source and target branches

to status/vote changes: