ManagesSites - Code Metrics - Inspection of "add start and stop maintenance endpoints" - ohdearapp/ohdear-php-sdk - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 8326c3...53ae2f )

by Freek

created 2020-01-08 15:09 UTC

ManagesSites A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

Metric	Value
wmc	7
lcom	0
cbo	1
dl	0
loc	48
rs	10
c	0
b	0
f	0

7 Methods

Rating	Name	Size	Complexity
A	sites()	7	1
A	site()	6	1
A	siteByUrl()	6	1
A	createSite()	6	1
A	deleteSite()	4	1
A	startSiteMaintenance()	6	1
A	stopSiteMaintenance()	4	1

<?php

namespace OhDear\PhpSdk\Actions;

use OhDear\PhpSdk\Resources\Site;

trait ManagesSites
{
    public function sites(): array
    {
        return $this->transformCollection(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $this->get('sites')['data'],
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            Site::class
        );
    }

    public function site(int $siteId): Site
    {
        $siteAttributes = $this->get("sites/{$siteId}");
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        return new Site($siteAttributes, $this);
    }

    public function siteByUrl(string $siteUrl): Site
    {
        $siteAttributes = $this->get("sites/url/{$siteUrl}");
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        return new Site($siteAttributes, $this);
    }

    public function createSite(array $data): Site
    {
        $siteAttributes = $this->post('sites', $data);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        return new Site($siteAttributes, $this);
    }

    public function deleteSite(int $siteId)
    {
        $this->delete("sites/$siteId");

    }

    public function startSiteMaintenance(int $siteId, int $stopMaintenanceAfterSeconds = 60 * 60)
    {
        $this->post("sites/{$siteId}/start-maintenance", [
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            'stop_maintenance_after_seconds' => $stopMaintenanceAfterSeconds,
        ]);
    }

    public function stopSiteMaintenance(int $siteId)
    {
        $this->post("sites/{$siteId}/stop-maintenance");
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }
}


1			<?php
2
3			namespace OhDear\PhpSdk\Actions;
4
5			use OhDear\PhpSdk\Resources\Site;
6
7			trait ManagesSites
8			{
9			public function sites(): array
10			{
11			return $this->transformCollection(
			0 ignored issues – show Bug introduced 2017-11-03 17:29 UTC by Report Bug Copy Issue Report It seems like `transformCollection()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
12			$this->get('sites')['data'],
			0 ignored issues – show Bug introduced 2017-11-03 17:29 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
13			Site::class
14			);
15			}
16
17			public function site(int $siteId): Site
18			{
19			$siteAttributes = $this->get("sites/{$siteId}");
			0 ignored issues – show Bug introduced 2017-11-03 17:29 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
20
21			return new Site($siteAttributes, $this);
22			}
23
24			public function siteByUrl(string $siteUrl): Site
25			{
26			$siteAttributes = $this->get("sites/url/{$siteUrl}");
			0 ignored issues – show Bug introduced 2018-09-01 11:49 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
27
28			return new Site($siteAttributes, $this);
29			}
30
31			public function createSite(array $data): Site
32			{
33			$siteAttributes = $this->post('sites', $data);
			0 ignored issues – show Bug introduced 2017-11-03 17:29 UTC by Report Bug Copy Issue Report It seems like `post()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
34
35			return new Site($siteAttributes, $this);
36			}
37
38			public function deleteSite(int $siteId)
39			{
40			$this->delete("sites/$siteId");
			0 ignored issues – show Bug introduced 2017-11-03 17:29 UTC by Report Bug Copy Issue Report The method `delete()` does not exist on `OhDear\PhpSdk\Actions\ManagesSites`. Did you maybe mean `deleteSite()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history...
41			}
42
43			public function startSiteMaintenance(int $siteId, int $stopMaintenanceAfterSeconds = 60 * 60)
44			{
45			$this->post("sites/{$siteId}/start-maintenance", [
			0 ignored issues – show Bug introduced 2020-01-08 15:23 UTC by Report Bug Copy Issue Report It seems like `post()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
46			'stop_maintenance_after_seconds' => $stopMaintenanceAfterSeconds,
47			]);
48			}
49
50			public function stopSiteMaintenance(int $siteId)
51			{
52			$this->post("sites/{$siteId}/stop-maintenance");
			0 ignored issues – show Bug introduced 2020-01-08 15:23 UTC by Report Bug Copy Issue Report It seems like `post()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
53			}
54			}
55

ohdearapp / ohdear-php-sdk

Push — master ( 8326c3...53ae2f )

ManagesSites A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

7 Methods

Duplication Side-by-Side

Filter issues like