It's a bad practice to include HTML in CLI output. It clutters results to the point of illegibility on occasion. However, since this same output ends up in our task logs, it can be very useful to use placeholders in dt() that will format strings for easier parsing.

Thankfully, Drush's logging capabilities provide us with a relatively simple way to get the best of both worlds, using a custom log callback.

Patch forthcoming.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ergonlogic created an issue. See original summary.

ergonlogic’s picture

Status: Active » Needs review
FileSize
1.21 KB

This appears to work nicely.

helmo’s picture

+1 for stripping this out.

However I'm still seeing html from my hosting-queued running with --debug after applying this patch.

<em class="placeholder">apache</em> on <em class="placeholder">opcvm01</em> has been restarted [9.43 sec, 16.17 MB]                                                                                                                                                                 [notice]
helmo’s picture

Status: Needs review » Needs work