Skip to content

New Crowdin updates#15676

Merged
calixtus merged 3 commits into
mainfrom
l10n_main
May 4, 2026
Merged

New Crowdin updates#15676
calixtus merged 3 commits into
mainfrom
l10n_main

Conversation

@Siedlerchr

@Siedlerchr Siedlerchr commented May 4, 2026

Copy link
Copy Markdown
Member

No description provided.

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

qodo-free-for-open-source-projects Bot commented May 4, 2026

Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Crowdin localization updates for French, Italian, and Portuguese

📝 Documentation

Grey Divider

Walkthroughs

Description 
• Add French translations for executable and push-related messages
• Add Italian translations for executable and push-related messages
• Add Portuguese (Brazilian) translations for library, import, export, and validation messages
Diagram 
flowchart LR
  A["Translation Source"] -->|French| B["JabRef_fr.properties"]
  A -->|Italian| C["JabRef_it.properties"]
  A -->|Portuguese BR| D["JabRef_pt_BR.properties"]
  B --> E["Executable & Push Messages"]
  C --> F["Executable & Push Messages"]
  D --> G["Library, Import, Export & Validation Messages"]
Loading

Grey Divider

File Changes

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

French translations for executable and push errors

• Added 4 new French translations for executable call errors and macOS command-line binary guidance
• Added 5 new French translations for Emacs connection errors and push operation failures
• Translations cover error messages for external application integration

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

2. jablib/src/main/resources/l10n/JabRef_it.properties 📝 Documentation +9/-0

Italian translations for executable and push errors

• Added 4 new Italian translations for executable call errors and macOS binary guidance
• Added 5 new Italian translations for Emacs connection and push operation error messages
• Translations mirror French additions for consistency across languages

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

3. jablib/src/main/resources/l10n/JabRef_pt_BR.properties 📝 Documentation +18/-0

Portuguese (Brazilian) translations for library and validation features

• Added 13 new Portuguese (Brazilian) translations for library operations (open, import, export)
• Added 6 translations for validation messages (URL format, unbalanced braces, booktitle checks)
• Added 4 translations for UI features (field formatters, CSL style management, cover image
 download)
• Added 2 translations for entry modification tracking and original entry references

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

Grey Divider

Qodo Logo

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

qodo-free-for-open-source-projects Bot commented May 4, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (2)

Grey Divider


Action required

1. Duplicate executable error string 📘 Rule violation ⚙ Maintainability
Description 
The PR adds both separate strings (Could not call executable '%0'. and `Please check the path in
the preferences.) and a near-duplicate combined string (Could not call executable '%0' Please
check...`) in the same locale files. This increases translation/maintenance burden and risks
inconsistent wording across the UI.
Code

jablib/src/main/resources/l10n/JabRef_fr.properties[2914]

+Could\ not\ call\ executable\ '%0'\ Please\ check\ the\ path\ in\ the\ preferences.=Impossible d'appeler l'exécutable '%0'. Veuillez vérifier le chemin dans les préférences.
Evidence 
PR Compliance requires reusing existing localization strings and avoiding near-duplicates. In
JabRef_fr.properties and JabRef_it.properties, the PR adds separate message parts and also adds
a combined message that duplicates those parts, violating the reuse/near-duplicate guidance.

AGENTS.md
jablib/src/main/resources/l10n/JabRef_fr.properties[66-69]
jablib/src/main/resources/l10n/JabRef_fr.properties[2914-2914]
jablib/src/main/resources/l10n/JabRef_it.properties[66-69]
jablib/src/main/resources/l10n/JabRef_it.properties[2885-2885]
Best Practice: Learned patterns

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 locale files add both split and combined variants of the same user-facing message (duplicate/near-duplicate strings).

## Issue Context
This violates the localization guidance to reuse strings and avoid near-duplicates, and it increases translation maintenance cost.

## Fix Focus Areas
- jablib/src/main/resources/l10n/JabRef_fr.properties[66-69]
- jablib/src/main/resources/l10n/JabRef_fr.properties[2914-2914]
- jablib/src/main/resources/l10n/JabRef_it.properties[66-69]
- jablib/src/main/resources/l10n/JabRef_it.properties[2885-2885]

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


Remediation recommended

2. PT-BR mid-sentence capitalization 📘 Rule violation ⚙ Maintainability
Description 
The new PT-BR translation for Journal-related capitalizes Revista mid-sentence, which conflicts
with the sentence-case localization style rule. This reduces consistency and translation quality in
user-facing UI text.
Code

jablib/src/main/resources/l10n/JabRef_pt_BR.properties[1173]

+Journal-related=Relacionado a uma Revista
Evidence 
The localization style rules require sentence case for UI text. The added PT-BR value `Relacionado a
uma Revista uses title-style capitalization for Revista` in the middle of the sentence.

AGENTS.md
jablib/src/main/resources/l10n/JabRef_pt_BR.properties[1173-1173]

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

## Issue description
A newly added PT-BR translation uses inconsistent capitalization (not sentence case).

## Issue Context
Localization copy should follow sentence case for consistency across the UI.

## Fix Focus Areas
- jablib/src/main/resources/l10n/JabRef_pt_BR.properties[1173-1173]

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

Grey Divider

Qodo Logo

@koppor koppor enabled auto-merge May 4, 2026 20:46
qodo-free-for-open-source-projects[bot]
qodo-free-for-open-source-projects Bot reviewed May 4, 2026
View reviewed changes
Comment thread jablib/src/main/resources/l10n/JabRef_fr.properties
@calixtus calixtus disabled auto-merge May 4, 2026 20:58
@calixtus calixtus merged commit 8e155a0 into main May 4, 2026
26 checks passed
@calixtus calixtus deleted the l10n_main branch May 4, 2026 20:58
Siedlerchr added a commit that referenced this pull request May 5, 2026
@Siedlerchr
Merge remote-tracking branch 'upstream/main' into add-oauth2
1292da7 
* upstream/main: (775 commits)
  Chore(deps): Bump com.konghq:unirest-modules-gson in /versions (#15682)
  Chore(deps): Bump org.glassfish.jaxb:jaxb-runtime in /versions (#15681)
  Update dependency com.konghq:unirest-modules-gson to v4.9.0 (#15679)
  Integrate with SearchRxiv  (#15373)
  Fix requirements (#15600)
  refactor: less objects during writing (#15677)
  Fix: suppress WARN for empty or blank column name in MainTableColumnModel#parse() (#15576)
  New Crowdin updates (#15676)
  Chore(deps): Bump com.github.ben-manes.caffeine:caffeine in /versions (#15673)
  Fix Nullwarnings - C (Mark bst package as nonnull by default) (#15663)
  Chore(deps): Bump com.github.javaparser:javaparser-symbol-solver-core (#15674)
  Chore(deps): Bump com.github.javaparser:javaparser-core in /versions (#15672)
  New Crowdin updates (#15669)
  Fix OpenRewrite (#15670)
  Udpate heylogs (and fix CHANGELOG.md) (#15671)
  Improve security and prevent shell injection for push2applications (#15628)
  Fix depdency analysis (#15668)
  Always use CI-local "gradle", instead of gradlew (#15667)
  Change OpenRewrite task to use rewriteDryRun (#15664)
  Add small documentation to parameter (#15666)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@qodo-free-for-open-source-projects qodo-free-for-open-source-projects[bot] qodo-free-for-open-source-projects[bot] left review comments

@calixtus calixtus calixtus approved these changes

@jabref-machine jabref-machine jabref-machine approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Siedlerchr @calixtus @jabref-machine