Skip to content
Please note that GitHub no longer supports your web browser.

We recommend upgrading to the latest Google Chrome or Firefox.

Learn more
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

Fix and improve BibTeX parsing #2020

Open
wants to merge 3 commits into
base: master
from

Conversation

@zuphilip
Copy link
Collaborator

commented Oct 12, 2019

  • In some cases the rawValue was undefined which caused
    a problem in the author field.
  • Handle electronic as webpage
  • Skip preamble
  • Improve inline documentation
- In some cases the rawValue was undefined which caused
a problem in the author field.
- Handle electronic as webpage
- Skip preamble
- Improve inline documentation
Copy link
Collaborator Author

left a comment

I tested this with the long bibtex test file from https://github.com/citation-js/bibtex-parser-experiments/blob/master/test/files/long.bib and everything worked well. Now, we are importing all 3345 entries and not other side effect was detected during the import. I will merge this the next days, if no one objects.

Moreover, allow special characters like colon
or underscore to be part of the string name.
@zuphilip zuphilip force-pushed the zuphilip:bibtex-edge-cases branch from a99aac4 to 1d7bc03 Oct 13, 2019
@adam3smith

This comment has been minimized.

Copy link
Collaborator

commented Oct 20, 2019

This looks good to me, but leaving for you to merge, not having read the code closely enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.