[eluser]boltsabre[/eluser]
Didn't actually look at your code to see if it is valid, but this is one way of reducing those nested if/else statements and making that code block about 50% smaller:
Code:
$data['details_error'] = "";
if( isset($this->session->userdata['logged_data']['data_error_message']) &&
($this->session->userdata['logged_data']['data_error_message']==TRUE )
{
$session_data = $this->session->userdata('logged_data');
$data['details_error'] = "<div class = 'alert alert-error'>Please fill the form with correct information and re submit</div>";
$session_data['data_error_message'] = FALSE;
$this->session->set_userdata('logged_data', $session_data);
}
Oh wait, I just had a look at your code, can't test this, I'm at work without a CI install, but this should work! Just assign your "data_error_message" straight in, no need to extract userdata('logged_data') first, manipulate it and then assign it back in.
Code:
$data['details_error'] = "";
if( isset($this->session->userdata['logged_data']['data_error_message']) &&
($this->session->userdata['logged_data']['data_error_message']==TRUE )
{
$data['details_error'] = "<div class = 'alert alert-error'>Please fill the form with correct information and re submit</div>";
$this->session->set_userdata("logged_data['data_error_message']", false);
}
7 lines of code instead of 18, much much better :-)