Issues in Widget.php (master) - Issues in master - Victoire/victoire - Measure and Improve Code Quality continuously with Scrutinizer

Issues (1704)

Security Analysis not enabled

Cross-Site Scripting

Cross-Site Scripting enables an attacker to inject code into the response of a web-request that is viewed by other users. It can for example be used to bypass access controls, or even to take over other users' accounts.

File Exposure

File Exposure allows an attacker to gain access to local files that he should not be able to access. These files can for example include database credentials, or other configuration files.

File Manipulation

File Manipulation enables an attacker to write custom data to files. This potentially leads to injection of arbitrary code on the server.

Object Injection

Object Injection enables an attacker to inject an object into PHP code, and can lead to arbitrary code execution, file exposure, or file manipulation attacks.

Code Injection

Code Injection enables an attacker to execute arbitrary code on the server.

Response Splitting

Response Splitting can be used to send arbitrary responses.

File Inclusion

File Inclusion enables an attacker to inject custom files into PHP's file loading mechanism, either explicitly passed to include, or for example via PHP's auto-loading mechanism.

Command Injection

Command Injection enables an attacker to inject a shell command that is execute with the privileges of the web-server. This can be used to expose sensitive data, or gain access of your server.

SQL Injection

SQL Injection enables an attacker to execute arbitrary SQL code on your database server gaining access to user data, or manipulating user data.

XPath Injection

XPath Injection enables an attacker to modify the parts of XML document that are read. If that XML document is for example used for authentication, this can lead to further vulnerabilities similar to SQL Injection.

LDAP Injection

LDAP Injection enables an attacker to inject LDAP statements potentially granting permission to run unauthorized queries, or modify content inside the LDAP tree.

Header Injection

Other Vulnerability

This category comprises other attack vectors such as manipulating the PHP runtime, loading custom extensions, freezing the runtime, or similar.

Regex Injection

Regex Injection enables an attacker to execute arbitrary code in your PHP process.

XML Injection

XML Injection enables an attacker to read files on your local filesystem including configuration files, or can be abused to freeze your web-server process.

Variable Injection

Variable Injection enables an attacker to overwrite program variables with custom data, and can lead to further vulnerabilities.

Unfortunately, the security analysis is currently not available for your project. If you are a non-commercial open-source project, please contact support to gain access.

Bundle/WidgetBundle/Entity/Widget.php (25 issues)

Labels

Severity

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php

namespace Victoire\Bundle\WidgetBundle\Entity;

use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\ORM\Mapping as ORM;
use Gedmo\Timestampable\Traits\TimestampableEntity;
use Victoire\Bundle\CoreBundle\Entity\EntityProxy;
use Victoire\Bundle\CriteriaBundle\Entity\Criteria;
use Victoire\Bundle\QueryBundle\Entity\QueryTrait;
use Victoire\Bundle\QueryBundle\Entity\VictoireQueryInterface;
use Victoire\Bundle\WidgetBundle\Entity\Traits\StyleTrait;
use Victoire\Bundle\WidgetBundle\Model\Widget as BaseWidget;
use Victoire\Bundle\WidgetMapBundle\Entity\WidgetMap;

/**
 * Widget.
 *
 * @ORM\Table("vic_widget")
 * @ORM\Entity(repositoryClass="Victoire\Bundle\WidgetBundle\Repository\WidgetRepository")
 * @ORM\InheritanceType("JOINED")
 * @ORM\DiscriminatorColumn(name="type", type="string")
 */
class Widget extends BaseWidget implements VictoireQueryInterface
{
    use StyleTrait;
    use QueryTrait;
    use TimestampableEntity;

    public function __construct()

    {
        $this->childrenSlot = uniqid('', true);
        $this->criterias = new ArrayCollection();
    }

    /**
     * @var int
     *
     * @ORM\Column(name="id", type="integer")
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    protected $id;


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


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


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


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


    /**
     * @var string
     *
     * @ORM\Column(name="fields", type="text")
     */
    protected $fields = 'a:0:{}';


    /**
     * @var string
     *
     * @ORM\Column(name="mode", type="string", length=255, nullable=false)
     */
    protected $mode = self::MODE_STATIC;


    /**
     * Auto simple mode: joined entity.
     *
     * @var EntityProxy
     *
     * @ORM\ManyToOne(targetEntity="\Victoire\Bundle\CoreBundle\Entity\EntityProxy", inversedBy="widgets", cascade={"persist"})
     * @ORM\JoinColumn(name="entityProxy_id", referencedColumnName="id", onDelete="CASCADE")
     */
    protected $entityProxy;


    /**
     * The entity linked to the widget.
     *
     * @var unknown
     */
    protected $entity;


    /**
     * @deprecated Remove Doctrine mapping and property
     *
     * @var string
     *
     * @ORM\ManyToOne(targetEntity="\Victoire\Bundle\CoreBundle\Entity\View", inversedBy="widgets", cascade={"persist"})
     * @ORM\JoinColumn(name="view_id", referencedColumnName="id", onDelete="CASCADE")
     */
    protected $view;


    /**
     * @var WidgetMap
     *
     * @ORM\ManyToOne(targetEntity="\Victoire\Bundle\WidgetMapBundle\Entity\WidgetMap", inversedBy="widgets")
     * @ORM\JoinColumn(name="widget_map_id", referencedColumnName="id", onDelete="SET NULL"))
     */
    protected $widgetMap;


    /**
     * @var [Criteria]
     *
     * @ORM\OneToMany(targetEntity="\Victoire\Bundle\CriteriaBundle\Entity\Criteria", mappedBy="widget", cascade={"persist", "remove"}, orphanRemoval=true)
     */
    protected $criterias;


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


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

    /**
     * @param string $asynchronous
     */
    public function setAsynchronous($asynchronous)
    {
        $this->asynchronous = $asynchronous;
    }

    /**
     * Set the entity proxy.
     *
     * @param EntityProxy $entityProxy
     */
    public function setEntityProxy(EntityProxy $entityProxy)
    {
        $this->entityProxy = $entityProxy;
    }

    /**
     * Get the entity proxy.
     *
     * @return EntityProxy
     */
    public function getEntityProxy()
    {
        return $this->entityProxy;
    }

    /**
     * Get businessEntityName.
     *
     * @return string
     */
    public function getBusinessEntityName()
    {
        if ($entityProxy = $this->getEntityProxy()) {
            return $entityProxy->getBusinessEntity()->getName();
        }

        return $this->businessEntityName;

    }

    /**
     * to string.
     *
     * @return string
     */
    public function __toString()
    {
        return (string) $this->getId();
    }

    /**
     * Get id.
     *
     * @return int
     */
    public function getId()
    {
        return $this->id;
    }

    /**
     * Set fields.
     *
     * @param string $fields
     *
     * @return Widget
     */
    public function setFields($fields)

    {
        $data = @unserialize($fields);
        if ($fields === 'b:0;' || $data !== false) {
            $this->fields = $fields;
        } else {
            $this->fields = serialize($fields);
        }

        return $this;
    }

    /**
     * Get fields.
     *
     * @return string
     */
    public function getFields()
    {
        return unserialize($this->fields);
    }

    /**
     * Set The Id.
     *
     * @param int $id The id
     */
    public function setId($id)
    {
        $this->id = $id;
    }

    /**
     * Set slot.
     *
     * @param string $slot
     *
     * @return Widget
     */
    public function setSlot($slot)
    {
        $this->slot = $slot;

        return $this;
    }

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

    /**
     * Set theme.
     *
     * @param string $theme
     *
     * @return Widget
     */
    public function setTheme($theme)
    {
        $this->theme = $theme;

        return $this;
    }

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

    /**
     * Get the type of the object.
     *
     * @return string The type
     */
    public function getType()
    {
        return $this->guessType();
    }

    /**
     * Guess the type of this by exploding and getting the last item.
     *
     * @return string The guessed type
     */
    protected function guessType()
    {
        $type = explode('\\', get_class($this));

        return strtolower(preg_replace('/Widget/', '', end($type)));
    }

    /**
     * Set the mode.
     *
     * @param string $mode
     */
    public function setMode($mode)

    {
        $this->mode = $mode;
    }

    /**
     * Get the mode.
     *
     * @return string
     */
    public function getMode()
    {
        return $this->mode;
    }

    /**
     * Get the view id.
     *
     * @return int The view id
     */
    public function getViewId()
    {
        $viewId = null;

        $view = $this->getView();


        if ($view !== null) {
            $viewId = $view->getId();
        }

        return $viewId;
    }

    /**
     * @return string
     */
    public function getChildrenSlot()
    {
        return $this->childrenSlot ?: $this->getId();
    }

    /**
     * @param string $childrenSlot
     */
    public function setChildrenSlot($childrenSlot)
    {
        $this->childrenSlot = $childrenSlot;
    }

    /**
     * Set widgets.
     *
     * @param [WidgetMap] $widgetMap

     *
     * @return Widget
     */
    public function setWidgetMap(WidgetMap $widgetMap = null)
    {
        if ($widgetMap) {
            $widgetMap->addWidget($this);
        }
        $this->widgetMap = $widgetMap;

        return $this;
    }

    /**
     * Get widgets.
     *
     * @return [WidgetMap]

     */
    public function getWidgetMap()
    {
        return $this->widgetMap;
    }

    /**
     * Set the entity.
     *
     * @param object $entity
     */
    public function setEntity($entity)
    {
        $this->entity = $entity;
    }

    /**
     * Get the entity.
     *
     * @return number
     */
    public function getEntity()
    {
        //if there is no entity
        if ($this->entity === null) {
            //we try to get one from the proxy
            $entityProxy = $this->getEntityProxy();

            //if there is a proxy
            if ($entityProxy !== null) {
                $this->entity = $entityProxy->getEntity();
            }
        }

        return $this->entity;
    }

    /**
     * Clone a widget.
     */
    public function __clone()
    {
        // if there is a proxy
        if ($this->entityProxy) {
            // we clone this one
            $this->entityProxy = clone $this->entityProxy;
        }

        // This check should be in the __constructor, but Doctrine does not use __constructor to
        // instanciate entites but __clone method.
        if (property_exists(get_called_class(), 'widget')) {
            throw new \Exception(sprintf('A property $widget was found in %s object.
                The $widget property is reserved for Victoire.
                You should chose a different property name.', get_called_class()));
        }
    }

    /**
     * @deprecated
     * Get view
     *
     * @return string
     */
    public function getView()
    {
        return $this->view;

    }

    /**
     * @return [Criteria]

     */
    public function getCriterias()
    {
        return $this->criterias;
    }

    /**
     * @param [Criteria] $criterias

     */
    public function setCriterias($criterias)
    {
        $this->criterias = $criterias;
    }

    /**
     * @param Criteria $criteria
     */
    public function addCriteria($criteria)
    {
        $criteria->setWidget($this);
        $this->criterias[] = $criteria;
    }

    /**
     * @param Criteria $criteria
     */
    public function removeCriteria(Criteria $criteria)
    {
        $criteria->setWidget(null);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->criterias->removeElement($criteria);
    }

    /**
     * @param Criteria $criteria
     *
     * @return bool
     */
    public function hasCriteria(Criteria $criteria)
    {
        return $this->criterias->contains($criteria);
    }

    /**
     * @param string $criteriaAlias
     *
     * @return bool
     */
    public function hasCriteriaNamed($criteriaAlias)
    {
        return $this->criterias->exists(function ($key, $element) use ($criteriaAlias) {
            return $criteriaAlias === $element->getName();
        });
    }

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

    /**
     * @param string $quantum
     */
    public function setQuantum($quantum)
    {
        $this->quantum = $quantum;
    }

    /**
     * Generate the CacheId, insert params that can invalid the cache.
     *
     * @return string
     */
    public function generateCacheId()
    {
        if (!$this->getCurrentView()) {
            throw new \Exception(sprintf('Cannot generate an hash for widget %s if currentView is not defined.',
                $this->getId()
            ));
        }

        return sprintf('%s-%s-%s',
            $this->getId(),
            $this->getUpdatedAt()->getTimestamp(),
            $this->getCurrentView()->getReference()->getId()
        );
    }

    /**
     * This method parse the widget criterias to discover in which locale this widget is used.
     *
     * @param $defaultLocale

     *
     * @return string|null
     */
    public function guessLocale()
    {
        if ($this->hasCriteriaNamed('locale')) {
            foreach ($this->getCriterias() as $criteria) {
                if ($criteria->getName() === 'locale') {
                    return $criteria->getValue();
                }
            }
        }
    }
}


1		<?php
2
3		namespace Victoire\Bundle\WidgetBundle\Entity;
4
5		use Doctrine\Common\Collections\ArrayCollection;
6		use Doctrine\ORM\Mapping as ORM;
7		use Gedmo\Timestampable\Traits\TimestampableEntity;
8		use Victoire\Bundle\CoreBundle\Entity\EntityProxy;
9		use Victoire\Bundle\CriteriaBundle\Entity\Criteria;
10		use Victoire\Bundle\QueryBundle\Entity\QueryTrait;
11		use Victoire\Bundle\QueryBundle\Entity\VictoireQueryInterface;
12		use Victoire\Bundle\WidgetBundle\Entity\Traits\StyleTrait;
13		use Victoire\Bundle\WidgetBundle\Model\Widget as BaseWidget;
14		use Victoire\Bundle\WidgetMapBundle\Entity\WidgetMap;
15
16		/**
17		* Widget.
18		*
19		* @ORM\Table("vic_widget")
20		* @ORM\Entity(repositoryClass="Victoire\Bundle\WidgetBundle\Repository\WidgetRepository")
21		* @ORM\InheritanceType("JOINED")
22		* @ORM\DiscriminatorColumn(name="type", type="string")
23		*/
24		class Widget extends BaseWidget implements VictoireQueryInterface
25		{
26		use StyleTrait;
27		use QueryTrait;
28		use TimestampableEntity;
29
30		public function __construct()
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Missing function doc comment Loading history...
31		{
32		$this->childrenSlot = uniqid('', true);
33		$this->criterias = new ArrayCollection();
34		}
35
36		/**
37		* @var int
38		*
39		* @ORM\Column(name="id", type="integer")
40		* @ORM\Id
41		* @ORM\GeneratedValue(strategy="AUTO")
42		*/
43		protected $id;
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare class properties before methods Loading history...
44
45		/**
46		* @var string
47		*
48		* @ORM\Column(name="slot", type="string", length=255, nullable=true)
49		*/
50		protected $slot;
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare class properties before methods Loading history...
51
52		/**
53		* @var string
54		*
55		* @ORM\Column(name="childrenSlot", type="string", length=100, nullable=true)
56		*/
57		protected $childrenSlot;
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare class properties before methods Loading history...
58
59		/**
60		* @var string
61		*
62		* @ORM\Column(name="theme", type="string", length=255, nullable=true)
63		*/
64		protected $theme;
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare class properties before methods Loading history...
65
66		/**
67		* @var string
68		*
69		* @ORM\Column(name="asynchronous", type="boolean", nullable=true)
70		*/
71		protected $asynchronous;
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare class properties before methods Loading history...
72
73		/**
74		* @var string
75		*
76		* @ORM\Column(name="fields", type="text")
77		*/
78		protected $fields = 'a:0:{}';
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare class properties before methods Loading history...
79
80		/**
81		* @var string
82		*
83		* @ORM\Column(name="mode", type="string", length=255, nullable=false)
84		*/
85		protected $mode = self::MODE_STATIC;
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare class properties before methods Loading history...
86
87		/**
88		* Auto simple mode: joined entity.
89		*
90		* @var EntityProxy
91		*
92		* @ORM\ManyToOne(targetEntity="\Victoire\Bundle\CoreBundle\Entity\EntityProxy", inversedBy="widgets", cascade={"persist"})
93		* @ORM\JoinColumn(name="entityProxy_id", referencedColumnName="id", onDelete="CASCADE")
94		*/
95		protected $entityProxy;
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare class properties before methods Loading history...
96
97		/**
98		* The entity linked to the widget.
99		*
100		* @var unknown
101		*/
102		protected $entity;
		0 ignored issues – show introduced 2017-04-25 15:53 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare class properties before methods Loading history...
103
104		/**
105		* @deprecated Remove Doctrine mapping and property
106		*
107		* @var string
108		*
109		* @ORM\ManyToOne(targetEntity="\Victoire\Bundle\CoreBundle\Entity\View", inversedBy="widgets", cascade={"persist"})
110		* @ORM\JoinColumn(name="view_id", referencedColumnName="id", onDelete="CASCADE")
111		*/
112		protected $view;
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare class properties before methods Loading history...
113
114		/**
115		* @var WidgetMap
116		*
117		* @ORM\ManyToOne(targetEntity="\Victoire\Bundle\WidgetMapBundle\Entity\WidgetMap", inversedBy="widgets")
118		* @ORM\JoinColumn(name="widget_map_id", referencedColumnName="id", onDelete="SET NULL"))
119		*/
120		protected $widgetMap;
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare class properties before methods Loading history...
121
122		/**
123		* @var [Criteria]
124		*
125		* @ORM\OneToMany(targetEntity="\Victoire\Bundle\CriteriaBundle\Entity\Criteria", mappedBy="widget", cascade={"persist", "remove"}, orphanRemoval=true)
126		*/
127		protected $criterias;
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare class properties before methods Loading history...
128
129		/**
130		* @var string
131		*
132		* @ORM\Column(name="quantum", type="string", length=255, nullable=true)
133		*/
134		protected $quantum;
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare class properties before methods Loading history...
135
136		/**
137		* @return string
138		*/
139		public function isAsynchronous()
140		{
141		return $this->asynchronous;
142		}
143
144		/**
145		* @param string $asynchronous
146		*/
147		public function setAsynchronous($asynchronous)
148		{
149		$this->asynchronous = $asynchronous;
150		}
151
152		/**
153		* Set the entity proxy.
154		*
155		* @param EntityProxy $entityProxy
156		*/
157		public function setEntityProxy(EntityProxy $entityProxy)
158		{
159		$this->entityProxy = $entityProxy;
160		}
161
162		/**
163		* Get the entity proxy.
164		*
165		* @return EntityProxy
166		*/
167		public function getEntityProxy()
168		{
169		return $this->entityProxy;
170		}
171
172		/**
173		* Get businessEntityName.
174		*
175		* @return string
176		*/
177		public function getBusinessEntityName()
178		{
179		if ($entityProxy = $this->getEntityProxy()) {
180		return $entityProxy->getBusinessEntity()->getName();
181		}
182
183		return $this->businessEntityName;
		0 ignored issues – show Deprecated Code introduced 2017-03-08 18:26 UTC by Report Bug Copy Issue Report Show Similar Issues like this The property `Victoire\Bundle\QueryBun...it::$businessEntityName` has been deprecated with message: Auto list mode: businessentity type. 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...
184		}
185
186		/**
187		* to string.
188		*
189		* @return string
190		*/
191		public function __toString()
192		{
193		return (string) $this->getId();
194		}
195
196		/**
197		* Get id.
198		*
199		* @return int
200		*/
201		public function getId()
202		{
203		return $this->id;
204		}
205
206		/**
207		* Set fields.
208		*
209		* @param string $fields
210		*
211		* @return Widget
212		*/
213	View Code Duplication	public function setFields($fields)
		0 ignored issues – show Duplication introduced 2017-03-08 18:26 UTC by Report Bug Copy Issue Report Show Similar Issues like this This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
214		{
215		$data = @unserialize($fields);
216		if ($fields === 'b:0;' \|\| $data !== false) {
217		$this->fields = $fields;
218		} else {
219		$this->fields = serialize($fields);
220		}
221
222		return $this;
223		}
224
225		/**
226		* Get fields.
227		*
228		* @return string
229		*/
230		public function getFields()
231		{
232		return unserialize($this->fields);
233		}
234
235		/**
236		* Set The Id.
237		*
238		* @param int $id The id
239		*/
240		public function setId($id)
241		{
242		$this->id = $id;
243		}
244
245		/**
246		* Set slot.
247		*
248		* @param string $slot
249		*
250		* @return Widget
251		*/
252		public function setSlot($slot)
253		{
254		$this->slot = $slot;
255
256		return $this;
257		}
258
259		/**
260		* Get slot.
261		*
262		* @return string
263		*/
264		public function getSlot()
265		{
266		return $this->slot;
267		}
268
269		/**
270		* Set theme.
271		*
272		* @param string $theme
273		*
274		* @return Widget
275		*/
276		public function setTheme($theme)
277		{
278		$this->theme = $theme;
279
280		return $this;
281		}
282
283		/**
284		* Get theme.
285		*
286		* @return string
287		*/
288		public function getTheme()
289		{
290		return $this->theme;
291		}
292
293		/**
294		* Get the type of the object.
295		*
296		* @return string The type
297		*/
298		public function getType()
299		{
300		return $this->guessType();
301		}
302
303		/**
304		* Guess the type of this by exploding and getting the last item.
305		*
306		* @return string The guessed type
307		*/
308		protected function guessType()
309		{
310		$type = explode('\\', get_class($this));
311
312		return strtolower(preg_replace('/Widget/', '', end($type)));
313		}
314
315		/**
316		* Set the mode.
317		*
318		* @param string $mode
319		*/
320		public function setMode($mode)
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Declare public methods first,then protected ones and finally private ones Loading history...
321		{
322		$this->mode = $mode;
323		}
324
325		/**
326		* Get the mode.
327		*
328		* @return string
329		*/
330		public function getMode()
331		{
332		return $this->mode;
333		}
334
335		/**
336		* Get the view id.
337		*
338		* @return int The view id
339		*/
340		public function getViewId()
341		{
342		$viewId = null;
343
344		$view = $this->getView();
		0 ignored issues – show Deprecated Code introduced 2017-03-08 09:49 UTC by Report Bug Copy Issue Report Show Similar Issues like this The method `Victoire\Bundle\WidgetBu...ntity\Widget::getView()` has been deprecated with message: Get view This method 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 method will be removed from the class and what other method or class to use instead. Loading history...
345
346		if ($view !== null) {
347		$viewId = $view->getId();
348		}
349
350		return $viewId;
351		}
352
353		/**
354		* @return string
355		*/
356		public function getChildrenSlot()
357		{
358		return $this->childrenSlot ?: $this->getId();
359		}
360
361		/**
362		* @param string $childrenSlot
363		*/
364		public function setChildrenSlot($childrenSlot)
365		{
366		$this->childrenSlot = $childrenSlot;
367		}
368
369		/**
370		* Set widgets.
371		*
372		* @param [WidgetMap] $widgetMap
		0 ignored issues – show Documentation introduced 2016-02-01 08:33 UTC by Report Bug Copy Issue Report Show Similar Issues like this The doc-type `[WidgetMap]` could not be parsed: Unknown type name "" at position 0. [(view supported doc-types) This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types. Loading history...
373		*
374		* @return Widget
375		*/
376		public function setWidgetMap(WidgetMap $widgetMap = null)
377		{
378		if ($widgetMap) {
379		$widgetMap->addWidget($this);
380		}
381		$this->widgetMap = $widgetMap;
382
383		return $this;
384		}
385
386		/**
387		* Get widgets.
388		*
389		* @return [WidgetMap]
		0 ignored issues – show Documentation introduced 2016-02-01 08:33 UTC by Report Bug Copy Issue Report Show Similar Issues like this The doc-type `[WidgetMap]` could not be parsed: Unknown type name "" at position 0. [(view supported doc-types) This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types. Loading history...
390		*/
391		public function getWidgetMap()
392		{
393		return $this->widgetMap;
394		}
395
396		/**
397		* Set the entity.
398		*
399		* @param object $entity
400		*/
401		public function setEntity($entity)
402		{
403		$this->entity = $entity;
404		}
405
406		/**
407		* Get the entity.
408		*
409		* @return number
410		*/
411		public function getEntity()
412		{
413		//if there is no entity
414		if ($this->entity === null) {
415		//we try to get one from the proxy
416		$entityProxy = $this->getEntityProxy();
417
418		//if there is a proxy
419		if ($entityProxy !== null) {
420		$this->entity = $entityProxy->getEntity();
421		}
422		}
423
424		return $this->entity;
425		}
426
427		/**
428		* Clone a widget.
429		*/
430		public function __clone()
431		{
432		// if there is a proxy
433		if ($this->entityProxy) {
434		// we clone this one
435		$this->entityProxy = clone $this->entityProxy;
436		}
437
438		// This check should be in the __constructor, but Doctrine does not use __constructor to
439		// instanciate entites but __clone method.
440		if (property_exists(get_called_class(), 'widget')) {
441		throw new \Exception(sprintf('A property $widget was found in %s object.
442		The $widget property is reserved for Victoire.
443		You should chose a different property name.', get_called_class()));
444		}
445		}
446
447		/**
448		* @deprecated
449		* Get view
450		*
451		* @return string
452		*/
453		public function getView()
454		{
455		return $this->view;
		0 ignored issues – show Deprecated Code introduced 2017-03-13 13:33 UTC by Report Bug Copy Issue Report Show Similar Issues like this The property `Victoire\Bundle\WidgetBundle\Entity\Widget::$view` has been deprecated with message: Remove Doctrine mapping and property 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...
456		}
457
458		/**
459		* @return [Criteria]
		0 ignored issues – show Documentation introduced 2016-04-01 22:40 UTC by Report Bug Copy Issue Report Show Similar Issues like this The doc-type `[Criteria]` could not be parsed: Unknown type name "" at position 0. [(view supported doc-types) This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types. Loading history...
460		*/
461		public function getCriterias()
462		{
463		return $this->criterias;
464		}
465
466		/**
467		* @param [Criteria] $criterias
		0 ignored issues – show Documentation introduced 2016-04-01 22:40 UTC by Report Bug Copy Issue Report Show Similar Issues like this The doc-type `[Criteria]` could not be parsed: Unknown type name "" at position 0. [(view supported doc-types) This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types. Loading history...
468		*/
469		public function setCriterias($criterias)
470		{
471		$this->criterias = $criterias;
472		}
473
474		/**
475		* @param Criteria $criteria
476		*/
477		public function addCriteria($criteria)
478		{
479		$criteria->setWidget($this);
480		$this->criterias[] = $criteria;
481		}
482
483		/**
484		* @param Criteria $criteria
485		*/
486		public function removeCriteria(Criteria $criteria)
487		{
488		$criteria->setWidget(null);
		0 ignored issues – show Documentation introduced 2016-04-01 22:40 UTC by Report Bug Copy Issue Report Show Similar Issues like this `null` is of type `null`, but the function expects a `object<Victoire\Bundle\W...etBundle\Entity\Widget>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
489		$this->criterias->removeElement($criteria);
490		}
491
492		/**
493		* @param Criteria $criteria
494		*
495		* @return bool
496		*/
497		public function hasCriteria(Criteria $criteria)
498		{
499		return $this->criterias->contains($criteria);
500		}
501
502		/**
503		* @param string $criteriaAlias
504		*
505		* @return bool
506		*/
507		public function hasCriteriaNamed($criteriaAlias)
508		{
509		return $this->criterias->exists(function ($key, $element) use ($criteriaAlias) {
510		return $criteriaAlias === $element->getName();
511		});
512		}
513
514		/**
515		* @return string
516		*/
517		public function getQuantum()
518		{
519		return $this->quantum;
520		}
521
522		/**
523		* @param string $quantum
524		*/
525		public function setQuantum($quantum)
526		{
527		$this->quantum = $quantum;
528		}
529
530		/**
531		* Generate the CacheId, insert params that can invalid the cache.
532		*
533		* @return string
534		*/
535		public function generateCacheId()
536		{
537		if (!$this->getCurrentView()) {
538		throw new \Exception(sprintf('Cannot generate an hash for widget %s if currentView is not defined.',
539		$this->getId()
540		));
541		}
542
543		return sprintf('%s-%s-%s',
544		$this->getId(),
545		$this->getUpdatedAt()->getTimestamp(),
546		$this->getCurrentView()->getReference()->getId()
547		);
548		}
549
550		/**
551		* This method parse the widget criterias to discover in which locale this widget is used.
552		*
553		* @param $defaultLocale
		0 ignored issues – show introduced 2017-03-21 09:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Missing parameter name Loading history...
554		*
555		* @return string\|null
556		*/
557		public function guessLocale()
558		{
559		if ($this->hasCriteriaNamed('locale')) {
560		foreach ($this->getCriterias() as $criteria) {
561		if ($criteria->getName() === 'locale') {
562		return $criteria->getValue();
563		}
564		}
565		}
566		}
567		}
568

Victoire / victoire

Issues (1704)

Security Analysis not enabled

Bundle/WidgetBundle/Entity/Widget.php (25 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like