[#45106] - [5.3] cleared miss match between default option and label in articles filter
- Fixed in Code Base
- 20 Apr 2025
- Medium
- Build: 5.2-dev
- # 45106
- Diff
- Dileepadari:I_37841
User tests: Successful: Unsuccessful:
Pull Request for Issue #37841
Summary of Changes
The default label for the articles filter in the user login has been updated to align with the actual default option (ID Descending). Previously, the label was incorrectly set to TITLE Ascending, while the functionality still followed ID Descending. This fix ensures that the label correctly reflects the selected sorting order.
Testing Instructions
- Navigate to: your.domain/index.php?option=com_content&view=form&layout=edit
- Select Article as the Menu Item Type (default is CMS content).
- A modal window with articles will open.
- Check the default filter dropdown—it should now correctly display ID Descending as both the label and the actual sorting option.
Actual result BEFORE applying this Pull Request
- The default filter label was incorrectly set to TITLE Ascending, while the actual sorting used ID Descending.
Expected result AFTER applying this Pull Request
- Both the label and the selected option now correctly show ID Descending (ensuring that the latest articles appear first).
Link to Documentation
-
Documentation link for docs.joomla.org:
<link> -
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
<link> -
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End com_content |
| Title |
|
||||||
| Labels |
Added:
PR-5.2-dev
|
||
Thanks for pointing it out, I changed the permissions back to 644. Also please let me know should I use the same pr to clear the other two issues too or can we merge this?
I have tested this item ✅ successfully on 37cf38b
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45106.
as far as i know:
Merging a branch in your Pull Request invalidate test result (you see the result of @QuyTon is gone in the checks).
Better to wait for a second successfully test and set to "Ready to Commit" (RTC) by a maintainer. After setting RTC merging the branch doesn't harm test results.
I have tested this item ✅ successfully on 1bf3379
Test by using custom update server of prebuilt packages.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45106.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45106.
| Labels |
Added:
RTC
|
||
This pull request has been automatically rebased to 5.3-dev.
| Title |
|
||||||
| Labels |
Added:
bug
PR-5.3-dev
Removed: PR-5.2-dev |
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-04-20 11:32:36 |
| Closed_By | ⇒ | richard67 |
@Dileepadari Thank you very much and congratulations for your first merged pull request. We are always happy to see new contributors.
And thanks to testers.
@Dileepadari If you check the changed files of this PR on GitHub here, you will see that you also changed the file permissions from 644 to 755. Could you revert that?
