Contrib.social

Resolve @todo in SearchApiSort::query()

Open on Drupal.org →
Created on 23 May 2024, about 1 month ago
Updated 8 June 2024, 20 days ago

I recently had the dubious pleasure of debugging Views exposed sorting and can now confirm that the code in \Drupal\search_api\Plugin\views\sort\SearchApiSort::query() is indeed required. Furthermore, we should let \Drupal\search_api\Plugin\views\query\SearchApiQuery::$orderby default to NULL, not [], to make the code work exactly as intended.
It also seems like we can remove the reference to the (ten years old) #2145547: Exposed sort order doesn't work when combined with non-exposed sort on the same field → .

📌 Task
Status

Fixed

Version

1.0

Component

Views integration

Created by

🇦🇹Austria drunken monkey Vienna, Austria

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

contrib.social Blog FAQ Discussions
Production build 0.69.0 2024