StyleMDTrait - 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

StyleMDTrait 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	getContainerMarginMD()	4	1
A	getContainerPaddingMD()	4	1
A	getContainerWidthMD()	4	1
A	getContainerHeightMD()	4	1
A	getTextAlignMD()	4	1
A	getContainerBackgroundMD()	4	1
A	getContainerBackgroundTypeMD()	4	1
A	getContainerBackgroundRepeatMD()	4	1
A	getContainerBackgroundPositionMD()	4	1
A	getContainerBackgroundSizeMD()	4	1
A	getContainerBackgroundImageMD()	4	1
A	getContainerBackgroundOverlayMD()	4	1
A	setContainerMarginMD()	6	1
A	setContainerPaddingMD()	6	1
A	setContainerWidthMD()	6	1
A	setContainerHeightMD()	6	1
A	setTextAlignMD()	6	1
A	setContainerBackgroundMD()	6	1
A	setContainerBackgroundTypeMD()	6	1
A	setContainerBackgroundRepeatMD()	6	1
A	setContainerBackgroundPositionMD()	6	1
A	setContainerBackgroundSizeMD()	6	1
A	getContainerBackgroundColorMD()	4	1
A	setContainerBackgroundColorMD()	6	1
A	setContainerBackgroundImageMD()	6	1
A	setContainerBackgroundOverlayMD()	6	1

<?php

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

use Victoire\Bundle\MediaBundle\Entity\Media;

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

        return $this;
    }

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

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

        return $this;
    }

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

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

        return $this;
    }

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

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

        return $this;
    }

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

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

        return $this;
    }

    /**
     * @return string
     */
    public function getContainerBackgroundMD()
    {
        return $this->containerBackgroundMD;

    }

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


        return $this;
    }

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

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

        return $this;
    }

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

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

        return $this;
    }

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

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

        return $this;
    }

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

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

        return $this;
    }

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

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

        return $this;
    }

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

    /**
     * Set image.
     *
     * @param Media $image
     *
     * @return $this
     */
    public function setContainerBackgroundImageMD(Media $image = null)
    {
        $this->containerBackgroundImageMD = $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 getContainerBackgroundOverlayMD()
    {
        return $this->containerBackgroundOverlayMD;
    }

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

        return $this;
    }
}


1			<?php
2
3			namespace Victoire\Bundle\WidgetBundle\Entity\Traits\StyleTraits;
4
5			use Victoire\Bundle\MediaBundle\Entity\Media;
6
7			trait StyleMDTrait
8			{
9			/**
10			* @var string
11			*
12			* @ORM\Column(name="container_margin_md", type="string", length=255, nullable=true)
13			*/
14			protected $containerMarginMD;
15
16			/**
17			* @var string
18			*
19			* @ORM\Column(name="container_padding_md", type="string", length=255, nullable=true)
20			*/
21			protected $containerPaddingMD;
22
23			/**
24			* @var string
25			*
26			* @ORM\Column(name="container_width_md", type="string", length=255, nullable=true)
27			*/
28			protected $containerWidthMD;
29
30			/**
31			* @var string
32			*
33			* @ORM\Column(name="container_height_md", type="string", length=255, nullable=true)
34			*/
35			protected $containerHeightMD;
36
37			/**
38			* @var string
39			*
40			* @ORM\Column(name="text_align_md", type="string", length=15, nullable=true)
41			*/
42			protected $textAlignMD;
43
44			/**
45			* @var string
46			*
47			* @deprecated
48			* @ORM\Column(name="container_background_md", type="string", length=255, nullable=true)
49			*/
50			protected $containerBackgroundMD;
51
52			/**
53			* @var string
54			*
55			* @ORM\Column(name="container_background_type_md", type="string", length=255, nullable=true)
56			*/
57			protected $containerBackgroundTypeMD;
58
59			/**
60			* @var string
61			*
62			* @ORM\Column(name="container_background_repeat_md", type="string", length=255, nullable=true)
63			*/
64			protected $containerBackgroundRepeatMD;
65
66			/**
67			* @var string
68			*
69			* @ORM\Column(name="container_background_position_md", type="string", length=255, nullable=true)
70			*/
71			protected $containerBackgroundPositionMD;
72
73			/**
74			* @var string
75			*
76			* @ORM\Column(name="container_background_size_md", type="string", length=255, nullable=true)
77			*/
78			protected $containerBackgroundSizeMD;
79
80			/**
81			* @var string
82			*
83			* @ORM\Column(name="container_background_color_md", type="string", length=255, nullable=true)
84			*/
85			protected $containerBackgroundColorMD;
86
87			/**
88			* @var string
89			*
90			* @ORM\ManyToOne(targetEntity="\Victoire\Bundle\MediaBundle\Entity\Media")
91			* @ORM\JoinColumn(name="container_background_image_md_id", referencedColumnName="id", onDelete="SET NULL", nullable=true)
92			*/
93			protected $containerBackgroundImageMD;
94
95			/**
96			* @var string
97			*
98			* @ORM\Column(name="container_background_overlay_md", type="string", length=255, nullable=true)
99			*/
100			protected $containerBackgroundOverlayMD;
101
102			/**
103			* @return string
104			*/
105			public function getContainerMarginMD()
106			{
107			return $this->containerMarginMD;
108			}
109
110			/**
111			* @param string $containerMarginMD
112			*
113			* @return $this
114			*/
115			public function setContainerMarginMD($containerMarginMD)
116			{
117			$this->containerMarginMD = $containerMarginMD;
118
119			return $this;
120			}
121
122			/**
123			* @return string
124			*/
125			public function getContainerPaddingMD()
126			{
127			return $this->containerPaddingMD;
128			}
129
130			/**
131			* @param string $containerPaddingMD
132			*
133			* @return $this
134			*/
135			public function setContainerPaddingMD($containerPaddingMD)
136			{
137			$this->containerPaddingMD = $containerPaddingMD;
138
139			return $this;
140			}
141
142			/**
143			* @return string
144			*/
145			public function getContainerWidthMD()
146			{
147			return $this->containerWidthMD;
148			}
149
150			/**
151			* @param string $containerWidthMD
152			*
153			* @return $this
154			*/
155			public function setContainerWidthMD($containerWidthMD)
156			{
157			$this->containerWidthMD = $containerWidthMD;
158
159			return $this;
160			}
161
162			/**
163			* @return string
164			*/
165			public function getContainerHeightMD()
166			{
167			return $this->containerHeightMD;
168			}
169
170			/**
171			* @param string $containerHeightMD
172			*
173			* @return $this
174			*/
175			public function setContainerHeightMD($containerHeightMD)
176			{
177			$this->containerHeightMD = $containerHeightMD;
178
179			return $this;
180			}
181
182			/**
183			* @return string
184			*/
185			public function getTextAlignMD()
186			{
187			return $this->textAlignMD;
188			}
189
190			/**
191			* @param string $textAlignMD
192			*
193			* @return $this
194			*/
195			public function setTextAlignMD($textAlignMD)
196			{
197			$this->textAlignMD = $textAlignMD;
198
199			return $this;
200			}
201
202			/**
203			* @return string
204			*/
205			public function getContainerBackgroundMD()
206			{
207			return $this->containerBackgroundMD;
			0 ignored issues – show Deprecated Code introduced 2015-11-24 15:51 UTC by Report Bug Copy Issue Report The property `Victoire\Bundle\WidgetBu...:$containerBackgroundMD` 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 $containerBackgroundMD
212			*
213			* @return $this
214			*/
215			public function setContainerBackgroundMD($containerBackgroundMD)
216			{
217			$this->containerBackgroundMD = $containerBackgroundMD;
			0 ignored issues – show Deprecated Code introduced 2015-11-24 15:51 UTC by Report Bug Copy Issue Report The property `Victoire\Bundle\WidgetBu...:$containerBackgroundMD` 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 getContainerBackgroundTypeMD()
226			{
227			return $this->containerBackgroundTypeMD;
228			}
229
230			/**
231			* @param string $containerBackgroundTypeMD
232			*
233			* @return $this
234			*/
235			public function setContainerBackgroundTypeMD($containerBackgroundTypeMD)
236			{
237			$this->containerBackgroundTypeMD = $containerBackgroundTypeMD;
238
239			return $this;
240			}
241
242			/**
243			* @return string
244			*/
245			public function getContainerBackgroundRepeatMD()
246			{
247			return $this->containerBackgroundRepeatMD;
248			}
249
250			/**
251			* @param string $containerBackgroundRepeatMD
252			*
253			* @return $this
254			*/
255			public function setContainerBackgroundRepeatMD($containerBackgroundRepeatMD)
256			{
257			$this->containerBackgroundRepeatMD = $containerBackgroundRepeatMD;
258
259			return $this;
260			}
261
262			/**
263			* @return string
264			*/
265			public function getContainerBackgroundPositionMD()
266			{
267			return $this->containerBackgroundPositionMD;
268			}
269
270			/**
271			* @param string $containerBackgroundPositionMD
272			*
273			* @return $this
274			*/
275			public function setContainerBackgroundPositionMD($containerBackgroundPositionMD)
276			{
277			$this->containerBackgroundPositionMD = $containerBackgroundPositionMD;
278
279			return $this;
280			}
281
282			/**
283			* @return string
284			*/
285			public function getContainerBackgroundSizeMD()
286			{
287			return $this->containerBackgroundSizeMD;
288			}
289
290			/**
291			* @param string $containerBackgroundSizeMD
292			*
293			* @return $this
294			*/
295			public function setContainerBackgroundSizeMD($containerBackgroundSizeMD)
296			{
297			$this->containerBackgroundSizeMD = $containerBackgroundSizeMD;
298
299			return $this;
300			}
301
302			/**
303			* @return string
304			*/
305			public function getContainerBackgroundColorMD()
306			{
307			return $this->containerBackgroundColorMD;
308			}
309
310			/**
311			* @param string $containerBackgroundColorMD
312			*
313			* @return $this
314			*/
315			public function setContainerBackgroundColorMD($containerBackgroundColorMD)
316			{
317			$this->containerBackgroundColorMD = $containerBackgroundColorMD;
318
319			return $this;
320			}
321
322			/**
323			* @return string
324			*/
325			public function getContainerBackgroundImageMD()
326			{
327			return $this->containerBackgroundImageMD;
328			}
329
330			/**
331			* Set image.
332			*
333			* @param Media $image
334			*
335			* @return $this
336			*/
337			public function setContainerBackgroundImageMD(Media $image = null)
338			{
339			$this->containerBackgroundImageMD = $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 `$containerBackgroundImageMD` 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 getContainerBackgroundOverlayMD()
348			{
349			return $this->containerBackgroundOverlayMD;
350			}
351
352			/**
353			* @param string $containerBackgroundOverlayMD
354			*
355			* @return $this
356			*/
357			public function setContainerBackgroundOverlayMD($containerBackgroundOverlayMD)
358			{
359			$this->containerBackgroundOverlayMD = $containerBackgroundOverlayMD;
360
361			return $this;
362			}
363			}
364

Victoire / victoire

Pull Request — master (#302)

StyleMDTrait A

Complexity

Size/Duplication

Coupling/Cohesion

26 Methods

Duplication Side-by-Side

Filter issues like