-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[4.0] re-do temporary change of filter published-condition #31155
[4.0] re-do temporary change of filter published-condition #31155
Conversation
I can not replicate this. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31155. |
I have tested this item ✅ successfully on f97121c The code is a refactoring of the filter selector, so no change to what appears is expected for the test case. Tried various ordering options for the module, all were successful. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31155. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31155. |
@alikon RTC with only 1 test? Or have you tested, too, and just forgotten to submit the test result? |
I have tested this item ✅ successfully on 9702057 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31155. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31155. |
Previous test by @particthistle was still valid because the commit after that test was just a clean branch update. I've restored the test result in the issue tracker so it's correctly counted there and here. |
Thanks! |
Summary of Changes
As required in #30467 the filter.condition is changed to filter.published.
Testing Instructions
As in #30445
In a module "Articles latest" only published items my be displayed.
Documentation Changes Required
no