Login

08-06-2009, 09:01 PM

[eluser]mr_coffee[/eluser]
This worked perfect for me (the new one) after making a couple of adjustments, though I see how this would be a special case. The previous comment tipped me off. The form_validation config file was not working as advertised for me, but it's how I have the controller set up:

Code:
function index($recipient) {

== fail. I tried variations on the config file but couldn't make it work (even when I knew the value of $recipient). The following adjustment was sufficient (adding this to the controller):

Code:
$this->form_validation->set_rules('recaptcha_response_field','lang:recaptcha_field_name','required|callback_check_captcha');

As I also like to sort things of this nature into folder - specifically one labeled 'forms' - I was surprised to find that placing the recaptcha.php view in the forms folder failed until I figured out its name and location was set in the recaptcha.php library file... should be line 158:

Code:
return $this->_CI->load->view('forms/recaptcha',$html_data,TRUE);

Ok, back to my controller implementation: I'd decided to create a single controller to deal with e-mail for multiple accounts. Therefore 'email/name' as an address generates the appropriate form. All sorts of goofy things happen in the controller... variables set up for the page view... if there's a phone number they want displayed, etc... it all gets dumped into the form view. The generic example assumes - at least I think it does - that the controller would only ever display the form, and that it's the "only" form. As this was not true in my case, I also had to make this adjustment to the form view:

Code:
&lt;?php echo form_open('contact/'.$contactpage,array('class' => 'emailForm')); ?&gt;

$contactpage is the same as $recipient... but just being the data variable name I passed to the view. 'contact' is the name of the view (contact.php). The way I have CI set up in general... as I have a main controller dealing with everything (:any)... if the address is not found, it goes home. I have to specify the address of the form or you'd never see the errors if there were any. The class is in there just so I could style the placement of the labels and images.

The only suck part is that the final code will fail xhtml validation as there is a script in the body of the page. It might be possible to make an adjustment using the reCaptcha API and having a predetermined div where the table will be written to, but I haven't tried it. The error is kind of worth it to curtail spam, on the other hand.

I've read through the comments here. To the neigh-sayers: the old "captcha" method was not very good. reCaptcha - on the other hand - IS very good. It isn't an "end all" to spam, but it's very effective. I had the old version installed to a forum and still got at least one SPAM sign up per week... and since switching to the reCaptcha verification method: have only had one in a year. I'd wager that user had to manually read the image and sign up with their spam URL in their profile... and if a spammer is willing to go through all THAT trouble... they deserved the ability to sign up even if I did ban their IP after. While the forum isn't advertised or anything... I'd call that a pretty decent reason to use the reCaptcha method: it is quite effective.

My thanks to the author of the library!