Support Thread for Google reCAPTCHA plugin

[DrByte]

David,
For v1.5.1 and newer (since some captcha-related notifier hooks were added to v1.5.1), you can drop the changes your v1.1 plugin makes to the /includes/modules/ files and simply add the attached notifier support files to accomplish the same results.
Makes integration a lot simpler that way because there's fewer files to edit (esp since some of those files are often edited by other plugins), and simpler to update in the future too.

I might also suggest that using zen_get_ip_address() instead of $_SERVER['REMOTE_ADDR'] might provide more accurate results especially if the visitor is behind some sort of proxy or forwarder, or the store's hosting company is using some sort of proxying or caching engine that incorrectly passes the store's IP instead of the visitor's IP.

[David Allen]

David,
For v1.5.1 and newer (since some captcha-related notifier hooks were added to v1.5.1), you can drop the changes your v1.1 plugin makes to the /includes/modules/ files and simply add the attached notifier support files to accomplish the same results.
Makes integration a lot simpler that way because there's fewer files to edit (esp since some of those files are often edited by other plugins), and simpler to update in the future too.

Thanks for that. I've just d/loaded today's new 1.51 and see the added notifier hooks. Brilliant. It will, as you say, make things a lot simpler.
In order not to have to maintain 2 versions (pre-1.51 and 1.51+) going forward, I'm thinking that I should make the pre-1.51 version also use the observer class. A quick question if I may. Am I correct in that all I would need (apart from the check on $error) to add to those header_php.php files would be the line

$zco_notifier->notify('NOTIFY_CONTACT_US_CAPTCHA_CHECK');

I like the sugestion of using zen_get_ip_address(). I'll add that to the observer class.

Many thanks
David

[DrByte]

The only requirements you'd need to backport would be the changes that added the notifier points to those modules files.

[David Allen]

Thanks DrByte - new version being tested, docs still to do :)
Regards
David