CategoryTerm::getChildren() - Code Metrics - Inspection of "Adding category term for managing hierarchies of t..." - nkostadinov/yii2-taxonomy - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#4)

unknown

created 2016-10-21 11:10 UTC

CategoryTerm::getChildren() A

↳ Parent: CategoryTerm

Complexity

Conditions	3
Paths	3

Size

Total Lines	19
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	19
c	0
b	0
f	0
rs	9.4285
cc	3
eloc	12
nc	3
nop	1

<?php

namespace nkostadinov\taxonomy\components\terms;

use insight\core\helpers\ArrayHelper;
use nkostadinov\taxonomy\components\interfaces\IHierarchicalTerm;
use nkostadinov\taxonomy\models\TaxonomyDef;
use nkostadinov\taxonomy\models\TaxonomyTerms;
use Yii;
use yii\db\Query;

class CategoryTerm extends TagTerm implements IHierarchicalTerm
{
    public $templateFile = '@nkostadinov/taxonomy/migrations/template/category.php' ;

    public function getTerms($object_id, $name = [])
    {
        $query = (new Query())
            ->select(TaxonomyTerms::tableName() . '.term')
            ->from(TaxonomyTerms::tableName())
            ->innerJoin($this->table, $this->table . '.term_id = taxonomy_terms.id',
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
                [':object_id' => $object_id])
            ->andFilterWhere(['taxonomy_terms.term' => $name]);

        if ($object_id) {
            $query->onCondition("$this->table.object_id = $object_id");
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        }

        $result = [];
        foreach($query->all() as $v) {
            $result[] = $v['term'];
        }

        return $result;
    }

    public function addTerm($object_id, $params)
    {
        $cachedParents = [];

        $addTerm = function ($parent, $item) use ($object_id, &$cachedParents) {
            $term = $this->getTaxonomyTerm($item);
            $data['term_id'] = $term->id;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
            $data['object_id'] = $object_id;

            if (array_key_exists($parent, $cachedParents)) {
                $term->parent_id = $cachedParents[$parent]->id;
            } else if (is_string($parent)) {
                $parentTerm = $this->getTaxonomyTerm($parent);
                $cachedParents[$parent] = $parentTerm;
                $term->parent_id = $parentTerm->id;
            }

            if (!(new Query())->from($this->table)->where($data)->exists(CategoryTerm::getDb())) {
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
                Yii::$app->db->transaction(function() use ($data, $term) {
                    if ($term->getDirtyAttributes(['parent_id'])) {
                        $term->save(false);
                    }
                    CategoryTerm::getDb()->createCommand()->insert($this->table, $data)->execute();
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

                    $term->updateCounters(['total_count' => 1]);
                    TaxonomyDef::updateAllCounters(['total_count' => 1], ['id' => $this->id]);
                });
            }
        };

        foreach ($params as $parent => $item) {
            if (is_array($item)) {
                foreach ($item as $child) {
                    $addTerm($parent, $child);
                }
            } else {
                $addTerm($parent, $item);
            }
        }
    }

    public function getParent($term)
    {
        $childTerm = $this->getTaxonomyTerm($term);
        if (!$childTerm || is_null($childTerm->parent_id)) {
            return null;
        }

        $parentTerm = (new Query())
            ->select('term')
            ->from(TaxonomyTerms::tableName())
            ->where("id = $childTerm->parent_id")
            ->one(self::getDb());

        return $parentTerm ? $parentTerm['term'] : null;
    }

    public function hasParent($term)
    {
        return $this->getParent($term) != null;
    }

    public function getChildren($term)
    {
        $parentTerm = $this->getTaxonomyTerm($term);
        if (!$parentTerm) {
            return [];
        }

        $query = (new Query())
            ->select('term')
            ->from(TaxonomyTerms::tableName())
            ->where("parent_id = $parentTerm->id");

        $result = [];
        foreach ($query->all(self::getDb()) as $row) {
            $result[] = $row['term'];
        }

        return $result;
    }
    
    public function hasChildren($term)
    {
        $parentTerm = $this->getTaxonomyTerm($term);
        if (!$parentTerm) {
            return false;
        }
        
        return (new Query())
            ->from(TaxonomyTerms::tableName())
            ->where("parent_id = $parentTerm->id")
            ->exists(self::getDb());
    }
}


1			<?php
2
3			namespace nkostadinov\taxonomy\components\terms;
4
5			use insight\core\helpers\ArrayHelper;
6			use nkostadinov\taxonomy\components\interfaces\IHierarchicalTerm;
7			use nkostadinov\taxonomy\models\TaxonomyDef;
8			use nkostadinov\taxonomy\models\TaxonomyTerms;
9			use Yii;
10			use yii\db\Query;
11
12			class CategoryTerm extends TagTerm implements IHierarchicalTerm
13			{
14			public $templateFile = '@nkostadinov/taxonomy/migrations/template/category.php' ;
15
16			public function getTerms($object_id, $name = [])
17			{
18			$query = (new Query())
19			->select(TaxonomyTerms::tableName() . '.term')
20			->from(TaxonomyTerms::tableName())
21			->innerJoin($this->table, $this->table . '.term_id = taxonomy_terms.id',
			0 ignored issues – show Documentation introduced 2016-10-21 11:13 UTC by Report Bug Copy Issue Report The property `table` does not exist on `object<nkostadinov\taxon...nts\terms\CategoryTerm>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
22			[':object_id' => $object_id])
23			->andFilterWhere(['taxonomy_terms.term' => $name]);
24
25			if ($object_id) {
26			$query->onCondition("$this->table.object_id = $object_id");
			0 ignored issues – show Documentation introduced 2016-10-21 11:13 UTC by Report Bug Copy Issue Report The property `table` does not exist on `object<nkostadinov\taxon...nts\terms\CategoryTerm>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
27			}
28
29			$result = [];
30			foreach($query->all() as $v) {
31			$result[] = $v['term'];
32			}
33
34			return $result;
35			}
36
37			public function addTerm($object_id, $params)
38			{
39			$cachedParents = [];
40
41			$addTerm = function ($parent, $item) use ($object_id, &$cachedParents) {
42			$term = $this->getTaxonomyTerm($item);
43			$data['term_id'] = $term->id;
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-10-21 11:13 UTC by Report Bug Copy Issue Report `$data` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$data = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
44			$data['object_id'] = $object_id;
45
46			if (array_key_exists($parent, $cachedParents)) {
47			$term->parent_id = $cachedParents[$parent]->id;
48			} else if (is_string($parent)) {
49			$parentTerm = $this->getTaxonomyTerm($parent);
50			$cachedParents[$parent] = $parentTerm;
51			$term->parent_id = $parentTerm->id;
52			}
53
54			if (!(new Query())->from($this->table)->where($data)->exists(CategoryTerm::getDb())) {
			0 ignored issues – show Documentation introduced 2016-10-21 11:13 UTC by Report Bug Copy Issue Report The property `table` does not exist on `object<nkostadinov\taxon...nts\terms\CategoryTerm>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
55			Yii::$app->db->transaction(function() use ($data, $term) {
56			if ($term->getDirtyAttributes(['parent_id'])) {
57			$term->save(false);
58			}
59			CategoryTerm::getDb()->createCommand()->insert($this->table, $data)->execute();
			0 ignored issues – show Documentation introduced 2016-10-21 11:13 UTC by Report Bug Copy Issue Report The property `table` does not exist on `object<nkostadinov\taxon...nts\terms\CategoryTerm>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
60
61			$term->updateCounters(['total_count' => 1]);
62			TaxonomyDef::updateAllCounters(['total_count' => 1], ['id' => $this->id]);
63			});
64			}
65			};
66
67			foreach ($params as $parent => $item) {
68			if (is_array($item)) {
69			foreach ($item as $child) {
70			$addTerm($parent, $child);
71			}
72			} else {
73			$addTerm($parent, $item);
74			}
75			}
76			}
77
78			public function getParent($term)
79			{
80			$childTerm = $this->getTaxonomyTerm($term);
81			if (!$childTerm \|\| is_null($childTerm->parent_id)) {
82			return null;
83			}
84
85			$parentTerm = (new Query())
86			->select('term')
87			->from(TaxonomyTerms::tableName())
88			->where("id = $childTerm->parent_id")
89			->one(self::getDb());
90
91			return $parentTerm ? $parentTerm['term'] : null;
92			}
93
94			public function hasParent($term)
95			{
96			return $this->getParent($term) != null;
97			}
98
99			public function getChildren($term)
100			{
101			$parentTerm = $this->getTaxonomyTerm($term);
102			if (!$parentTerm) {
103			return [];
104			}
105
106			$query = (new Query())
107			->select('term')
108			->from(TaxonomyTerms::tableName())
109			->where("parent_id = $parentTerm->id");
110
111			$result = [];
112			foreach ($query->all(self::getDb()) as $row) {
113			$result[] = $row['term'];
114			}
115
116			return $result;
117			}
118
119			public function hasChildren($term)
120			{
121			$parentTerm = $this->getTaxonomyTerm($term);
122			if (!$parentTerm) {
123			return false;
124			}
125
126			return (new Query())
127			->from(TaxonomyTerms::tableName())
128			->where("parent_id = $parentTerm->id")
129			->exists(self::getDb());
130			}
131			}
132

nkostadinov / yii2-taxonomy

Pull Request — master (#4)

CategoryTerm::getChildren() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like