-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[5.4] naming convention language string #45785
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[5.4] naming convention language string #45785
Conversation
|
You can't change it this way. You need to deprecate the existing string and add a new one. See #45725 for an example |
|
@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. |
|
@brianteeman @richard67 thank you for the clear explanation. |
|
@hans2103 Thanks. |
|
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. |
|
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. |
|
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45785. |
|
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. |
|
Thank you @hans2103 for your contribution. Thank you @brianteeman and @exlemor for testing. |
Pull Request for no existing Issue .
Summary of Changes
Two language strings for a description use
DESCR.All other use
DESCRIPTIONorDESC.We should not have a third flavor.
Therefor this PR will change the language string from
_DESCRto_DESCThis PR will require a change in all languages.
Testing Instructions
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