#52: Automatically set the label separator to the list separator when csv, etc. is selected with @sites (or other multi-site commands)

Author: greg-1-anderson

commands/core/outputformat.drush.inc

@@ -379,6 +379,15 @@ class drush_outputformat {
     $supported = $this->supported_data_types($metadata);
     return (count($supported) == 1) && ($supported[0] == 'format-single');
   }
+  function get_info($key) {
+    if (array_key_exists($key, $this->engine_config)) {
+      return $this->engine_config[$key];
+    }
+    elseif (isset($this->sub_engine)) {
+      return $this->sub_engine->get_info($key);
+    }
+    return FALSE;
+  }
   /**
    * Perform pre-processing and then format() the $input.
    */

includes/drush.inc

@@ -1189,6 +1189,29 @@ function drush_preflight_command_dispatch() {
     if (drush_get_option('no-label', FALSE)) {
       $backend_options['no-label'] = TRUE;
     }
+    // If the user specified a format, try to look up the
+    // default list separator for the specified format.
+    // If the user did not specify a different label separator,
+    // then pass in the default as an option, so that the
+    // separtor between the items in the list and the site
+    // name will be consistant.
+    $format = drush_get_option('format', FALSE);
+    if ($format) {
+      $implemented = drush_get_commands();
+      if (array_key_exists($command_name, $implemented)) {
+        $command = $implemented[$command_name];
+        if ($command) {
+          $result = drush_load_command_engines($command);
+          $formatter = drush_get_outputformat();
+          if (!array_key_exists('label-separator', $multi_options)) {
+            $list_separator = $formatter->get_info('list-separator');
+            if ($list_separator) {
+              $multi_options['label-separator'] = $list_separator;
+            }
+          }
+        }
+      }
+    }
     $values = drush_invoke_process($site_record, $command_name, $args, $multi_options, $backend_options);
     return TRUE;
   }