Add drush commands to get, set, del environment variables [#1673086]

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.

Comment	File	Size	Author
#2	1673086-2.patch	14.33 KB	pwolanin
#2
#1	1673086-1.patch	10.25 KB	pwolanin
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

pwolanin CreditAttribution: pwolanin commented 5 July 2012 at 21:54

Status:

Active

» Needs review

File	Size
1673086-1.patch	10.25 KB

Comment #2

pwolanin CreditAttribution: pwolanin commented 9 July 2012 at 13:53

File	Size
1673086-2.patch	14.33 KB

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

Comment #3

Nick_vh

he/him

Ghent

CreditAttribution: Nick_vh commented 9 July 2012 at 16:01

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

Comment #4

Nick_vh

he/him

Ghent

CreditAttribution: Nick_vh commented 9 July 2012 at 16:10

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

Comment #5

pwolanin CreditAttribution: pwolanin commented 10 July 2012 at 02:18

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.