We have a system of apachesolr environment variables that are like core Drupal variables. We should add drush commands so they can be directly manipulated as needed.

CommentFileSizeAuthor
#2 1673086-2.patch14.33 KBpwolanin
#1 1673086-1.patch10.25 KBpwolanin

Comments

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
StatusFileSize
new 1673086-1.patch10.25 KB
pwolanin’s picture

pwolanin commented
StatusFileSize
new 1673086-2.patch14.33 KB

Better patch to apply consistency with an --id option for most commands.

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Needs review » Needs work 
+++ b/drush/apachesolr.drush.incundefined
@@ -168,55 +250,265 @@ function apachesolr_drush_solr_search() {
 }
+

this apparently adds a whitespace error. (Complete bottom of the file)

Testing out the rest as we speak

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Needs work » Reviewed & tested by the community

Looking good to me

Nick-Veenhofs-MacBook-Pro:apachesolr nickveenhof$ drush solr-vset acquia_search 1
Using environment ID: "solr"                                                                                                                                                                                                     [success]
acquia_search was set to 1                                                                                                                                                                                                       [success]
Nick-Veenhofs-MacBook-Pro:apachesolr nickveenhof$ drush solr-vset acquia_search 1 --id=acquia_search_server_1_0
Using environment ID: "acquia_search_server_1_0"                                                                                                                                                                                 [success]
acquia_search was set to 1                                                                                                                                                                                                       [success]
Nick-Veenhofs-MacBook-Pro:apachesolr nickveenhof$ drush solr-vset testing_search 1 --id=acquia_search_server_1_0
Using environment ID: "acquia_search_server_1_0"                                                                                                                                                                                 [success]
Enter a number to choose which variable to set.
 [0]  :  Cancel                        
 [1]  :  testing_search (new variable) 

1
testing_search was set to 1

No errors and explanation is good and correct

pwolanin’s picture

pwolanin commented
Status: Reviewed & tested by the community » Fixed

committed to 7.x. The patch didn't apply 100% cleanly to 6.x-3.x so I just copied the final version from 7.x-1.x to 6.x-3.x and committed. I'm not sure what was out of sync.

Need to verify 6.x-3.x.

Status: Fixed » Closed (fixed)

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