Should be able to:

  • drush stats-ls to list all available fields. Perhaps output as pipe-able data, also perhaps use flags to list only enabled/disabled.
  • drush stats-en to enable a particular field. Like a module enable, there should be a confirmation dialogue.
  • drush stats-dis to remove a particular field. Like a module disable, there should be a confirmation dialogue and data loss warning.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

iamEAP’s picture

Status: Active » Needs work
FileSize
7.07 KB

Field list functionality is about 90% there. Still an issue with the flags.

Field enable/disable functionality is 50% there. Confirmation dialogue and error reporting are pretty much there, but work needs to be done to abstract the field enable/disable out of the form submit in better_statistics.admin.inc so that drush can make use of them easily.

iamEAP’s picture

Status: Needs work » Needs review
FileSize
22.54 KB

Here's an initial stab at the abstraction. Curious to see what TestBot thinks.

Status: Needs review » Needs work

The last submitted patch, better_statistics-add_drush_support-1800648-2.patch, failed testing.

iamEAP’s picture

Status: Needs work » Needs review
FileSize
16.1 KB

Previous patch was based on an older version of the module. Restored the deleted portions and removed the remaining @todos.

iamEAP’s picture

Status: Needs review » Fixed

Testbot approves; committed in lieu of interested folks for review.

c8cd850

Status: Fixed » Closed (fixed)

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