App\Providers\ErrorHandlerServiceProvider

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	register()	0	24	3

<?php

namespace App\Providers;

use App\Common\JsonException;
use Pimple\Container;

final class ErrorHandlerServiceProvider extends BaseServiceProvider
{
    /**
     * Register error handlers service provider.
     *
     * @param Container $container
     */
    public function register(Container $container)
    {
        $container['errorHandler'] = function (Container $c) {

Avoid variables with short names like $c. Configured minimum length is 3.

            return function($request, $response, $exception) use ($c) {
                $details = (defined('DEBUG_MODE') && DEBUG_MODE == 1) ? $exception->getMessage() : 'Internal server error';
                $e       = new JsonException(null, 500, 'Internal server error', $details);

Avoid variables with short names like $e. Configured minimum length is 3.

                return $c->get('renderer')->jsonApiRender($response, $e->statusCode, $e->encodeError());

It seems like you code against a specific sub-type and not the parent class Pimple\Container as the method get() does only exist in the following sub-classes of Pimple\Container: Slim\Container. Maybe you want to instanceof check for one of these explicitly?

            };
        };

        $container['notAllowedHandler'] = function (Container $c) {
            return function($request, $response, $methods) use ($c) {
                throw new JsonException(null, 405, 'Method Not Allowed', 'Method must be one of: '.implode(', ', $methods));
            };
        };

        $container['notFoundHandler'] = function (Container $c) {
            return function() use ($c) {
                throw new JsonException(null, 404, 'Not found', 'Entity not found');
            };
        };

    }
}