Skip Menu |

This queue is for tickets about the Finance-OFX-Parse-Simple CPAN distribution.

Report information
The Basics
Id: 42599
Status: resolved
Priority: 0/
Queue: Finance-OFX-Parse-Simple

People
Owner: Nobody in particular
Requestors: bitcard [...] duncanprime.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.01
Fixed in: 0.02

Attachments


Subject: ACCTID regex mis-match
My financial institution uses <ACCTID> values of the form xxxxxxx-txx where x is a digit, dash is a dash, and t is a type (S or L) which won't match the current regex, and errors out. I have worked around this issue with the following. You may be able to do better.... I don't know if my financial institution may not be following the spec if this field should only allow digits.... at line 103... if ($all =~ m:<ACCTID>([^<]+)<:s) { $aa = $1; $aa =~ s/\s*$//; } $aa; My setup: perl version 5.8.8 ubuntu 8.04 w/ all updates as of 1/20/2009 Linux arachnae 2.6.24-23-generic #1 SMP Thu Nov 27 18:44:42 UTC 2008 i686 GNU/Linux enjoy...
RT-Send-CC: jeremy.jones [...] pennyphoto.com
Thanks for your bug report. I have revised the Finance::OFX::Parse::Simple module in line with your code sample: if ($all =~ m:<ACCTID>([^<]+?)\s*<:s) and I think that should capture everything you need, and it's non-greedy so you hopefully don't have to do extra trimming. This fix is in for version 0.02 which I have just uploaded, it may not be available on CPAN yet so I'll also attach it in case it's useful. Thanks again for the report and for using the module. Cheers!

Message body not shown because it is not plain text.