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

Quickfix for import into current [shared] database #1777

Merged
merged 1 commit into from
Aug 18, 2016

Conversation

obraliar
Copy link
Contributor

@obraliar obraliar commented Aug 18, 2016

Quickfix for the comments in #1748:

JabRef shows "Aktualisierung verweigert. Den Eintrag, den Sie zur Zeit bearbeiten, ist auf der geminsam genutzen Datenbank nicht mehr vorhanden." for each entry.

and

After the import, all entries appear twice.

Bug occured due to change of a BibEntry before adding it into local database.

@tschechlovdev tschechlovdev added status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers and removed stupro-ready-for-internal-review labels Aug 18, 2016
@koppor
Copy link
Member

koppor commented Aug 18, 2016

Quick fix is OK. A discussion with @obraliar revealed that the reason of the wrong event is entry.setId(IdGenerator.next()); in gui/importer/ImportInspectionDialog.java line 474. might be

BibtexKeyPatternUtil.makeLabel(localMetaData, database, entry,
and
private final EventList<BibEntry> entries = new BasicEventList<>();
.

One has to investigate further if the event system has some huger issue. An event for an entry not being present in the database should not be posted.

@koppor koppor merged commit 0c16669 into JabRef:master Aug 18, 2016
Siedlerchr added a commit that referenced this pull request Aug 19, 2016
* master:
  Fix NPE when closing the last tab (#1783)
  Updated java-string-similarity to 0.16 (#1784)
  Fix Imports
  Move logic to ImportFileFilter
  Some OS didn't show directories when a file type was selected in the import dialog
  GvkFetcher (#1656)
  Add some more Objects.requireNonNull checks
  Fixed startup problem
  Add isPresentLocalBibEntry (#1777)
  Injected LayoutFormatterPreferences in ExportFormat (#1695)
  Added UpdateFieldPreferences (#1773)
  Fix typo
  Optimize imports
  Fix #1771 Show all supported import types as default
Siedlerchr added a commit to Siedlerchr/jabref that referenced this pull request Aug 19, 2016
* master:
  Fix NPE when closing the last tab (JabRef#1783)
  Updated java-string-similarity to 0.16 (JabRef#1784)
  Fix Imports
  Move logic to ImportFileFilter
  Some OS didn't show directories when a file type was selected in the import dialog
  GvkFetcher (JabRef#1656)
  Add some more Objects.requireNonNull checks
  Fixed startup problem
  Add isPresentLocalBibEntry (JabRef#1777)
  Injected LayoutFormatterPreferences in ExportFormat (JabRef#1695)
  Added UpdateFieldPreferences (JabRef#1773)
  Fix typo
  Optimize imports
  Fix JabRef#1771 Show all supported import types as default
ayanai1 pushed a commit to ayanai1/jabref that referenced this pull request Sep 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants