I believe solrsort is broken in beta11. Links like /search/site/test?solrsort=is_read_count desc no longer result in the sort being passed to Solr via $params and $queryString. This appears to be due to the refactoring of apachesolr_search_search_execute to apachesolr_search_search results and in particular the change from:

  $solrsort = isset($_GET['solrsort']) ? $_GET['solrsort'] : '';

to

  $solrsort = isset($search_page->settings['apachesolr_search_sort']) ? $search_page->settings['apachesolr_search_sort'] : '';

I may be missing something, but I can definitely report that the default sort links no longer work.

CommentFileSizeAuthor
#1 1347092.patch767 bytesnick_vh

Comments

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
StatusFileSize
new 1347092.patch767 bytes

The change was indeed a mistake but not as you describe it. Attached patch solves this. Can you confirm?
This looks like it got lost during a cleanup.

In apachesolr_search_conditions_default()

// Add the sort from the page to our conditions
  $sort = isset($_GET['solrsort']) ? $_GET['solrsort'] : '';
  $conditions['apachesolr_search_sort'] = $sort;
nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Active » Fixed

Thanks for reporting! Committed it :-)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.