Feature Language Change PR-5.0-dev PR-5.1-dev Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
2 Nov 2022

Pull Request for Issue #6687 .

Summary of Changes

Tags in Joomla have so far partial support for multilanguage capabilities and there was partial, non-working code for associations. This PR introduces associations completely, cleans up the code around this and the multilanguage features are brought up to par with the rest of the Joomla core components.

  • When multilanguage is enabled, the views in the component now only show tags and content items for the current language and *.
  • The tag_list_language_filter option was removed, since it conflicts with how multilinguality works in all of the rest of Joomla. Now it automatically behaves like the rest when multilanguage is enabled.
  • The TagModel::save() method has been largely removed and replaced with the call to AdminModel::save(). The only thing that was kept was the rebuild of the nested set structure. The old code didn't have the code for associations included and I'm pretty sure some other stuff was missing here, too.
  • Tag associations aren't just one-on-one mappings, but there are cases where you have selected more than one tag per menu item. In that case, we have to find the association for each tag individually and then create a URL out of all those associated IDs combined. If in case of multiple tag IDs a tag doesn't have an associated tag, it will use the original tag to build the new URL. This assumes that such a tag will have * as language value.
  • The branch is based on the changes from #39114. Please keep that PR open as well and test them separately! These changes are pretty large and if we push this all into one giant PR, this will never get tested and merged. That would be very unfortunate, so please keep it the way it is. This PR already is gigantic.
  • The component currently does not contain the Associations toolbar button.

Testing Instructions

Please first test the general functionality of tags. This means to create a few tags in the backend both via the component and by the input field in for example articles. Assign those tags to content items, create menu items in the frontend for these tags or the the "all tags" menu item type. Surf those pages in the frontend and check if everything is displayed correctly. Also create menu items with more than one tag selected for it.

In the next step, enable multilanguage mode and mark several content items with different content languages. Mark at least one tag as from one content language, create a second tag with another content language and associate these 2 tags. Check in the frontend that a tag view only shows content matching the current language and if you click on a flag, it should redirect to the associated tag.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

I don't know what we have to change here...

avatar Hackwar Hackwar - open - 2 Nov 2022
avatar Hackwar Hackwar - change - 2 Nov 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Nov 2022
Category com_tags Administration Language & Strings Front End Libraries
avatar Hackwar Hackwar - change - 3 Nov 2022
Labels Added: Language Change PR-4.3-dev
avatar joomla-cms-bot joomla-cms-bot - change - 3 Nov 2022
Category com_tags Administration Language & Strings Front End Libraries com_tags Administration Language & Strings Front End com_content Libraries Modules
avatar Hackwar Hackwar - change - 3 Nov 2022
The description was changed
avatar Hackwar Hackwar - edited - 3 Nov 2022
avatar jamfx
jamfx - comment - 3 Nov 2022

I have tested this item 🔴 unsuccessfully on 0d7ed82

Ok, i tried the following and didn't succeed. The Test failed at least partly.
Steps I took.

  1. Joomla Default Installation with Multilanguage Sample Data
  2. Installed German and Greek languagepacks
  3. Opened Tags and Translated Millions to Millionen for Association.
  4. now set Millions to English and Millionen to German
  5. Opened "Millions"-Tag and clicked on Associations. Now I wanted to choose Millionen and typed in "Mill" but only the item IDs showed up. I could save the IDs successfully, but after reopening nothing was shown in Associations.

I stopped the test here.
Thanks for your work so far!


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

avatar jamfx jamfx - test_item - 3 Nov 2022 - Tested unsuccessfully
avatar Hackwar
Hackwar - comment - 9 Nov 2022

@jamfx I did some fixes. Please test this again.

avatar Hackwar Hackwar - change - 23 Mar 2023
Title
[4.3] Tags: Adding associations
[4.4] Tags: Adding associations
avatar Hackwar Hackwar - edited - 23 Mar 2023
avatar Hackwar Hackwar - change - 23 Mar 2023
Labels Added: PR-4.4-dev
avatar laoneo
laoneo - comment - 28 Mar 2023

Thank you for your contribution, unfortunately the 4.4-dev branch doesn't get new features. When #40181 is merged this pr should be rebased to 5.0-dev. Thanks for understanding.

avatar Hackwar Hackwar - change - 28 Mar 2023
Title
[4.4] Tags: Adding associations
[5.0] Tags: Adding associations
avatar Hackwar Hackwar - edited - 28 Mar 2023
avatar Hackwar Hackwar - change - 28 Mar 2023
Labels Removed: PR-4.3-dev
avatar Hackwar Hackwar - change - 28 Mar 2023
Labels Added: PR-5.0-dev
avatar Hackwar
Hackwar - comment - 28 Mar 2023

Since I had to rebase this from 4.3 to 4.4 and now to 5.0, I first have to check all the changes here if they are still correct. Switching back to draft for now.

avatar Hackwar Hackwar - change - 18 Apr 2023
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2023-04-18 12:06:34
Closed_By Hackwar
Labels Added: ?
Removed: PR-4.4-dev
avatar Hackwar Hackwar - close - 18 Apr 2023
avatar richard67
richard67 - comment - 18 Apr 2023

Why closed? Should we reopen the original issue?

avatar Hackwar Hackwar - change - 18 Apr 2023
Status Closed New
Closed_Date 2023-04-18 12:06:34
Closed_By Hackwar
avatar Hackwar Hackwar - change - 18 Apr 2023
Status New Pending
avatar Hackwar Hackwar - reopen - 18 Apr 2023
avatar HLeithner
HLeithner - comment - 30 Sep 2023

This pull request has been automatically rebased to 5.1-dev.

avatar Hackwar Hackwar - change - 4 Jan 2024
Labels Added: Feature
Removed: ?
avatar Hackwar Hackwar - change - 3 Apr 2024
Title
[5.0] Tags: Adding associations
[5.2] Tags: Adding associations
avatar Hackwar Hackwar - edited - 3 Apr 2024
avatar Hackwar Hackwar - change - 3 Apr 2024
Labels Added: PR-5.1-dev

Add a Comment

Login with GitHub to post a comment