Hi Rob,

I'm busy to get the CAPTCHA module for Drupal 6 past Beta and RC state and one of the last issues I'd like to iron out for CAPTCHA 6.x-2.0 is removing the preprocess operation from hook_captcha. For more discussion and details see #463002: Drop preprocess op of hook_captcha?. Feel free to tune in.

In attachment I have a patch that should make the reCAPTCHA module compatible with this API change: it replaces the usage of the custom hook_captcha-preprocess with standard Drupal Form API stuff (mainly a '#process' callback).
Note that this change is also backwards compatible with the "old" CAPTCHA API (CAPTCHA 5.x-3.x and 6.x-1.x), so nobody gets hurt.

CommentFileSizeAuthor
drop_preprocess.patch2.93 KBsoxofaan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RobLoach’s picture

Wow, I like that process array. Committed. When you release the 6.x-2.0 version, I'll hit up a release on reCAPTCHA. Well done, soxofaan!

soxofaan’s picture

Hi,
I just released CAPTCHA 6.x-2.0-rc1 in which the preprocess op was dropped, so I think you should hit up that reCAPTCHA release.

keep up your good work, rob!

RobLoach’s picture

Status: Active » Fixed

Great! Thanks a lot for reminding me.... reCAPTCHA 6.x-1.2.

I didn't do any work, you did it all. Also, met a Dutch person at the Drupal NYC meet and he said that his girlfriend knows you! Don't know how that works, but it was pretty funny.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.