CSFCloud\Containers\Manager

Complexity

Total Complexity

2

Size/Duplication

Total Lines	9
Duplicated Lines	0 %

Test Coverage

Coverage

80%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	GetContainer()	0	1	1
A	ListContainers()	0	4	1

<?php

namespace CSFCloud\Containers;

use CSFCloud\KeyManager;
use CSFCloud\Resource;
use CSFCloud\Containers\Container;
use Httpful\Request;

class Manager extends Resource {

    public function ListContainers() : array {
        $request = Request::get("https://api.csfcloud.com/container?key=" . urlencode($this->keymanager->GetServerKey()))->expectsText()->send();

The method expectsText() does not exist on Httpful\Request. Since you implemented __call, consider adding a @method annotation.

        $cntlist = json_decode($request->body, true);
        return $cntlist;
    }

    public function GetContainer(string $id) : Container {

The parameter $id is not used and could be removed.

    }

In this branch, the function will implicitly return null which is incompatible with the type-hinted return CSFCloud\Containers\Container. Consider adding a return statement or allowing null as return value.

}