CODEWITHSUNDEEP
cakephppagination
klark
klark

Reputation: 494

Cakephp Paginate Find

I want to list the posts of a given user. It work but paginate is not accurate.

My code is the following

public function index($userid = null) {

     if ($this->Post->exists($userid)) {

        $this->set('posts',$this->Post->find('all',array('conditions'=>array('user_id'=>$userid))), 
            $this->paginate());

        } else
        {

        $this->Post->recursive = 0;
        $this->set('posts', $this->paginate());
    }

The result give the correct list --> 3 posts, but the paginator display page number 1 and 2

Can you help me? Thank you

Upvotes: 1

Views: 5114

Answers (3)

bowlerae
bowlerae

Reputation: 944

First off, you're checking to see if the post exists but using the $userid. Are you trying to see "if the user exists, get the posts for that user, or else get posts for ALL users"? As you have it right now, say you have the $userid = 159, but the max Post.id in your database is 28, then the condition is not being met because it is checking to see whether or not there is a Post with the id = 159 that exists, which it doesn't.

Second, your conditions are wrong. You are performing a find and then a paginate which are two separate queries. The conditions are being implemented on the find query but not the paginate but you are only displaying the find results.

public function index($userid = null) {
    // setting recursive outside of if statement makes it applicable either way
    $this->Post->recursive = 0;

    // check if user exists
    if ($this->Post->User->exists($userid)) {
        // get posts for user
        $this->set('posts', $this->paginate('Post', array('Post.user_id' => $userid));
    }
    else{
        // get all posts
        $this->set('posts', $this->paginate('Post'));
    }    

} // end index function

Upvotes: 0

AD7six
AD7six

Reputation: 66169

Refer to the documentation

The code in the question is quite confused.

find

The find method only has two parameters:

find(string $type = 'first', array $params = array())

The third parameter (the result of calling paginate) isn't used and will be ignored - but it will setup the view variables for the pagination helper, based on the conditions used in the paginate call - there are no conditions being used.

It is not possible to paginate the result of a find call - to do so restructure the code to call paginate instead of find.

paginate

The paginate method is just a proxy for the paginator component - it can be used in several ways, this one (controller code example):

$this->paginate($conditions)

Is the most appropriate usage for the case in the question i.e. the complete action code should be similar to:

public function index($userId = null) {
    $conditions = array();
    if ($userId) {
        $conditions['Post.user_id'] = $userId;
    }
    $this->set('posts',$this->paginate($conditions));
}

Note that logically, if a user id is requested that doesn't exist the response should be nothing - not everything.

Upvotes: 3

johhniedoe
johhniedoe

Reputation: 173

I'm quite sure that conditions for paginate do now work that way.

If you want to set conditions for paginations you should do it as follows:

$this->paginate = array('conditions' => array('Post.user_id' => $userid)));
$this->set('posts', $this->paginate());

And yes, the result stored in $posts ( in view ) will be proper as you assigned proper find result to it, meanwhile you've paginated post model without any conditions whatsoever.

Upvotes: 1

Related Questions