Closed (won't fix)
Project:
CAPTCHA
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
9 Apr 2007 at 18:05 UTC
Updated:
9 Apr 2007 at 19:01 UTC
The attached simple patch introduces a new hook_captcha that allows modules to inform the captcha.module what captcha points need captcha.
To use this, it is very simple, for example, feedback module would have this:
function feedback_captcha() {
return array(
'feedback_mail_form' => t('Feedback Form'),
);
}
Then the new captcha point will appear in captcha's settings, and the site admin can enable it.
No more editing of captcha module to add your points with each release.
This has been tested and works well.
| Comment | File | Size | Author |
|---|---|---|---|
| captcha-hook.patch | 1005 bytes | kbahey |
Comments
Comment #1
wundo commentedYour patch doesn't work properly with 5.x-1.x-dev and HEAD.
Head already implements that, and uses an different api, please check it.
thanks
fabiano
Comment #2
kbahey commentedRe: http://heine.familiedeelstra.com/math-captcha-form-store
The advantage of Heine's FormStore is that it requires no changes in the modules that want to implement captchas, so that is a definite plus.
Re: patch not applying, I checked out DRUPAL-5. Seems that DRUPAL-5--1-x are branches, and hence I would not get them when I do cvs update. This is one issue with the new release system, as opposed to the old system where we get the latest by doing cvs update.
Do you plan on releasing this form store solution for 5.x any time soon?
Comment #3
wundo commentedI'm just waiting for more user feedback before releasing Captcha 5.x.2