Solr_Base_Query is constructed with a $sortstring initialized from $_GET['solrsort'] (ie. without the solrsort= prefix), while get_url_querystring() expects Solr_Base_Query::solrsort to contain the prefix.

CommentFileSizeAuthor
#3 366959-solrsort-inconsistencies.patch1.86 KBdamien tournoud
#2 366959-solrsort-inconsistencies.patch1.86 KBdamien tournoud
#1 366959-solrsort-inconsistencies.patch1.86 KBdamien tournoud

Comments

damien tournoud’s picture

damien tournoud commented
Status: Active » Needs review
StatusFileSize
new 366959-solrsort-inconsistencies.patch1.86 KB

Here is a patch.

damien tournoud’s picture

damien tournoud commented
StatusFileSize
new 366959-solrsort-inconsistencies.patch1.86 KB

And a non broken patch.

damien tournoud’s picture

damien tournoud commented
StatusFileSize
new 366959-solrsort-inconsistencies.patch1.86 KB

Uploaded the same patch by mistake.

pwolanin’s picture

pwolanin commented

need to test, but consistency++

dww’s picture

dww
we/he/they
commented
Issue tags: +drupal.org upgrade

This patch is being used to let project_solr.module work on d6.d.o. So, getting this into the upstream apachesolr.module before we really go live would be a big help.

pwolanin’s picture

pwolanin commented
Status: Needs review » Fixed

committed to 6.x

Status: Fixed » Closed (fixed)
Issue tags: -drupal.org upgrade

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

Issue tags: +drupal.org upgrade

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.