Coding Standards for modern Drupal

+++ b/modules/tarpit_ban/tarpit_ban.services.yml
@@ -3,4 +3,4 @@ services:
+    arguments: ['@requestStack', '@ban.ip_manager', '@logger.channel.tarpit']

Are you sure about this ?

No it has nothing to do with variables.

@request_stack is a service !

I applied the 'coding_standards_for_d8-2833166-7.patch' and it was removing a function commentary.
I generated a new patch file ('coding_standards_for_d8-2833166-8.patch ') to fix this problem.

Then rechecked the code standard using the following command without further complains:
find . -type f \( -name '*.php' -or -name '*.yml' -or -name '*.module' \) -exec phpcs --standard=Drupal {} \+

Then I load the patched module in my environment without mistakes using drupal console as shown in the picture 'loading_tarpit.png'.

Some dependency injection and standards fixes

Hello People !

I apply the patch and check the standards.

All looks fine by me !

Thanks @faline !

Updating the version to the latest release.

Thanks everyone for working on this, now more than eight years ago :)

The Tarpit module has become potentially very useful, since it can help block bot attacks for web sites using Facets 2, by offering them Honeypot links, and a ban if they follow them:

- https://www.drupal.org/docs/administering-a-drupal-site/security-in-drup...
- https://www.drupal.org/forum/support/post-installation/2025-02-20/drupal...

I created a GitLab Merge Request based on the last patch from comment #10. There were a few additional deprecations since then, which are also fixed in the MR, and the module works well in Drupal 10.