User tests: Successful: Unsuccessful:
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
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
No indication that the article can not be displayed
Text as shown in the image below
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_content |
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"
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.
This should do the trick: rdeutz@847c041
Labels |
Added:
?
Maintainers Checked
|
Category | Administration com_content | ⇒ | Administration com_content Language & Strings Libraries |
Labels |
Added:
Language Change
|
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
I have tested this item
The patch workes exactly as described
I have tested this item
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.
Status | Pending | ⇒ | Ready to Commit |
RTC
I really like this PR, but it will not make it in 4.1, but for sure in 4.2!
Thanks for the improvement.
Labels |
Added:
?
|
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 |
Title |
|
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).
No problem
Labels |
Added:
?
?
Removed: ? |
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 |
Labels |
Removed:
?
|
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 |
Thanks everybody
Looks like a good idea, I think this is something we should add and can be added to the next minor version.