Skip to content

Conversation

@koppor
Copy link
Member

@koppor koppor commented Nov 23, 2025

Triggered by #14252 (comment)

Follow-up to #13506.

Introduces a factor for the comparators.

Moreover, fixes field properties. YEARDIVISON had only one field --> removed. We have two different YEARS --> added property.

Steps to test

Do some test cases mentioned at #13506.

Mandatory checks

  • I own the copyright of the code submitted and I license it under the MIT license
  • [/] I manually tested my changes in running JabRef (always required)
  • [/] I added JUnit tests for changes (if applicable)
  • [/] I added screenshots in the PR description (if change is visible to the user)
  • [/] I described the change in CHANGELOG.md in a way that is understandable for the average user (if change is visible to the user)
  • [/] I checked the user documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request updating file(s) in https://github.com/JabRef/user-documentation/tree/main/en.

@koppor koppor added the dev: code-quality Issues related to code or architecture decisions label Nov 23, 2025
Siedlerchr
Siedlerchr previously approved these changes Nov 24, 2025
…bility/EntryTypePlausibilityComparator.java

Co-authored-by: Christoph <siedlerkiller@gmail.com>
@koppor koppor enabled auto-merge November 24, 2025 20:48
@koppor koppor added this pull request to the merge queue Nov 24, 2025
Merged via the queue into main with commit bc9868f Nov 24, 2025
53 checks passed
@koppor koppor deleted the add-plausibility-package branch November 24, 2025 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev: code-quality Issues related to code or architecture decisions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants