Module purpose & overview

The main functionality of this module is to automatically add an specific captcha challenge to all the forms in the site where this module is enabled.

Why we created this module?

The idea to create this module was that we always use catpcha for webforms due to a security process on our company and we can save time with this solution and we don't have to care about specific IDs.

Is it Features/Deploy friendly?

Yes, it is. Because it doesn't store any ID or form name.

Configurations

We can configure different challenges (if we want) to each of type of form (System ones and webforms). Of course, this configuration, won't override configurations set on the captcha module. So if you have configured by ID any form in the captcha module, it won't override that challenge. If there isn't anything set, it will set the captcha we've configured on this module.

Git url

git clone --branch 7.x-1.x nbouhid@git.drupal.org:sandbox/nbouhid/2154411.git auto_captcha

Comments

PA robot’s picture

PA robot commented
Status: Active » Needs work

There are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpgitdrupalorgsandboxnbouhid2154411git

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

beljaako’s picture

beljaako commented

Hi there,

Here's my review
- Please remove your username from the git url, as we don'y have your password :).
- Please add dependency on webform.
- You're adding a captcha to every form. It's also adding a captcha to the user login form. I don't think that's useful.
- It would be much nicer if you would just add an option to the captcha module "enable captch for all forms". That would have the exact same effect, and that way you wouldn't need another administration part.
- Take a look at the order of fields. The captcha is added on top of the form. It would be nicer to put it on the bottom.
- Reconsider the texts on your admin form. A title like "How to configure it" isn't really helpful.
- What if a user already configured captcha's for some forms, and decides to use your module. I can see in the code you're trying to prevent double captcha's, but in my case I get 2 captcha's on one form. Please check that.

That't it so far. Hope it helps. Good luck on you're module.

Alexxikon’s picture

Alexxikon commented

Hi nbouhid, here are my comments on your project:

  • Remove the "master" branch from Git. This is not used in the Drupal flow.
  • You are missing a README.txt file.
  • You are using variable_set('auto_captcha_default_challenge_webform',... but you don't have an uninstall hook implemented to clean up.
PA robot’s picture

PA robot commented
Status: Needs work » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application (see also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.