Skip to content

[4.0] Fix Tags Compact list bug#31093

Merged
drmenzelit merged 1 commit intojoomla:4.0-devfrom
infograf768:4.0_tags_compactlistbug
Oct 15, 2020
Merged

[4.0] Fix Tags Compact list bug#31093
drmenzelit merged 1 commit intojoomla:4.0-devfrom
infograf768:4.0_tags_compactlistbug

Conversation

@infograf768
Copy link
Member

@infograf768 infograf768 commented Oct 14, 2020

Summary of Changes

Users Notes Categories and Banners Categories can be tagged.
I guess because of the common code for category params.

As they do not have a frontend display layout (normal) they should not have a link to display them in the Tag lists (Compact and normal list).
The code is OK in the normal list

<?php if (($item->type_alias === 'com_users.category') || ($item->type_alias === 'com_banners.category')) : ?>
<h3>
<?php echo $this->escape($item->core_title); ?>
</h3>
<?php else : ?>
<h3>
<a href="<?php echo Route::_($item->link); ?>">
<?php echo $this->escape($item->core_title); ?>
</a>
</h3>

But was forgotten in the Compact list menu item layout, therefore getting a link for these Titles and evidently a 404 if clicked.
This PR corrects list_items to solve the bug.

Testing Instructions

Create a tag.
Tag a few items with that tag, including a banners category and a User Notes Category.
Create a Compact List of tagged items menu item.
Display in frontend.

Actual result BEFORE applying this Pull Request

Clickable titles

Screen Shot 2020-10-14 at 18 42 18

Expected result AFTER applying this Pull Request

No more clickable link

Screen Shot 2020-10-14 at 18 40 48

Note:

There are other issues with the queries in /libraries/src/Helper/TagsHelper.php. Will create an issue for this.

@ChristineWk
Copy link

ChristineWk commented Oct 14, 2020

I have tested this item ✅ successfully on ab3eadf

I don't usually use tags. That's why I'm always happy to receive your exact test instructions :-)


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

@ChristineWk
Copy link

screen shot 2020-10-14 at 17 31 35


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

@ghost
Copy link

ghost commented Oct 15, 2020

I have tested this item ✅ successfully on ab3eadf

thanks for clear test instructions.


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

@infograf768
Copy link
Member Author

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Oct 15, 2020
@infograf768 infograf768 added this to the Joomla 4.0 milestone Oct 15, 2020
@drmenzelit drmenzelit merged commit f865a3f into joomla:4.0-dev Oct 15, 2020
@joomla-cms-bot joomla-cms-bot added PR-4.0-dev and removed RTC This Pull Request is Ready To Commit labels Oct 15, 2020
@infograf768 infograf768 deleted the 4.0_tags_compactlistbug branch October 15, 2020 09:13
@drmenzelit
Copy link
Collaborator

Thanks

sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants