? Language Change PR-4.3-dev Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
23 Aug 2022

You should be able to create tags with parents and children such as

  • Sport
    -- adults
    -- children
  • games
    -- adults
    -- children

Before this PR you could not as you would get a duplicate alias message error even though the tags have different parents and there is no reason that I can see to prevent this I may be very wrong as I dont use tags that often

Pull Request for Issue #21326 .

avatar brianteeman brianteeman - open - 23 Aug 2022
avatar brianteeman brianteeman - change - 23 Aug 2022
Status New Pending
avatar brianteeman brianteeman - change - 23 Aug 2022
The description was changed
avatar brianteeman brianteeman - edited - 23 Aug 2022
avatar joomla-cms-bot joomla-cms-bot - change - 23 Aug 2022
Category com_tags Administration Language & Strings Libraries
avatar brianteeman brianteeman - change - 23 Aug 2022
Labels Added: Language Change ?
46f47fb 23 Aug 2022 avatar brianteeman cs
avatar Franzwohlkoenig
Franzwohlkoenig - comment - 27 Aug 2022

I have tested this item successfully on 46f47fb


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

avatar Franzwohlkoenig Franzwohlkoenig - test_item - 27 Aug 2022 - Tested successfully
avatar jwaisner
jwaisner - comment - 27 Aug 2022

I have tested this item successfully on 46f47fb


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

avatar jwaisner jwaisner - test_item - 27 Aug 2022 - Tested successfully
avatar jwaisner jwaisner - change - 27 Aug 2022
Status Pending Ready to Commit
avatar jwaisner
jwaisner - comment - 27 Aug 2022

RTC


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

avatar fancyFranci
fancyFranci - comment - 6 Sep 2022

To be sure that there are no side effects we do not see, please move it to 4.3. Especially with your new tag feature this fix will have the chance to receive deeper tests.

avatar brianteeman
brianteeman - comment - 6 Sep 2022

please move it to 4.3.

You already did ;(

avatar obuisard obuisard - change - 7 Sep 2022
Labels Added: ? PR-4.3-dev
Removed: ?
avatar obuisard obuisard - change - 7 Sep 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-09-07 19:47:53
Closed_By obuisard
avatar obuisard obuisard - close - 7 Sep 2022
avatar obuisard obuisard - merge - 7 Sep 2022
avatar obuisard
obuisard - comment - 7 Sep 2022

Thank you Brian @brianteeman

avatar Hackwar
Hackwar - comment - 3 Nov 2022

Please be aware that this change means that routing is going to be broken for the duplicate aliases. The routing currently (and with my rewrite in #39114) simply takes the alias of the tag to identify the tag to load. I would propose to add a switch to either allow single level tags or to use nestable tags and change the routing in that case.

Add a Comment

Login with GitHub to post a comment