mahm17 / ramverk1-module

src/Weather/WeatherController.php

<?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,
        ]);
    }
}