Skip to content

New Crowdin updates#15208

Merged
koppor merged 11 commits into
mainfrom
l10n_main
Feb 24, 2026
Merged

New Crowdin updates#15208
koppor merged 11 commits into
mainfrom
l10n_main

Conversation

@Siedlerchr

Copy link
Copy Markdown
Member

No description provided.

@qodo-free-for-open-source-projects

Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Update localization files with Crowdin translations

📝 Documentation

Grey Divider

Walkthroughs

Description
• Remove deprecated field validation message across multiple languages
• Add new French translations for field formatters and entry cleanup
• Add new Italian translations for search configuration and import features
• Update Italian field validation message with improved wording
Diagram
flowchart LR
  A["Crowdin Updates"] --> B["Remove deprecated message"]
  A --> C["Add new translations"]
  B --> D["9 language files"]
  C --> E["French translations"]
  C --> F["Italian translations"]
  D --> G["Updated l10n files"]
  E --> G
  F --> G
Loading

Grey Divider

File Changes

1. jablib/src/main/resources/l10n/JabRef_de.properties 📝 Documentation +0/-1

Remove field validation message

jablib/src/main/resources/l10n/JabRef_de.properties


2. jablib/src/main/resources/l10n/JabRef_es.properties 📝 Documentation +0/-1

Remove field validation message

jablib/src/main/resources/l10n/JabRef_es.properties


3. jablib/src/main/resources/l10n/JabRef_fr.properties 📝 Documentation +4/-1

Add new translations for features

jablib/src/main/resources/l10n/JabRef_fr.properties


View more (8)
4. jablib/src/main/resources/l10n/JabRef_it.properties 📝 Documentation +10/-1

Update field validation and add translations

jablib/src/main/resources/l10n/JabRef_it.properties


5. jablib/src/main/resources/l10n/JabRef_ja.properties 📝 Documentation +0/-1

Remove field validation message

jablib/src/main/resources/l10n/JabRef_ja.properties


6. jablib/src/main/resources/l10n/JabRef_ko.properties 📝 Documentation +0/-1

Remove field validation message

jablib/src/main/resources/l10n/JabRef_ko.properties


7. jablib/src/main/resources/l10n/JabRef_nl.properties 📝 Documentation +0/-1

Remove field validation message

jablib/src/main/resources/l10n/JabRef_nl.properties


8. jablib/src/main/resources/l10n/JabRef_pt_BR.properties 📝 Documentation +0/-1

Remove field validation message

jablib/src/main/resources/l10n/JabRef_pt_BR.properties


9. jablib/src/main/resources/l10n/JabRef_ru.properties 📝 Documentation +0/-1

Remove field validation message

jablib/src/main/resources/l10n/JabRef_ru.properties


10. jablib/src/main/resources/l10n/JabRef_tr.properties 📝 Documentation +0/-1

Remove field validation message

jablib/src/main/resources/l10n/JabRef_tr.properties


11. jablib/src/main/resources/l10n/JabRef_zh_CN.properties 📝 Documentation +0/-1

Remove field validation message

jablib/src/main/resources/l10n/JabRef_zh_CN.properties


Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented Feb 24, 2026

Copy link
Copy Markdown
Contributor

Code Review by Qodo

🐞 Bugs (1) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider


Action required

1. Removed locale validation key 🐞 Bug ✓ Correctness
Description
The GUI still requests the key "Field cannot be empty. Please enter a name.", but this PR removes
that key from multiple locale bundles, so users in those locales will see the raw English key text
instead of a translation.
Code

jablib/src/main/resources/l10n/JabRef_de.properties[2219]

-Field\ cannot\ be\ empty.\ Please\ enter\ a\ name.=Feld darf nicht leer sein. Bitte geben Sie einen Namen ein.
Evidence
The custom entry type field-name validator still uses the removed localization key. The English
bundle still defines the key, but e.g. the German bundle no longer contains it. JabRef’s
Localization implementation returns the key itself when a translation is missing, so this regression
will silently show English text to non-English users.

jabgui/src/main/java/org/jabref/gui/preferences/customentrytypes/CustomEntryTypesTabViewModel.java[74-82]
jablib/src/main/resources/l10n/JabRef_en.properties[2246-2252]
jablib/src/main/resources/l10n/JabRef_de.properties[2210-2221]
jablib/src/main/java/org/jabref/logic/l10n/Localization.java[104-107]
jablib/src/main/java/org/jabref/logic/l10n/Localization.java[130-137]
jablib/src/main/java/org/jabref/logic/l10n/Localization.java[164-168]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
The localization key `Field cannot be empty. Please enter a name.` is still used by `CustomEntryTypesTabViewModel`, but translations for this key were removed from several locale bundles. This causes untranslated English UI text for users in those locales.

### Issue Context
JabRef’s localization layer returns the key itself when no translation is found, so the regression is silent and won’t crash, but it degrades UX.

### Fix Focus Areas
- jabgui/src/main/java/org/jabref/gui/preferences/customentrytypes/CustomEntryTypesTabViewModel.java[74-82]
- jablib/src/main/resources/l10n/JabRef_de.properties[2216-2221]
- jablib/src/main/resources/l10n/JabRef_es.properties[2204-2210]
- jablib/src/main/resources/l10n/JabRef_fr.properties[2246-2252]
- jablib/src/main/resources/l10n/JabRef_ja.properties[1775-1781]
- jablib/src/main/resources/l10n/JabRef_ko.properties[1680-1685]
- jablib/src/main/resources/l10n/JabRef_nl.properties[1836-1841]
- jablib/src/main/resources/l10n/JabRef_pt_BR.properties[2192-2197]
- jablib/src/main/resources/l10n/JabRef_ru.properties[1776-1781]
- jablib/src/main/resources/l10n/JabRef_tr.properties[1868-1873]
- jablib/src/main/resources/l10n/JabRef_zh_CN.properties[2214-2219]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

@koppor koppor enabled auto-merge February 24, 2026 21:47
Optional=Optional
Required=Erforderlich
Entry\ type\ cannot\ be\ empty\ and\ must\ not\ contain\ spaces.=Eintragstyp darf nicht leer sein und darf keine Leerzeichen enthalten.
Field\ cannot\ be\ empty.\ Please\ enter\ a\ name.=Feld darf nicht leer sein. Bitte geben Sie einen Namen ein.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Action required

1. Removed locale validation key 🐞 Bug ✓ Correctness

The GUI still requests the key "Field cannot be empty. Please enter a name.", but this PR removes
that key from multiple locale bundles, so users in those locales will see the raw English key text
instead of a translation.
Agent Prompt
### Issue description
The localization key `Field cannot be empty. Please enter a name.` is still used by `CustomEntryTypesTabViewModel`, but translations for this key were removed from several locale bundles. This causes untranslated English UI text for users in those locales.

### Issue Context
JabRef’s localization layer returns the key itself when no translation is found, so the regression is silent and won’t crash, but it degrades UX.

### Fix Focus Areas
- jabgui/src/main/java/org/jabref/gui/preferences/customentrytypes/CustomEntryTypesTabViewModel.java[74-82]
- jablib/src/main/resources/l10n/JabRef_de.properties[2216-2221]
- jablib/src/main/resources/l10n/JabRef_es.properties[2204-2210]
- jablib/src/main/resources/l10n/JabRef_fr.properties[2246-2252]
- jablib/src/main/resources/l10n/JabRef_ja.properties[1775-1781]
- jablib/src/main/resources/l10n/JabRef_ko.properties[1680-1685]
- jablib/src/main/resources/l10n/JabRef_nl.properties[1836-1841]
- jablib/src/main/resources/l10n/JabRef_pt_BR.properties[2192-2197]
- jablib/src/main/resources/l10n/JabRef_ru.properties[1776-1781]
- jablib/src/main/resources/l10n/JabRef_tr.properties[1868-1873]
- jablib/src/main/resources/l10n/JabRef_zh_CN.properties[2214-2219]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

@koppor koppor added this pull request to the merge queue Feb 24, 2026
@testlens-app

testlens-app Bot commented Feb 24, 2026

Copy link
Copy Markdown

✅ All tests passed ✅

🏷️ Commit: 669aa90
▶️ Tests: 11198 executed
⚪️ Checks: 65/65 completed


Learn more about TestLens at testlens.app.

@github-actions github-actions Bot added the status: to-be-merged PRs which are accepted and should go into the merge-queue. label Feb 24, 2026
Merged via the queue into main with commit 929789c Feb 24, 2026
66 checks passed
@koppor koppor deleted the l10n_main branch February 24, 2026 22:30
Siedlerchr added a commit that referenced this pull request Feb 25, 2026
* refs/heads/main:
  Increase max assignments from 2 to 3
  Chore(deps): Bump io.zonky.test:embedded-postgres in /versions (#15213)
  Chore(deps): Bump jablib/src/main/resources/csl-styles (#15211)
  New Crowdin updates (#15208)
  Fix: Prevent creating empty or duplicate fields  (#15168)
  chore(deps): update jackson monorepo to v3.1.0 (#15203)
  Update KeywordEditor to work with escaping (#14929)
  Chore(deps): Bump com.dlsc.gemsfx:gemsfx in /versions (#15205)
  Chore(deps): Bump tools.jackson:jackson-bom in /versions (#15206)
  Fix: Reset External File Type to default (#15167)
  docs: fix link formatting in ADR 0012 (#15201)
RakockiW pushed a commit to RakockiW/jabref that referenced this pull request Mar 1, 2026
* New translations jabref_en.properties (French)

* New translations jabref_en.properties (Spanish)

* New translations jabref_en.properties (German)

* New translations jabref_en.properties (Italian)

* New translations jabref_en.properties (Japanese)

* New translations jabref_en.properties (Korean)

* New translations jabref_en.properties (Dutch)

* New translations jabref_en.properties (Russian)

* New translations jabref_en.properties (Turkish)

* New translations jabref_en.properties (Chinese Simplified)

* New translations jabref_en.properties (Portuguese, Brazilian)
priyanshu16095 pushed a commit to priyanshu16095/jabref that referenced this pull request Mar 3, 2026
* New translations jabref_en.properties (French)

* New translations jabref_en.properties (Spanish)

* New translations jabref_en.properties (German)

* New translations jabref_en.properties (Italian)

* New translations jabref_en.properties (Japanese)

* New translations jabref_en.properties (Korean)

* New translations jabref_en.properties (Dutch)

* New translations jabref_en.properties (Russian)

* New translations jabref_en.properties (Turkish)

* New translations jabref_en.properties (Chinese Simplified)

* New translations jabref_en.properties (Portuguese, Brazilian)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: to-be-merged PRs which are accepted and should go into the merge-queue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants