admin/config/search/apachesolr/settings/acquia_search_server_1/edit

When I check "Make this Solr search environment multisite capable" and save and edit, the checkbox is not checked. However, the setting does get saved to the DB.

CommentFileSizeAuthor
#5 2236045-5.patch908 byteslokapujya
multsite_checkbox.patch788 byteslokapujya
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lokapujya’s picture

lokapujya
Primary language English
Location Boston
CreditAttribution: lokapujya commented
Issue summary: View changes
Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs review » Needs work

This doesn't work. The apachesolr_environment_variable_get() clearly expects an environment ID as the first parameter, not an environment object.

lokapujya’s picture

lokapujya
Primary language English
Location Boston
CreditAttribution: lokapujya commented

Sure. But, in my setup, the line before that:
$environment = reset($form_state['build_info']['args']);
gives me the environment ID, not an environment object.

lokapujya’s picture

lokapujya
Primary language English
Location Boston
CreditAttribution: lokapujya commented

Maybe the variable name should be changed to $env_id. Possibly, at some point $environment was an object. I'm using apachesolr version 1.6 and $environment is just the ID.

lokapujya’s picture

lokapujya
Primary language English
Location Boston
CreditAttribution: lokapujya commented
Assigned: lokapujya » Unassigned
Status: Needs work » Needs review
FileSize
2236045-5.patch908 bytes

Something like this.

shahinam’s picture

shahinam CreditAttribution: shahinam commented
Status: Needs review » Reviewed & tested by the community

#5 works very well.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Reviewed & tested by the community » Closed (works as designed)
Related issues: +#1827868: Improve usability of admin interface when using multiple environments

Actually I see what is the issue here. The patch in https://drupal.org/node/1827868#comment-7941799 was committed to 7.x-1.x which changes the menu arguments for the settings path and causes this regression. This was reverted prior to the 7.x-1.6 release of the Apache Solr module. You just need to update to the latest version (of the ApacheSolr module) and then this is resolved.

shahinam’s picture

shahinam CreditAttribution: shahinam commented

Yup, I upgraded to 7.x-1.6 and it fixes the problem.
Thanks!