Laravel Error Calling Member Function

Question

I followed some tutorial on http://laravel.com/docs/ and it just won't work. Here's what I tried:

Role model:

<?php

class Role extends Eloquent {

    protected $table = 'accounts';

    public function users()
    {
        return $this->belongsToMany('User');
    }
}

RoleController.php

<?php

class RoleController extends BaseController {

    public function get_role()
    {
        $roles = User::find(16)->group_id;

        if ($roles->contains(3))
        {
            echo 'this works';
        }
        else
        {
            return Redirect::to('news/index');
        }
    }
}

And my route:

Route::get('dash', 'RoleController@get_role');

And here's my error:

Call to a member function contains() on a non-object

It says the error is on line:

if ($roles->contains(3))

Like the contains method is... I don't know hah.

Also, $roles = User::find(16)->group_id; The 16 is the id of the account, right? And the group_id is the table?

Thanks in advance.

EDIT:

Here's the solution, I've changed it in RoleController.php:

<?php

class RoleController extends BaseController {

    public function get_role()
    {
        $roles = User::find(16)->group_id;

        if (if ($roles == '1')
        {
            echo 'this works';
        }
        else
        {
            return Redirect::to('news/index');
        }
    }
}

I did it like this: class RoleController extends BaseController { public function get_role() { $roles = User::find(16)->group_id; print_r($roles); } } and it just displays 1 :p Also I tried with the contain method to set it to 1, but it still does that error. — dynamitem
– dynamitem, Commented Aug 2, 2013 at 13:33
Your script expects $roles to be an object, as shown by your debugging, $roles contains 1 (not an object) which is why you're getting the error. — ɴᴀᴛᴇ ᴅᴇᴠ
– ɴᴀᴛᴇ ᴅᴇᴠ, Commented Aug 2, 2013 at 13:35
Aha, got it. But how could I make it a non-object, or whatever does it take to make it works? Should change the method or something? — dynamitem
– dynamitem, Commented Aug 2, 2013 at 13:38
Try changing public function users() (plural) to public function user() (singular) in your Role class — ɴᴀᴛᴇ ᴅᴇᴠ
– ɴᴀᴛᴇ ᴅᴇᴠ, Commented Aug 2, 2013 at 13:42

Laurence · Accepted Answer · 2013-08-02 13:45:49Z

1

This line:

$roles = User::find(16)->group_id;

means you are asking "find user 16, and give me the group_id of that user". This is the "1" that you refer to (in your comments).

Since you have the group_id - change

if ($roles->contains(3))

to

if ($roles == '3')

answered Aug 2, 2013 at 13:45

Laurence

60.3k22 gold badges178 silver badges215 bronze badges

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

Comments

dynamitem · Accepted Answer · 2013-08-02 14:58:39Z

0

here's the solution, I've changed it in RoleController.php:

    <?php

class RoleController extends BaseController {

    public function get_role() {
$roles = User::find(16)->group_id;

            if (if ($roles == '1')
             {
               echo 'this works';
             } else {

               return Redirect::to('news/index');
                    }
             }
}

edited Aug 2, 2013 at 14:58

answered Aug 2, 2013 at 13:50

dynamitem

1,6796 gold badges25 silver badges46 bronze badges

1 Comment

Nunser Over a year ago

Could you update your answer to include where you change that code? A bigger snippet (I know you can infer it by reading the question, but it's better if the answer is self-sufficient)

Collectives™ on Stack Overflow

Laravel Error Calling Member Function

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related