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

Fix aux duplicates #2480

Merged
merged 2 commits into from
Jan 23, 2017
Merged

Fix aux duplicates #2480

merged 2 commits into from
Jan 23, 2017

Conversation

lenhard
Copy link
Member

@lenhard lenhard commented Jan 23, 2017

Fixes #2475

A very simple change. Our aux export had some problems when there were crossrefs to other entries. Such references could result in multiple additions of the referenced entry to the generated bib file. A simple check avoids this

  • Change in CHANGELOG.md described
  • Manually tested changed features in running JabRef

@tobiasdiez tobiasdiez merged commit d615880 into master Jan 23, 2017
@tobiasdiez tobiasdiez deleted the fix-aux-duplicates branch January 23, 2017 19:07
@tobiasdiez
Copy link
Member

A test would be nice...but well, I merge it 😄

@ghost
Copy link

ghost commented Jan 24, 2017

I confirm, it solved the problem. Many thanks!

Siedlerchr added a commit that referenced this pull request Jan 26, 2017
* upstream/master:
  Fix medline tests...again (#2492)
  Make sure that unregistered event sources do not stop JabRef from shu… (#2487)
  Fix #2481: ClassCastException because of wrong cast (#2490)
  Catch NumberFormatException if context can't be parsed in groups (#2488)
  Improve CHANGELOG formatting
  Update guava from 20.0 to 21.0 and mockito-core from 2.5.5 to 2.6.2
  Fix aux duplicates (#2480)
  add update from DOI to the entryeditor sidebar (#2476)
  Remove obsolete import
  Add CHANGELOG entry (and one more link)
  Resolves #2309 JabRef freezes when importing unlinked PDF files into Database
  Update CHANGELOG.md
  Fixed bug when assigning refs to groups.
  emove html code from ACM fetcher before calling parser to prevent junk in bib file (#2473)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants