Hi,

Is there plans to introduce D8 version of this module?
We would like to help you with development of this version or to implement it from scratch.

CommentFileSizeAuthor
#60 flood_control-drupal_8_port-2221025-60.patch4.5 KBfabianderijk
#47 flood_control-drupal_8_port-2221025-47.patch9.7 KBshkiper
#17 2221025-drupal-8-port.patch14.14 KBshabirahmad
#9 2626748-drupal-8-port-4.patch5.03 KBshabirahmad
#6 flood_control-drupal_8-2221025-6.patch25.82 KBshkiper
#5 flood_control-d8-2221025-5.patch9.75 KBjeqq
#4 flood_control-drupal_8-2221025-4.patch19.79 KBshkiper

Comments

anavarre’s picture

anavarre
Primary language French
Location 🇪🇺
commented

FYI there's a port here that will be contributed back to the d.o. issue queue.

Background here: http://www.ciandt.com/card/hackathon-drupal-8

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Yeah I'm not able to commit that code currently, if we want to attribute the git commits, as it wasn't actually created or forked from the existing Git source.

anavarre’s picture

anavarre
Primary language French
Location 🇪🇺
commented

Right, this would need some git format-patch magic or using the script I put together at https://gist.github.com/anavarre/7486059ea510e71d24c2

shkiper’s picture

shkiper commented
StatusFileSize
new flood_control-drupal_8-2221025-4.patch19.79 KB

Here is the patch to port Flood control module to Drupal 8.

jeqq’s picture

jeqq commented
Status: Active » Needs review
StatusFileSize
new flood_control-d8-2221025-5.patch9.75 KB

Fixed patch #4, added menu link.

shkiper’s picture

shkiper commented
StatusFileSize
new flood_control-drupal_8-2221025-6.patch25.82 KB

Hi, I made a version of this module that works with Drupal 8 beta
Here is my patch

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Needs work

Please do not include the LICENSE.txt. It should not be included in the repo because it gets packaged automatically by Drupal.org.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Version: 7.x-1.x-dev » 8.x-1.x-dev

Now have an 8.x-1.x branch for targeting with tests with an updated patch.

shabirahmad’s picture

shabirahmad commented
Status: Needs work » Needs review
StatusFileSize
new 2626748-drupal-8-port-4.patch5.03 KB

I have ported the module. you can find the code here. I am submitting the patch too.

Status: Needs review » Needs work

The last submitted patch, 9: 2626748-drupal-8-port-4.patch, failed testing.

The last submitted patch, 9: 2626748-drupal-8-port-4.patch, failed testing.

The last submitted patch, 9: 2626748-drupal-8-port-4.patch, failed testing.

The last submitted patch, 9: 2626748-drupal-8-port-4.patch, failed testing.

The last submitted patch, 9: 2626748-drupal-8-port-4.patch, failed testing.

The last submitted patch, 9: 2626748-drupal-8-port-4.patch, failed testing.

The last submitted patch, 9: 2626748-drupal-8-port-4.patch, failed testing.

shabirahmad’s picture

shabirahmad commented
StatusFileSize
new 2221025-drupal-8-port.patch14.14 KB
shabirahmad’s picture

shabirahmad commented
Status: Needs work » Needs review

The last submitted patch, 4: flood_control-drupal_8-2221025-4.patch, failed testing.

The last submitted patch, 4: flood_control-drupal_8-2221025-4.patch, failed testing.

The last submitted patch, 4: flood_control-drupal_8-2221025-4.patch, failed testing.

The last submitted patch, 4: flood_control-drupal_8-2221025-4.patch, failed testing.

The last submitted patch, 4: flood_control-drupal_8-2221025-4.patch, failed testing.

The last submitted patch, 4: flood_control-drupal_8-2221025-4.patch, failed testing.

The last submitted patch, 4: flood_control-drupal_8-2221025-4.patch, failed testing.

The last submitted patch, 5: flood_control-d8-2221025-5.patch, failed testing.

The last submitted patch, 5: flood_control-d8-2221025-5.patch, failed testing.

The last submitted patch, 5: flood_control-d8-2221025-5.patch, failed testing.

The last submitted patch, 5: flood_control-d8-2221025-5.patch, failed testing.

The last submitted patch, 5: flood_control-d8-2221025-5.patch, failed testing.

The last submitted patch, 5: flood_control-d8-2221025-5.patch, failed testing.

The last submitted patch, 5: flood_control-d8-2221025-5.patch, failed testing.

The last submitted patch, 6: flood_control-drupal_8-2221025-6.patch, failed testing.

The last submitted patch, 6: flood_control-drupal_8-2221025-6.patch, failed testing.

The last submitted patch, 6: flood_control-drupal_8-2221025-6.patch, failed testing.

The last submitted patch, 6: flood_control-drupal_8-2221025-6.patch, failed testing.

The last submitted patch, 6: flood_control-drupal_8-2221025-6.patch, failed testing.

The last submitted patch, 6: flood_control-drupal_8-2221025-6.patch, failed testing.

The last submitted patch, 6: flood_control-drupal_8-2221025-6.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 17: 2221025-drupal-8-port.patch, failed testing.

The last submitted patch, 17: 2221025-drupal-8-port.patch, failed testing.

The last submitted patch, 17: 2221025-drupal-8-port.patch, failed testing.

The last submitted patch, 17: 2221025-drupal-8-port.patch, failed testing.

The last submitted patch, 17: 2221025-drupal-8-port.patch, failed testing.

The last submitted patch, 17: 2221025-drupal-8-port.patch, failed testing.

The last submitted patch, 17: 2221025-drupal-8-port.patch, failed testing.

shkiper’s picture

shkiper commented
Status: Needs work » Needs review
StatusFileSize
new flood_control-drupal_8_port-2221025-47.patch9.7 KB

Could you please review my patch? It works with current Drupal 8.1.1 version

Status: Needs review » Needs work

The last submitted patch, 47: flood_control-drupal_8_port-2221025-47.patch, failed testing.

The last submitted patch, 47: flood_control-drupal_8_port-2221025-47.patch, failed testing.

The last submitted patch, 47: flood_control-drupal_8_port-2221025-47.patch, failed testing.

The last submitted patch, 47: flood_control-drupal_8_port-2221025-47.patch, failed testing.

The last submitted patch, 47: flood_control-drupal_8_port-2221025-47.patch, failed testing.

The last submitted patch, 47: flood_control-drupal_8_port-2221025-47.patch, failed testing.

The last submitted patch, 47: flood_control-drupal_8_port-2221025-47.patch, failed testing.

tamerzg’s picture

tamerzg commented

In meantime you can check my blog to see how to change flood variables using PHP snippet:
http://zoubi.me/blog/increase-flood-limit-and-flood-interval-drupal-8

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Title: Drupal 8 version » Port Flood Control to Drupal 8

Standardized the issue title so it's easier to find.

mrpauldriver’s picture

mrpauldriver commented

This is popular module which is badly needed for Drupal 8

wombatbuddy’s picture

wombatbuddy commented

@MrPaulDriver
Take a look at the 'Flood settings' module.

mrpauldriver’s picture

mrpauldriver commented

Thanks for that @wombatbuddy

fabianderijk’s picture

fabianderijk
him/his
Primary language Dutch
Location Alphen aan den Rijn
commented
StatusFileSize
new flood_control-drupal_8_port-2221025-60.patch4.5 KB

I've created a new patch based on the one in #47, but a bit cleaner with dependency injection.

fabianderijk’s picture

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

I've committed the patch, and created a 2.0.0 release for the port. The module now works for D8 and D9!

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Fixed » Active

Woah! This is unexpected.

Per #3176717-2: Incorporate functionality from Flood Unblock, @fabianderijk, could you please bring clarity on the flood_control vs flood_unblock situation? 🙏

Reopening for that 🤓

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Related issues: +#3176717: Incorporate functionality from Flood Unblock, +#2626748: [flood_control]Flood Control
wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Related issues: +#3176714: Move functionality to Flood Control
fabianderijk’s picture

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

@Wim, I'll answer your question in #3176717: Incorporate functionality from Flood Unblock, that should clarify some things :)

Status: Fixed » Closed (fixed)

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