Kunstmaan\Skylab\Skeleton\StatusCakeSkeleton

Complexity

Total Complexity

28

Size/Duplication

Total Lines	190
Duplicated Lines	11.58 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	6
Bugs	2	Features	0

17 Methods

Rating	Name	Duplication	Size	Complexity
A	getName()	0	4	1
A	create()	0	3	1
A	preMaintenance()	0	3	1
A	postMaintenance()	0	3	1
A	maintenance()	13	13	4
A	getCheckName()	0	3	1
A	getTestArray()	0	20	4
A	preBackup()	0	3	1
A	postBackup()	0	3	1
A	preRemove()	0	3	1
A	dependsOn()	0	4	1
A	getTestId()	0	11	2
A	createTest()	0	5	1
A	updateTest()	0	5	1
A	deleteTest()	0	5	1
B	performAPICall()	0	24	3
A	postRemove()	9	9	3

<?php
namespace Kunstmaan\Skylab\Skeleton;

/**
 * StatusCakeSkeleton
 */
class StatusCakeSkeleton extends AbstractSkeleton
{

    const NAME = "statuscake";

    /**
     * @return string
     */
    public function getName()
    {
        return self::NAME;
    }

    /**
     * @param \ArrayObject $project
     *
     * @return mixed
     */
    public function create(\ArrayObject $project)
    {
    }

    /**
     * @return mixed
     */
    public function preMaintenance()
    {
    }

    /**
     * @return mixed
     */
    public function postMaintenance()
    {
    }

    /**
     * @param \ArrayObject $project
     *
     * @return mixed
     */
    public function maintenance(\ArrayObject $project)

This method seems to be duplicated in your project.

    {
        if (isset($this->app["config"]["statuscake"]["enabled"]) && $this->app["config"]["statuscake"]["enabled"]){
            $id = $this->getTestId($project);
            if ($id) {
                $this->updateTest($id, $project);
            } else {
                $this->createTest($project);
            }
        } else {
            $this->dialogProvider->logConfig("Statuscake is disabled");
        }
    }

    private function getCheckName(\ArrayObject $project){
        return $project["name"] . "." . str_replace(".kunstmaan.com", "", $this->app["config"]["webserver"]["hostmachine"]);
    }

    private function getTestId(\ArrayObject $project)
    {
        $tests = $this->performAPICall("Tests/", "GET");
        $found = false;
        array_walk($tests, function ($item, $key) use (&$found, $project) {

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

            if ($item["WebsiteName"] == $this->getCheckName($project)) {
                $found = $item["TestID"];
            }
        });
        return $found;
    }

    private function createTest(\ArrayObject $project)
    {
        $data = $this->getTestArray($project);
        $this->performAPICall("Tests/Update", "PUT", $data);
    }

    private function updateTest($id, \ArrayObject $project)
    {
        $data = $this->getTestArray($project, $id);
        $this->performAPICall("Tests/Update", "PUT", $data);
    }

    private function deleteTest($id)
    {
        $data = $this->getTestArray(null, $id);
        $this->performAPICall("Tests/Details/", "DELETE", $data);
    }

    /**
     * @param \ArrayObject $project

Should the type for parameter $project not be null|\ArrayObject?

     * @return array
     */
    private function getTestArray(\ArrayObject $project = null, $id = false)
    {
        $data = array();
        if (!is_null($project)) {
            $data = array(
                "WebsiteName" => $this->getCheckName($project),
                "Paused" => 0,
                "WebsiteURL" => ($this->skeletonProvider->hasSkeleton($project, $this->skeletonProvider->findSkeleton("ssl"))?"https://":"http://") . $project["url"],
                "CheckRate" => "60",
                "TestType" => "HTTP",
                "WebsiteHost" => $this->app["config"]["webserver"]["hostmachine"],
                "ContactGroup" => 22636,
                "TestTags" => $this->app["config"]["webserver"]["hostmachine"]
            );
        }
        if ($id) {
            $data["TestID"] = $id;
        }
        return $data;
    }

    private function performAPICall($url, $method, $data = null)
    {

        // The data to insert
        $API = $this->app["config"]["statuscake"]["apikey"];
        $Username = $this->app["config"]["statuscake"]["username"];

        // Create the CURL String
        $ch = curl_init("https://www.statuscake.com/API/" . $url);
        curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
        curl_setopt($ch, CURLOPT_CUSTOMREQUEST, $method);
        if ($data) {
            curl_setopt($ch, CURLOPT_POSTFIELDS, http_build_query($data));
        }
        curl_setopt($ch, CURLOPT_HTTPHEADER, array(
            "API: " . $API,
            "Username: " . $Username
        ));

        $response = curl_exec($ch);
        $response = json_decode($response, true);
        $this->dialogProvider->logConfig("Sent a " . $method . " call to " . "https://www.statuscake.com/API/" . $url . " and got a response: " . ($method == "GET"?"with " . sizeof($response) . " checks":$response["Message"]));
        return $response;
    }

    /**
     * @param \ArrayObject $project
     *
     * @return mixed
     */
    public function preBackup(\ArrayObject $project)
    {
    }

    /**
     * @param \ArrayObject $project
     *
     * @return mixed
     */
    public function postBackup(\ArrayObject $project)
    {
    }

    /**
     * @param \ArrayObject $project
     *
     * @return mixed
     */
    public function preRemove(\ArrayObject $project)
    {
    }

    /**
     * @param \ArrayObject $project
     *
     * @return mixed
     */
    public function postRemove(\ArrayObject $project)

This method seems to be duplicated in your project.

    {
        if (isset($this->app["config"]["statuscake"]["enabled"]) && $this->app["config"]["statuscake"]["enabled"]) {
            $id = $this->getTestId($project["name"]);
            $this->deleteTest($id);
        } else {
            $this->dialogProvider->logConfig("Statuscake is disabled");
        }
    }

    /**
     * @return string[]
     */
    public function dependsOn()
    {
        return array("base", "apache");
    }

}