StyleLGTrait - Code Metrics - Inspection of "move viewReference from array to object in Analyti..." - Victoire/victoire - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#302)

by Leny

created 2015-12-10 14:56 UTC

StyleLGTrait A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	357
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Metric	Value
wmc	26
lcom	0
cbo	0
dl	0
loc	357
rs	10

26 Methods

Rating	Name	Size	Complexity
A	getContainerMarginLG()	4	1
A	getContainerPaddingLG()	4	1
A	getContainerWidthLG()	4	1
A	getContainerHeightLG()	4	1
A	getTextAlignLG()	4	1
A	getContainerBackgroundLG()	4	1
A	getContainerBackgroundTypeLG()	4	1
A	getContainerBackgroundRepeatLG()	4	1
A	getContainerBackgroundPositionLG()	4	1
A	getContainerBackgroundSizeLG()	4	1
A	getContainerBackgroundImageLG()	4	1
A	getContainerBackgroundOverlayLG()	4	1
A	setContainerMarginLG()	6	1
A	setContainerPaddingLG()	6	1
A	setContainerWidthLG()	6	1
A	setContainerHeightLG()	6	1
A	setTextAlignLG()	6	1
A	setContainerBackgroundLG()	6	1
A	setContainerBackgroundTypeLG()	6	1
A	setContainerBackgroundRepeatLG()	6	1
A	setContainerBackgroundPositionLG()	6	1
A	setContainerBackgroundSizeLG()	6	1
A	getContainerBackgroundColorLG()	4	1
A	setContainerBackgroundColorLG()	6	1
A	setContainerBackgroundImageLG()	6	1
A	setContainerBackgroundOverlayLG()	6	1

<?php

namespace Victoire\Bundle\WidgetBundle\Entity\Traits\StyleTraits;

use Victoire\Bundle\MediaBundle\Entity\Media;

trait StyleLGTrait
{
    /**
     * @var string
     *
     * @ORM\Column(name="container_margin_lg", type="string", length=255, nullable=true)
     */
    protected $containerMarginLG;

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

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

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

    /**
     * @var string
     *
     * @ORM\Column(name="text_align_lg", type="string", length=15, nullable=true)
     */
    protected $textAlignLG;

    /**
     * @var string
     *
     * @deprecated
     * @ORM\Column(name="container_background_lg", type="string", length=255, nullable=true)
     */
    protected $containerBackgroundLG;

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

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

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

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

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

    /**
     * @var string
     *
     * @ORM\ManyToOne(targetEntity="\Victoire\Bundle\MediaBundle\Entity\Media")
     * @ORM\JoinColumn(name="container_background_image_lg_id", referencedColumnName="id", onDelete="SET NULL", nullable=true)
     */
    protected $containerBackgroundImageLG;

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

    /**
     * @return string
     */
    public function getContainerMarginLG()
    {
        return $this->containerMarginLG;
    }

    /**
     * @param string $containerMarginLG
     *
     * @return $this
     */
    public function setContainerMarginLG($containerMarginLG)
    {
        $this->containerMarginLG = $containerMarginLG;

        return $this;
    }

    /**
     * @return string
     */
    public function getContainerPaddingLG()
    {
        return $this->containerPaddingLG;
    }

    /**
     * @param string $containerPaddingLG
     *
     * @return $this
     */
    public function setContainerPaddingLG($containerPaddingLG)
    {
        $this->containerPaddingLG = $containerPaddingLG;

        return $this;
    }

    /**
     * @return string
     */
    public function getContainerWidthLG()
    {
        return $this->containerWidthLG;
    }

    /**
     * @param string $containerWidthLG
     *
     * @return $this
     */
    public function setContainerWidthLG($containerWidthLG)
    {
        $this->containerWidthLG = $containerWidthLG;

        return $this;
    }

    /**
     * @return string
     */
    public function getContainerHeightLG()
    {
        return $this->containerHeightLG;
    }

    /**
     * @param string $containerHeightLG
     *
     * @return $this
     */
    public function setContainerHeightLG($containerHeightLG)
    {
        $this->containerHeightLG = $containerHeightLG;

        return $this;
    }

    /**
     * @return string
     */
    public function getTextAlignLG()
    {
        return $this->textAlignLG;
    }

    /**
     * @param string $textAlignLG
     *
     * @return $this
     */
    public function setTextAlignLG($textAlignLG)
    {
        $this->textAlignLG = $textAlignLG;

        return $this;
    }

    /**
     * @return string
     */
    public function getContainerBackgroundLG()
    {
        return $this->containerBackgroundLG;

    }

    /**
     * @param string $containerBackgroundLG
     *
     * @return $this
     */
    public function setContainerBackgroundLG($containerBackgroundLG)
    {
        $this->containerBackgroundLG = $containerBackgroundLG;


        return $this;
    }

    /**
     * @return string
     */
    public function getContainerBackgroundTypeLG()
    {
        return $this->containerBackgroundTypeLG;
    }

    /**
     * @param string $containerBackgroundTypeLG
     *
     * @return $this
     */
    public function setContainerBackgroundTypeLG($containerBackgroundTypeLG)
    {
        $this->containerBackgroundTypeLG = $containerBackgroundTypeLG;

        return $this;
    }

    /**
     * @return string
     */
    public function getContainerBackgroundRepeatLG()
    {
        return $this->containerBackgroundRepeatLG;
    }

    /**
     * @param string $containerBackgroundRepeatLG
     *
     * @return $this
     */
    public function setContainerBackgroundRepeatLG($containerBackgroundRepeatLG)
    {
        $this->containerBackgroundRepeatLG = $containerBackgroundRepeatLG;

        return $this;
    }

    /**
     * @return string
     */
    public function getContainerBackgroundPositionLG()
    {
        return $this->containerBackgroundPositionLG;
    }

    /**
     * @param string $containerBackgroundPositionLG
     *
     * @return $this
     */
    public function setContainerBackgroundPositionLG($containerBackgroundPositionLG)
    {
        $this->containerBackgroundPositionLG = $containerBackgroundPositionLG;

        return $this;
    }

    /**
     * @return string
     */
    public function getContainerBackgroundSizeLG()
    {
        return $this->containerBackgroundSizeLG;
    }

    /**
     * @param string $containerBackgroundSizeLG
     *
     * @return $this
     */
    public function setContainerBackgroundSizeLG($containerBackgroundSizeLG)
    {
        $this->containerBackgroundSizeLG = $containerBackgroundSizeLG;

        return $this;
    }

    /**
     * @return string
     */
    public function getContainerBackgroundColorLG()
    {
        return $this->containerBackgroundColorLG;
    }

    /**
     * @param string $containerBackgroundColorLG
     *
     * @return $this
     */
    public function setContainerBackgroundColorLG($containerBackgroundColorLG)
    {
        $this->containerBackgroundColorLG = $containerBackgroundColorLG;

        return $this;
    }

    /**
     * @return string
     */
    public function getContainerBackgroundImageLG()
    {
        return $this->containerBackgroundImageLG;
    }

    /**
     * Set image.
     *
     * @param Media $image
     *
     * @return $this
     */
    public function setContainerBackgroundImageLG(Media $image = null)
    {
        $this->containerBackgroundImageLG = $image;
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}

        return $this;
    }

    /**
     * @return string
     */
    public function getContainerBackgroundOverlayLG()
    {
        return $this->containerBackgroundOverlayLG;
    }

    /**
     * @param string $containerBackgroundOverlayLG
     *
     * @return $this
     */
    public function setContainerBackgroundOverlayLG($containerBackgroundOverlayLG)
    {
        $this->containerBackgroundOverlayLG = $containerBackgroundOverlayLG;

        return $this;
    }
}


1			<?php
2
3			namespace Victoire\Bundle\WidgetBundle\Entity\Traits\StyleTraits;
4
5			use Victoire\Bundle\MediaBundle\Entity\Media;
6
7			trait StyleLGTrait
8			{
9			/**
10			* @var string
11			*
12			* @ORM\Column(name="container_margin_lg", type="string", length=255, nullable=true)
13			*/
14			protected $containerMarginLG;
15
16			/**
17			* @var string
18			*
19			* @ORM\Column(name="container_padding_lg", type="string", length=255, nullable=true)
20			*/
21			protected $containerPaddingLG;
22
23			/**
24			* @var string
25			*
26			* @ORM\Column(name="container_width_lg", type="string", length=255, nullable=true)
27			*/
28			protected $containerWidthLG;
29
30			/**
31			* @var string
32			*
33			* @ORM\Column(name="container_height_lg", type="string", length=255, nullable=true)
34			*/
35			protected $containerHeightLG;
36
37			/**
38			* @var string
39			*
40			* @ORM\Column(name="text_align_lg", type="string", length=15, nullable=true)
41			*/
42			protected $textAlignLG;
43
44			/**
45			* @var string
46			*
47			* @deprecated
48			* @ORM\Column(name="container_background_lg", type="string", length=255, nullable=true)
49			*/
50			protected $containerBackgroundLG;
51
52			/**
53			* @var string
54			*
55			* @ORM\Column(name="container_background_type_lg", type="string", length=255, nullable=true)
56			*/
57			protected $containerBackgroundTypeLG;
58
59			/**
60			* @var string
61			*
62			* @ORM\Column(name="container_background_repeat_lg", type="string", length=255, nullable=true)
63			*/
64			protected $containerBackgroundRepeatLG;
65
66			/**
67			* @var string
68			*
69			* @ORM\Column(name="container_background_position_lg", type="string", length=255, nullable=true)
70			*/
71			protected $containerBackgroundPositionLG;
72
73			/**
74			* @var string
75			*
76			* @ORM\Column(name="container_background_size_lg", type="string", length=255, nullable=true)
77			*/
78			protected $containerBackgroundSizeLG;
79
80			/**
81			* @var string
82			*
83			* @ORM\Column(name="container_background_color_lg", type="string", length=255, nullable=true)
84			*/
85			protected $containerBackgroundColorLG;
86
87			/**
88			* @var string
89			*
90			* @ORM\ManyToOne(targetEntity="\Victoire\Bundle\MediaBundle\Entity\Media")
91			* @ORM\JoinColumn(name="container_background_image_lg_id", referencedColumnName="id", onDelete="SET NULL", nullable=true)
92			*/
93			protected $containerBackgroundImageLG;
94
95			/**
96			* @var string
97			*
98			* @ORM\Column(name="container_background_overlay_lg", type="string", length=255, nullable=true)
99			*/
100			protected $containerBackgroundOverlayLG;
101
102			/**
103			* @return string
104			*/
105			public function getContainerMarginLG()
106			{
107			return $this->containerMarginLG;
108			}
109
110			/**
111			* @param string $containerMarginLG
112			*
113			* @return $this
114			*/
115			public function setContainerMarginLG($containerMarginLG)
116			{
117			$this->containerMarginLG = $containerMarginLG;
118
119			return $this;
120			}
121
122			/**
123			* @return string
124			*/
125			public function getContainerPaddingLG()
126			{
127			return $this->containerPaddingLG;
128			}
129
130			/**
131			* @param string $containerPaddingLG
132			*
133			* @return $this
134			*/
135			public function setContainerPaddingLG($containerPaddingLG)
136			{
137			$this->containerPaddingLG = $containerPaddingLG;
138
139			return $this;
140			}
141
142			/**
143			* @return string
144			*/
145			public function getContainerWidthLG()
146			{
147			return $this->containerWidthLG;
148			}
149
150			/**
151			* @param string $containerWidthLG
152			*
153			* @return $this
154			*/
155			public function setContainerWidthLG($containerWidthLG)
156			{
157			$this->containerWidthLG = $containerWidthLG;
158
159			return $this;
160			}
161
162			/**
163			* @return string
164			*/
165			public function getContainerHeightLG()
166			{
167			return $this->containerHeightLG;
168			}
169
170			/**
171			* @param string $containerHeightLG
172			*
173			* @return $this
174			*/
175			public function setContainerHeightLG($containerHeightLG)
176			{
177			$this->containerHeightLG = $containerHeightLG;
178
179			return $this;
180			}
181
182			/**
183			* @return string
184			*/
185			public function getTextAlignLG()
186			{
187			return $this->textAlignLG;
188			}
189
190			/**
191			* @param string $textAlignLG
192			*
193			* @return $this
194			*/
195			public function setTextAlignLG($textAlignLG)
196			{
197			$this->textAlignLG = $textAlignLG;
198
199			return $this;
200			}
201
202			/**
203			* @return string
204			*/
205			public function getContainerBackgroundLG()
206			{
207			return $this->containerBackgroundLG;
			0 ignored issues – show Deprecated Code introduced 2015-11-24 15:51 UTC by Report Bug Copy Issue Report The property `Victoire\Bundle\WidgetBu...:$containerBackgroundLG` has been deprecated. This property has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the property will be removed from the class and what other property to use instead. Loading history...
208			}
209
210			/**
211			* @param string $containerBackgroundLG
212			*
213			* @return $this
214			*/
215			public function setContainerBackgroundLG($containerBackgroundLG)
216			{
217			$this->containerBackgroundLG = $containerBackgroundLG;
			0 ignored issues – show Deprecated Code introduced 2015-11-24 15:51 UTC by Report Bug Copy Issue Report The property `Victoire\Bundle\WidgetBu...:$containerBackgroundLG` has been deprecated. This property has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the property will be removed from the class and what other property to use instead. Loading history...
218
219			return $this;
220			}
221
222			/**
223			* @return string
224			*/
225			public function getContainerBackgroundTypeLG()
226			{
227			return $this->containerBackgroundTypeLG;
228			}
229
230			/**
231			* @param string $containerBackgroundTypeLG
232			*
233			* @return $this
234			*/
235			public function setContainerBackgroundTypeLG($containerBackgroundTypeLG)
236			{
237			$this->containerBackgroundTypeLG = $containerBackgroundTypeLG;
238
239			return $this;
240			}
241
242			/**
243			* @return string
244			*/
245			public function getContainerBackgroundRepeatLG()
246			{
247			return $this->containerBackgroundRepeatLG;
248			}
249
250			/**
251			* @param string $containerBackgroundRepeatLG
252			*
253			* @return $this
254			*/
255			public function setContainerBackgroundRepeatLG($containerBackgroundRepeatLG)
256			{
257			$this->containerBackgroundRepeatLG = $containerBackgroundRepeatLG;
258
259			return $this;
260			}
261
262			/**
263			* @return string
264			*/
265			public function getContainerBackgroundPositionLG()
266			{
267			return $this->containerBackgroundPositionLG;
268			}
269
270			/**
271			* @param string $containerBackgroundPositionLG
272			*
273			* @return $this
274			*/
275			public function setContainerBackgroundPositionLG($containerBackgroundPositionLG)
276			{
277			$this->containerBackgroundPositionLG = $containerBackgroundPositionLG;
278
279			return $this;
280			}
281
282			/**
283			* @return string
284			*/
285			public function getContainerBackgroundSizeLG()
286			{
287			return $this->containerBackgroundSizeLG;
288			}
289
290			/**
291			* @param string $containerBackgroundSizeLG
292			*
293			* @return $this
294			*/
295			public function setContainerBackgroundSizeLG($containerBackgroundSizeLG)
296			{
297			$this->containerBackgroundSizeLG = $containerBackgroundSizeLG;
298
299			return $this;
300			}
301
302			/**
303			* @return string
304			*/
305			public function getContainerBackgroundColorLG()
306			{
307			return $this->containerBackgroundColorLG;
308			}
309
310			/**
311			* @param string $containerBackgroundColorLG
312			*
313			* @return $this
314			*/
315			public function setContainerBackgroundColorLG($containerBackgroundColorLG)
316			{
317			$this->containerBackgroundColorLG = $containerBackgroundColorLG;
318
319			return $this;
320			}
321
322			/**
323			* @return string
324			*/
325			public function getContainerBackgroundImageLG()
326			{
327			return $this->containerBackgroundImageLG;
328			}
329
330			/**
331			* Set image.
332			*
333			* @param Media $image
334			*
335			* @return $this
336			*/
337			public function setContainerBackgroundImageLG(Media $image = null)
338			{
339			$this->containerBackgroundImageLG = $image;
			0 ignored issues – show Documentation Bug introduced 2015-11-24 15:51 UTC by Report Bug Copy Issue Report It seems like `$image` can also be of type `object<Victoire\Bundle\MediaBundle\Entity\Media>`. However, the property `$containerBackgroundImageLG` is declared as type `string`. Maybe add an additional type check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly. For example, imagine you have a variable `$accountId` that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to the `id` property of an instance of the `Account` class. This class holds a proper account, so the id value must no longer be false. Either this assignment is in error or a type check should be added for that assignment. class Id { public $id; public function __construct($id) { $this->id = $id; } } class Account { /** @var Id $id */ public $id; } $account_id = false; if (starsAreRight()) { $account_id = new Id(42); } $account = new Account(); if ($account instanceof Id) { $account->id = $account_id; } Loading history...
340
341			return $this;
342			}
343
344			/**
345			* @return string
346			*/
347			public function getContainerBackgroundOverlayLG()
348			{
349			return $this->containerBackgroundOverlayLG;
350			}
351
352			/**
353			* @param string $containerBackgroundOverlayLG
354			*
355			* @return $this
356			*/
357			public function setContainerBackgroundOverlayLG($containerBackgroundOverlayLG)
358			{
359			$this->containerBackgroundOverlayLG = $containerBackgroundOverlayLG;
360
361			return $this;
362			}
363			}
364

Victoire / victoire

Pull Request — master (#302)

StyleLGTrait A

Complexity

Size/Duplication

Coupling/Cohesion

26 Methods

Duplication Side-by-Side

Filter issues like