Comment 5 for bug 487402

Thank you Jan for the quick reply.
1. I did comment it out and I had no issues at all
2. OK
3. The problem here is that the module did install a new top level account without getting the demo accounts nested in the account with the ID 0 of Openerp. This is what I meant before. This way you will only be able to select accounts from the "Automated Tests" chart and it would be good to just use the chart that is already installed.
4. I agree, however if you refer to the MT940 Specs, you will see as an example that the Debit / Credit sign is at index 10 so that the line checking the debit would read:

if bank_statement[current][':61:'][10] == "D":
    st_line['account_id'] = bank['def_pay_acc']
    st_line['amount'] = - st_line['amount']

The function get_number should have a pointer set to 11 to read the amount

This is according to the specs from swift_mt940_942-1.pdf

I agree with you however that every bank is dealing differently with MT940 which makes the job a bit difficult. I think a profile per bank would a good idea.