Anax\Weather\WeatherController

Complexity

Total Complexity

2

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

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

<?php

namespace Anax\Weather;

use Anax\Commons\ContainerInjectableInterface;
use Anax\Commons\ContainerInjectableTrait;

class WeatherController implements ContainerInjectableInterface
{
    use ContainerInjectableTrait;

    public function indexAction() : object
    {
        $title = "Väder data";

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

        $page->add("weather/weather");

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

    public function validateAction() : object
    {
        $title = "Väder";
        $this->api = require ANAX_INSTALL_PATH . "/config/keys.php";

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

        $this->address = $this->di->get("request")->getPost("address");

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

        $page = $this->di->get("page");
        $curl = $this->di->get("curl");

        $curl->getIpData($this->address, $this->api["ip_key"]);
        $curl->getWeatherData($this->api["weather_key"]);

        $resOne = $curl->weatherData;
        $resTwo = $curl->ipData;
        $map = $curl->map;

        $page->add("weather/weather_data", [
            "resOne" => $resOne,
            "resTwo" => $resTwo,
            "map" => $map,
        ]);

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