PHP nested function gives 'Using $this when not in object context' error

Question

I am trying to filter an array using the built-in array_filter function. According to the examples in PHP manual, I have to provide the name of the callback function. So, here is what I tried:

// This function is a method 
public function send($user_id, $access, $content, $aids)
{
    // check if user is allowed to see the attachments
    function is_owner($var)
    {
        return $this->attachment_owner($var, $user_id);
    }

    $aids = implode(';', array_filter(explode(';', $aids), 'is_owner'));
}

Here is the error I'm getting:

Fatal error: Using $this when not in object context in filename line number.

How to solve this?

Use anonymous functions: array_filter(..., function($var) { ... }); (Sorry, didn't really understand the problem first.) — Niko Hujanen
– Niko Hujanen, Commented Sep 4, 2013 at 16:56
$this is no longer within scope. Out of curiosity, why make a nested function here and not just a private function in the class? — Kai Qing
– Kai Qing, Commented Sep 4, 2013 at 16:58
The "nested" function ends up in the global scope; you can't use $this inside of it. — lafor
– lafor, Commented Sep 4, 2013 at 16:59
i have similar issue, but i can't make it anonymous because it is used many times. and i don't want it to be a method because it's only relevant to one specific method. any ideas? — oriadam
– oriadam, Commented Jun 23, 2020 at 9:57

Community · Accepted Answer · 2017-05-23 11:52:43Z

3

You could avoid using the nested function by making it a member of your class

... inside a class
function is_owner($var)
{
    return $this->attachment_owner($var, $user_id);
}

public function send($user_id, $access, $content, $aids)
{
// check if user is allowed to see the attachments


$aids = implode(';', array_filter(explode(';', $aids), array($this, 'is_owner')));
...

References:

What are nested functions for

Array_filter in the context of an object, with private callback

edited May 23, 2017 at 11:52

CommunityBot

11 silver badge

answered Sep 4, 2013 at 16:59

tim

3,3792 gold badges17 silver badges17 bronze badges

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

Comments

ComFreek · Accepted Answer · 2013-09-04 17:03:14Z

2

Since you're using PHP > 5.4.0, you could create an anonymous function and even use $this:

public function send($user_id, $access, $content, $aids)
{   
    // check if user is allowed to see the attachments
    $is_owner = function ($var)
    {
        return $this->attachment_owner($var, $user_id);
    }
    $aids = implode(';', array_filter(explode(';', $aids), $is_owner));

Nevertheless I would go with tim's solution because it is cleaner.

answered Sep 4, 2013 at 17:03

ComFreek

29.5k18 gold badges107 silver badges157 bronze badges

Comments

Niko Hujanen · Accepted Answer · 2013-09-04 17:02:49Z

1

One solution is to use anonymous functions:

$aids = implode(';', array_filter(explode(';', $aids), function($var) { return $this->attachment_owner($var, $user_id); }));

answered Sep 4, 2013 at 17:02

Niko Hujanen

7335 silver badges12 bronze badges

Collectives™ on Stack Overflow

PHP nested function gives 'Using $this when not in object context' error

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related