CodeIgniter/PHP Error Message

Question

The goal of this application is to be able to click on a link from one view to get data from another. The first view works just fine and I'm getting the correct PK as well. When I click the link I am having issues.

Getting a 'Message: Invalid argument supplied for foreach()' from this view:

<?php
    echo "<h2>View</h2>";
    foreach ($rows as $r){
        echo '<br /><h3>'; echo $r->tHandle; echo'</h3>';
        echo "<li>Sent at:  "; echo $r->content; echo"</li><br />";
        echo'<li>Created:  '; echo $r->created; echo'</li><br />';
    }
?>

This is the model and function where the DB query is taking place:

<?php
class Tweets_model extends CI_Model{

    public function __construct() {

        $this->load->database();

    }

    public function getTweetDetails($id){
    $this->db->select('*')->from('tweets')->where('tweetId', $id);
    $q = $this->db->get();
        if($q->num_rows > 0){

            foreach($q->result() as $row){

                $data[]=$row;

            }
            return $data;

        }

    }
}
?>

Controller:

<?php

class Tweets extends CI_Controller{

    public function __construct() {
        parent::__construct();
        $this->load->model('tweets_model');
    }

    public function details(){

        $data['rows'] = $this->tweets_model->getTweetDetails($this->uri->segment(2));
        $this->load->view('header');
        $this->load->view('tweet_details', $data);
        $this->load->view('footer');
    }
}

?

>

Can anyone help me get past my error message?

Thanks.

What's the result of print_r($rows); before the foreach? — Rolando Isidoro
– Rolando Isidoro, Commented Aug 1, 2013 at 16:43

Erman Belegu · Accepted Answer · 2013-08-01 17:09:13Z

1

Try this:

VIEW:

<?php
    echo "<h2>View</h2>";
    foreach ($rows as $r){
        echo '<br /><h3>'; echo $r['tHandle']; echo'</h3>'; // EDIT THIS LINE
        echo "<li>Sent at:  "; echo $r['content']; echo"</li><br />";
        echo'<li>Created:  '; echo $r['created']; echo'</li><br />';
    }
?>

MODEL:

<?php
class Tweets_model extends CI_Model{

    public function __construct() {

        $this->load->database();

    }

    public function getTweetDetails($id){
    $this->db->select('*')
    $this->db->from('tweets')
    $this->db->where('tweetId', $id);
    return $this->db->get()->result_array();

    }
}
?>

CONTROLLER:

<?php

class Tweets extends CI_Controller{

    public function __construct() {
        parent::__construct();
        $this->load->model('tweets_model');
    }

    public function details(){

        $data['rows'] = $this->tweets_model->getTweetDetails($this->uri->segment(2));
        $this->load->view('header');
        $this->load->view('tweet_details', $data);
        $this->load->view('footer');
    }
}

?>

UPDATE:

Segments are numbered from left to right. For example, if your full URL is this:

http://example.com/index.php/news/local/metro/crime_is_up

The segment numbers would be this:

$this->uri->segment(1) // with return -> news
$this->uri->segment(2) // with return -> local
$this->uri->segment(3) // with return -> metro
$this->uri->segment(4) // with return -> crime_is_up

In your case $this->uri->segment(2) will return details. And than the query will return 0 rows. Just for testing, you can do like this:

public function details(){
            $tweet_id = 1 // for example
            $data['rows'] = $this->tweets_model->getTweetDetails($tweet_id);
            $this->load->view('header');
            $this->load->view('tweet_details', $data);
            $this->load->view('footer');
        }

Or you can take it from url, for example:

public function details($id){ // Here you will have the $tweet_id

                $data['rows'] = $this->tweets_model->getTweetDetails($this->uri->segment(3));
                $this->load->view('header');
                $this->load->view('tweet_details', $data);
                $this->load->view('footer');
            }

edited Aug 1, 2013 at 17:09

answered Aug 1, 2013 at 16:45

Erman Belegu

4,08928 silver badges39 bronze badges

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

8 Comments

user2642958 Over a year ago

Error message is gone but instead of any of the results I am just getting 'Array()' in the browser.

Erman Belegu Over a year ago

@user2642958, please check the view again... I make one edit please, and tell me the result

Erman Belegu Over a year ago

Do you have data in DB?

user2642958 Over a year ago

Yes. My first view pulls and displays the same data except for one field without a problem.

Erman Belegu Over a year ago

Because $this->uri->segment(2) in your case is null. So when you do the query, you didn't receive results!

|

Rick Calder · Accepted Answer · 2013-08-01 17:03:29Z

0

You need to do some troubleshooting here. All your errors are because the database isn't returning any data.

Ensure that the tweetId is actually in segment 2, the segments start counting after the base_url so if your base url is example.com then segment 2 would be something like example.com/tweet/2 with 2 being the right segment.
Pass a tweetId you KNOW exists to the model in this line:

$data['rows'] = $this->tweets_model->getTweetDetails($this->uri->segment(2));

So something like:

$data['rows'] = $this->tweets_model->getTweetDetails(1);

answered Aug 1, 2013 at 17:03

Rick Calder

18.8k3 gold badges27 silver badges41 bronze badges

3 Comments

user2642958 Over a year ago

right on - I changed it to: $data['rows'] = $this->tweets_model->getTweetDetails($this->uri->segment(3)); and it worked!! Thanks!!

Rick Calder Over a year ago

lol no worries, the uri segments are almost always the issue with stuff like this.

user2642958 Over a year ago

I am brand new to PHP and ci so I'm still making newbie mistakes. Thanks again.

Collectives™ on Stack Overflow

CodeIgniter/PHP Error Message

2 Answers 2

8 Comments

3 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

8 Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related