thomasrawiel / powermailcaptcha

GNU General Public License v3.0
4 stars 4 forks source link

Feature request: Make it possible to always enable captcha without having to manually add the field to each form #14

Closed sypets closed 5 months ago

sypets commented 11 months ago

I would like to make it possible to always activate spam proctection via "friendly captcha". We have several forms (more than 200), where it would be very tedious to have to add the "powermailcaptcha" field manually in every form.

I would like to enable this "always on" behaviour via configuration, e.g. TypoScript. Ideally, the forms which already have the field should not show it twice (this could however be manually updated in the DB to deactivate or soft delete the fields with type "powermailcapcha").

I have used this extension with "friendly captcha" for a while as a "trial run" and seems to be working fine (several attempts already blocked, as seen in spam log).

Also, from the usability point of view, I think it would be beneficial, to always look the same, e.g.

I think it would be possible for me to do this by overriding the Fluid partials "Friendlycaptcha.html", "Submit.html" and the function CaptchaMethod::isFormWithCaptchaField.

But having this functionality might be helpful for others as well, so I am submitting it as feature request.

thomasrawiel commented 11 months ago

Interesting idea! I see your reasons behind this request and think this would make a good addition.

But I don't think this would work for all use cases.

A few thoughts:

I can have a look into it when I have the time but I can't promise this will be added in the future.

sypets commented 11 months ago

@thomasrawiel Thanks for looking into, I understand. I managed to do what I attempted in the sitepackage extension, overriding the Page Fluid and the CaptchaMethod. Even with these changes, using your extension is already a huge help, compared to having to write everything from scratch. I am happy with the solution for now.

thomasrawiel commented 5 months ago

As described in my previous comment I think it's not feasible to add this without changing the way the extension currently ("just") works.