Skip to content

Conversation

@hans2103
Copy link
Contributor

Pull Request for no existing Issue .

Summary of Changes

Two language strings for a description use DESCR.
All other use DESCRIPTION or DESC.
We should not have a third flavor.
Therefor this PR will change the language string from _DESCR to _DESC

This PR will require a change in all languages.

Testing Instructions

  • Joomla Administrator > System > Plugins > Editor - Codemirror
  • open plugin to edit
  • go to tab advanced
  • press green button to add custom extensions
  • read the description below both input fields

Actual result BEFORE applying this Pull Request

There is a description between both input fields

Expected result AFTER applying this Pull Request

The description is still there and the same.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-5.4-dev labels Jul 25, 2025
@brianteeman
Copy link
Contributor

You can't change it this way. You need to deprecate the existing string and add a new one. See #45725 for an example

@richard67
Copy link
Member

@hans2103 You cannot just remove a language key (changing a key is like removing the old key and adding the new key).

You have to deprecate the language string for 6.0, and when this PR got merged and merged up into 6.0-dev, someone has to make a PR for 6.0-dev to finally remove the deprecated string.

You can find examples for this here:

The changes in the spell check GitHub action would not be needed as your PR does not handle language stings with spelling errors in the key.

@richard67 richard67 added the Updates Requested Indicates that this pull request needs an update from the author and should not be tested. label Jul 25, 2025
@hans2103
Copy link
Contributor Author

@brianteeman @richard67 thank you for the clear explanation.
Used your feedback and adjusted the PR based on the way Brian created his in https://github.com/joomla/joomla-cms/pull/45725/files

@richard67
Copy link
Member

@hans2103 Thanks.

@richard67 richard67 removed the Updates Requested Indicates that this pull request needs an update from the author and should not be tested. label Jul 25, 2025
@brianteeman
Copy link
Contributor

I have tested this item ✅ successfully on bf1a180


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45785.

@exlemor
Copy link

exlemor commented Jul 26, 2025

I have tested this item ✅ successfully on bf1a180

I have successfully tested this. Thanks.@hans2013


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45785.

@richard67
Copy link
Member

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45785.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jul 26, 2025
@muhme
Copy link
Contributor

muhme commented Jul 26, 2025

Final check before merge: Each language key is only used once and this usage has been changed. Old language keys are still existing and marked as deprecated in 6.0.

@muhme muhme merged commit 3e0ea1a into joomla:5.4-dev Jul 26, 2025
32 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jul 26, 2025
@muhme muhme added this to the Joomla! 5.4.0 milestone Jul 26, 2025
@muhme
Copy link
Contributor

muhme commented Jul 26, 2025

Thank you @hans2103 for your contribution. Thank you @brianteeman and @exlemor for testing.

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

Labels

Language Change This is for Translators PR-5.4-dev

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants