Using $this when not in object context, when using call_user_func_array()

Question

I am having a hard time trying to figure the error in the code. There some similar issues on SO, but they did not help that much with my specific problem. I also googled every possible phrases regarding the error, yet still no joy. In ProductCategoryController.php I have:

namespace App\controllers\admin;


use App\classes\CSRFToken;
use App\classes\Request;
use App\classes\ValidateRequest;
use App\models\Category;

class ProductCategoryController
{
    public $table_name = 'categories';
    public $categories;
    public $links;

    public function __construct()
    {
        $obj = new Category();
        $total = Category::all()->count(); // total number of rows
        list($this->categories, $this->links) = paginate(3, $total, $this->table_name, $obj);
    }

    public function show() {
        return view('admin/products/categories',
            [
                'categories' => $this->categories,
                'links' => $this->links
            ]);
    }

}

I get error

using $this when not in object context

on line 27 where I assign 'categories' => $this->categories and 'links' => $this->links

When I try setting 'categories' and 'links' to an empty array, everything work fine as expected.

In the RouteDispatcher.php I have:

Perhaps I may be missing something very obvious, any support with my problem is well appreciated.

How are you calling ProductCategoryController's show method? — Progrock
– Progrock, Commented Feb 17, 2018 at 17:06
Stack in image suggests RouteDispatcher uses call_user_func_array, to call the controller's show method. Would be good to see more of this code. — Progrock
– Progrock, Commented Feb 17, 2018 at 17:14

Progrock · Accepted Answer · 2018-02-17 19:27:55Z

3

In your dispatcher, you are calling your controller's method statically.

In your code you test if your method is callable on a new instance. And then don't go on to reuse that newly created instance when calling. Instead you use the class and method name in call_user_func_array - and therefore call it statically, which results in your error.

Try and change your code to something more like this:

$controller = new $this->controller;
$method     = $this->method;

if(is_callable(array($controller, $method)))
    call_user_func_array(array($controller, $method), $params);

Or move the new:

if(is_callable(array($this->controller, $this->method)))
    call_user_func_array(
        array(new $this->controller, $this->method),
        $params
    );

edited Feb 17, 2018 at 19:27

answered Feb 17, 2018 at 17:52

Progrock

7,4831 gold badge21 silver badges25 bronze badges

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

1 Comment

Baba Zannah Over a year ago

You are a king @Progrock. Thank you so so very much. Thank you.

Collectives™ on Stack Overflow

Using $this when not in object context, when using call_user_func_array()

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related