It showes error); ?> on the suggestions page and then shows the form, but no captcha is loaded
It showes error); ?> on the suggestions page and then shows the form, but no captcha is loaded
looks like your php tags are out of synch.
You will also need the recaptchalib.php file in the extra functions folder.
it is there
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.
.
Zen Cart - putting the dream of business ownership within reach of anyone!
Donate to: DrByte directly or to the Zen Cart team as a whole
Remember: Any code suggestions you see here are merely suggestions. You assume full responsibility for your use of any such suggestions, including any impact ANY alterations you make to your site may have on your PCI compliance.
Furthermore, any advice you see here about PCI matters is merely an opinion, and should not be relied upon as "official". Official PCI information should be obtained from the PCI Security Council directly or from one of their authorized Assessors.
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 lineI like the sugestion of using zen_get_ip_address(). I'll add that to the observer class.Code:$zco_notifier->notify('NOTIFY_CONTACT_US_CAPTCHA_CHECK');
Many thanks
David
The only requirements you'd need to backport would be the changes that added the notifier points to those modules files.
.
Zen Cart - putting the dream of business ownership within reach of anyone!
Donate to: DrByte directly or to the Zen Cart team as a whole
Remember: Any code suggestions you see here are merely suggestions. You assume full responsibility for your use of any such suggestions, including any impact ANY alterations you make to your site may have on your PCI compliance.
Furthermore, any advice you see here about PCI matters is merely an opinion, and should not be relied upon as "official". Official PCI information should be obtained from the PCI Security Council directly or from one of their authorized Assessors.
Thanks DrByte - new version being tested, docs still to do :)
Regards
David