From 27daf1ad140c79a8512f927c4a03fa74ff53173b Mon Sep 17 00:00:00 2001 From: Joey Hess Date: Sun, 22 Jun 2008 09:42:08 -0400 Subject: [PATCH] web commit by http://willu.myopenid.com/: Some info on tracking down the remaining issue --- doc/todo/require_CAPTCHA_to_edit.mdwn | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/doc/todo/require_CAPTCHA_to_edit.mdwn b/doc/todo/require_CAPTCHA_to_edit.mdwn index 0e32afc65..d87c73260 100644 --- a/doc/todo/require_CAPTCHA_to_edit.mdwn +++ b/doc/todo/require_CAPTCHA_to_edit.mdwn @@ -30,7 +30,20 @@ A second issue is that the OpenID login system resets the 'required' flags of all the other fields, so using OpenID will cause the CAPTCHA to be ignored. -> This is still a todo. +> This is still not fixed. I would have thought the following patch would +> have fixed this second issue, but it doesn't. + +--- a/IkiWiki/Plugin/openid.pm ++++ b/IkiWiki/Plugin/openid.pm +@@ -61,6 +61,7 @@ sub formbuilder_setup (@) { #{{{ + # Skip all other required fields in this case. + foreach my $field ($form->field) { + next if $field eq "openid_url"; ++ next if $field eq "recaptcha"; + $form->field(name => $field, required => 0, + validate => '/.*/'); + } + Instructions ===== -- 2.26.2