ArticleResource::publish() - Code Metrics - Inspection of "feat: add articles resource" - signifly/laravel-shopify - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#16)

unknown

created 2021-10-06 13:28 UTC

ArticleResource::publish() A

↳ Parent: ArticleResource

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

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

<?php

namespace Signifly\Shopify\REST\Resources;

class ArticleResource extends ApiResource
{
    public function update(array $data): self
    {
        return $this->shopify->updateArticle($this->id, $data);

    }

    public function publish(): self
    {
        return $this->update(['published' => true]);
    }

    public function unpublish(): self
    {
        return $this->update(['published' => false]);
    }

    public function delete(): void
    {
        $this->shopify->deleteBlog($this->id);

    }
}


1			<?php
2
3			namespace Signifly\Shopify\REST\Resources;
4
5			class ArticleResource extends ApiResource
6			{
7			public function update(array $data): self
8			{
9			return $this->shopify->updateArticle($this->id, $data);
			0 ignored issues – show Bug Best Practice introduced 2021-10-06 13:30 UTC by Report Bug Copy Issue Report The property `id` does not exist on `Signifly\Shopify\REST\Resources\ArticleResource`. Since you implemented `__get`, consider adding a @property annotation. Loading history...
10			}
11
12			public function publish(): self
13			{
14			return $this->update(['published' => true]);
15			}
16
17			public function unpublish(): self
18			{
19			return $this->update(['published' => false]);
20			}
21
22			public function delete(): void
23			{
24			$this->shopify->deleteBlog($this->id);
			0 ignored issues – show Bug Best Practice introduced 2021-10-06 13:30 UTC by Report Bug Copy Issue Report The property `id` does not exist on `Signifly\Shopify\REST\Resources\ArticleResource`. Since you implemented `__get`, consider adding a @property annotation. Loading history...
25			}
26			}
27

signifly / laravel-shopify

Pull Request — master (#16)

ArticleResource::publish() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like