Closed (fixed)
Project:
Apache Solr Panels
Version:
7.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
24 Jan 2012 at 21:39 UTC
Updated:
22 Mar 2012 at 01:53 UTC
In the latest version of apachesolr, the apachesolr_current_query function must specify the environment to get the query from otherwise it returns NULL.
This patch is not backwards compatible and will require user's to upgrade their version of apachesolr if committed.
| Comment | File | Size | Author |
|---|---|---|---|
| apachesolr_panels-apachesolr-beta15-support.patch | 1.78 KB | josh waihi |
Comments
Comment #1
Silicon.Valet commentedThe apachesolr_has_searched() has to use the environment ID as a first argument as well, might add that to the patch ~line 43 in apachesolr_spellchecker.inc.
I went ahead and replaced the variable_get with an assignment to a local var at the top of the function for legibility eg.
Comment #2
davidneedhamThis patch works for me!
For reference, here are the errors I used to be getting that this patch fixes:
The Apache Solr search engine is not available. Please contact your site administrator.Warning: Missing argument 1 for apachesolr_current_query(), called in ...sites/all/modules/apachesolr_panels/content_types/apachesolr_info.inc on line 32 and defined in apachesolr_current_query() (line 1479 of ...sites/all/modules/apachesolr/apachesolr.module).Notice: Undefined variable: env_id in apachesolr_current_query() (line 1484 of ...sites/all/modules/apachesolr/apachesolr.module).Comment #3
acbramley commentedCommited to 7.x-1.x branch:
Initial patch by Josh Commit 8b551e5
Patch #1 by Silicon.Valet Commit 77aae70