WebViewTrait::getSeo() - Code Metrics - Inspection of "Feature/use knp translatable" - Victoire/victoire - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#433)

by Paul

created 2016-05-30 07:17 UTC

WebViewTrait::getSeo() A

↳ Parent: WebViewTrait

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace Victoire\Bundle\PageBundle\Entity\Traits;

use Symfony\Component\PropertyAccess\PropertyAccess;
use Victoire\Bundle\CoreBundle\Annotations as VIC;
use Victoire\Bundle\PageBundle\Entity\PageStatus;
use Victoire\Bundle\SeoBundle\Entity\PageSeo;

/**
 * This trait make a view displayable for public.
 */
trait WebViewTrait
{
    /**
     * @ORM\OneToOne(targetEntity="\Victoire\Bundle\SeoBundle\Entity\PageSeo", cascade={"persist", "remove"})
     * @ORM\JoinColumn(name="seo_id", referencedColumnName="id", onDelete="SET NULL")
     */
    protected $seo;

    /**
     * @ORM\OneToMany(targetEntity="Victoire\Bundle\SeoBundle\Entity\PageSeo", mappedBy="redirectTo")
     */
    protected $referers;

    /**
     * @ORM\Column(name="status", type="string", nullable=false)
     */
    protected $status = PageStatus::PUBLISHED;

    /**
     * @var \Datetime
     *
     * @ORM\Column(name="publishedAt", type="datetime")
     * @VIC\BusinessProperty("date")
     */
    protected $publishedAt;

    /**
     * @var string
     *
     * @ORM\Column(name="homepage", type="boolean", nullable=false)
     */
    protected $homepage;

    /**
     * Set seo.
     *
     * @param PageSeo $seo
     *
     * @return WebViewTrait

     */
    public function setSeo(PageSeo $seo)
    {
        $this->seo = $seo;

        return $this;
    }

    /**
     * Get seo.
     *
     * @return PageSeo
     */
    public function getSeo()
    {
        return $this->seo;
    }

    /**
     * Get referers.
     *
     * @return string
     */
    public function getReferers()
    {
        return $this->referers;
    }

    /**
     * Set the refere.
     *
     * @param string $referers
     */
    public function setReferers($referers)
    {
        $this->referers = $referers;
    }

    /**
     * Set status.
     *
     * @param string $status
     */
    public function setStatus($status)
    {
        $this->status = $status;
    }

    /**
     * Get status.
     *
     * @return string
     */
    public function getStatus()
    {
        return $this->status;
    }

    /**
     * Set publishedAt.
     *
     * @param \DateTime $publishedAt
     */
    public function setPublishedAt($publishedAt)
    {
        $this->publishedAt = $publishedAt;
    }

    /**
     * Get publishedAt.
     *
     * @return \DateTime
     */
    public function getPublishedAt()
    {
        return $this->publishedAt;
    }

    /**
     * Is this page published.
     *
     * @return bool
     */
    public function isPublished()
    {
        if (
            $this->getStatus() === PageStatus::PUBLISHED ||
            ($this->getStatus() === PageStatus::SCHEDULED &&
            $this->getPublishedAt() < new \DateTime())
            ) {
            return true;
        } else {
            return false;
        }
    }

    /**
     * Get homepage.
     *
     * @return bool
     */
    public function isHomepage()
    {
        return $this->homepage;
    }

    /**
     * Set homepage.
     *
     * @param bool $homepage
     *
     * @return $this
     */
    public function setHomepage($homepage)
    {
        $this->homepage = $homepage;
$answer = 42;

$correct = false;

$correct = (bool) $answer;

        return $this;
    }

    public function getUrl()
    {
        return PropertyAccess::createPropertyAccessor()->getValue($this->translate(), 'getUrl');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    public function setUrl($name, $locale = null)
    {
        $this->translate($locale)->setUrl($name);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }
}


1			<?php
2
3			namespace Victoire\Bundle\PageBundle\Entity\Traits;
4
5			use Symfony\Component\PropertyAccess\PropertyAccess;
6			use Victoire\Bundle\CoreBundle\Annotations as VIC;
7			use Victoire\Bundle\PageBundle\Entity\PageStatus;
8			use Victoire\Bundle\SeoBundle\Entity\PageSeo;
9
10			/**
11			* This trait make a view displayable for public.
12			*/
13			trait WebViewTrait
14			{
15			/**
16			* @ORM\OneToOne(targetEntity="\Victoire\Bundle\SeoBundle\Entity\PageSeo", cascade={"persist", "remove"})
17			* @ORM\JoinColumn(name="seo_id", referencedColumnName="id", onDelete="SET NULL")
18			*/
19			protected $seo;
20
21			/**
22			* @ORM\OneToMany(targetEntity="Victoire\Bundle\SeoBundle\Entity\PageSeo", mappedBy="redirectTo")
23			*/
24			protected $referers;
25
26			/**
27			* @ORM\Column(name="status", type="string", nullable=false)
28			*/
29			protected $status = PageStatus::PUBLISHED;
30
31			/**
32			* @var \Datetime
33			*
34			* @ORM\Column(name="publishedAt", type="datetime")
35			* @VIC\BusinessProperty("date")
36			*/
37			protected $publishedAt;
38
39			/**
40			* @var string
41			*
42			* @ORM\Column(name="homepage", type="boolean", nullable=false)
43			*/
44			protected $homepage;
45
46			/**
47			* Set seo.
48			*
49			* @param PageSeo $seo
50			*
51			* @return WebViewTrait
			0 ignored issues – show Comprehensibility Bug introduced 2015-11-24 15:51 UTC by Report Bug Copy Issue Report The return type `WebViewTrait` is a trait, and thus cannot be used for type-hinting in PHP. Maybe consider adding an interface and use that for type-hinting? In PHP traits cannot be used for type-hinting as they do not define a well-defined structure. This is because any class that uses a trait can rename that trait’s methods. If you would like to return an object that has a guaranteed set of methods, you could create a companion interface that lists these methods explicitly. Loading history...
52			*/
53			public function setSeo(PageSeo $seo)
54			{
55			$this->seo = $seo;
56
57			return $this;
58			}
59
60			/**
61			* Get seo.
62			*
63			* @return PageSeo
64			*/
65			public function getSeo()
66			{
67			return $this->seo;
68			}
69
70			/**
71			* Get referers.
72			*
73			* @return string
74			*/
75			public function getReferers()
76			{
77			return $this->referers;
78			}
79
80			/**
81			* Set the refere.
82			*
83			* @param string $referers
84			*/
85			public function setReferers($referers)
86			{
87			$this->referers = $referers;
88			}
89
90			/**
91			* Set status.
92			*
93			* @param string $status
94			*/
95			public function setStatus($status)
96			{
97			$this->status = $status;
98			}
99
100			/**
101			* Get status.
102			*
103			* @return string
104			*/
105			public function getStatus()
106			{
107			return $this->status;
108			}
109
110			/**
111			* Set publishedAt.
112			*
113			* @param \DateTime $publishedAt
114			*/
115			public function setPublishedAt($publishedAt)
116			{
117			$this->publishedAt = $publishedAt;
118			}
119
120			/**
121			* Get publishedAt.
122			*
123			* @return \DateTime
124			*/
125			public function getPublishedAt()
126			{
127			return $this->publishedAt;
128			}
129
130			/**
131			* Is this page published.
132			*
133			* @return bool
134			*/
135			public function isPublished()
136			{
137			if (
138			$this->getStatus() === PageStatus::PUBLISHED \|\|
139			($this->getStatus() === PageStatus::SCHEDULED &&
140			$this->getPublishedAt() < new \DateTime())
141			) {
142			return true;
143			} else {
144			return false;
145			}
146			}
147
148			/**
149			* Get homepage.
150			*
151			* @return bool
152			*/
153			public function isHomepage()
154			{
155			return $this->homepage;
156			}
157
158			/**
159			* Set homepage.
160			*
161			* @param bool $homepage
162			*
163			* @return $this
164			*/
165			public function setHomepage($homepage)
166			{
167			$this->homepage = $homepage;
			0 ignored issues – show Documentation Bug introduced 2015-12-01 20:20 UTC by Report Bug Copy Issue Report The property `$homepage` was declared of type `string`, but `$homepage` is of type `boolean`. Maybe add a type cast? This check looks for assignments to scalar types that may be of the wrong type. To ensure the code behaves as expected, it may be a good idea to add an explicit type cast. $answer = 42; $correct = false; $correct = (bool) $answer; Loading history...
168
169			return $this;
170			}
171
172			public function getUrl()
173			{
174			return PropertyAccess::createPropertyAccessor()->getValue($this->translate(), 'getUrl');
			0 ignored issues – show Bug introduced 2016-05-27 15:10 UTC by Report Bug Copy Issue Report It seems like `translate()` 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...
175			}
176
177			public function setUrl($name, $locale = null)
178			{
179			$this->translate($locale)->setUrl($name);
			0 ignored issues – show Bug introduced 2016-05-27 15:10 UTC by Report Bug Copy Issue Report It seems like `translate()` 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...
180			}
181			}
182

Victoire / victoire

Pull Request — master (#433)

WebViewTrait::getSeo() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like