? Language Change ? Maintainers Checked Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
19 Apr 2022

Articles that are in a category which is unpublished or trashed are not visible on the site but there is no way you can tell this from the article manager

Summary of Changes

  1. Adds an indicator if the category is unpublished or trashed.
  2. Changes the published icon to a disabled icon and adds a tooltip to the icon

Testing Instructions

Create a catgory called test and create 4 featured articles in that category.
Unpublish or Trash the category

Check the list of articles and the list of featured articles

Actual result BEFORE applying this Pull Request

No indication that the article can not be displayed

Expected result AFTER applying this Pull Request

Text as shown in the image below

image

Documentation Changes Required

avatar brianteeman brianteeman - open - 19 Apr 2022
avatar brianteeman brianteeman - change - 19 Apr 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 19 Apr 2022
Category Administration com_content
avatar rdeutz
rdeutz - comment - 20 Apr 2022

Looks like a good idea, I think this is something we should add and can be added to the next minor version.

avatar rdeutz
rdeutz - comment - 20 Apr 2022

one addition: maybe better to solve this in the PublishedButton class. Then you only have to fix it one time and not in all views. Could be something like "Published but category unpublished or trashed"

avatar brianteeman
brianteeman - comment - 20 Apr 2022

@rdeutz That was my first thought but then it gets complicated by workflows

avatar brianteeman
brianteeman - comment - 20 Apr 2022

@rdeutz took another look but I dont see how to do it in the class as it is not aware of the category

avatar brianteeman
brianteeman - comment - 20 Apr 2022

@rdeutz do you have any pointers as to how to solve it in the PublishedButton. If not I will continue with this approach.

avatar rdeutz
rdeutz - comment - 22 Apr 2022

I looked at the class and (beside Workflow, I don't see how this effects the buttom but maybe I missed something) it should be doable. I will check over the weekend and let you know.

avatar brianteeman
brianteeman - comment - 22 Apr 2022

@rdeutz thanks

avatar rdeutz
rdeutz - comment - 22 Apr 2022

This should do the trick: rdeutz@847c041

avatar brianteeman brianteeman - change - 22 Apr 2022
The description was changed
avatar brianteeman brianteeman - edited - 22 Apr 2022
avatar brianteeman brianteeman - change - 22 Apr 2022
Labels Added: ? Maintainers Checked
avatar joomla-cms-bot joomla-cms-bot - change - 22 Apr 2022
Category Administration com_content Administration com_content Language & Strings Libraries
avatar brianteeman brianteeman - change - 22 Apr 2022
Labels Added: Language Change
avatar brianteeman brianteeman - change - 22 Apr 2022
The description was changed
avatar brianteeman brianteeman - edited - 22 Apr 2022
avatar brianteeman
brianteeman - comment - 22 Apr 2022

one addition: maybe better to solve this in the PublishedButton class. Then you only have to fix it one time and not in all views. Could be something like "Published but category unpublished or trashed"

as even your code requires a chang in the view I decided to combin the two

avatar BertaOctech
BertaOctech - comment - 22 Apr 2022

I have tested this item successfully on d3bd885

The patch workes exactly as described


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

avatar BertaOctech BertaOctech - test_item - 22 Apr 2022 - Tested successfully
avatar pabloarias
pabloarias - comment - 22 Apr 2022

I have tested this item successfully on d3bd885

Tested in Joomla 4.1.3-dev with PHP 8.1.

Expected behaviour when category is unpublished or trashed.

It works in Featured Articles list too.


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

avatar pabloarias pabloarias - test_item - 22 Apr 2022 - Tested successfully
avatar chmst chmst - change - 22 Apr 2022
Status Pending Ready to Commit
avatar chmst
chmst - comment - 22 Apr 2022

RTC


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

avatar bembelimen
bembelimen - comment - 23 Apr 2022

I really like this PR, but it will not make it in 4.1, but for sure in 4.2!

Thanks for the improvement.

avatar bembelimen bembelimen - change - 23 Apr 2022
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 23 Apr 2022
Category Administration com_content Language & Strings Libraries Unit Tests Administration com_cache com_config com_contact com_content com_finder com_languages com_media com_menus com_redirect com_templates com_users Language & Strings Modules
avatar bembelimen bembelimen - change - 23 Apr 2022
Title
[4.1] Indicate if category not published
[4.2] Indicate if category not published
avatar bembelimen bembelimen - edited - 23 Apr 2022
avatar bembelimen
bembelimen - comment - 23 Apr 2022

JFYI: the "too much files" will be fixed on Tuesday, then there will be the next upmerge from 4.1 (which is already in your branch) after the relese of J! 4.2 Alpha 2. Then we're back to normal (no additional human tests needed).

avatar brianteeman
brianteeman - comment - 23 Apr 2022

No problem

avatar roland-d roland-d - change - 26 Apr 2022
Labels Added: ? ?
Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 26 Apr 2022
Category Administration com_content Language & Strings Unit Tests com_cache com_config com_contact com_finder com_languages com_media com_menus com_redirect com_templates com_users Modules Administration com_content Language & Strings Libraries
avatar roland-d roland-d - change - 26 Apr 2022
Labels Removed: ?
avatar roland-d roland-d - change - 26 Apr 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-04-26 19:16:32
Closed_By roland-d
avatar roland-d roland-d - close - 26 Apr 2022
avatar roland-d roland-d - merge - 26 Apr 2022
avatar roland-d
roland-d - comment - 26 Apr 2022

Thanks everybody

Add a Comment

Login with GitHub to post a comment