Issue 13 in iso8583py: Data Field 18 has LLL instead of N

[iso8...@googlecode.com]

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 13 by rob...@valepresente.com.br: Data Field 18 has LLL instead
of N
https://code.google.com/p/iso8583py/issues/detail?id=13

According to the ISO spec, DE 18 should be N -> 4, but your class is
setting LLL and cause an exception. Fixed it subclassing ISO8583, but would
be nice to get it patched:

self._BITS_VALUE_TYPE[18] = ['18','Message error indicator','N',4,'n']

Regards,


--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

[iso8...@googlecode.com]

Updates:
Status: Accepted
Labels: OpSys-All

Comment #1 on issue 13 by igo...@gmail.com: Data Field 18 has LLL instead
of N
https://code.google.com/p/iso8583py/issues/detail?id=13

Hi, This is a "Copy and Paste" bug.

Will be fixed.

Best regards,

[iso8...@googlecode.com]

Comment #2 on issue 13 by igo...@gmail.com: Data Field 18 has LLL instead

of N
https://code.google.com/p/iso8583py/issues/detail?id=13

Hi,

The Fix Whithout the new lib is using the redefine Method:

Example:

from ISO8583.ISO8583 import ISO8583
#(...)
i2 = ISO8583()
i2.redefineBit(18, '18', i2.getLargeBitName(18), 'N', 4, 'N' )

Visit the example page to more info:
https://code.google.com/p/iso8583py/wiki/ExamplesPage

Best Regards,

[iso8...@googlecode.com]

Comment #3 on issue 13 by rob...@valepresente.com.br: Data Field 18 has LLL

instead of N
https://code.google.com/p/iso8583py/issues/detail?id=13

Actually, I did something different. Subclassed ISO8583 class, redefined
the bit and added support to EBCDIC, as I'm testing the package using
Mastercard messages.

Till now, everything is looking good. Would be nice to move this project to
github and send some pull requests.

BR

Robson

[iso8...@googlecode.com]

Updates:
Status: Fixed

Comment #4 on issue 13 by igo...@gmail.com: Data Field 18 has LLL instead
of N
https://code.google.com/p/iso8583py/issues/detail?id=13

Fixed in revire #52 https://code.google.com/p/iso8583py/source/detail?r=52