-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Complete rebranding of bibtexkey
as citationkey
#6875
Conversation
The JabRef maintainers will add the following name to the AUTHORS file. In case you want to use a different one, please comment here and adjust your name in your git configuration for future commits.
|
@@ -37,6 +37,11 @@ | |||
} | |||
|
|||
public static Optional<InternalField> fromName(String name) { | |||
if (name.equalsIgnoreCase("bibtexkey")) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Migration of preferences etc is not necessary because we have such a beautiful architecture that every field name is passed trough here. 😆 😄
src/test/java/org/jabref/logic/importer/fetcher/GrobidCitationFetcherTest.java
Show resolved
Hide resolved
In general looks good, but I would prefer a preferences migration for the preview and the default patterns etc to be consistent. Otherwise it looks confusing when we always refer to citation key but some pieces still use bibtexkey |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please test if the layout files still work and follow the checklist.
I left the bibtexkey in the old version of the migration tests since this is the form they are encountered usually for a user, so the tests are really covering the use case. The custom layout files I didn't check, but with custom preview it did worked. |
We should merge this soon, so conflicts with new PRs can be avoided. I scrolled through this one, but did not see anything. |
Agreed. Let's merge it now, and hopefully everything keeps working. If not, feel free to assign the new issues on my name ;-) |
There are failing tests related to the Preferences MIgration:
|
* upstream/master: Fix sort order in main table not stored (#6898) Window "Manage field names & content": make title consistent with menu (+ localization) (#6895) Complete rebranding of `bibtexkey` as `citationkey` (#6875) Dev-doc: ADR list (#6887) Bump lucene-queryparser from 8.6.1 to 8.6.2 (#6881) Bump fontbox from 2.0.20 to 2.0.21 (#6883) Bump org.beryx.jlink from 2.21.3 to 2.21.4 (#6882) Bump com.github.ben-manes.versions from 0.29.0 to 0.30.0 (#6884) Bump pdfbox from 2.0.20 to 2.0.21 (#6885) Bump mockito-core from 3.5.7 to 3.5.10 (#6886) lint changelog Dev Doc: Moving around ADR (end?) (#6873) Add MD031 to lint ignore list Add mlep's other email adress Fetch by ID: SAO/NASA Astrophysics Data System -> SAO/NASA ADS (#6876)
* upstream/master: (55 commits) Fix sort order in main table not stored (#6898) Window "Manage field names & content": make title consistent with menu (+ localization) (#6895) Complete rebranding of `bibtexkey` as `citationkey` (#6875) Dev-doc: ADR list (#6887) Bump lucene-queryparser from 8.6.1 to 8.6.2 (#6881) Bump fontbox from 2.0.20 to 2.0.21 (#6883) Bump org.beryx.jlink from 2.21.3 to 2.21.4 (#6882) Bump com.github.ben-manes.versions from 0.29.0 to 0.30.0 (#6884) Bump pdfbox from 2.0.20 to 2.0.21 (#6885) Bump mockito-core from 3.5.7 to 3.5.10 (#6886) lint changelog Dev Doc: Moving around ADR (end?) (#6873) Add MD031 to lint ignore list Add mlep's other email adress Fetch by ID: SAO/NASA Astrophysics Data System -> SAO/NASA ADS (#6876) Reimport -> Reload (#6874) GitBook: [master] 19 pages and 4 assets modified Add new author Delete .Rhistory Fix icon issue (#6858) ...
Follow-up from #6545.