Laravel trying to get property of non-object in an object

Question

Simple code will not work! I keep getting the error Trying to get property of non-object on the line with the if statement. I tried accessing this like an array I get a message saying it can't access a stdClass as an Array.

 public function isAllowed($perm) { $cando = 0; $groups = DB::table('group_user')->where('user_id', $this->id)->get(); foreach ($groups as $mygroup) { $group_can = DB::table('group_permission')->where([ ['permission_id', $permission->id], ['group_id', $mygroup->group_id] ])->first(); $setting = $group_can->setting; // Error returns for this line // if ($setting > $cando) { $cando = $setting; } } }

print_r, var_dump, and dd of $group_can give this:

stdClass Object ( [group_id] => 1 [permission_id] => 50 [setting] => 1 ) object(stdClass)#555 (3) { ["group_id"]=> int(1) ["permission_id"]=> int(50) ["setting"]=> int(1) } {#555 ▼ +"group_id": 1 +"permission_id": 50 +"setting": 1 }

Using $setting = $group_can->setting; returns the error Trying to get property of non-object

Using $setting = $group_can['setting']; returns the error Cannot use object of type stdClass as array

The details of the laravel error are:

at HandleExceptions->handleError(8, 'Trying to get property of non-object', '/home/mwsubmissions/public_html/jon/MWSubmissionManager/app/User.php', 91, array('perm' => 'manage.projects', 'cando' => 1, 'groups' => object(Collection), 'permission' => object(stdClass), 'mygroup' => object(stdClass), 'group' => null, 'group_can' => null, 'setting' => 1))

EDIT I removed the first part of the code that I was having errors with and then got to this, another example of the same thing, but using a smaller object and this line is more important than the last was. All details updated.

$group = Group::find($mygroup->group_id); should be $group = Group::find($mygroup->id); — Anar Bayramov
– Anar Bayramov, Commented Sep 13, 2017 at 3:07
@AnarBayramov negative. $mygroup comes from the before mentioned $groups query, which is a pivot table with only 2 columns, group_id and user_id. The $group object is finding the appropriate data, I just can't access it. — JLZenor
– JLZenor, Commented Sep 13, 2017 at 3:11
@user2486 I updated the question with more detail and a dd inside the foreach. I removed the previous line of code and continued on to the next, more important line of code that was returning the same error. — JLZenor
– JLZenor, Commented Sep 13, 2017 at 3:40

Vision Coderz · Accepted Answer · 2017-09-13 03:33:00Z

1

Better you can check isset

public function isAllowed($perm) { $cando = 0; $groups = DB::table('group_user')->where('user_id', $this->id)->get(); if(isset($groups)&&count($groups)>0){ foreach ($groups as $mygroup) { if(isset($mygroup->group_id)){ $group = Group::find($mygroup->group_id); } if (!is_null($group->project_id)) { continue; } } } }

answered Sep 13, 2017 at 3:33

Vision Coderz

8,0786 gold badges40 silver badges56 bronze badges

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

5 Comments

JLZenor Over a year ago

I just tried using isset for $group_can (see my updated question) and it passed, and then I got an error as soon as I tried to access any of the data in $group_can.

Vision Coderz Over a year ago

i can say you are getting data in loop may be any one of the row has no data .can you print_r $group_can) and post result

JLZenor Over a year ago

That information is in the original question, but here it is again: stdClass Object ( [group_id] => 1 [permission_id] => 50 [setting] => 1 )

JLZenor Over a year ago

This was the final solution... In the for loop the first loop would go through, which is why my print_r worked. But the second iteration through is where it failed, when the $group_can query failed to return a column. Dang... 2 hours wasted on this simple thing.

Vision Coderz Over a year ago

glad to here answer is helped you.

flauntster · Accepted Answer · 2017-09-13 03:09:22Z

0

The property you're trying to access is an element of an array 'attributes', so try:

public function isAllowed($perm) { $cando = 0; $groups = DB::table('group_user')->where('user_id', $this->id)->get(); foreach ($groups as $mygroup) { $group = Group::find($mygroup->group_id); if (!is_null($group->attributes['project_id'])) { continue; } } }

Hope this helps :)

answered Sep 13, 2017 at 3:09

flauntster

2,01614 silver badges20 bronze badges

4 Comments

JLZenor Over a year ago

Nope. Just tried that and got the same Trying to get property of non-object error at the same line.

flauntster Over a year ago

Sorry I just noticed the attributes array is protected.. check the class to see if there is a getter method to retrieve attributes (eg $group->getAttributes()) :)

JLZenor Over a year ago

Throughout every DB query in this application with Laravel I have accessed the same data in the same way. I cannot figure out why this one is different...

flauntster Over a year ago

Weird, if you're accessing protected members from outside of that class. You could cast the object as an array and then access the protected members, as shown here: stackoverflow.com/a/27754169/3106662

Oyewole A. Samuel · Accepted Answer · 2017-09-13 03:37:45Z

First, ensure you add "project_id" to $fillable array in Group Model
Try this

public function isAllowed($perm) { $cando = 0; $groups = DB::table('group_user')->where('user_id', $this->id)->get(); foreach ($groups as $mygroup) { $group = Group::whereId($mygroup->group_id)->first(); if (!is_null($group->attributes['project_id'])) { continue; } } }

Collectives™ on Stack Overflow

Laravel trying to get property of non-object in an object

3 Answers 3

5 Comments

4 Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

5 Comments

4 Comments

Comments

Linked

Related