Anax\Controller\weatherApiController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Test Coverage

Coverage

90.91%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	indexAction()	0	34	4

<?php

namespace Anax\Controller;

use Anax\Commons\ContainerInjectableInterface;

use Anax\Commons\ContainerInjectableTrait;

// use Anax\Route\Exception\ForbiddenException;

// use Anax\Route\Exception\NotFoundException;

// use Anax\Route\Exception\InternalErrorException;

/**

 * A sample controller to show how a controller class can be implemented.

 */

class weatherApiController implements ContainerInjectableInterface

{

    use ContainerInjectableTrait;

    /**

     * This is the index method action, it handles:

     * ANY METHOD mountpoint

     *

     * @return array

     */

    public function indexAction() : array

    {

        $weather = $this->di->get("weather");

        $search = $this->di->request->getGet("search");

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

        $when = $this->di->request->getGet("when")  ?? "future";


        $ip = $this->di->request->getGet("search");

        $validator = new \Anax\Model\ipValidation;

        $res = $validator->toJson($ip);


        if ($res["valid"] == "valid IP"){

            $locationInfo = $weather->getLocationData($res["city"]);

        } else {

            if (!isset($search)){

                $locationInfo = $weather->getLocationData("");

            } else{

                $locationInfo = $weather->getLocationData($search);

            }

        }

        $lon = $locationInfo["lon"] ?? null;

        $lat = $locationInfo["lat"] ?? null;

        if (!isset($locationInfo["error"])){


            $weatherInfo = $weather->getWeatherMultiCurl($when,$lon,$lat);

        } else {

            $weatherInfo = ["error" => "that is not a city"];

        }

        $data = [

            "location" => $search ?? null,

            "lon" => $lon ?? null,

            "lat" => $lat ?? null,

            "weatherInfo" => $weatherInfo,

        ];

        return [$data];

    }

}