Hi,

I was wondering, how do I redirect the user if a slug is not found in the database?

In my Controller, I have:

This retrieves the page details based on the slug.

In my Model, I have:

Would I have an else in my Model function and then let the Controller know if no results were found?

Many thanks for any guidance.

invision - 11 May 2010 10:53 AM

Hi,

I was wondering, how do I redirect the user if a slug is not found in the database?

In my Controller, I have:

function pages($slug){
    $data['page_data'] = $this->MPages->getPageBySlug($slug);
    $data['title'] = $data['page_data']['title'];
    $data['id'] = $data['page_data']['id'];
    $data['body'] = $data['page_data']['body'];
    $data['main'] = 'public_page';
    $this->load->vars($data);
    $this->load->view('template');  
  } 

This retrieves the page details based on the slug.

In my Model, I have:

function getPageBySlug($slug){
        $data = array();
        $this->db->where('slug',$slug);
        $this->db->limit(1);
        $Q = $this->db->get('webpages');
        if ($Q->num_rows() > 0){
          $data = $Q->row_array();
        }
        $Q->free_result();    
        return $data;    
    } 

Would I have an else in my Model function and then let the Controller know if no results were found?

Many thanks for any guidance.

I usually make my models return FALSE on failure, and let the controller ask this:

To make show_404() show a custom 404 you will have to do a lot of work (there are some topics about that in this forum). Another option is redirecting like you said

Many thanks Buso.

So perhaps something like this:

Model

Controller

I’m a bit new to this so any help at all is super appreciated.

Thank you.

be careful with assignations and comparations

if($data = false) will set it to false

if($data == false) will work, but using === is considered a good practice, so you don’t have things like 0 evaluating to false

Many thanks Buso. I will try this shortly. I’m getting an unrelated Database Error right now. Nice

Only issue now is that it doesn’t seem to be redirecting the user.

It just the template, without any of the variables in place.

In my pages function in the Controller, I’m using:

invision - 11 May 2010 11:36 AM

Only issue now is that it doesn’t seem to be redirecting the user.

It just the template, without any of the variables in place.

In my pages function in the Controller, I’m using:

if ($data === false) { 
      redirect('/index.php/page/aboutus/','refresh'); 
} 

That code block should be:

Never add index.php and you don’t really need to add preceding or trailing slashes.

Hi Phil,

Many thanks for your input.

Just to confirm this pages function is part of my Page controller:

It’s still not redirecting. Is it because of the line above the redirect?

The rest of my Page Controller is as follows:

Thanks again for your help.

invision - 11 May 2010 12:31 PM

It’s still not redirecting. Is it because of the line above the redirect?

Of course!

You are testing that $data is empty after populating it. Even if this model fails you will have:

That is a full variable.

So…

Apologies again for these simple queries.

Is this correct?

Whoop! It is

Thanks again for your help and patience Phil.