ManagesMaintenancePeriods::stopSiteMaintenance() - Code Metrics - SierraTecnologia/integrations - Measure and Improve Code Quality continuously with Scrutinizer

ManagesMaintenancePeriods::stopSiteMaintenance() A
last analyzed 2021-05-27 22:55 UTC

↳ Parent: ManagesMaintenancePeriods

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	1
dl	0
loc	4
rs	10
c	0
b	0
f	0

<?php

namespace Integrations\Traits\Actions;

use Integrations\Resources\MaintenancePeriod;

trait ManagesMaintenancePeriods
{
    /**
     * @param int $siteId
     *
     * @return array
     */
    public function maintenancePeriods(int $siteId): array
    {
        return $this->transformCollection(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $this->get("sites/{$siteId}/maintenance-periods")['data'],
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            MaintenancePeriod::class
        );
    }

    /**
     * @param int $siteId
     * @param int $stopMaintenanceAfterSeconds Stops after one hour by default
     *
     * @return MaintenancePeriod
     */
    public function startSiteMaintenance(int $siteId, int $stopMaintenanceAfterSeconds = 60 * 60): MaintenancePeriod
    {
        $attributes = $this->post(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            "sites/{$siteId}/start-maintenance", [
            'stop_maintenance_after_seconds' => $stopMaintenanceAfterSeconds,
            ]
        );

        return new MaintenancePeriod($attributes, $this);
    }

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

    /**
     * @param int    $siteId
     * @param string $startsAt Y:m:d H:i
     * @param string $endsAt   Y:m:d H:i
     *
     * @return MaintenancePeriod
     */
    public function createSiteMaintenance(int $siteId, string $startsAt, string $endsAt): MaintenancePeriod
    {
        $payload = [
            'site_id' => $siteId,
            'starts_at' => $startsAt,
            'ends_at' => $endsAt,
        ];

        $attributes = $this->post('maintenance-periods', $payload);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        return new MaintenancePeriod($attributes, $this);
    }

    /**
     * @param int $maintenancePeriodId
     */
    public function deleteSiteMaintenance(int $maintenancePeriodId)
    {
        $this->delete("maintenance-periods/{$maintenancePeriodId}");

    }
}


1			<?php
2
3			namespace Integrations\Traits\Actions;
4
5			use Integrations\Resources\MaintenancePeriod;
6
7			trait ManagesMaintenancePeriods
8			{
9			/**
10			* @param int $siteId
11			*
12			* @return array
13			*/
14			public function maintenancePeriods(int $siteId): array
15			{
16			return $this->transformCollection(
			0 ignored issues – show Bug introduced 2020-07-28 00:54 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...
17			$this->get("sites/{$siteId}/maintenance-periods")['data'],
			0 ignored issues – show Bug introduced 2020-07-28 00:54 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...
18			MaintenancePeriod::class
19			);
20			}
21
22			/**
23			* @param int $siteId
24			* @param int $stopMaintenanceAfterSeconds Stops after one hour by default
25			*
26			* @return MaintenancePeriod
27			*/
28			public function startSiteMaintenance(int $siteId, int $stopMaintenanceAfterSeconds = 60 * 60): MaintenancePeriod
29			{
30			$attributes = $this->post(
			0 ignored issues – show Bug introduced 2020-09-19 14: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...
31			"sites/{$siteId}/start-maintenance", [
32			'stop_maintenance_after_seconds' => $stopMaintenanceAfterSeconds,
33			]
34			);
35
36			return new MaintenancePeriod($attributes, $this);
37			}
38
39			/**
40			* @param int $siteId
41			*/
42			public function stopSiteMaintenance(int $siteId)
43			{
44			$this->post("sites/{$siteId}/stop-maintenance");
			0 ignored issues – show Bug introduced 2020-07-28 00:54 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...
45			}
46
47			/**
48			* @param int $siteId
49			* @param string $startsAt Y:m:d H:i
50			* @param string $endsAt Y:m:d H:i
51			*
52			* @return MaintenancePeriod
53			*/
54			public function createSiteMaintenance(int $siteId, string $startsAt, string $endsAt): MaintenancePeriod
55			{
56			$payload = [
57			'site_id' => $siteId,
58			'starts_at' => $startsAt,
59			'ends_at' => $endsAt,
60			];
61
62			$attributes = $this->post('maintenance-periods', $payload);
			0 ignored issues – show Bug introduced 2020-07-28 00:54 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...
63
64			return new MaintenancePeriod($attributes, $this);
65			}
66
67			/**
68			* @param int $maintenancePeriodId
69			*/
70			public function deleteSiteMaintenance(int $maintenancePeriodId)
71			{
72			$this->delete("maintenance-periods/{$maintenancePeriodId}");
			0 ignored issues – show Bug introduced 2020-07-28 00:54 UTC by Report Bug Copy Issue Report The method `delete()` does not exist on `Integrations\Traits\Acti...nagesMaintenancePeriods`. Did you maybe mean `deleteSiteMaintenance()`? 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...
73			}
74			}
75

SierraTecnologia / integrations

ManagesMaintenancePeriods::stopSiteMaintenance() A last analyzed 2021-05-27 22:55 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ManagesMaintenancePeriods::stopSiteMaintenance() A
last analyzed 2021-05-27 22:55 UTC