nkostadinov / yii2-taxonomy

components/terms/BaseTerm.php

Braces +8 added lines, -5 removed lines

@@ -74,8 +74,9 @@ 
     }
 
     public function canInstall() {
-        if(!$this->getTable())
-            return 'Missing "table" property';
+        if(!$this->getTable()) {
+                    return 'Missing "table" property';
+        }
         return true;
     }
 
@@ -95,10 +96,12 @@ 
 
     public function getRefTableName()
     {
-        if(strpos($this->refTable, '\\') === FALSE) //not an AR class but a table name
+        if(strpos($this->refTable, '\\') === FALSE) {
+            //not an AR class but a table name
             return $this->refTable;
-        else
-            return call_user_func([$this->refTable, 'tableName']);
+        } else {
+                    return call_user_func([$this->refTable, 'tableName']);
+        }
     }
 
     public function getTable()

Spacing +10 added lines, -10 removed lines

@@ -35,11 +35,11 @@ 
 
     public function removeTerm($object_id, $params = [])
     {
-        if(empty($params)) {
+        if (empty($params)) {
             $params = $this->getTerms($object_id);
         }
 
-        foreach($params as $item) {
+        foreach ($params as $item) {
             $term = $this->getTaxonomyTerm($item);
             $data['term_id'] = $term->id;
             $data['object_id'] = $object_id;
@@ -49,7 +49,7 @@ 
                 $this->getDb()->createCommand()->delete($this->table, $data)->execute();
 
                 $term->updateCounters(['total_count' => -1]);
-                Taxonomydef::updateAllCounters(['total_count' => -1], [ 'id' => $this->id ]);
+                Taxonomydef::updateAllCounters(['total_count' => -1], ['id' => $this->id]);
             }
         }
     }
@@ -62,7 +62,7 @@ 
             ->andFilterWhere(['taxonomy_terms.term' => $name]);
 
         if ($object_id) {
-            $query->innerJoin($this->table, $this->table . '.term_id = taxonomy_terms.id')
+            $query->innerJoin($this->table, $this->table.'.term_id = taxonomy_terms.id')
                   ->onCondition("$this->table.object_id = $object_id");
         }
         
@@ -107,7 +107,7 @@ 
     }
 
     public function canInstall() {
-        if(!$this->getTable())
+        if (!$this->getTable())
             return 'Missing "table" property';
         return true;
     }
@@ -115,7 +115,7 @@ 
     public function getTaxonomyTerm($name, $create = true)
     {
         $term = TaxonomyTerms::findOne(['term'=>$name, 'taxonomy_id' => $this->id]);
-        if($create and !isset($term))
+        if ($create and !isset($term))
         {
             $term = new TaxonomyTerms();
             $term->taxonomy_id = $this->id;
@@ -128,7 +128,7 @@ 
 
     public function getRefTableName()
     {
-        if(strpos($this->refTable, '\\') === FALSE) //not an AR class but a table name
+        if (strpos($this->refTable, '\\') === FALSE) //not an AR class but a table name
             return $this->refTable;
         else
             return call_user_func([$this->refTable, 'tableName']);
@@ -149,7 +149,7 @@ 
         if (!preg_match('/^\w+$/', $this->name)) {
             throw new Exception('The migration name should contain letters, digits and/or underscore characters only.');
         }
-        $name = 'm' . gmdate('ymd_His') . '_' . $this->name;
+        $name = 'm'.gmdate('ymd_His').'_'.$this->name;
         return $name;
     }
 
@@ -157,7 +157,7 @@ 
     {
 
         $name = $this->getMigrationFile();
-        $file = Yii::getAlias($this->migrationPath . DIRECTORY_SEPARATOR . $name . '.php');
+        $file = Yii::getAlias($this->migrationPath.DIRECTORY_SEPARATOR.$name.'.php');
 
         //$data = get_object_vars($this);
         $data = [];
@@ -168,7 +168,7 @@ 
         $data['migration'] = $name;
 
         $this->migration = $name;
-        $content = Yii::$app->getView()->renderFile(Yii::getAlias($this->templateFile), [ 'data' => $data ]);
+        $content = Yii::$app->getView()->renderFile(Yii::getAlias($this->templateFile), ['data' => $data]);
         file_put_contents($file, $content);
         return $name;
     }

behaviors/PropertySupportBehavior.php

Unused Use Statements -3 removed lines

@@ -4,9 +4,6 @@
 
 use Yii;
 use yii\base\Behavior;
-use yii\base\Event;
-use yii\db\ActiveRecord;
-use yii\db\AfterSaveEvent;
 
 /**
  * @author Nikolay Traykov

behaviors/TagSupportBehavior.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Yii;
 use yii\base\Behavior;
-use yii\base\Event;
 use yii\db\ActiveRecord;
 use yii\db\AfterSaveEvent;
 

components/terms/PropertyTerm.php

Spacing +10 added lines, -10 removed lines

@@ -17,7 +17,7 @@ 
 
 class PropertyTerm extends BaseTerm
 {
-    public $templateFile = '@nkostadinov/taxonomy/migrations/template/properties.php' ;
+    public $templateFile = '@nkostadinov/taxonomy/migrations/template/properties.php';
 
     public $updateOnExist = true;
 
@@ -33,14 +33,14 @@ 
             'value' => Schema::TYPE_STRING,
         ]);
         if ($migration->db->driverName === 'mysql') {
-            $migration->addForeignKey('fk_' . $this->getTable() . '_' . $this->getRefTableName(), $this->getTable(), 'object_id', $this->getRefTableName(), 'id', 'CASCADE');
-            $migration->addForeignKey('fk_' . $this->getTable() . '_' . TaxonomyTerms::tableName(), $this->getTable(), 'term_id', TaxonomyTerms::tableName(), 'id', 'CASCADE');
+            $migration->addForeignKey('fk_'.$this->getTable().'_'.$this->getRefTableName(), $this->getTable(), 'object_id', $this->getRefTableName(), 'id', 'CASCADE');
+            $migration->addForeignKey('fk_'.$this->getTable().'_'.TaxonomyTerms::tableName(), $this->getTable(), 'term_id', TaxonomyTerms::tableName(), 'id', 'CASCADE');
         }
     }
 
     public function addTerm($object_id, $params)
     {
-        foreach($params as $item => $value) {
+        foreach ($params as $item => $value) {
             $term = TaxonomyTerms::findOne(['term' => $item, 'taxonomy_id' => $this->id]);
             if (!isset($term)) {
                 $term = new TaxonomyTerms();
@@ -66,7 +66,7 @@ 
                 } catch (Exception $e) {
                     $transaction->rollBack();
                 }
-            } elseif($this->updateOnExist) {
+            } elseif ($this->updateOnExist) {
                 $this->getDb()->createCommand()->update($this->getTable(), ['value' => $value], $data)->execute();
             }
         }
@@ -75,7 +75,7 @@ 
     public function removeTerm($object_id, $params = [])
     {
         $terms = $this->getTerms($object_id, $params);
-        foreach($terms as $term => $value) {
+        foreach ($terms as $term => $value) {
             $term = $this->getTaxonomyTerm($term);
             $data['term_id'] = $term->id;
             $data['object_id'] = $object_id;
@@ -85,7 +85,7 @@ 
                 $this->getDb()->createCommand()->delete($this->getTable(), $data)->execute();
 
                 $term->updateCounters(['total_count' => -1]);
-                TaxonomyDef::updateAllCounters(['total_count' => -1], [ 'id' => $this->id ]);
+                TaxonomyDef::updateAllCounters(['total_count' => -1], ['id' => $this->id]);
             }
         }
     }
@@ -93,11 +93,11 @@ 
     public function getTerms($object_id, $name = [])
     {
         $query = (new Query())
-            ->select(TaxonomyTerms::tableName() . '.term, ' . $this->getTable() . '.value')
+            ->select(TaxonomyTerms::tableName().'.term, '.$this->getTable().'.value')
             ->from(TaxonomyTerms::tableName())
-            ->innerJoin($this->getTable(), $this->getTable() . '.term_id = taxonomy_terms.id and ' . $this->getTable() . '.object_id=:object_id',
+            ->innerJoin($this->getTable(), $this->getTable().'.term_id = taxonomy_terms.id and '.$this->getTable().'.object_id=:object_id',
                 [':object_id' => $object_id])
-            ->andFilterWhere([TaxonomyTerms::tableName() . '.term' => $name]);
+            ->andFilterWhere([TaxonomyTerms::tableName().'.term' => $name]);
 
         return ArrayHelper::map($query->all(), 'term', 'value');
     }

components/terms/CategoryTerm.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
     {
         $cachedParents = [];
 
-        $addTerm = function ($parent, $item) use ($object_id, &$cachedParents, &$addTerm) {
+        $addTerm = function($parent, $item) use ($object_id, &$cachedParents, &$addTerm) {
             if ($this->detectLoop($parent, $item)) {
                 throw new InvalidCallException('Loop detected! Cannot add parent as a child!');
             }