Download & Extend
Apache Solr Search Integration » Issues

Add drush commands to get, set, del environment variables

Posted by pwolanin on July 5, 2012 at 9:52pm
Project:Apache Solr Search Integration
Version:7.x-1.x-dev
Component:Miscellaneous
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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.

Login or register to post comments

Comments

#1

Posted by pwolanin on July 5, 2012 at 9:54pm
Status:active» needs review
AttachmentSizeStatusTest resultOperations
1673086-1.patch10.25 KBIdlePASSED: [[SimpleTest]]: [MySQL] 501 pass(es).View details

#2

Posted by pwolanin on July 9, 2012 at 1:53pm

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

AttachmentSizeStatusTest resultOperations
1673086-2.patch14.33 KBIdlePASSED: [[SimpleTest]]: [MySQL] 501 pass(es).View details

#3

Posted by Nick_vh on July 9, 2012 at 4:01pm
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

#4

Posted by Nick_vh on July 9, 2012 at 4:10pm
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

#5

Posted by pwolanin on July 10, 2012 at 2:18am
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.

#6

Posted by System Message on July 24, 2012 at 2:21am
Status:fixed» closed (fixed)

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