? ? Pending

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
17 Mar 2017

Pull Request for Issue #14677

Summary of Changes

Reducing the number of language strings by using global values.
30 strings are replaced by 8 strings
7 in lib, one in com_fields

Testing Instructions

Enable Language debug
patch and test that fields plugin do not miss any lang values
test that when creating each type of field, we also get all values.

@laoneo @brianteeman @Bakual

avatar infograf768 infograf768 - open - 17 Mar 2017
avatar infograf768 infograf768 - change - 17 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Mar 2017
Category Administration Language & Strings Front End Plugins
avatar laoneo
laoneo - comment - 17 Mar 2017

Would it be wrong to turn COM_FIELDS_FIELD_USE_GLOBAL into JUSE_GLOBAL_FROM_PLUGIN? For me it looks wrong to have a com_fields string in a plugin XML file.

avatar Bakual
Bakual - comment - 17 Mar 2017

Would it be wrong to turn COM_FIELDS_FIELD_USE_GLOBAL into JUSE_GLOBAL_FROM_PLUGIN?

JGLOBAL_USE_FROM_PLUGIN then, not JUSE_... ?

For me it looks wrong to have a com_fields string in a plugin XML file.

It's in the params.xml one used only by com_fields, not in the plugin manifest file. But I understand your point. Personally I could live with it in this specific case.

avatar laoneo
laoneo - comment - 17 Mar 2017

It's in the params.xml one used only by com_fields

Correctly it is used only by the plugin https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_fields/libraries/fieldsplugin.php#L238 .

avatar infograf768
infograf768 - comment - 17 Mar 2017

as @Bakual rightfully said, this is only used by com_fields. i see no reason to add it in en-GB.ini

avatar AlexRed AlexRed - test_item - 17 Mar 2017 - Tested successfully
avatar AlexRed
AlexRed - comment - 17 Mar 2017

I have tested this item successfully on 1f7a2c4

Patch ok for me


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

avatar Bakual Bakual - change - 18 Mar 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-03-18 11:59:49
Closed_By Bakual
Labels Added: ? ?
avatar Bakual Bakual - close - 18 Mar 2017
avatar Bakual Bakual - merge - 18 Mar 2017

Add a Comment

Login with GitHub to post a comment