Like this.

CommentFileSizeAuthor
#3 views_analyze_drush.patch1.45 KBchx
#1 views_analyze.patch1.45 KBchx
views_analyze.patch1.47 KBchx

Comments

chx’s picture

chx commented
StatusFileSize
new views_analyze.patch1.45 KB

Removed a leftover $printed variable.

moshe weitzman’s picture

moshe weitzman commented

At least the last summary line should use drush_log() with a type of 'ok'. That works better when run on remote sites. A bit more programmable. Not a big deal.

chx’s picture

chx commented
StatusFileSize
new views_analyze_drush.patch1.45 KB

Certainly.

drewish’s picture

drewish commented

You can swap:

+        drush_print('  ' . $message['type'] .': '. $message['message']);

for

+        drush_print($message['type'] .': '. $message['message'], 2);

the 2nd parameter is indent level.

dawehner’s picture

dawehner
Primary language German
commented
Version: 6.x-3.x-dev » 7.x-3.x-dev
Status: Needs review » Fixed

Commited with this tiny change to 6.x-3.x and 7.x-3.x

Status: Fixed » Closed (fixed)

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