Port to Drupal 7

#670810: Port to Drupal 7 :-)

subscribing

subscribe. This is an important module.

Agreed.

Subscribing

Here is a patch for reCaptcha and mailhide. On my computer it works correctly with Drupal 7. More testing would be appreciated.

I downloaded the patch applied it on the latest 6.x-1.x-dev (as of today) and copied it to a new installation. I got the following error on the first load of modules page. I was however able to install and use the module without any problem.

Documenting this error here. I have reported this under Drupal core (#981818: Error PDOException: SQLSTATE[23000] Integrity constraint violation: 1062 Duplicate entry while loading modules page)

PDOException: SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'sites/all/modules/recaptcha/recaptcha.module' for key 1: INSERT INTO {system} (filename, name, type, owner, info) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2, :db_insert_placeholder_3, :db_insert_placeholder_4), (:db_insert_placeholder_5, :db_insert_placeholder_6, :db_insert_placeholder_7, :db_insert_placeholder_8, :db_insert_placeholder_9); Array ( [:db_insert_placeholder_0] => sites/all/modules/recaptcha/recaptcha.module [:db_insert_placeholder_1] => recaptcha [:db_insert_placeholder_2] => module [:db_insert_placeholder_3] => [:db_insert_placeholder_4] => a:9:{s:4:"name";s:9:"reCAPTCHA";s:11:"description";s:61:"Uses the reCAPTCHA web service to improve the CAPTCHA system.";s:12:"dependencies";a:1:{i:0;s:7:"captcha";}s:7:"package";s:12:"Spam control";s:4:"core";s:3:"7.x";s:5:"files";a:3:{i:0;s:19:"recaptcha.admin.inc";i:1;s:16:"recaptcha.module";i:2;s:17:"recaptcha.install";}s:7:"version";N;s:3:"php";s:5:"5.2.4";s:9:"bootstrap";i:0;} [:db_insert_placeholder_5] => sites/all/modules/recaptcha/recaptcha_mailhide.module [:db_insert_placeholder_6] => recaptcha_mailhide [:db_insert_placeholder_7] => module [:db_insert_placeholder_8] => [:db_insert_placeholder_9] => a:9:{s:4:"name";s:18:"reCAPTCHA Mailhide";s:11:"description";s:58:"Uses the reCAPTCHA web service to protect email addresses.";s:7:"package";s:12:"Spam control";s:4:"core";s:3:"7.x";s:5:"files";a:1:{i:0;s:25:"recaptcha_mailhide.module";}s:12:"dependencies";a:0:{}s:7:"version";N;s:3:"php";s:5:"5.2.4";s:9:"bootstrap";i:0;} ) in system_update_files_database() (line 2232 of /home/buddyerp/public_html/modules/system/system.module).

I can't reproduce this. I tried uninstalling and reinstalling on my existing install and a new install. Both times it worked fine with no errors.

Can you reproduce this error?

I have tested this patch and have found no issues - everything is working as I would expect.

Here's a reroll of #6, which is working for me with the alpha1 of captcha. I'm deploying this on randyfay.com.

This reroll just:

1. Patches from the root of the project against the D6 dev version.
2. Removes the LICENSE.txt (Drupal projects aren't supposed to have these any more)
3. Removes a spurious (I think) .cvsignore

Also attaching a tarball for people who just want to take it for a spin.

+1

Committed! Drupal 7 reCAPTCHA now open. If there are any additional issues, let's open up new nodes. Thank you guys so much for the help on this.

Randy Fay is so awesome that I gave him CVS access on this project.

Looks to me like @will_in_wi did all the work. Thanks, will_in_wi! Nice work.

I thought getting CVS access was punishment, not reward :-)

Thanks, Rob.