Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem with ampersand character "&" when importing ofx-file #61

Closed
fahrplaner opened this issue Oct 5, 2018 · 3 comments
Closed

Problem with ampersand character "&" when importing ofx-file #61

fahrplaner opened this issue Oct 5, 2018 · 3 comments

Comments

@fahrplaner
Copy link

fahrplaner commented Oct 5, 2018

Hello, it seems as if the problem #35 exists again in the jGnash-version (2.34.0, 2.35.1 and) 2.36.0 we supply new testfiles -- the files supplied in #35 works fine though.

Using java 1.8.0 and swing ui.

Cheers

ofx.zip

@ccavanaugh
Copy link
Owner

Looking into it. 35 addressed incorrectly OFX v2 files with incorrectly escape characters

Your files are OFX v2, and contain incorrectly escape characters. Should be a relatively easy fix.

@fahrplaner
Copy link
Author

OK, thanks for the hint. We'll address the problem at the source. You can close the issue or change it in something regarding error handling (XML validating?). Just in case other people run into this problem, it would be nice to have an error message, instead missing entries. Cheers.

ccavanaugh added a commit that referenced this issue Oct 7, 2018
@ccavanaugh
Copy link
Owner

A fix has been committed. Ideally, it should be corrected upstream, but historically, multiple banks have produced non-compliant files. It's easier to handle it gracefully instead of dealing with the noise.

Thanks for reporting and submitting the sample file. I have included it in a unit test.

ccavanaugh added a commit that referenced this issue Oct 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants