No Code Attached Yet
avatar HLeithner
HLeithner
20 Dec 2023

Steps to reproduce the issue

if ($this->state->get('filter.published') == -2 && $canDo->get('core.delete')) {

only checks for filter.published but since we support (or should replace not sure about the bigger picture) filter.state we also should support it in the ListView.

avatar HLeithner HLeithner - open - 20 Dec 2023
avatar joomla-cms-bot joomla-cms-bot - change - 20 Dec 2023
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 20 Dec 2023
avatar HLeithner
HLeithner - comment - 20 Dec 2023

the reason why they work is because non of the joomla core htmlview classes extends from the ListHtmlView class, they have all a copy of this. The class has been introduced but no one had the time to move our own classes to the new utility class.

avatar brianteeman
brianteeman - comment - 20 Dec 2023

This is a perfect example then of where not using the issue template and/or not giving a detailed post causes confusion and wastes time.

If nothing else the title of the issue is factually incorrect

avatar Quy
Quy - comment - 20 Dec 2023

PR #38797

avatar HLeithner HLeithner - change - 20 Dec 2023
Status New Closed
Closed_Date 0000-00-00 00:00:00 2023-12-20 20:18:19
Closed_By HLeithner
avatar HLeithner HLeithner - close - 20 Dec 2023
avatar HLeithner
HLeithner - comment - 20 Dec 2023

yeah sadly it's not funny, your wording remembers me why I don't like to go to the tracker anymore and reduce my engagement. Second which is also not funny is that not many people in joomla care about developers.

thanks @Quy finding my pr...

Add a Comment

Login with GitHub to post a comment