User tests: Successful: Unsuccessful:
Pull Request for Issue #46371 & #46334 .
This reverts PR #37140 as due update of choices #46133 the value for single space is now handle differently.
In J5.4 the parameter featured_categories is saved as string(0) "" and in J6.0 the parameter is saved as string(1) " "
Here is the relevant code for set the filter filter.frontpage.categories
J5.4: var_dump((bool) ''); -> filter not set when "All Categories" selected
bool(false)
J6.0: var_dump((bool) ' '); -> filter is set when "All Categories" selected
bool(true)
joomla-cms/components/com_content/src/Model/FeaturedModel.php
Lines 95 to 99 in 2afb305
create menu item Featured Articles and set filter Select Categories to All Categories
visit menuitem in frontend
see also issue #46371 for more information
additional check if issue #33556 still solved
no featured articles are displayed
featured articles are displayed
Please select:
| Status | New | ⇒ | Pending |
| Category | ⇒ | Front End com_content |
This change will not fix existing menu items
Do you mean we need an update sql here?
Info: This only affects menu entries that were newly created or re-saved under J6.0
yes it will need an update sql - otherwise you have to delete the settings and reselect "all categories" and save
| Labels |
Added:
PR-6.0-dev
|
||
| Category | Front End com_content | ⇒ | SQL Administration com_admin Front End com_content |
@richard67 Can you advise me on whether I'm on the right track with the SQL update and offer some support with PostgreSQL?
@richard67 Can you advise me on whether I'm on the right track with the SQL update and offer some support with PostgreSQL?
@heelc29 On quick view the MySQL files doe not look wrong, but I won't have time before weekend to check deeper and do something for PostgreSQL.
| Category | Front End com_content SQL Administration com_admin | ⇒ | Repository SQL Administration com_admin Postgresql Front End com_content JavaScript Unit Tests |
| Labels |
Added:
Unit/System Tests
|
||
| Category | Front End com_content SQL Administration com_admin Repository Postgresql JavaScript Unit Tests | ⇒ | SQL Administration com_admin Postgresql Front End com_content |
I have tested this item ✅ successfully on 3bba340
I have tested this item ✅ successfully on 3bba340
Tested with mysql update, not postgresql
@alikon or @brianteeman : Could you test this one with PostgreSQL with an update from 5.4 using the update package created by Drone (or by applying the update SQL script)? That would be really appreciated.
@ceford @chmst Have you only tested with a new installation? Or have you also tested updating with the update package created by Drone or with applying the update SQL script in phpMyAdmin?
I tested with a joomla-cms clone with the Testing Sample Data installed. So No! I did not test running the updated SQL scripts.
@alikon or @brianteeman : Could you test this one with PostgreSQL with an update from 5.4 using the update package created by Drone (or by applying the update SQL script)? That would be really appreciated.
@alikon or @brianteeman Sorry, I just see it needs to test that with an update from 6.0.0, not from 5.4.
| Labels |
Removed:
Unit/System Tests
|
||
I have tested this item ✅ successfully on 42fd986
I've successfully verified the issue on a clean, current 6.0-dev installation with the "Featured Articles" menu item, and I've also tested the "Archived Articles" menu item, and I've updated to the patched package for this PR with live update using the custom update URL created by drone.
After the update the issue was fixed.
I've tested that with MySQL and PostgreSQL, so we can be sure both update SQL scripts are working.
| Status | Pending | ⇒ | Ready to Commit |
RTC
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-11-11 12:55:42 |
| Closed_By | ⇒ | Bodge-IT | |
| Labels |
Added:
RTC
bug
|
||
Thanks for the fix @Heelc, good job! Thanks to testers @richard67 & @ceford @chmst for your diligience
Thanks for the fix @heelc29 , good job! Thanks to testers @richard67 & @ceford @chmst for your diligience
This change will not fix existing menu items