Allow recaptcha library to exist outside of the modules directory

Actually, it's packed as part of the module in 6.x-1.x-dev ;-) . Libraries API is definitely a great solution around this though.

I double check with latest CVS of recaptcha.module but still not able to find the Library API integration (http://drupalcode.org/viewvc/drupal/contributions/modules/recaptcha/reca...).

Is there any schedule for this update? Or may I contribute for this issue?

Patch for above issues, including changes for:

• Rename target library folder name as recaptcha-php, as unzip from original package.
• Dynamic figure out recaptcha-php from: 1. preconfigured path from admin page, 2. recaptcha module path, and 3. from libraries_get_path() result if function exists.
• Additional admin UI options for above preconfigure library path, plus checking during save configuration.
• Code cleanup with code-clean.
• Coding style cleanup with coder.
• Remove the embed recaptcha/recaptchalib.php. As the original library is licensed under MIT, should it be a good idea to let end-user download and install it manually?

Completely tested with recaptcha-6.x-1.x-dev CVS. Please kindly review and feel free to comment with :D

Couple of minor comments on the patch:

-/* $Id: recaptcha.install,v 1.2.4.2.2.3.2.2 2009/07/28 15:39:20 robloach Exp $ */
+// $Id: recaptcha.install,v 1.2.4.2.2.3.2.2 2009/07/28 15:39:20 robloach Exp $

Should be

-/* $Id: recaptcha.install,v 1.2.4.2.2.3.2.2 2009/07/28 15:39:20 robloach Exp $ */
+// $Id$

And when giving error feedback:

+  else {
+    drupal_set_message(t('The recaptcha-php files <em>recaptchalib.php</em> could not be found in any possible directories.'));
+  }

drupal_set_message should be called with an error parameter: drupal_set_message('message', 'error');

@manarth: Totally agree, and patch update as attached. Thank you very much ;-)

The patch no longer applies. Please re-roll this patch for 7.x-1.x-dev. Once we have it working there, it can be backported to D6.

recaptchalib is supplied with the module so I don't see any need for this. If you feel this is an important feature, please provide a patch and justification.