Easier adding of forms that are invisible to the "add CAPTCHA administration links to forms" feature [#214557]

CAPTCHA isn't able to add administration links to forms that are only visible to anonymous users (such as the password_reset_form form created by Password reset), making it necessary to add the form manually in the database.

Would it be possible to add a freetext field somewhere in the CAPTCHA settings page to make it easier to add custom forms?

Comment	File	Size	Author
#10	captchapointtab.png	26.84 KB	soxofaan
#10	214557_annonadd_D5_6.patch	14.74 KB	soxofaan
#10
#8	214557_annonadd_D5_4.patch	14.17 KB	soxofaan
#8
#5	captcha_point_add_snapshot6.png	16.98 KB	soxofaan
#4	214557_annonadd_3.patch	14.17 KB	soxofaan
#4
#3	214557_annonadd_2.patch	13.38 KB	soxofaan
#3
#1	214557_annonadd_1.patch	6.38 KB	soxofaan
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

soxofaan CreditAttribution: soxofaan commented 3 February 2008 at 16:09

Component:	Code	» User interface
Assigned:	Unassigned	» soxofaan
Status:	Active	» Needs review

File	Size
214557_annonadd_1.patch	6.38 KB

First iteration of a patch that adds a form for adding form_id's

Comment #2

soxofaan CreditAttribution: soxofaan commented 3 February 2008 at 17:58

This issue is already solved in version 6.x-1.x dev by http://drupal.org/cvs?commit=99120

Comment #3

soxofaan CreditAttribution: soxofaan commented 21 March 2008 at 11:58

File	Size
214557_annonadd_2.patch	13.38 KB

this is the port of http://drupal.org/cvs?commit=99120 to 5.x-3.x

includes:
* refactored captcha point administration,
* admin-invisible form_ids can now be added through UI
* new internal menu structure for captcha point administration
* added confirmation forms for disabling and deleting captcha points

Comment #4

soxofaan CreditAttribution: soxofaan commented 21 March 2008 at 12:46

File	Size
214557_annonadd_3.patch	14.17 KB

new version of the patch from #3:
added an update function for clearing the menu cache,
which is needed because the internal menu structure changed a bit

Comment #5

soxofaan CreditAttribution: soxofaan commented 21 March 2008 at 13:11

File	Size
captcha_point_add_snapshot6.png	16.98 KB

screenshot of adding a new CAPTCHA point

Comment #6

christefano CreditAttribution: christefano commented 21 March 2008 at 13:35

This is excellent!

Comment #7

soxofaan CreditAttribution: soxofaan commented 21 March 2008 at 13:58

This is excellent!

Thanks, did you try the patch yourself, or is your appreciation based on the screenshot? ;-)

Be welcome to test it thoroughly, so we can get this feature in the 5.x-3.x branch soon.

Maybe http://drupal.org/node/237147 could also interest you

Comment #8

soxofaan CreditAttribution: soxofaan commented 29 March 2008 at 17:32

File	Size
214557_annonadd_D5_4.patch	14.17 KB

new version of patch #4 to solve the issue from http://drupal.org/node/240088

Comment #9

christefano CreditAttribution: christefano commented 29 March 2008 at 18:06

Yes, I am testing the patch. :)

May I suggest changing the path of the CAPTCHA point configuration form to be at admin/user/captcha/captcha_point (instead of admin/user/captcha/captcha/captcha_point) so that it gets its own tab? I'm also up for moving the whole "Challenge type per fom" configuration fieldset to that same tab. I think that would help organize the interface a bit.