? ? Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
6 Nov 2021

Pull Request for Issue #35946 .

Same as #35981 but for 3.10-dev.

Summary of Changes

This pull request (PR) changes the input filter type from "cmd" to "string" in the extension installer's language adapter so that spaces and parenthesis are not removed anymore from the extension name in database so that error messages show it right.

In opposite to other extension types where language strings might be provided to translate the name, names of languages should never be translated, neither the English name nor the native name.

That's why there is no need to use the "cmd" filter like it would be when using the name to construct a language string constant from it.

Hint for maintainers

PR #35981 which does the same as this one here but for 4.0-dev has already been merged.

Testing Instructions

  1. On a clean 3.10-dev branch or latest 3.10.3 release or 3.10 nightly build, install an old version of a language pack where the extension names for the site and the admin language contain spaces and/or parenthesis.
    You can download e.g. the old French and German packages here:
  1. Check the names of the site and the admin languages in database e.g. with phpMyAdmin.
    Result: In opposite to the core English languages, the names of the just installed site and admin languages have spaces and parenthesis stripped off.
    j3-db-after-new-install-without-patch

  2. Go to "Extensions - Manage - Manage" and try to uninstall some of the previously installed language packs' admin or site languages.
    Result: The warning alert shows the name as it is in database.
    j3-db-try-uninstall-language-without-patch

  3. Apply the patch of this PR.

  4. Make sure that you have switched on "Debug System" and set "Error Reporting" to "Maximum" or "Development" in Global Configuration so you would see any errors happening in the next steps.

  5. Go to "Extensions - Manage - Update", if necessary use the buttons to clear cache and to check for updates and then update the 2 old language packs.

  6. Go to "Extensions - Manage - Install Languages" and install some more language(s) suitable for the test, e.g. French (Canada) and German (Switzerland).

  7. Check again the names in database.
    Result: The names are as specified in the site and admin language's manifest XML files, no spaces or parenthesis have been stripped off. This is true for both the updated and the newly installed languages.
    j3-db-after-update-with-patch

  8. Go to "Extensions - Manage - Manage" and try again to uninstall some of the site or admin languages.
    Result: The warning alert shows the name as it is in database, and that's ok now.
    j3-db-try-uninstall-language-with-patch

Actual result BEFORE applying this Pull Request

See testing instructions steps 2 and 3.

Expected result AFTER applying this Pull Request

See testing instructions steps 8 and 9.

Documentation Changes Required

None.

avatar richard67 richard67 - open - 6 Nov 2021
avatar richard67 richard67 - change - 6 Nov 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Nov 2021
Category Libraries
avatar richard67 richard67 - change - 6 Nov 2021
The description was changed
avatar richard67 richard67 - edited - 6 Nov 2021
avatar richard67 richard67 - change - 6 Nov 2021
The description was changed
avatar richard67 richard67 - edited - 6 Nov 2021
avatar infograf768 infograf768 - test_item - 7 Nov 2021 - Tested successfully
avatar infograf768
infograf768 - comment - 7 Nov 2021

I have tested this item successfully on d35fc59


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

avatar imanickam imanickam - test_item - 7 Nov 2021 - Tested successfully
avatar imanickam
imanickam - comment - 7 Nov 2021

I have tested this item successfully on d35fc59

I have test this successfully on Joomla! 3.10.3


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

avatar richard67 richard67 - change - 7 Nov 2021
Status Pending Ready to Commit
Labels Added: ?
avatar richard67
richard67 - comment - 7 Nov 2021

RTC


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

avatar richard67 richard67 - change - 7 Nov 2021
The description was changed
avatar richard67 richard67 - edited - 7 Nov 2021
avatar richard67 richard67 - change - 7 Nov 2021
The description was changed
avatar richard67 richard67 - edited - 7 Nov 2021
avatar pritam825 pritam825 - test_item - 20 Nov 2021 - Tested successfully
avatar pritam825
pritam825 - comment - 20 Nov 2021

I have tested this item successfully on d35fc59


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

avatar zero-24
zero-24 - comment - 21 Nov 2021

Taking this in thanks folks :)

avatar zero-24 zero-24 - close - 21 Nov 2021
avatar zero-24 zero-24 - merge - 21 Nov 2021
avatar zero-24 zero-24 - change - 21 Nov 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-11-21 13:32:14
Closed_By zero-24
Labels Added: ?
avatar richard67
richard67 - comment - 21 Nov 2021

Thanks all.

Add a Comment

Login with GitHub to post a comment