The error messages for the math captcha field (which comes bundled with the CAPTCHA module) are not appearing inline. I've made some attempts at adding support for it, but haven't figured it out yet.

CommentFileSizeAuthor
#4 ife_captcha_support_2.patch1.99 KBmatt v.
#3 ife_captcha_support.patch2.14 KBmatt v.

Comments

matt v.’s picture

matt v. commented
Title: Match captcha errors not appearing inline » Math captcha errors not appearing inline
javaguystuckwithdrupal’s picture

javaguystuckwithdrupal commented

Hello,

I'm also stuck on this issue. Were you able to find a solution for this? thanks much

matt v.’s picture

matt v. commented
Status: Active » Needs review
StatusFileSize
new ife_captcha_support.patch2.14 KB

I've written a patch to add captcha support. It seems to be working, so far.

matt v.’s picture

matt v. commented
StatusFileSize
new ife_captcha_support_2.patch1.99 KB

I was getting an error message, if the CAPTCHA had been entered correctly but there were additional errors on the page:

Warning: implode() [function.implode]: Invalid arguments passed in form_get_error() (line 837 of /var/www/vhosts/example.com/htdocs/includes/form.inc).

I've updated the patch to address that issue.

matt v.’s picture

matt v. commented

removing tag I accidentally added

cburyta’s picture

cburyta commented

I just wanted to chime in and note that the patched worked bug free (used _2.patch) Thanks!

spikeerob’s picture

spikeerob commented

Patch worked for me too. Great stuff! :)

matt v.’s picture

matt v. commented

@cburyta & @spikeerob,

Thanks for the feedback. Did you use the patch specifically with the Math CAPTCHA? I haven't had a chance to test it, but I'm curious if it works with any of the other CAPTCHA types.

spikeerob’s picture

spikeerob commented

I was using it with the reCaptcha plugin actually. Not tried it with any of the others.

kristofvanroy’s picture

kristofvanroy commented

Patch #4 is working with captcha/Math

stijndm’s picture

stijndm commented
Version: 6.x-1.1 » 7.x-1.x-dev
Status: Needs review » Patch (to be ported)

Has been committed to the D6 branch. Leaving this open as to be ported to D7.

stijndm’s picture

stijndm commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
Issue summary: View changes
Status: Patch (to be ported) » Needs review

This might be fixed in the latest dev version. The code has been overhauled. Can you please try out the latest dev?

kenorb’s picture

kenorb commented
Status: Needs review » Postponed (maintainer needs more info)
kenorb’s picture

kenorb commented

See also: #1922668: Add support for reCaptcha