Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Slim controller issue : must be an instance of ContainerInterface, instance of Slim\\Container given

I am trying to use controller in Slim however keep getting the error

PHP Catchable fatal error: Argument 1 passed to
TopPageController::__construct() must be an instance of ContainerInterface,
instance of Slim\Container given

My index.php

<?php
use \Psr\Http\Message\ServerRequestInterface as Request;
use \Psr\Http\Message\ResponseInterface as Response;

require '../vendor/autoload.php';
require 'settings.php';

spl_autoload_register(function ($classname) {
    require ("../classes/" . $classname . ".php");
});

$app = new \Slim\App(["settings" => $config]);
$app->get('/', function (Request $request, Response $response) {
    $response->getBody()->write("Welcome");
    return $response;
});
$app->get('/method1', '\TopPageController:method1');
$app->run();
?>

My TopPageController.php

<?php
class TopPageController {
   protected $ci;
   //Constructor
   public function __construct(ContainerInterface $ci) {
       $this->ci = $ci;
   }

   public function method1($request, $response, $args) {
        //your code
        //to access items in the container... $this->ci->get('');
        $response->getBody()->write("Welcome1");
        return $response;
   }

   public function method2($request, $response, $args) {
        //your code
        //to access items in the container... $this->ci->get('');
        $response->getBody()->write("Welcome2");
        return $response;
   }

   public function method3($request, $response, $args) {
        //your code
        //to access items in the container... $this->ci->get('');
        $response->getBody()->write("Welcome3");
        return $response;
   }
}
?>

Thanks. I am using Slim 3.

like image 838
lost111in Avatar asked Jun 19 '16 10:06

lost111in


2 Answers

Based on a later change in Slim3 (from version 3.12.2 to 3.12.3) a slightly different ContainerInterface is required. This changes \Interop\ to \Psr\. Add the following on top of you code or change the existing use:

use Psr\Container\ContainerInterface;

Or change the constructor:

public function __construct(\Psr\Container\ContainerInterface $container)
{
    $this->container = $container;
}
like image 105
Trendfischer Avatar answered Oct 22 '22 23:10

Trendfischer


Your code seems to be based on the Slim 3 documentation at http://www.slimframework.com/docs/objects/router.html which does not contain all the required code to avoid the Exception being thrown.

You basically have two options to make it work.

Option 1:

Import the namespace in your index.php, just like it is being done for Request and Response:

use \Interop\Container\ContainerInterface as ContainerInterface;

Option 2:

Change the constructor of the TopPageController to:

public function __construct(Interop\Container\ContainerInterface $ci) {
    $this->ci = $ci;
}

TL;DR

The reason the Exception is thrown is that the Slim\Container class is using the Interop\Container\ContainerInterface interface (see the source code):

use Interop\Container\ContainerInterface;

Since Slim\Container is extending Pimple\Container, the following should all be valid (working) type declarations for your controller's method:

public function __construct(Pimple\Container $ci) {
    $this->ci = $ci;
}

...or even...

public function __construct(ArrayAccess $ci) {
    $this->ci = $ci;
}
like image 17
Werner Avatar answered Oct 22 '22 22:10

Werner