? ? Pending

User tests: Successful: Unsuccessful:

avatar phproberto
phproberto
24 Jan 2017

Pull Request for Issue #12038 .

Summary of Changes

#10304 introduced a B/C issue with extensions that use core tag system without using core categories system. Making an inner join like:

->join('INNER', '#__categories AS tc ON tc.id = c.core_catid AND tc.published = 1')

That causes that 3rd part extensions are broken and tags used on them aren't shown in core tag views because c.core_catid is always 0.

Patch changes the query to use something like:

LEFT JOIN boy8n_categories AS tc ON tc.id = c.core_catid
WHERE (c.core_catid = 0 OR tc.published = 1)

So that it only returns tags assigned to content not using the category system or using it and having the category published.

Testing Instructions

  1. Create a content category
  2. Create a content item and assign it to category created in step 1. Also assign a sample tag on it.
  3. Create a menu item of type Tags > Compact list of tagged items and in the Tag menu setting select the tag created in step 2.
  4. In frontend go to menu item created in step 3 and check that you see the content item.
  5. Unpublish category created in step 2.
  6. In frontend go to menu item created in step 3 and check that you DON'T see the content item.

Additional tests if you have a custom extension experiencing the issue ( see #12038 ) also test this:

  1. Publish category created in step 2.
  2. Create an item in your extension and assign the same tag assigned to the content item in step 2.
  3. In frontend go to menu item created in step 3 and check that you see your extension item + core content item.
  4. Unpublish category created in step 2.
  5. In frontend go to menu item created in step 3 and check that you see your extension item but you DON'T see core content item.

Documentation Changes Required

avatar phproberto phproberto - open - 24 Jan 2017
avatar phproberto phproberto - change - 24 Jan 2017
Status New Pending
avatar phproberto phproberto - edited - 24 Jan 2017
avatar joomla-cms-bot joomla-cms-bot - change - 24 Jan 2017
Category Libraries
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 24 Jan 2017

I have tested this item successfully on d9cb627

Menu Itam Type Taga > Compact list of tagged items: With or without PR tagged Content isn't shown if Category is unpublished.


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 24 Jan 2017 - Tested successfully
avatar alikon
alikon - comment - 24 Jan 2017

I have tested this item successfully on d9cb627

without 3dp extensions


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

avatar alikon alikon - test_item - 24 Jan 2017 - Tested successfully
avatar jameswadsworth
jameswadsworth - comment - 25 Jan 2017

I have tested this item successfully on joomla 3.6.5 against our 3rd party extension. If we remove our hack to set core_catid to 1 in #__ucm_content and let is default to 0 upon save with this patch by @phproberto all items show in the tags list view.

avatar jeckodevelopment jeckodevelopment - change - 26 Jan 2017
The description was changed
avatar jeckodevelopment jeckodevelopment - edited - 26 Jan 2017
avatar jeckodevelopment jeckodevelopment - change - 26 Jan 2017
Status Pending Ready to Commit
avatar jeckodevelopment
jeckodevelopment - comment - 26 Jan 2017

RTC


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

avatar rdeutz rdeutz - close - 27 Jan 2017
avatar rdeutz rdeutz - merge - 27 Jan 2017
avatar rdeutz rdeutz - change - 27 Jan 2017
The description was changed
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-01-27 18:27:12
Closed_By rdeutz
Labels Added: ? ?

Add a Comment

Login with GitHub to post a comment