Problem/Motivation

As discussed in #1845626: Provide UI to remove blocks from flood it would be a good idea to merge the Flood Unblock (https://www.drupal.org/project/flood_unblock) and Flood Control modules.

Proposed resolution

- Incorporate all the functionality of Flood Unblock in Flood Control.

Remaining tasks

- Incorporate all the functionality of Flood Unblock in Flood Control.

User interface changes

A table will be added that allows users to delete items from the flood table.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

batigolix created an issue. See original summary.

wim leers’s picture

+1, it's really confusing right now to figure out which module to use when. Even more confusingly, @fabianderijk seems to maintain both! :D

So … @fabianderijk, could you update both project pages to describe when to use which module, or ideally, merge the two modules?

fabianderijk’s picture

@Wim,

I do maintain both, but only since a short while including flood_control. We (@batigolix and me) ar maintainers of the flood_unblock module. We came across the flood_control module a while ago. I even already asked to co-maintain or incorporate the two modules a few years ago without any response.

A few weeks / months ago we revitalised the flood_unblock module and started the progress to actually become co-maintainer for the flood_control module with the idea to eventually integrate the flood_unblock module into it. Functionality of both modules are not the same, but extending each other and should be placed in one module. And because we think Flood Control is a better name for the functionality, we are moving everything to this module and are deprecating the flood_unblock module.

FYI, @batigolix and I have discussed a exiting strategy for flood_unblock where we will warn existing users to start using the flood_control module when the functionality has been moved.

batigolix’s picture

Assigned: Unassigned » batigolix

batigolix’s picture

Assigned: batigolix » fabianderijk
Status: Active » Needs review

Please review MR 2

All the functionality from Flood Unblock has been moved to Flood Control

You cannot have both modules installed at the same time.

  • batigolix committed 4543d99 on 2.0.x
    Issue #3176717 by batigolix: Incorporate functionality from Flood...
batigolix’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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