heidipatja / ramverk1-weather-module

config/di/ipstack.php

<?php
/**
 * Configuration file for IPStack
 */
return [
    // Services to add to the container.
    "services" => [
        "ipstack" => [
            "shared" => true,
            "callback" => function () {
                $ipstack = new \Hepa19\IPGeo\IPStack();
                $curl = new \Hepa19\Curl\Curl();
                $cfg = $this->get("configuration");

The variable $this seems to be never defined.

                if (file_exists(ANAX_INSTALL_PATH . "/config/api_ip_config.php")) {
                    $config = $cfg->load("api_ip_config.php");
                    $key = $config["config"]["apiKey"] ?? null;
                    $baseUrl = $config["config"]["baseUrl"] ?? null;
                    $ipstack->setCurl($curl);
                    $ipstack->setApiKey($key);
                    $ipstack->setBaseUrl($baseUrl);

It seems like $baseUrl can also be of type null; however, parameter $baseUrl of Hepa19\IPGeo\IPStack::setBaseUrl() does only seem to accept string, maybe add an additional type check?

                }

                return $ipstack;
            }
        ],
    ],
];