Issue with my 'Sign in' script

[eluser]Bas Vermeulen[/eluser]
It's because you do a redirect, a whole new page request with no post data or form validation errors. If you really want to do a redirect you should set the form validation errors as flashdata before the redirect and check/display the flashdata in your view. I would prefer to just load the method of the comments view page (I guess that's in the same controller?):

Code:

function addcomment(){
  
      $this->load->library('form_validation');

      $this->form_validation->set_rules('post_id', 'post_id', 'required|numeric');
      $this->form_validation->set_rules('comment_body', 'Comments', 'trim|required');
  
      if($this->form_validation->run() == FALSE)
      {                
            $this->viewcomment($this->input->post('post_id')); // or w/e you called it ;)
      }
      else
      {  
            $this->MComments->addComment();
            $this->viewcomment($this->input->post('post_id'));
      }
  }

I also added a validation rule for the post_id, I think it's good practice to always validate all incoming postdata, even if it's a hidden input and you set the value's yourself. People can spoof forms and do bad things


Also a little update in the view which I recently learned myself:

Code:

<?php if(!empty($this->form_validation->_error_array)):?>
    <div class="error">
        &lt;?php echo validation_errors(); ?&gt;  
    </div>  
&lt;?php endif; ?&gt;

The following works as well (shorter & cleaner):

Code:

<?php if(validation_errors()):?>
    <div class="error">
        &lt;?php echo validation_errors(); ?&gt;  
    </div>  
&lt;?php endif; ?&gt;

Good luck!