Codeigniter Message: Undefined variable: query

Question

SOLVED!

thanks to Fabio i realized that i missed something in my index function of my controller.

i am working on a view that show's me the data from the database.

Now i get the error: Message: Undefined variable: query.

my Model:

<?php
class categorieen_model extends CI_Model{

    function categorieen_getall()
    {
        $this->load->database();
        $query = $this->db->get('Categorieen');

        return $query->result();
    }
}

?>

My controller

public function get_All()
{
    $this->load->model('categorieen_model');
    $data['query'] =
    $this->categorieen_model->categorieen_getall();

    $this->load->view('sidebar', $data);
}

My view

<ul>
<?php foreach ($query as $row): ?>
    <?php echo $row->Categorie; ?>
<?php endforeach; ?>
</ul>

Last week i tried this with the same database, another table and other data and names in my controllers/views and it worked. when i changed it today to my new table it did not work anymore.

Please help

Thanks

Does the query return any rows. Try doing a print_r($query->result()); before you return the results; — Jeemusu
– Jeemusu, Commented Feb 27, 2013 at 9:33
Sounds like your query doesn't return any results, and so the array passed to the view holds no data, and can't be looped through. — Jeemusu
– Jeemusu, Commented Feb 27, 2013 at 9:38
Don't know why this happens because there is data in my table and i Thought the code was good. — Kees Sonnema
– Kees Sonnema, Commented Feb 27, 2013 at 9:39

Fabio Antunes · Accepted Answer · 2018-11-11 21:45:10Z

1

You're not passing any data on the index method to the views

<?php
  class Home extends CI_Controller{

    public function index() {
      //add this code
      $this->load->model('Categorieen_Model');
      $data['query'] = $this->Categorieen_Model->categorieen_getall();
      $this->load->vars($data);
      //end of new code
      $this->load->view('header');
      $this->load->view('navmenu');
      $this->load->view('sidebar');
      $this->load->view('home');
      $this->load->view('sidebar2');
      $this->load->view('sidebar3');
      $this->load->view('footer');
    }

    public function get_All(){
      $this->load->model('Categorieen_Model');
      $data['query'] = $this->Categorieen_Model->categorieen_getall();

      $this->load->view('sidebar', $data);
    }

  }
?>

edited Nov 11, 2018 at 21:45

answered Feb 27, 2013 at 10:43

Fabio Antunes

22.9k17 gold badges83 silver badges96 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Kees Sonnema Over a year ago

allright. i am very stupid. please hit me with a stick. thanks so much :D accepted

Fabio Antunes Over a year ago

No problem at all, by the way instead of loading all that views, in every method you could adopt a "templating" system, in this answer I gave a very basic way of do it stackoverflow.com/questions/15093949/… And you also have this way stackoverflow.com/questions/8601054/…

Rahul Chipad · Accepted Answer · 2013-02-27 09:38:57Z

1

Use this in your controller:

<?php 
if(is_array($query) && count($query)  > 0 ) { ?>
<ul>
<?php foreach ($query as $row): ?>
    <?php echo $row->Categorie; ?>
<?php endforeach; ?>
</ul>
<?php }?>

answered Feb 27, 2013 at 9:38

Rahul Chipad

2,41118 silver badges21 bronze badges

4 Comments

Leri Over a year ago

&& count($query) > 0 is odd here. If there're no elements loop body will never execute.

Kees Sonnema Over a year ago

You mean views? and this doesn't work either. still the same error.

Kees Sonnema Over a year ago

Hmm something changed now. the second error is gone. but the first error is still showing up

Kees Sonnema Over a year ago

Can you help me any further with this?

narendar · Accepted Answer · 2013-02-27 09:40:04Z

-1

Change your controller to this

public function get_All()
{
    $this->load->model('categorieen_model');
    $data['list'] =
    $this->categorieen_model->categorieen_getall();

    $this->load->view('sidebar', $data);
}

And change view to

<ul>
<?php foreach ($list as $row){?>
    <?php echo $row->Categorie; ?>
<?php } ?>
</ul>

answered Feb 27, 2013 at 9:40

narendar

631 gold badge2 silver badges7 bronze badges

1 Comment

Kees Sonnema Over a year ago

Doesn't work. get the same error only list is different Severity: Notice Message: Undefined variable: list

Collectives™ on Stack Overflow

Codeigniter Message: Undefined variable: query

3 Answers 3

2 Comments

4 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

4 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related