Skip to content

fix missing symbol - #1094

Open
purplegrape wants to merge 1 commit into
Combodo:support/3.2from
purplegrape:support/3.2
Open

purplegrape wants to merge 1 commit into
Combodo:support/3.2from
purplegrape:support/3.2

Conversation

@purplegrape

Copy link
Copy Markdown
Contributor

fix missing symbol.
other branches also has the same issue.

Copilot AI balanced review requested due to automatic review settings October 24, 2026 00:56

Copilot AI left a comment

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.

🟢 Approval recommended

All ten changes correctly restore the numbered placeholder, with no unrelated changes or unresolved issues.

No open issues 🎉

What changed in this PR

Fixes date/time range labels in iTop’s localized search interface by restoring a missing % in the first argument placeholder.

Changes:

  • Replaces literal 1$s with %1$s in ten language dictionaries so the field name is displayed correctly.
File Description
dictionaries/​zh_cn.dictionary.itop.ui.php Fixes the Simplified Chinese range placeholder.
dictionaries/​ru.dictionary.itop.ui.php Fixes the Russian range placeholder.
dictionaries/​pt_br.dictionary.itop.ui.php Fixes the Brazilian Portuguese range placeholder.
dictionaries/​pl.dictionary.itop.ui.php Fixes the Polish range placeholder.
dictionaries/​nl.dictionary.itop.ui.php Fixes the Dutch range placeholder.
dictionaries/​hu.dictionary.itop.ui.php Fixes the Hungarian range placeholder.
dictionaries/​fr.dictionary.itop.ui.php Fixes the French range placeholder.
dictionaries/​es_cr.dictionary.itop.ui.php Fixes the Costa Rican Spanish range placeholder.
dictionaries/​de.dictionary.itop.ui.php Fixes the German range placeholder.
dictionaries/​cs.dictionary.itop.ui.php Fixes the Czech range placeholder.

Review effort: Balanced

@purplegrape

Copy link
Copy Markdown
Contributor Author

@jf-cbd

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Pending review

Development

Successfully merging this pull request may close these issues.

4 participants