Ps2alerts\Api\Controller\Endpoint\Admin\AdminEndpointController

Complexity

Total Complexity

3

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	createAlert()	0	11	1
A	errorTest()	0	5	1

<?php

namespace Ps2alerts\Api\Controller\Endpoint\Admin;

use Ps2alerts\Api\Controller\Endpoint\AbstractEndpointController;
use Psr\Http\Message\ServerRequestInterface;
use Psr\Http\Message\ResponseInterface;

class AdminEndpointController extends AbstractEndpointController
{
    /**
     * Construct
     *
    */
    public function __construct()
    {

    }

    public function createAlert(ServerRequestInterface $request, ResponseInterface $response, array $args)

The parameter $response is not used and could be removed.

The parameter $args is not used and could be removed.

    {
        $world = $request->get('world');

The method get() does not seem to exist on object<Psr\Http\Message\ServerRequestInterface>.

        $zone = $request->get('zone');

The method get() does not seem to exist on object<Psr\Http\Message\ServerRequestInterface>.

        $apikey = $request->get('apikey');

The method get() does not seem to exist on object<Psr\Http\Message\ServerRequestInterface>.

$apikey is not used, you could remove the assignment.

        $user = $request->get('user');

The method get() does not seem to exist on object<Psr\Http\Message\ServerRequestInterface>.

$user is not used, you could remove the assignment.

        var_dump($world);

var_dump($world); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

        var_dump($zone);
        var_dump($headers);

The variable $headers does not exist. Did you forget to declare it?

    }

    public function errorTest(ServerRequestInterface $request, ResponseInterface $response, array $args)

The parameter $request is not used and could be removed.

The parameter $response is not used and could be removed.

The parameter $args is not used and could be removed.

    {
        throw new \Exception('Test');
        trigger_error('Test');

trigger_error('Test'); does not seem to be reachable.

    }
}