- 🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
Wonder if this is still an issue for D10
Tried replicating in D10 on a standard install
Editing the content view
Add sort for published (ASC) (unexposed)
Add sort for title (exposed)
Add sort for changed date (Exposed)The sorting appears to be working as expected?
Did I miss a step?
- Status changed to Needs review
12 months ago 7:03pm 12 January 2024 - 🇺🇸United States caesius
I was able to replicate the issue on a vanilla Drupal 10 install. See screenshots.
Unexposed DESC sort followed by exposed sort; note that the SQL shows "ORDER BY DESC" for both sorts:
However, the default sort for the exposed filter is actually ASC:
- Status changed to Needs work
12 months ago 7:31pm 12 January 2024 - 🇺🇸United States smustgrave
Issue summary is incomplete.
testExposedFilterSortOrder should have return type.
Recommend using MR vs patches as patches are phased out.
- Assigned to ravimane23
- 🇮🇳India ravimane23
ravimane23 → changed the visibility of the branch 9.2.x to hidden.
- Merge request !9205Issue #3120847: Non-exposed and exposed sorts cause issues with the default sort order → (Open) created by ravimane23
- Issue was unassigned.
- Status changed to Needs review
4 months ago 10:02am 14 August 2024 - Status changed to Needs work
4 months ago 11:11am 14 August 2024 - 🇺🇸United States smustgrave
Test coverage appears to have been lost and issue summary update is still needed