Merge lp:~therp-nl/account-banking/account-banking-lp544466 into lp:account-banking

[Stefan Rijnhart (Opener)]

Hi Pieter,

You fixed this one before, but the fix disappeared in revision 33. Let's try again.

Regards,
Stefan.

[Pieter J. Kersten (EduSense BV) (pieterj)]

Hi Stefan,

Thanks for noting. The comment in point 2 of this method got lost too. It still needs a restore.

I'm not sure why line 17 is needed. Never saw any problems with the current sniffing behavior.

[Stefan Rijnhart (Opener) (stefan-opener)]

On 26-04-11 23:51, Pieter J. Kersten (EduSense BV) wrote:
> Hi Stefan,
>
> Thanks for noting. The comment in point 2 of this method got lost too. It still needs a restore.
>
> I'm not sure why line 17 is needed. Never saw any problems with the current sniffing behavior.

Hi Pieter,

that line is actually unrelated to SNS, I encountered the problem while
doing some testing and it seemed worthwhile keeping it. If you insist, I
can remove it from the current branch.

Cheers,
Stefan.

--
Therp - Maatwerk in open ontwikkeling

Stefan Rijnhart - Ontwerp en implementatie

mail: <email address hidden>
tel: [phone]
web: http://therp.nl

=== modified file 'account_banking_nl_multibank/multibank.py'
--- account_banking_nl_multibank/multibank.py	2011-02-01 21:48:34 +0000
+++ account_banking_nl_multibank/multibank.py	2011-04-26 21:25:09 +0000
@@ -155,7 +155,7 @@
                 self.effective_date) and (
                     self.remote_account or
                     self.transfer_type in [
-                        'KST', 'PRV', 'BTL', 'BEA', 'OPN', 'KNT',
+                        'KST', 'PRV', 'BTL', 'BEA', 'OPN', 'KNT', 'DIV',
                     ]
                 and not self.error_message
                 )
@@ -306,6 +306,7 @@
         # Probe first record to find out which format we are parsing.
         if lines and lines[0].count(',') > lines[0].count(';'):
             dialect.delimiter = ','
+        if lines and lines[0].count("'") > lines[0].count('"'):
             dialect.quotechar = "'"
         # Transaction lines are not numbered, so keep a tracer
         subno = 0