PiedWeb\CMSBundle\Entity\PageTrait

Complexity

Total Complexity

21

Size/Duplication

Total Lines	123
Duplicated Lines	0 %

Test Coverage

Coverage

20.93%

Importance

Changes	1
Bugs	0	Features	0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	setH1()	0	5	1
A	setUpdatedAt()	0	5	1
A	setMainContent()	0	5	1
A	setSlug()	0	16	4
A	getCreatedAt()	0	3	1
A	setCreatedAt()	0	5	1
A	getMainContent()	0	3	1
A	getRealSlug()	0	7	2
A	getUpdatedAt()	0	3	1
A	getH1()	0	7	2
A	__toString()	0	3	1
A	__constructPage()	0	5	3
A	getSlug()	0	5	2

<?php

namespace PiedWeb\CMSBundle\Entity;

use Cocur\Slugify\Slugify;
use Doctrine\ORM\Mapping as ORM;
use PiedWeb\CMSBundle\Service\ShortCodeConverter;
use Symfony\Component\Validator\Constraints as Assert;

trait PageTrait
{
    /**
     * @ORM\Column(type="string", length=150)
     */
    protected $slug;

    /**
     * @ORM\Column(type="string", length=255, nullable=true)
     */
    protected $h1;

    public function getH1($elseTitle = false): ?string
    {
        if ($elseTitle) {
            return ShortCodeConverter::do($this->h1 ?? $this->title ?? '', $this->getLocale());

It seems like getLocale() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return $this->h1;
    }

    public function setH1(?string $h1): self
    {
        $this->h1 = $h1;

        return $this;
    }

    /**
     * @ORM\Column(type="text", nullable=true)
     */
    protected $mainContent;

    /**
     * @ORM\Column(type="datetime")
     */
    protected $createdAt;

    /**
     * @ORM\Column(type="datetime")
     */
    protected $updatedAt;

    public function __toString()
    {
        return trim($this->host.'/'.$this->slug.' ');
    }

    public function __constructPage()
    {
        $this->updatedAt = null !== $this->updatedAt ? $this->updatedAt : new \DateTime();
        $this->createdAt = null !== $this->createdAt ? $this->createdAt : new \DateTime();
        $this->slug = '';
    }

    public function getSlug(): ?string
    {
        if (!$this->slug) return $this->id;

        return $this->slug;
    }

    public function getRealSlug(): ?string
    {
        if ('homepage' == $this->slug) {
            return '';
        }

        return $this->slug;
    }

    public function setSlug($slug, $set = false): self
    {
        if (true === $set) {
            $this->slug = $slug;
        } elseif (null === $slug) { // work around for disabled input in sonata admin
            if (null === $this->slug) {
                throw new \ErrorException('slug cant be null');
            }
        } else {
            $slg = new Slugify(['regexp' => '/[^A-Za-z0-9\/]+/']);
            $slug = $slg->slugify($slug);
            $slug = trim($slug, '/');
            $this->slug = $slug; //$this->setSlug(trim($slug, '/'), true);
        }

        return $this;
    }

    public function getMainContent(): ?string
    {
        return $this->mainContent;
    }

    public function setMainContent(?string $mainContent): self
    {
        $this->mainContent = $mainContent;

        return $this;
    }

    public function getCreatedAt(): ?\DateTimeInterface
    {
        return $this->createdAt;
    }

    public function setCreatedAt(\DateTimeInterface $createdAt): self
    {
        $this->createdAt = $createdAt;

        return $this;
    }

    public function getUpdatedAt(): ?\DateTimeInterface
    {
        return $this->updatedAt;
    }

    public function setUpdatedAt(\DateTimeInterface $updatedAt): self
    {
        $this->updatedAt = $updatedAt;

        return $this;
    }
}