The module Drush commands in the legacy drush.inc format are not compatible with the current Drush 9 format and need to be converted.

CommentFileSizeAuthor
#5 interdiff-2952220-2-4.txt1.1 KBfgm
#4 interdiff-2952220-2-4.txt1.1 KBfgm
#4 0001-Issue-2952220-by-fgm-convert-to-Drush-9.patch2.75 KBfgm
#2 0001-Issue-2952220-by-fgm-convert-to-Drush-9.patch2.75 KBfgm

Comments

fgm created an issue. See original summary.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Active » Needs review
StatusFileSize
new 0001-Issue-2952220-by-fgm-convert-to-Drush-9.patch2.75 KB

Suggested patch.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

LGTM.

If anyone rerolls this, I would use $this->logger()->notice() instead of writeln(). Its a trivial point.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
StatusFileSize
new 0001-Issue-2952220-by-fgm-convert-to-Drush-9.patch2.75 KB
new interdiff-2952220-2-4.txt1.1 KB

Rerolled accordingly.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
StatusFileSize
new interdiff-2952220-2-4.txt1.1 KB

Oops, bad interdiff.

  • 652b9f8 committed on 8.x-1.x 
    Issue #2952220 by fgm: Convert commands to Drush 9
yce’s picture

yce commented
Status: Reviewed & tested by the community » Fixed

Hi,

Thanks for the patch, I've applied it.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented

Thanks. Any hope for the #2842933: View error width field tags patch too ?

Status: Fixed » Closed (fixed)

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