banking-addons-team team mailing list archive

["Guewen Baconnier @ Camptocamp" <guewen.baconnier@xxxxxxxxxxxxxx>]

Guewen Baconnier @ Camptocamp has proposed merging lp:~camptocamp/banking-addons/bank-statement-reconcile-7.0-could-not-determine-delimiters-1285075 into lp:banking-addons/bank-statement-reconcile-7.0.

Requested reviews:
  Banking Addons Core Editors (banking-addons-team)
Related bugs:
  Bug #1285075 in Banking Addons: "bank-statement-reconcile: import of bank statement fails to sniff the csv"
  https://bugs.launchpad.net/banking-addons/+bug/1285075

For more details, see:
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-could-not-determine-delimiters-1285075/+merge/208322

Details on the bug report: lp:1285075
-- 
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-could-not-determine-delimiters-1285075/+merge/208322
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~camptocamp/banking-addons/bank-statement-reconcile-7.0-could-not-determine-delimiters-1285075 into lp:banking-addons/bank-statement-reconcile-7.0.

=== modified file 'account_statement_base_import/parser/parser.py'
--- account_statement_base_import/parser/parser.py	2014-01-21 16:46:36 +0000
+++ account_statement_base_import/parser/parser.py	2014-02-26 10:26:29 +0000
@@ -25,7 +25,7 @@
 def UnicodeDictReader(utf8_data, **kwargs):
     sniffer = csv.Sniffer()
     pos = utf8_data.tell()
-    sample_data = utf8_data.read(1024)
+    sample_data = utf8_data.read(2048)
     utf8_data.seek(pos)
     dialect = sniffer.sniff(sample_data, delimiters=',;\t')
     csv_reader = csv.DictReader(utf8_data, dialect=dialect, **kwargs)