Merge lp:~camptocamp/openerp-swiss-localization/fix-v11-parser into lp:~camptocamp/openerp-swiss-localization/7.0

[Nicolas Bessi - Camptocamp]

Fix of bug no: 1173149
+ v11 parser can support new line at end of file or empty line in body

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

LGTM

review: Approve (code review, no test)

=== modified file 'l10n_ch_payment_slip/wizard/bvr_import.py'
--- l10n_ch_payment_slip/wizard/bvr_import.py	2013-02-15 13:28:41 +0000
+++ l10n_ch_payment_slip/wizard/bvr_import.py	2013-04-26 11:50:40 +0000
@@ -66,13 +66,15 @@
         else:
             return []
 
-    def _parse_lines(self, cursor, uid, lines, context=None):
+    def _parse_lines(self, cursor, uid, inlines, context=None):
         """Parses raw v11 line and populate records list with dict"""
         records = []
         total_amount = 0
         total_cost = 0
         find_total = False
-        for lines in lines:
+        for lines in inlines:
+            if not lines:  # manage new line at end of file
+                continue
             (line, lines) = (lines[:128], lines[128:])
             record = {}
             if line[0:3] in ('999', '995'):
@@ -114,7 +116,7 @@
                 total_amount += record['amount']
                 total_cost += record['cost']
                 records.append(record)
-            return records
+        return records
 
     def _create_voucher_from_record(self, cursor, uid, record,
                                     statement, line_ids, context=None):
@@ -203,6 +205,7 @@
         statement_id = data['id']
         lines = base64.decodestring(file).split("\n")
         records = self._parse_lines(cursor, uid, lines, context=context)
+
         if context is None:
             context = {}