Code review comment for lp:~hbrunn/banking-addons/lp-1106831-handle-P

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Good catch, but did you check whether this makes for a valid Clieop file? The most recent Clieop specifications don't mention it, but the specifications from March 2006 rule it out explicitely in page 15 of

    http://www-old.adslweb.net/tools/clieop2psv/ClieOp_03_EN.pdf

It may be bank specific as well. Rabobank rules it out in page 12 of

    https://www.rabobank.com/en/images/Format_description__RCM6.pdf

So does ING in page 10 of

    http://www.ing.nl/Images/ClieOp_03_tcm7-69019.pdf

All of this seems to indicate that a better approach would be to replace the P by a zero. What do you think?

review: Needs Information

« Back to merge proposal