Skip to content
New issue

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

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

Already on GitHub? # to your account

Import Format Reader Fix #531

Merged
merged 1 commit into from
Dec 21, 2015
Merged

Conversation

boceckts
Copy link
Contributor

The Problem is, that the stream can not be resetted without a BufferedInputStream.
However this is necessary since we check if a import file is in the right format with
looking into the stream and searching for specific patterns.

Result of the above mentioned issue is that I always got an error message saying
"No entries found" when trying to import a file with a valid format and entries.

This fixes the issue without changing any method interfaces.

@simonharrer
Copy link
Contributor

Please add a CHANGELOG entry as this fixes #479

@simonharrer simonharrer added stupro [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs labels Dec 21, 2015
@boceckts boceckts force-pushed the IsRecognizedFormatFix branch from e94db1c to 2b929c7 Compare December 21, 2015 11:58
@oscargus
Copy link
Contributor

Probably some line ending issue as all lines in ImportFormatReader are marked as changed and a new copy of the CHANGELOG is appended at the start of it.

The stream could not be resetted without a BufferedInputStream. However
this is necessary since we check if a import file is in the right format
with looking into the stream and searching for specific patterns. This
fixes the issue without changing any method interfaces.
@boceckts boceckts force-pushed the IsRecognizedFormatFix branch from 2b929c7 to c0ec6d1 Compare December 21, 2015 13:43
simonharrer added a commit that referenced this pull request Dec 21, 2015
@simonharrer simonharrer merged commit a54e565 into JabRef:master Dec 21, 2015
@boceckts boceckts deleted the IsRecognizedFormatFix branch January 11, 2016 13:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
[outdated] type: bug Confirmed bugs or reports that are very likely to be bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants