Anax\Weather\JsonApiController

Complexity

Total Complexity

1

Size/Duplication

Total Lines	14
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	indexAction()	0	10	1

<?php

namespace Anax\Weather;

use Anax\Commons\ContainerInjectableInterface;
use Anax\Commons\ContainerInjectableTrait;
use Anax\Route\Exception\NotFoundException;

/**
 * A controller to ease with development and debugging information.
 */
class JsonApiController implements ContainerInjectableInterface
{
    use ContainerInjectableTrait;

    public function indexAction() : array
    {
        $city = $this->di->request->getGet("city");

Accessing request on the interface Psr\Container\ContainerInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        $cnt = $this->di->request->getGet("cnt");
        $weatherModel = $this->di->get("weatherhelper");
        $days = explode(" ", $cnt);

        $info = $weatherModel->getUsersThroughMultiCurl($days, $city);

        return [$info];
    }
}