Route::setPath() - Code Metrics - Inspection of "Allow setRedirectedFrom in Route to be an array" - components-web-app/api-components-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 62e1a3...672be2 )

by Daniel

created 2021-09-07 15:53 UTC

Route::setPath() A

↳ Parent: Route

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
eloc	2
nc	1
nop	1
dl	0
loc	5
ccs	3
cts	3
cp	1
crap	1
rs	10
c	0
b	0
f	0

<?php

/*
 * This file is part of the Silverback API Components Bundle Project
 *
 * (c) Daniel West <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace Silverback\ApiComponentsBundle\Entity\Core;

use ApiPlatform\Core\Annotation\ApiResource;
use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\Common\Collections\Collection;
use Silverback\ApiComponentsBundle\Annotation as Silverback;
use Silverback\ApiComponentsBundle\Entity\Utility\IdTrait;
use Silverback\ApiComponentsBundle\Entity\Utility\TimestampedTrait;
use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntity;
use Symfony\Component\Validator\Constraints as Assert;

/**
 * Although a user will be able to get the routes and the tree of data down to getting the ID for a component
 * fetching a component will be restricted based on the route it is within.
 *
 * @author Daniel West <[email protected]>
 *
 * @Silverback\Timestamped
 * @ApiResource(
 *     mercure=true,
 *     collectionOperations={
 *         "get",
 *         "post",
 *         "generate"={ "method"="POST", "path"="/routes/generate" }
 *     },
 *     itemOperations={
 *         "get"={ "requirements"={"id"="(.+)"}, "security"="is_granted('read_route', object)" },
 *         "delete"={ "requirements"={"id"="(.+)"}, "security"="is_granted('read_route', object)" },
 *         "put"={ "requirements"={"id"="(.+)"}, "security"="is_granted('read_route', object)" },
 *         "patch"={ "requirements"={"id"="(.+)"}, "security"="is_granted('read_route', object)" }
 *     }
 * )
 * @Assert\Expression(
 *     "!(this.getPage() == null & this.getPageData() == null)",
 *     message="Please specify either page or pageData."
 * )
 * @Assert\Expression(
 *     "!(this.getPage() != null & this.getPageData() != null)",
 *     message="Please specify either page or pageData, not both."
 * )
 * @UniqueEntity("name", message="The route name must be unique.")
 * @UniqueEntity("path", message="The route path must be unique.")
 */
class Route
{
    use IdTrait;
    use TimestampedTrait;

    /**
     * @Assert\NotBlank
     */
    private string $path = '';

    /**
     * @Assert\NotNull
     */
    private string $name;

    private ?Route $redirect = null;

    private Collection $redirectedFrom;

    private ?Page $page = null;

    private ?AbstractPageData $pageData = null;

    public function __construct()
    {
        $this->redirectedFrom = new ArrayCollection();
    }

    public function getPath(): string
    {
        return $this->path;
    }

    public function setPath(string $path): self
    {
        $this->path = $path;

        return $this;
    }

    public function getName(): string
    {
        return $this->name;
    }

    public function setName(string $name): self
    {
        $this->name = $name;

        return $this;
    }

    public function getRedirect(): ?self
    {
        return $this->redirect;
    }

    public function setRedirect(?self $redirect): self
    {
        $this->redirect = $redirect;
        if ($redirect) {
            $redirect->addRedirectedFrom($this);
        }

        return $this;
    }

    /**
     * @return Collection|Route[]
     */
    public function getRedirectedFrom()
    {
        return $this->redirectedFrom;
    }

    /**
     * @param array|Collection $redirectedFrom
     */
    public function setRedirectedFrom($redirectedFrom): self
    {
        $isArray = \is_array($redirectedFrom);
        if (!$isArray && !$redirectedFrom instanceof Collection) {

            throw new \InvalidArgumentException('setRedirectedFrom requires an array or Collection');
        }
        $this->redirectedFrom = $isArray ? new ArrayCollection($redirectedFrom) : $redirectedFrom;
        $this->redirectedFrom = $isArray ? new ArrayCollection(/** @scrutinizer ignore-type */ $redirectedFrom) : $redirectedFrom;

        return $this;
    }

    public function addRedirectedFrom(self $redirectedFrom): self
    {
        if (!$this->redirectedFrom->contains($redirectedFrom)) {
            $this->redirectedFrom->add($redirectedFrom);
        }

        return $this;
    }

    public function getPage(): ?Page
    {
        return $this->page;
    }

    public function setPage(?Page $page): self
    {
        $this->page = $page;
        if ($this->page) {
            $this->page->setRoute($this);
        }

        return $this;
    }

    public function getPageData(): ?AbstractPageData
    {
        return $this->pageData;
    }

    public function setPageData(?AbstractPageData $pageData): self
    {
        $this->pageData = $pageData;
        if ($this->pageData) {
            $this->pageData->setRoute($this);
        }

        return $this;
    }
}


components-web-app / api-components-bundle

Push — master ( 62e1a3...672be2 )

Route::setPath() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like