? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
30 Sep 2019

Pull Request for Issue # .

Summary of Changes

Currently "Edit Category" string used in articles/featured view is glued out of two separate strings. This is not translatable in some languages.

Testing Instructions

"Edit Category" text still appears when hovering over category in articles/featured view in backend.

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 30 Sep 2019
avatar SharkyKZ SharkyKZ - change - 30 Sep 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 Sep 2019
Category Administration com_content Language & Strings
avatar Quy Quy - test_item - 30 Sep 2019 - Tested successfully
avatar Quy
Quy - comment - 30 Sep 2019

I have tested this item successfully on d467522


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

avatar alikon alikon - test_item - 1 Oct 2019 - Tested successfully
avatar alikon
alikon - comment - 1 Oct 2019

I have tested this item successfully on d467522


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

avatar alikon alikon - change - 1 Oct 2019
Status Pending Ready to Commit
avatar alikon
alikon - comment - 1 Oct 2019

RTC


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

avatar brianteeman
brianteeman - comment - 1 Oct 2019

I am sure there will be other strings constructed this way

avatar bembelimen
bembelimen - comment - 2 Oct 2019

I am sure there will be other strings constructed this way

Shouldn't we then go a way like:
JEDIT_ITEM="Edit %s"

avatar infograf768
infograf768 - comment - 2 Oct 2019

Shouldn't we then go a way like:
JEDIT_ITEM="Edit %s"

I’m afraid it would not be a good idea, as the semantics depend on the language.

avatar bembelimen
bembelimen - comment - 2 Oct 2019

Shouldn't we then go a way like:
JEDIT_ITEM="Edit %s"

I’m afraid it would not be a good idea, as the semantics depend on the language.

Yeah, that's why the placeholder %s. So English would be Edit %s and German would be
%s bearbeiten

avatar SharkyKZ
SharkyKZ - comment - 2 Oct 2019

In some languages the placeholder may be different depending on its use in a sentence. This, for example, would not work in some languages:

Text::sprintf('JEDIT_ITEM', Text::_('JCATEGORY'))

avatar infograf768
infograf768 - comment - 2 Oct 2019

The point is not the placeholder variable here, but the verb “edit” which may need to be written differently depending on the variable. (This PR is fine.)

avatar uthorat uthorat - test_item - 19 Oct 2019 - Tested successfully
avatar uthorat
uthorat - comment - 19 Oct 2019

I have tested this item successfully on d467522


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

avatar snehaM26 snehaM26 - test_item - 19 Oct 2019 - Tested successfully
avatar snehaM26
snehaM26 - comment - 19 Oct 2019

I have tested this item successfully on d467522

Tested successfully


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

avatar ttplpoojak ttplpoojak - test_item - 19 Oct 2019 - Tested successfully
avatar ttplpoojak
ttplpoojak - comment - 19 Oct 2019

I have tested this item successfully on d467522


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

avatar wilsonge wilsonge - change - 19 Oct 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-10-19 12:55:33
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 19 Oct 2019
avatar wilsonge wilsonge - merge - 19 Oct 2019
avatar wilsonge
wilsonge - comment - 19 Oct 2019

Thanks guys!

Add a Comment

Login with GitHub to post a comment