Anax\Weather\WeatherController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Test Coverage

Coverage

79.31%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	initialize()	0	3	1
A	getQuery()	0	6	1
A	indexAction()	0	22	2
A	getInfo()	0	12	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 WeatherController implements ContainerInjectableInterface
{
    use ContainerInjectableTrait;


    // private $ipHelper;

    public function initialize()
    {
        $this->GetLocation = new Weatherhelper();

The property GetLocation does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }


    public function indexAction() : object
    {
        $title = "Weather";
        $page = $this->di->get("page");

        $weatherModel = $this->di->get("weatherhelper");


        if ($this->di->get("request")->hasGet("ipaddress")) {
            $session = $this->di->get("session");
            $session->set("ipaddress", $this->di->get("request")->getGet("ipaddress"));
            // $userContainer = new Weatherhelper();

            $data = $weatherModel->getUsersThroughMultiCurl([30], $this->getQuery());

            $ipInfo = $this->getInfo();
            $page->add("weather/weather-result", $ipInfo);
            $page->add("weather/weather-past", $data);
        }
        $page->add("weather/weather-form", []);
        return $page->render([
            "title" => $title,
        ]);
    }

    public function getInfo()
    {
        $session = $this->di->get("session");

        $weatherModel = $this->di->get("weatherhelper");

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

        $ipInfo = [];

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

        $ipAddress = $session->get("ipaddress");

        $weatherModel = $this->di->get("weatherhelper");
        $ipInfo = $weatherModel->getLocation($ipAddress);

        return $ipInfo;
    }

    public function getQuery()
    {
        $session = $this->di->get("session");
        $ipAddress = $session->get("ipaddress");

        return $ipAddress;
    }
}