-
-
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
Support CrossRef in CSL styles #8454
Conversation
Fixes #7378 Add new test Fix existing test
Failing fetcher test are not relevant (RFC the url format changed) |
src/main/java/org/jabref/logic/citationstyle/CitationStyleGenerator.java
Show resolved
Hide resolved
src/main/java/org/jabref/logic/citationstyle/CitationStyleGenerator.java
Outdated
Show resolved
Hide resolved
src/test/java/org/jabref/logic/citationstyle/CitationStyleGeneratorTest.java
Outdated
Show resolved
Hide resolved
todo need db context
* upstream/main: Bump h2-mvstore from 2.0.206 to 2.1.210 in /buildSrc (#8458) Bump mariadb-java-client from 2.7.4 to 2.7.5 (#8457) Bump com.github.andygoossens.modernizer from 1.6.1 to 1.6.2 (#8459) Bump mockito-core from 4.2.0 to 4.3.0 (#8460) Bump xmlunit-matchers from 2.8.3 to 2.8.4 (#8461) Improve error handling for browser extension (#8432)
* upstream/main: Made search button in Web search wider. Also skewed side panel titles to the left (#8463)
add tests for issue and number modify example tobe able to distinguish between issue and number
fix tests
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.
Small wish added - otherwise LGTM!
factory.createMenuItem(StandardActions.COPY_CITATION_RTF, new CopyCitationAction(CitationStyleOutputFormat.RTF, dialogService, stateManager, clipBoardManager, taskExecutor, previewPreferences)), | ||
factory.createMenuItem(StandardActions.COPY_CITATION_ASCII_DOC, new CopyCitationAction(CitationStyleOutputFormat.ASCII_DOC, dialogService, stateManager, clipBoardManager, taskExecutor, previewPreferences)), | ||
factory.createMenuItem(StandardActions.COPY_CITATION_XSLFO, new CopyCitationAction(CitationStyleOutputFormat.XSL_FO, dialogService, stateManager, clipBoardManager, taskExecutor, previewPreferences))); | ||
copyCitationMenu.getItems().add( |
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.
Can we add COPY_CITATION_HTML
? - This is IMHO a proper replacement for RTF, ASCII_DOC, XSLFO
factory.createMenuItem(StandardActions.COPY_CITATION_HTML, new CopyCitationAction(CitationStyleOutputFormat.HTML, dialogService, stateManager, clipBoardManager, taskExecutor, previewPreferences)),
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.
See two lines above, HTML has been already available
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.
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.
@Siedlerchr How can i help you? Sorry, at least ~90% of this pull request is something i would need to look up, though. It is above my current skill level.
BibEntry entry = new BibEntry(StandardEntryType.Article); | ||
entry.setField(StandardField.AUTHOR, "Last, First and\nDoe, Jane"); | ||
entry.setField(StandardField.ISSUE, "7"); | ||
entry.setField(StandardField.ISSUE, "28"); |
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.
Is this a typo? Issue x2. Above it says "withIssueAndNumber".
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.
Oh thanks for spotting! This was not intended.
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.
As you can now see at the tests, the bibtex number gets now rendered as Art no. This PR is the preparation for the other issue, as we now have complete database information
void testBiblatexWitNumber() { | ||
BibEntry entry = new BibEntry(StandardEntryType.Article); | ||
entry.setField(StandardField.AUTHOR, "Last, First and Doe, Jane"); | ||
entry.setField(StandardField.NUMBER, "7"); |
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.
At the top you used "3" for number. Maybe here too?
* upstream/main: Update to Javafx 17.02 (#8470)
* upstream/main: (24 commits) Support CrossRef in CSL styles (#8454) Update FUNDING.yml (#8473) Update to Javafx 17.02 (#8470) Made search button in Web search wider. Also skewed side panel titles to the left (#8463) Bump h2-mvstore from 2.0.206 to 2.1.210 in /buildSrc (#8458) Bump mariadb-java-client from 2.7.4 to 2.7.5 (#8457) Bump com.github.andygoossens.modernizer from 1.6.1 to 1.6.2 (#8459) Bump mockito-core from 4.2.0 to 4.3.0 (#8460) Bump xmlunit-matchers from 2.8.3 to 2.8.4 (#8461) Improve error handling for browser extension (#8432) New Crowdin updates (#8451) Fix library tab exception when saving prefs (#8450) Rename Groups interface into Groups (#8449) New Crowdin updates (#8445) update snap url Update bug_report.yml for 5.5 Show development information\n\n+semver: minor Release v5.5 Update journal abbrev list New Crowdin updates (#8439) ...
Fixes #7378 and also provides groundwork for any fix regarding #8372
Removed copy of CSL to ASCIIDOC, RTF and XSL_FO as these are no longer supported
CHANGELOG.md
described in a way that is understandable for the average user (if applicable)