Moody\weather_and_position\indexs

Complexity

Total Complexity

1

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	indexAction()	0	15	1

<?php

namespace Moody\weather_and_position;

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 indexs implements ContainerInjectableInterface
{
    use ContainerInjectableTrait;

    protected $ipAddress;
    protected $object;


    public function indexAction() : object
    {
        $host = null;

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

        $title = "Ip validator";
        $page = $this->di->get("page");
        $request = $this->di->get("request");

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

        $this->object = new IpValidate();

        $this->openWeatherMapModel = $this->di->get("openWeatherMap");

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

        $host = $this->object->getDomain($this->ipAddress);
        $data["host"] = $host;

$data was never initialized. Although not strictly required by PHP, it is generally a good practice to add $data = array(); before regardless.

        $page->add("weather/index", $data);
        return $page->render([
            "title" => $title,
        ]);
    }
}