Skip to content

Remove resolve duplicate keys dialog#4696

Merged
Siedlerchr merged 1 commit into
masterfrom
removeDuplicateKeyDialog
Feb 23, 2019
Merged

Remove resolve duplicate keys dialog#4696
Siedlerchr merged 1 commit into
masterfrom
removeDuplicateKeyDialog

Conversation

@tobiasdiez

@tobiasdiez tobiasdiez commented Feb 23, 2019 •
edited by Siedlerchr
Loading

Copy link
Copy Markdown
Member

Fixes #4614

Removes the resolve duplicate keys dialog. This feature is made obsolete by the integrity check and the user can generate new keys very simply (for all entries or only for the duplicated ones) through the entry editor and/or menu.

  • Change in CHANGELOG.md described
  • Tests created for changes
  • Manually tested changed features in running JabRef
  • Screenshots added in PR description (for bigger UI changes)
  • Ensured that the git commit message is a good one
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

@tobiasdiez tobiasdiez added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Feb 23, 2019
@Siedlerchr

Siedlerchr commented Feb 23, 2019

Copy link
Copy Markdown
Member

Well, however on import there should be a warning and the option to resolve duplicate keys. It's essential to discover duplicates

@tobiasdiez

tobiasdiez commented Feb 23, 2019 •
edited
Loading

Copy link
Copy Markdown
Member Author

@Siedlerchr such a feature wasn't implemented (the only check was when opening a database) and is obsolete with my other PR #4692, where now keys are automatically generated for every imported entry. Duplicate entries are also highlighted in the import dialog.

@Siedlerchr

Siedlerchr commented Feb 23, 2019

Copy link
Copy Markdown
Member

Okay, then I see no problem!

@Siedlerchr Siedlerchr merged commit 4deb581 into master Feb 23, 2019
@Siedlerchr Siedlerchr deleted the removeDuplicateKeyDialog branch February 23, 2019 16:49
@Siedlerchr Siedlerchr mentioned this pull request Feb 24, 2019
Closed
36 tasks
Siedlerchr added a commit that referenced this pull request Feb 24, 2019
@Siedlerchr
Merge remote-tracking branch 'upstream/master' into libraryproperties…
466f40f 
…tofx

* upstream/master:
  Catch NumberFormatException in BibTeXML exporter (#4698)
  Remove resolve duplicate keys dialog (#4696)
  fixes #4610. Changed the way the GroupDialog description is displayed.  (#4660)
  Bump slf4j-api from 1.8.0-beta2 to 1.8.0-beta4 (#4675)
@luizphilipe2 luizphilipe2 mentioned this pull request Jun 26, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Siedlerchr Siedlerchr Siedlerchr approved these changes

Assignees

No one assigned

Labels

status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

BibTexKey Duplicate Check throws NPE

2 participants

@tobiasdiez @Siedlerchr