EntityTrait::unsetSelfFields() - Code Metrics - Inspection of "upload coverage.clover in php 7.1" - rhosocial/yii2-base-models - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 19b92d...15c8d0 )

by vistart

created 2017-01-14 16:54 UTC

EntityTrait::unsetSelfFields() A

↳ Parent: EntityTrait

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

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

<?php

/**
 *   _   __ __ _____ _____ ___  ____  _____
 *  | | / // // ___//_  _//   ||  __||_   _|
 *  | |/ // /(__  )  / / / /| || |     | |
 *  |___//_//____/  /_/ /_/ |_||_|     |_|
 * @link https://vistart.me/
 * @copyright Copyright (c) 2016 - 2017 vistart
 * @license https://vistart.me/license/
 */

namespace rhosocial\base\models\traits;

use Yii;
use yii\base\ModelEvent;
use yii\caching\Cache;
use yii\caching\TagDependency;

/**
 * This trait must be used in class extended from ActiveRecord. The ActiveRecord
 * supports [[\yii\db\ActiveRecord]], [[\yii\mongodb\ActiveRecord]], [[\yii\redis\ActiveRecord]].
 * @property array $entityRules
 * @property array $entityBehaviors
 * @version 1.0
 * @author vistart <[email protected]>
 */
trait EntityTrait
{
    use GUIDTrait, IDTrait, IPTrait, TimestampTrait;
    
    private $entityLocalRules = [];
    private $entityLocalBehaviors = [];

    /**
     * @var string cache key and tag prefix. the prefix is usually set to full
     * qualified class name.
     */
    public $cachePrefix = '';
    public static $eventNewRecordCreated = 'newRecordCreated';
    public static $cacheKeyEntityRules = 'entity_rules';
    public static $cacheTagEntityRules = 'tag_entity_rules';
    public static $cacheKeyEntityBehaviors = 'entity_behaviors';
    public static $cacheTagEntityBehaviors = 'tag_entity_behaviors';
    
    /**
     * @var string cache component id.
     */
    public $cacheId = 'cache';
    
    /**
     * @var boolean Determines to skip initialization.
     */
    public $skipInit = false;
    
    /**
     * @var string the name of query class or sub-class.
     */
    public $queryClass;
    
    /**
     * @return \static New self without any initializations.
     */
    public static function buildNoInitModel()
    {
        return new static(['skipInit' => true]);

    }
    
    /**
     * Populate and return the entity rules.
     * You should call this function in your extended class and merge the result
     * with your rules, instead of overriding it, unless you know the
     * consequences.
     * The classical rules are like following:
     * [
     *     ['guid', 'required'],
     *     ['guid', 'unique'],
     *     ['guid', 'string', 'max' => 36],
     *
     *     ['id', 'required'],
     *     ['id', 'unique'],
     *     ['id', 'string', 'max' => 4],
     *
     *     ['created_at', 'safe'],
     *     ['updated_at', 'safe'],
     *
     *     ['ip_type', 'in', 'range' => [4, 6]],
     *     ['ip', 'number', 'integerOnly' => true, 'min' => 0],
     * ]
     * @return array
     */
    public function rules()
    {
        return $this->getEntityRules();
    }
    
    /**
     * Populate and return the entity behaviors.
     * You should call this function in your extended class and merge the result
     * with your behaviors, instead of overriding it, unless you know the
     * consequences.
     * @return array
     */
    public function behaviors()
    {
        return $this->getEntityBehaviors();
    }
    
    /**
     * Get cache component. If cache component is not configured, Yii::$app->cache
     * will be given.
     * @return Cache cache component.
     */
    protected function getCache()
    {
        $cacheId = $this->cacheId;
        return empty($cacheId) ? Yii::$app->cache : Yii::$app->$cacheId;
    }
    
    /**
     * Get entity rules cache key.
     * @return string cache key.
     */
    public function getEntityRulesCacheKey()
    {
        return static::class . $this->cachePrefix . static::$cacheKeyEntityRules;
    }
    
    /**
     * Get entity rules cache tag.
     * @return string cache tag.
     */
    public function getEntityRulesCacheTag()
    {
        return static::class . $this->cachePrefix . static::$cacheTagEntityRules;
    }
    
    /**
     * Get entity rules.
     * @return array rules.
     */
    public function getEntityRules()
    {
        $cache = $this->getCache();
        if ($cache) {
            $this->entityLocalRules = $cache->get($this->getEntityRulesCacheKey());

        }
        if (empty($this->entityLocalRules) || !is_array($this->entityLocalRules)) {
            $rules = array_merge($this->getGuidRules(), $this->getIdRules(), $this->getCreatedAtRules(), $this->getUpdatedAtRules(), $this->getExpiredAfterRules(), $this->getIpRules());
            $this->setEntityRules($rules);
        }
        return $this->entityLocalRules;
    }
    
    /**
     * Set entity rules.
     * @param array $rules
     */
    protected function setEntityRules($rules = [])
    {
        $this->entityLocalRules = $rules;
        $cache = $this->getCache();
        if ($cache) {
            $tagDependency = new TagDependency(
                ['tags' => [$this->getEntityRulesCacheTag()]]
            );
            $cache->set($this->getEntityRulesCacheKey(), $rules, 0, $tagDependency);
        }
    }
    
    /**
     * Get entity behaviors cache key.
     * @return string cache key.
     */
    public function getEntityBehaviorsCacheKey()
    {
        return static::class . $this->cachePrefix . static::$cacheKeyEntityBehaviors;
    }
    
    /**
     * Get entity behaviors cache tag.
     * @return string cache tag.
     */
    public function getEntityBehaviorsCacheTag()
    {
        return static::class . $this->cachePrefix . static::$cacheTagEntityBehaviors;
    }
    
    /**
     * Get the entity behaviors.
     * @return array
     */
    public function getEntityBehaviors()
    {
        $cache = $this->getCache();
        if ($cache) {
            $this->entityLocalBehaviors = $cache->get($this->getEntityBehaviorsCacheKey());

        }
        if (empty($this->entityLocalBehaviors) || !is_array($this->entityLocalBehaviors)) {
            $this->setEntityBehaviors($this->getTimestampBehaviors());
        }
        return $this->entityLocalBehaviors;
    }
    
    /**
     * Set the entity behaviors.
     * @param array $behaviors
     */
    protected function setEntityBehaviors($behaviors)
    {
        $this->entityLocalBehaviors = $behaviors;
        $cache = $this->getCache();
        if ($cache) {
            $tagDependencyConfig = ['tags' => [$this->getEntityBehaviorsCacheTag()]];
            $tagDependency = new TagDependency($tagDependencyConfig);
            $cache->set($this->getEntityBehaviorsCacheKey(), $behaviors, 0, $tagDependency);
        }
    }
    
    /**
     * Reset cache key.
     * @param string $cacheKey
     * @param mixed $value
     * @return boolean whether the value is successfully stored into cache. if
     * cache component was not configured, then return false directly.
     */
    public function resetCacheKey($cacheKey, $value = false)
    {
        $cache = $this->getCache();
        if ($cache) {
            return $this->getCache()->set($cacheKey, $value);
        }
        return false;
    }
    
    /**
     * Attach events associated with entity model.
     */
    protected function initEntityEvents()
    {
        $this->on(static::EVENT_INIT, [$this, 'onInitCache']);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->attachInitGUIDEvent(static::$eventNewRecordCreated);
        $this->attachInitIDEvent(static::$eventNewRecordCreated);
        $this->attachInitIPEvent(static::$eventNewRecordCreated);
        if ($this->isNewRecord) {
class MyClass { }

$x = new MyClass();
$x->foo = true;
            $this->trigger(static::$eventNewRecordCreated);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        }
        $this->on(static::EVENT_AFTER_FIND, [$this, 'onRemoveExpired']);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }
    
    /**
     * Initialize the cache prefix.
     * @param ModelEvent $event
     */
    public function onInitCache($event)
    {
        $sender = $event->sender;
        $data = $event->data;
        if (isset($data['prefix'])) {
            $sender->cachePrefix = $data['prefix'];
        } else {
            $sender->cachePrefix = $sender::className();
        }
    }
    
    /**
     * Record warnings.
     */
    protected function recordWarnings()
    {
        if (YII_ENV !== YII_ENV_PROD || YII_DEBUG) {
            Yii::warning($this->errors);
class MyClass { }

$x = new MyClass();
$x->foo = true;
        }
    }
    
    /**
     * Get guid or id. if neither disabled, return null.
     * @return string
     */
    public function __toString()
    {
        if (is_string($this->guidAttribute) && !empty($this->guidAttribute)) {
            return $this->getGUID();
        }
        if (is_string($this->idAttribute) && !empty($this->idAttribute)) {
            return $this->getId();
        }
        return parent::__toString();
    }
    
    /**
     * @inheritdoc
     * -------------
     * if enable `$idAttribute` and $row[$idAttribute] set, the `idPreassigned`
     * will be assigned to true.
     */
    public static function instantiate($row)
    {
        $self = static::buildNoInitModel();
        if (isset($self->idAttribute) && isset($row[$self->idAttribute])) {
            $model = new static(['idPreassigned' => true]);

        } else {
            $model = new static;
        }
        return $model;
    }
    
    /**
     * unset entity attributes.
     * @return array result.
     */
    public function unsetSelfFields()
    {
        return static::unsetFields($this->attributes, $this->enabledFields());

    }
    
    /**
     * unset fields of array.
     * @param array $array
     * @param array $fields
     * @return array
     */
    public static function unsetFields($array, $fields = null)
    {
        if (!is_array($array)) {
            $fields = [];
        }
        foreach ($array as $key => $value) {
            if (is_string($key) && in_array($key, $fields)) {
                unset($array[$key]);
            }
        }
        return $array;
    }
    
    /**
     * Get enabled fields.
     * @return string[]
     */
    public function enabledFields()
    {
        return array_merge(
            (is_string($this->guidAttribute) && !empty($this->guidAttribute)) ? [$this->guidAttribute] : [],
            (is_string($this->idAttribute) && !empty($this->idAttribute)) ? [$this->idAttribute] : [],
            $this->enabledTimestampFields(),
            $this->enabledIPFields()
        );
    }
}


1		<?php
2
3		/**
4		* _ __ __ _____ _____ ___ ____ _____
5		* \| \| / // // ___//_ _// \|\| __\|\|_ _\|
6		* \| \|/ // /(__ ) / / / /\| \|\| \| \| \|
7		* \|___//_//____/ /_/ /_/ \|_\|\|_\| \|_\|
8		* @link https://vistart.me/
9		* @copyright Copyright (c) 2016 - 2017 vistart
10		* @license https://vistart.me/license/
11		*/
12
13		namespace rhosocial\base\models\traits;
14
15		use Yii;
16		use yii\base\ModelEvent;
17		use yii\caching\Cache;
18		use yii\caching\TagDependency;
19
20		/**
21		* This trait must be used in class extended from ActiveRecord. The ActiveRecord
22		* supports [[\yii\db\ActiveRecord]], [[\yii\mongodb\ActiveRecord]], [[\yii\redis\ActiveRecord]].
23		* @property array $entityRules
24		* @property array $entityBehaviors
25		* @version 1.0
26		* @author vistart <[email protected]>
27		*/
28		trait EntityTrait
29		{
30		use GUIDTrait, IDTrait, IPTrait, TimestampTrait;
31
32		private $entityLocalRules = [];
33		private $entityLocalBehaviors = [];
34
35		/**
36		* @var string cache key and tag prefix. the prefix is usually set to full
37		* qualified class name.
38		*/
39		public $cachePrefix = '';
40		public static $eventNewRecordCreated = 'newRecordCreated';
41		public static $cacheKeyEntityRules = 'entity_rules';
42		public static $cacheTagEntityRules = 'tag_entity_rules';
43		public static $cacheKeyEntityBehaviors = 'entity_behaviors';
44		public static $cacheTagEntityBehaviors = 'tag_entity_behaviors';
45
46		/**
47		* @var string cache component id.
48		*/
49		public $cacheId = 'cache';
50
51		/**
52		* @var boolean Determines to skip initialization.
53		*/
54		public $skipInit = false;
55
56		/**
57		* @var string the name of query class or sub-class.
58		*/
59		public $queryClass;
60
61		/**
62		* @return \static New self without any initializations.
63		*/
64	161	public static function buildNoInitModel()
65		{
66	161	return new static(['skipInit' => true]);
		0 ignored issues – show Unused Code introduced 2017-01-12 08:52 UTC by Report Bug Copy Issue Report The call to `EntityTrait::__construct()` has too many arguments starting with `array('skipInit' => true)`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
67		}
68
69		/**
70		* Populate and return the entity rules.
71		* You should call this function in your extended class and merge the result
72		* with your rules, instead of overriding it, unless you know the
73		* consequences.
74		* The classical rules are like following:
75		* [
76		* ['guid', 'required'],
77		* ['guid', 'unique'],
78		* ['guid', 'string', 'max' => 36],
79		*
80		* ['id', 'required'],
81		* ['id', 'unique'],
82		* ['id', 'string', 'max' => 4],
83		*
84		* ['created_at', 'safe'],
85		* ['updated_at', 'safe'],
86		*
87		* ['ip_type', 'in', 'range' => [4, 6]],
88		* ['ip', 'number', 'integerOnly' => true, 'min' => 0],
89		* ]
90		* @return array
91		*/
92	154	public function rules()
93		{
94	154	return $this->getEntityRules();
95		}
96
97		/**
98		* Populate and return the entity behaviors.
99		* You should call this function in your extended class and merge the result
100		* with your behaviors, instead of overriding it, unless you know the
101		* consequences.
102		* @return array
103		*/
104	171	public function behaviors()
105		{
106	171	return $this->getEntityBehaviors();
107		}
108
109		/**
110		* Get cache component. If cache component is not configured, Yii::$app->cache
111		* will be given.
112		* @return Cache cache component.
113		*/
114	171	protected function getCache()
115		{
116	171	$cacheId = $this->cacheId;
117	171	return empty($cacheId) ? Yii::$app->cache : Yii::$app->$cacheId;
118		}
119
120		/**
121		* Get entity rules cache key.
122		* @return string cache key.
123		*/
124	154	public function getEntityRulesCacheKey()
125		{
126	154	return static::class . $this->cachePrefix . static::$cacheKeyEntityRules;
127		}
128
129		/**
130		* Get entity rules cache tag.
131		* @return string cache tag.
132		*/
133	154	public function getEntityRulesCacheTag()
134		{
135	154	return static::class . $this->cachePrefix . static::$cacheTagEntityRules;
136		}
137
138		/**
139		* Get entity rules.
140		* @return array rules.
141		*/
142	154	public function getEntityRules()
143		{
144	154	$cache = $this->getCache();
145	154	if ($cache) {
146	154	$this->entityLocalRules = $cache->get($this->getEntityRulesCacheKey());
		0 ignored issues – show Documentation Bug introduced 2017-01-12 08:52 UTC by Report Bug Copy Issue Report It seems like `$cache->get($this->getEntityRulesCacheKey())` of type `*` is incompatible with the declared type `array` of property `$entityLocalRules`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
147		}
148	154	if (empty($this->entityLocalRules) \|\| !is_array($this->entityLocalRules)) {
149	154	$rules = array_merge($this->getGuidRules(), $this->getIdRules(), $this->getCreatedAtRules(), $this->getUpdatedAtRules(), $this->getExpiredAfterRules(), $this->getIpRules());
150	154	$this->setEntityRules($rules);
151		}
152	154	return $this->entityLocalRules;
153		}
154
155		/**
156		* Set entity rules.
157		* @param array $rules
158		*/
159	154	protected function setEntityRules($rules = [])
160		{
161	154	$this->entityLocalRules = $rules;
162	154	$cache = $this->getCache();
163	154	if ($cache) {
164	154	$tagDependency = new TagDependency(
165	154	['tags' => [$this->getEntityRulesCacheTag()]]
166		);
167	154	$cache->set($this->getEntityRulesCacheKey(), $rules, 0, $tagDependency);
168		}
169	154	}
170
171		/**
172		* Get entity behaviors cache key.
173		* @return string cache key.
174		*/
175	171	public function getEntityBehaviorsCacheKey()
176		{
177	171	return static::class . $this->cachePrefix . static::$cacheKeyEntityBehaviors;
178		}
179
180		/**
181		* Get entity behaviors cache tag.
182		* @return string cache tag.
183		*/
184	171	public function getEntityBehaviorsCacheTag()
185		{
186	171	return static::class . $this->cachePrefix . static::$cacheTagEntityBehaviors;
187		}
188
189		/**
190		* Get the entity behaviors.
191		* @return array
192		*/
193	171	public function getEntityBehaviors()
194		{
195	171	$cache = $this->getCache();
196	171	if ($cache) {
197	171	$this->entityLocalBehaviors = $cache->get($this->getEntityBehaviorsCacheKey());
		0 ignored issues – show Documentation Bug introduced 2017-01-12 08:52 UTC by Report Bug Copy Issue Report It seems like `$cache->get($this->getEntityBehaviorsCacheKey())` of type `*` is incompatible with the declared type `array` of property `$entityLocalBehaviors`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
198		}
199	171	if (empty($this->entityLocalBehaviors) \|\| !is_array($this->entityLocalBehaviors)) {
200	171	$this->setEntityBehaviors($this->getTimestampBehaviors());
201		}
202	171	return $this->entityLocalBehaviors;
203		}
204
205		/**
206		* Set the entity behaviors.
207		* @param array $behaviors
208		*/
209	171	protected function setEntityBehaviors($behaviors)
210		{
211	171	$this->entityLocalBehaviors = $behaviors;
212	171	$cache = $this->getCache();
213	171	if ($cache) {
214	171	$tagDependencyConfig = ['tags' => [$this->getEntityBehaviorsCacheTag()]];
215	171	$tagDependency = new TagDependency($tagDependencyConfig);
216	171	$cache->set($this->getEntityBehaviorsCacheKey(), $behaviors, 0, $tagDependency);
217		}
218	171	}
219
220		/**
221		* Reset cache key.
222		* @param string $cacheKey
223		* @param mixed $value
224		* @return boolean whether the value is successfully stored into cache. if
225		* cache component was not configured, then return false directly.
226		*/
227	1	public function resetCacheKey($cacheKey, $value = false)
228		{
229	1	$cache = $this->getCache();
230	1	if ($cache) {
231	1	return $this->getCache()->set($cacheKey, $value);
232		}
233		return false;
234		}
235
236		/**
237		* Attach events associated with entity model.
238		*/
239	171	protected function initEntityEvents()
240		{
241	171	$this->on(static::EVENT_INIT, [$this, 'onInitCache']);
		0 ignored issues – show Bug introduced 2017-01-12 08:52 UTC by Report Bug Copy Issue Report It seems like `on()` 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...
242	171	$this->attachInitGUIDEvent(static::$eventNewRecordCreated);
243	171	$this->attachInitIDEvent(static::$eventNewRecordCreated);
244	171	$this->attachInitIPEvent(static::$eventNewRecordCreated);
245	171	if ($this->isNewRecord) {
		0 ignored issues – show Bug introduced 2017-01-12 08:52 UTC by Report Bug Copy Issue Report The property `isNewRecord` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
246	171	$this->trigger(static::$eventNewRecordCreated);
		0 ignored issues – show Bug introduced 2017-01-12 08:52 UTC by Report Bug Copy Issue Report It seems like `trigger()` 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...
247		}
248	171	$this->on(static::EVENT_AFTER_FIND, [$this, 'onRemoveExpired']);
		0 ignored issues – show Bug introduced 2017-01-12 08:52 UTC by Report Bug Copy Issue Report It seems like `on()` 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...
249	171	}
250
251		/**
252		* Initialize the cache prefix.
253		* @param ModelEvent $event
254		*/
255	171	public function onInitCache($event)
256		{
257	171	$sender = $event->sender;
258	171	$data = $event->data;
259	171	if (isset($data['prefix'])) {
260		$sender->cachePrefix = $data['prefix'];
261		} else {
262	171	$sender->cachePrefix = $sender::className();
263		}
264	171	}
265
266		/**
267		* Record warnings.
268		*/
269		protected function recordWarnings()
270		{
271		if (YII_ENV !== YII_ENV_PROD \|\| YII_DEBUG) {
272		Yii::warning($this->errors);
		0 ignored issues – show Bug introduced 2017-01-12 08:52 UTC by Report Bug Copy Issue Report The property `errors` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
273		}
274		}
275
276		/**
277		* Get guid or id. if neither disabled, return null.
278		* @return string
279		*/
280	14	public function __toString()
281		{
282	14	if (is_string($this->guidAttribute) && !empty($this->guidAttribute)) {
283	14	return $this->getGUID();
284		}
285		if (is_string($this->idAttribute) && !empty($this->idAttribute)) {
286		return $this->getId();
287		}
288		return parent::__toString();
289		}
290
291		/**
292		* @inheritdoc
293		* -------------
294		* if enable `$idAttribute` and $row[$idAttribute] set, the `idPreassigned`
295		* will be assigned to true.
296		*/
297	67	public static function instantiate($row)
298		{
299	67	$self = static::buildNoInitModel();
300	67	if (isset($self->idAttribute) && isset($row[$self->idAttribute])) {
301	67	$model = new static(['idPreassigned' => true]);
		0 ignored issues – show Unused Code introduced 2017-01-12 08:52 UTC by Report Bug Copy Issue Report The call to `EntityTrait::__construct()` has too many arguments starting with `array('idPreassigned' => true)`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
302		} else {
303		$model = new static;
304		}
305	67	return $model;
306		}
307
308		/**
309		* unset entity attributes.
310		* @return array result.
311		*/
312	1	public function unsetSelfFields()
313		{
314	1	return static::unsetFields($this->attributes, $this->enabledFields());
		0 ignored issues – show Bug introduced 2017-01-12 08:52 UTC by Report Bug Copy Issue Report The property `attributes` does not seem to exist. Did you mean `idAttributeSafe`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
315		}
316
317		/**
318		* unset fields of array.
319		* @param array $array
320		* @param array $fields
321		* @return array
322		*/
323	1	public static function unsetFields($array, $fields = null)
324		{
325	1	if (!is_array($array)) {
326		$fields = [];
327		}
328	1	foreach ($array as $key => $value) {
329	1	if (is_string($key) && in_array($key, $fields)) {
330	1	unset($array[$key]);
331		}
332		}
333	1	return $array;
334		}
335
336		/**
337		* Get enabled fields.
338		* @return string[]
339		*/
340	40	public function enabledFields()
341		{
342	40	return array_merge(
343	40	(is_string($this->guidAttribute) && !empty($this->guidAttribute)) ? [$this->guidAttribute] : [],
344	40	(is_string($this->idAttribute) && !empty($this->idAttribute)) ? [$this->idAttribute] : [],
345	40	$this->enabledTimestampFields(),
346	40	$this->enabledIPFields()
347		);
348		}
349		}
350

rhosocial / yii2-base-models

Push — master ( 19b92d...15c8d0 )

EntityTrait::unsetSelfFields() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like