ntentan / nibii

src/adapters/PostgresqlAdapter.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
      */
     public function mapDataTypes($nativeType) 
     {
-        switch($nativeType)
+        switch ($nativeType)
         {
             case 'character varying':
                 return 'string';

src/behaviours/TimestampableBehaviour.php

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
     
     public function preSaveCallback($data)
     {
-        if(isset($this->model->getDescription()->getFields()['created']))
+        if (isset($this->model->getDescription()->getFields()['created']))
         {
             $data['created'] = date('Y-m-d H:i:s');
         }
@@ -25,7 +25,7 @@ 
     
     public function preUpdateCallback($data)
     {
-        if(isset($this->model->getDescription()->getFields()['updated']))
+        if (isset($this->model->getDescription()->getFields()['updated']))
         {
             $data['updated'] = date('Y-m-d H:i:s');
         }

src/DriverAdapter.php

Spacing +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
 
     public function insert($record)
     {
-        if($this->insertQuery === null) {
+        if ($this->insertQuery === null) {
             $this->initInsert();
         }
         return Db::getDriver()->query($this->insertQuery, $record);
@@ -86,7 +86,7 @@ 
 
     public function update($record)
     {
-        if($this->updateQuery === null) {
+        if ($this->updateQuery === null) {
             $this->initUpdate();
         }
         return Db::getDriver()->query($this->updateQuery, $record);

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace ntentan\nibii;
 
-use ntentan\utils\Text;
 use ntentan\atiaa\Db;
 
 /**

src/relationships/ManyHaveManyRelationship.php

Spacing +8 added lines, -8 removed lines

@@ -46,7 +46,7 @@ 
             ->filterBy($this->options['junction_local_key'], $data[$this->options['local_key']])
             ->fetch();
         $foreignKeys = [];
-        foreach($filter->toArray() as $foreignItem) {
+        foreach ($filter->toArray() as $foreignItem) {
             $foreignKeys[] = $foreignItem[$this->options['junction_foreign_key']];
         }
         $query = (new \ntentan\nibii\QueryParameters($this->getModelInstance()))
@@ -56,7 +56,7 @@ 
 
     public function runSetup()
     {
-        if(isset($this->options['through'])) {
+        if (isset($this->options['through'])) {
             $junctionModelName = $this->options['through'];
         } else {
             $junctionModelName = Nibii::joinModels($this->setupName, $this->options['model']);
@@ -64,22 +64,22 @@ 
         $this->options['junction_model'] = $junctionModelName;
         
         $foreignModel = Nibii::load($this->options['model']);
-        if($this->options['foreign_key'] == null) {
+        if ($this->options['foreign_key'] == null) {
             $this->options['foreign_key'] = $foreignModel->getDescription()->getPrimaryKey()[0];
         } 
         
-        if($this->options['local_key'] == null) {
+        if ($this->options['local_key'] == null) {
             $this->options['local_key'] = $this->setupPrimaryKey[0];
         }
         
-        if(!isset($this->options['junction_local_key'])) {
+        if (!isset($this->options['junction_local_key'])) {
             $this->options['junction_local_key'] = 
-                Text::singularize($this->setupTable) . '_id';
+                Text::singularize($this->setupTable).'_id';
         }
         
-        if(!isset($this->options['junction_foreign_key'])) {
+        if (!isset($this->options['junction_foreign_key'])) {
             $this->options['junction_foreign_key'] = 
-                Text::singularize($foreignModel->getTable()) . '_id';
+                Text::singularize($foreignModel->getTable()).'_id';
         }
     }
 }

src/relationships/HasManyRelationship.php

Spacing +3 added lines, -3 removed lines

@@ -42,10 +42,10 @@
 
     public function runSetup()
     {
-        if($this->options['foreign_key'] == null) {
-            $this->options['foreign_key'] = Text::singularize($this->setupTable) . '_id';
+        if ($this->options['foreign_key'] == null) {
+            $this->options['foreign_key'] = Text::singularize($this->setupTable).'_id';
         }
-        if($this->options['local_key'] == null) {
+        if ($this->options['local_key'] == null) {
             $this->options['local_key'] = $this->setupPrimaryKey[0];
         }
     }

src/validations/UniqueValidation.php

Spacing +5 added lines, -5 removed lines

@@ -45,19 +45,19 @@ 
     public function run($field, $data)
     {
         $test = [];
-        foreach($field['name'] as $name) {
+        foreach ($field['name'] as $name) {
             $test[$name] = isset($data[$name]) ? $data[$name] : null;
         }
         
-        if($this->mode == \ntentan\nibii\DataOperations::MODE_UPDATE) {
+        if ($this->mode == \ntentan\nibii\DataOperations::MODE_UPDATE) {
             $primaryKeyFields = $this->model->getDescription()->getPrimaryKey();
             $keys = [];
-            foreach($primaryKeyFields as $name) {
+            foreach ($primaryKeyFields as $name) {
                 $keys[$name] = $data[$name];
             }
             $testItem = $this->model->createNew()->fetchFirst($test);
             $intersection = array_intersect($testItem->toArray(), $keys);
-            if(!empty($intersection)) {
+            if (!empty($intersection)) {
                 return true;
             }                   
         } else {
@@ -67,7 +67,7 @@ 
         return $this->evaluateResult(
             $field, 
             $testItem->count() === 0,
-            "The value of " . implode(', ', $field['name']) . " must be unique"
+            "The value of ".implode(', ', $field['name'])." must be unique"
         );        
     }
 }

src/interfaces/TableNameResolverInterface.php

Doc Comments +3 added lines

@@ -9,5 +9,8 @@
  */
 interface TableNameResolverInterface
 {
+    /**
+     * @return string
+     */
     public function getTableName($instance);
 }

src/RecordWrapper.php

Doc Comments +5 added lines, -1 removed lines

@@ -101,7 +101,7 @@ 
 
     /**
      * @method
-     * @param type $name
+     * @param string $name
      * @param type $arguments
      * @return type
      */
@@ -136,6 +136,10 @@ 
         return $this->table;
     }
     
+    /**
+     * @param Relationship[] $relationships
+     * @param integer $depth
+     */
     private function expandArrayValue($array, $relationships, $depth, $index = null)
     {
         foreach($relationships as $name => $relationship) {

Unused Use Statements -1 removed lines

@@ -25,7 +25,6 @@
 
 namespace ntentan\nibii;
 
-use ntentan\utils\Utils;
 use ntentan\kaikai\Cache;
 use ntentan\panie\InjectionContainer;
 use ntentan\utils\Text;

Spacing +16 added lines, -16 removed lines

@@ -50,7 +50,7 @@ 
     {
         $this->table = Nibii::getModelTable($this);
         $this->adapter = $adapter;
-        foreach($this->behaviours as $behaviour) {
+        foreach ($this->behaviours as $behaviour) {
             $behaviourInstance = $this->getComponentInstance($behaviour);
             $behaviourInstance->setModel($this);
         }
@@ -63,7 +63,7 @@ 
     public function getDescription()
     {
         return Cache::read(
-            (new \ReflectionClass($this))->getName() . '::desc',
+            (new \ReflectionClass($this))->getName().'::desc',
             function() 
             {
                 return new ModelDescription($this);
@@ -84,7 +84,7 @@ 
     {
         $relationships = $this->getDescription()->getRelationships();
         $decamelizedKey = Text::deCamelize($key);
-        if(isset($relationships[$key])) {
+        if (isset($relationships[$key])) {
             return $this->fetchRelatedFields($relationships[$key]);
         }
         return isset($this->modelData[$decamelizedKey]) ? $this->modelData[$decamelizedKey] : null;
@@ -108,7 +108,7 @@ 
      */
     public function __call($name, $arguments)
     {
-        if($this->dynamicOperations === null) {
+        if ($this->dynamicOperations === null) {
             $this->dynamicOperations = new Operations(
                 $this, $this->adapter
             );
@@ -139,7 +139,7 @@ 
     
     private function expandArrayValue($array, $relationships, $depth, $index = null)
     {
-        foreach($relationships as $name => $relationship) {
+        foreach ($relationships as $name => $relationship) {
             $array[$name] = $this->fetchRelatedFields($relationship, $index)->toArray($depth);
         }
         return $array;
@@ -149,9 +149,9 @@ 
     {
         $relationships = $this->getDescription()->getRelationships();
         $array = $this->modelData;
-        if($depth > 0) {
-            if($this->hasMultipleData()) {
-                foreach($array as $i => $value) {
+        if ($depth > 0) {
+            if ($this->hasMultipleData()) {
+                foreach ($array as $i => $value) {
                     $array[$i] = $this->expandArrayValue($value, $relationships, $depth - 1, $i);
                 }
             } else {
@@ -170,7 +170,7 @@ 
 
     private function hasMultipleData()
     {
-        if(count($this->modelData) > 0) {
+        if (count($this->modelData) > 0) {
             return is_numeric(array_keys($this->modelData)[0]);
         } else {
             return false;
@@ -181,11 +181,11 @@ 
     {
         $data = [];
                 
-        if(count($this->modelData) == 0) {
+        if (count($this->modelData) == 0) {
             $data = $this->modelData;
-        } else if($this->hasMultipleData()) {
+        } else if ($this->hasMultipleData()) {
             $data = $this->modelData;
-        } else if(count($this->modelData) > 0) {
+        } else if (count($this->modelData) > 0) {
             $data[] = $this->modelData;
         }
         
@@ -200,7 +200,7 @@ 
     
     public function mergeData($data)
     {
-        foreach($data as $key => $value) {
+        foreach ($data as $key => $value) {
             $this->modelData[$key] = $value;
         }
         $this->dataSet = true;
@@ -233,7 +233,7 @@ 
 
     private function wrap($offset)
     {
-        if(isset($this->modelData[$offset])) {
+        if (isset($this->modelData[$offset])) {
             $newInstance = $this->createNew();
             $newInstance->setData($this->modelData[$offset]);
             return $newInstance;
@@ -289,13 +289,13 @@ 
 
     private function fetchRelatedFields($relationship, $index = null)
     {
-        if($index === null) {
+        if ($index === null) {
             $data = $this->modelData;
         } else {
             $data = $this->modelData[$index];
         }
         $model = $relationship->getModelInstance();
-        if(empty($data)) {
+        if (empty($data)) {
             return $model;
         } else {
             return $model->fetch($relationship->getQuery($data));

src/Resolver.php

Spacing +5 added lines, -5 removed lines

@@ -31,7 +31,7 @@ 
     {
         $arrayed = explode('\\', $className);
         $class = array_pop($arrayed);
-        if($arrayed[0] == '') {
+        if ($arrayed[0] == '') {
             array_shift($arrayed);
         }
         return ['class' => $class, 'namespace' => implode('\\', $arrayed)];
@@ -41,7 +41,7 @@ 
     {
         $classA = $this->getClassFileDetails($classA);
         $classB = $this->getClassFileDetails($classB);
-        if($classA['namespace'] != $classB['namespace']) {
+        if ($classA['namespace'] != $classB['namespace']) {
             throw new NibiiException(
                 "Cannot automatically join two classes of different "
                     . "namespaces. Please provide a model joiner or "
@@ -50,7 +50,7 @@ 
         }
         $classes = [$classA['class'], $classB['class']];
         sort($classes);
-        return "{$classA['namespace']}\\" . implode('', $classes);        
+        return "{$classA['namespace']}\\".implode('', $classes);        
     }
 
     public function getTableName($instance)
@@ -63,8 +63,8 @@ 
     public static function getDriverAdapterClassName()
     {
         $driver = Config::get('ntentan:db.driver', false);
-        if($driver) {
-            return __NAMESPACE__ . '\adapters\\' . Text::ucamelize(Config::get('ntentan:db.driver')) . 'Adapter';
+        if ($driver) {
+            return __NAMESPACE__.'\adapters\\'.Text::ucamelize(Config::get('ntentan:db.driver')).'Adapter';
         } 
         throw new NibiiException("Please specify a driver");
     }

Unused Use Statements +1 added lines, -1 removed lines

@@ -8,10 +8,10 @@
 
 namespace ntentan\nibii;
 
+use ntentan\config\Config;
 use ntentan\nibii\interfaces\ModelClassResolverInterface;
 use ntentan\nibii\interfaces\ModelJoinerInterface;
 use ntentan\nibii\interfaces\TableNameResolverInterface;
-use ntentan\config\Config;
 use ntentan\utils\Text;
 
 /**