? Pending

User tests: Successful: Unsuccessful:

avatar Bakual
Bakual
15 Dec 2019

Pull Request for Issue #27275

Summary of Changes

The Language Override Manager didn't recognise the non-prefixed language files.
This PR changes the language file loading logic in the StringsModel->refresh() function so it doesn't require a prefix anymore. It now just loads all INI files in a specific folder.

Testing Instructions

Try to create a new language override and check in the search if the string appears. Eg search fo "Articles" which should yield you several results for en-GB in the administrator client.

Expected result

Search yields all appropriate results.

Actual result

Search yields only results from files that are language prefixed. In core that is only the sampledata testing plugin.

Documentation Changes Required

None

avatar Bakual Bakual - open - 15 Dec 2019
avatar Bakual Bakual - change - 15 Dec 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Dec 2019
Category Administration com_languages
avatar infograf768 infograf768 - test_item - 15 Dec 2019 - Tested successfully
avatar infograf768
infograf768 - comment - 15 Dec 2019

I have tested this item successfully on 7a6dc8e


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

avatar alikon alikon - test_item - 15 Dec 2019 - Tested successfully
avatar alikon
alikon - comment - 15 Dec 2019

I have tested this item successfully on 7a6dc8e


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

avatar alikon alikon - change - 15 Dec 2019
Status Pending Ready to Commit
avatar alikon
alikon - comment - 15 Dec 2019

RTC


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

avatar infograf768 infograf768 - change - 15 Dec 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-12-15 10:48:34
Closed_By infograf768
Labels Added: ?
avatar infograf768 infograf768 - close - 15 Dec 2019
avatar infograf768 infograf768 - merge - 15 Dec 2019
avatar infograf768
infograf768 - comment - 15 Dec 2019

Tks.

avatar zero-24
zero-24 - comment - 16 Dec 2019

@Bakual @infograf768 isn't this something that needs to be backported to 3.10 too?

avatar infograf768
infograf768 - comment - 16 Dec 2019

Would not hurt, but not absolutely necessary imho as prefixes are still used in 3.9.10

avatar infograf768
infograf768 - comment - 17 Dec 2019

Hmm, I see that some advise users that it is not necessary anymore to add the prefix in their 3rd party extensions in 3.10.
May be therefore safer to indeed backport this.

@Bakual Can you do?

avatar Bakual
Bakual - comment - 17 Dec 2019

Yes, of course we should backport it. Good catch @zero-24
I'll do this.

Add a Comment

Login with GitHub to post a comment