Problem/Motivation

Drush 9 now uses yml and classes for its commands, and has dropped support for legacy drush.inc files.

Proposed resolution

The commands used by flood_unblock will need to be ported to the new syntax. The scaffolding can be generated by Drush, but the commands themselves will require some manual conversion (particularly drush_print() and drush_choice()

CommentFileSizeAuthor
#6 flood_unblock-2973091-6.patch2.19 KBsittard
#5 flood_unblock-2973091-5.patch2.12 KBndobromirov

Comments

Asacolips created an issue. See original summary.

fabianderijk’s picture

fabianderijk
him/his
Primary language Dutch
Location Alphen aan den Rijn
commented

True, I will start porting to Drush 9!

ndobromirov’s picture

ndobromirov commented
Priority: Normal » Major

Any progress on this one, as it's kind of critical for the module.
Once your IP was flooded, drush is the only way to do it unless you want to go manually in DB...

ndobromirov’s picture

ndobromirov commented
Category: Task » Bug report
Issue tags: +Drush-9

As per Drush 8 being unusable on sported Drupal 8 versions I am marking this now as a bug, as this functionality that was present is now missing.

ndobromirov’s picture

ndobromirov commented
Status: Active » Needs review
StatusFileSize
new flood_unblock-2973091-5.patch2.12 KB

Here is a patch that adds the simplest possible Drush 9 bindings...

Available commands are: 

drush flood_unblock:ip
drush flood_unblock:user
drush flood_unblock:all
sittard’s picture

sittard commented
StatusFileSize
new flood_unblock-2973091-6.patch2.19 KB

Updated patch to include drush command to clear flood when user has been blocked on REST request. Compatibility with https://www.drupal.org/project/rest_password

Please see related issue:
https://www.drupal.org/project/flood_unblock/issues/2983522

  • sittard authored 772e578 on 8.x-1.x 
    Issue #2973091 by ndobromirov, sittard: Add Support for Drush 9 Commands
fabianderijk’s picture

fabianderijk
him/his
Primary language Dutch
Location Alphen aan den Rijn
commented

The patch has been applied

fabianderijk’s picture

fabianderijk
him/his
Primary language Dutch
Location Alphen aan den Rijn
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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