Lefty\Weather\WeatherController

Complexity

Total Complexity

11

Size/Duplication

Total Lines	171
Duplicated Lines	0 %

Test Coverage

Coverage

92.21%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	initialize()	0	4	1
A	indexActionPost()	0	63	5
A	indexActionGet()	0	63	5

<?php

namespace Lefty\Weather;

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.
 * The controller will be injected with $di if implementing the interface
 * ContainerInjectableInterface, like this sample class does.
 * The controller is mounted on a particular route and can then handle all
 * requests for that mount point.
 *
 * @SuppressWarnings(PHPMD.TooManyPublicMethods)
 */
class WeatherController implements ContainerInjectableInterface
{
    use ContainerInjectableTrait;


    /**
     * @var string $db a sample member variable that gets initialised
     */
    private $db = "not active";



    /**
     * The initialize method is optional and will always be called before the
     * target method/action. This is a convienient method where you could
     * setup internal properties that are commonly used by several methods.
     *
     * @return void
     */
    public function initialize(): void
    {
        // Use to initialise member variables.
        $this->db = "active";
    }


    /**
    * This is the index method action, it handles:
    * ANY METHOD mountpoint
    * ANY METHOD mountpoint/
    * ANY METHOD mountpoint/index
    *
    * @return object
    */
    public function indexActionGet(): object
    {
        $request     = $this->di->get("request");
        $iptocheck = $request->getGet("ip") ?? "";

        $weatherInfo = [];

The assignment to $weatherInfo is dead and can be removed.

        $weatherInfoHist = [];

The assignment to $weatherInfoHist is dead and can be removed.

        // Prepare page
        $page = $this->di->get("page");
        $page->add("weather/index");
        $title = "Weather | ramverk1";

        // Validate IP
        $ipAddress = new WeatherIpValidation($iptocheck);
        $data = $ipAddress->answer();

        if (empty($iptocheck)) {
            return $page->render([
                "title" => $title,
            ]);
        } elseif (!$ipAddress->isValid()) {
            $page->add("weather/notvalidIP", $data);
            return $page->render([
                "title" => $title,
            ]);
        }
        // Get IP location
        $geoLocation = new WeatherGeoLocation($iptocheck);

The call to Lefty\Weather\WeatherGeoLocation::__construct() has too many arguments starting with $iptocheck.

        $geoLocation->setDI($this->di);
        $geoLocation->setAPI("ipstack");
        $geoLocation->checkGeoLocation($iptocheck);
        $locationInfo = $geoLocation->getGeoLocation();


        // Get Weather information Forecast
        $weatherRequest = new WeatherRequest();
        $weatherRequest->setDI($this->di);
        $weatherRequest->setAPI("openweathermap");
        $weatherRequest->checkWeather($geoLocation);
        $weatherInfo = (array)$weatherRequest->getWeather();
        
        // Get Weather information Historical Data
        
        $weatherInfoHist = array("weatherInfoHistorical" => $weatherRequest->checkWeatherMulti($geoLocation));
        $weatherInfo = array_merge($weatherInfo, $weatherInfoHist);
        
        // Merge location data with ip data
        $data = array_merge($data, (array)$locationInfo);


        if ($ipAddress->isValid() && $geoLocation->geoLocationOK()) {
            $page->add("weather/validIP", $data);

            $weatherInfo = array_merge($weatherInfo, (array)$weatherInfoHist);
            $page->add("weather/weather", $weatherInfo);
        } else {
            $page->add("weather/notvalidWeather", $data);
        }

        return $page->render([
            "title" => $title,
        ]);
    }

    /**
    * This is the index method action, it handles:
    * ANY METHOD mountpoint
    * ANY METHOD mountpoint/
    * ANY METHOD mountpoint/index
    *
    * @return object
    */
    public function indexActionPost(): object
    {
        $request     = $this->di->get("request");
        $iptocheck = $request->getPost("ip") ?? "";

        $weatherInfo = [];

The assignment to $weatherInfo is dead and can be removed.

        $weatherInfoHist = [];

The assignment to $weatherInfoHist is dead and can be removed.

        // Prepare page
        $page = $this->di->get("page");
        $page->add("weather/index");
        $title = "Weather | ramverk1";

        // Validate IP
        $ipAddress = new WeatherIpValidation($iptocheck);
        $data = $ipAddress->answer();

        if (empty($iptocheck)) {
            return $page->render([
                "title" => $title,
            ]);
        } elseif (!$ipAddress->isValid()) {
            $page->add("weather/notvalidIP", $data);
            return $page->render([
                "title" => $title,
            ]);
        }
        // Get IP location
        $geoLocation = new WeatherGeoLocation($iptocheck);

The call to Lefty\Weather\WeatherGeoLocation::__construct() has too many arguments starting with $iptocheck.

        $geoLocation->setDI($this->di);
        $geoLocation->setAPI("ipstack");
        $geoLocation->checkGeoLocation($iptocheck);
        $locationInfo = $geoLocation->getGeoLocation();


        // Get Weather information Forecast
        $weatherRequest = new WeatherRequest();
        $weatherRequest->setDI($this->di);
        $weatherRequest->setAPI("openweathermap");
        $weatherRequest->checkWeather($geoLocation);
        $weatherInfo = (array)$weatherRequest->getWeather();
        
        // Get Weather information Historical Data
        
        $weatherInfoHist = array("weatherInfoHistorical" => $weatherRequest->checkWeatherMulti($geoLocation));
        $weatherInfo = array_merge($weatherInfo, $weatherInfoHist);
        
        // Merge location data with ip data
        $data = array_merge($data, (array)$locationInfo);


        if ($ipAddress->isValid() && $geoLocation->geoLocationOK()) {
            $page->add("weather/validIP", $data);

            $weatherInfo = array_merge($weatherInfo, (array)$weatherInfoHist);
            $page->add("weather/weather", $weatherInfo);
        } else {
            $page->add("weather/notvalidWeather", $data);
        }

        return $page->render([
            "title" => $title,
        ]);
    }
}