ntentan / nibii

src/Relationship.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
     }
     
     public function getQuery() {
-        if(!$this->query) {
+        if (!$this->query) {
             $this->query = new QueryParameters();
         }
         return $this->query;

src/QueryParameters.php

Spacing +10 added lines, -10 removed lines

@@ -67,11 +67,11 @@ 
     }
 
     public function getWhereClause() {
-        if($this->whereClause) {
-            foreach($this->boundArrays as $boundArray) {
+        if ($this->whereClause) {
+            foreach ($this->boundArrays as $boundArray) {
                 $where = "";
                 $comma = "";
-                for($i = 0; $i < count($this->boundData[$boundArray]); $i++) {
+                for ($i = 0; $i < count($this->boundData[$boundArray]); $i++) {
                     $where .= "{$comma}:{$boundArray}_{$i}";
                     $comma = ', ';
                 }
@@ -82,11 +82,11 @@ 
     }
 
     public function getBoundData() {
-        if($this->preparedBoundData === false) {
+        if ($this->preparedBoundData === false) {
             $this->preparedBoundData = [];
-            foreach($this->boundData as $key => $value) {
-                if(in_array($key, $this->boundArrays)) {
-                    foreach($value as $i => $v) {
+            foreach ($this->boundData as $key => $value) {
+                if (in_array($key, $this->boundArrays)) {
+                    foreach ($value as $i => $v) {
                         $this->preparedBoundData["{$key}_{$i}"] = $v;
                     }
                 } else {
@@ -98,10 +98,10 @@ 
     }
 
     public function setBoundData($key, $value) {
-        if(isset($this->boundData[$key])){
+        if (isset($this->boundData[$key])) {
             $isArray = is_array($value);
             $boundArray = in_array($key, $this->boundArrays);
-            if($isArray && !$boundArray) {
+            if ($isArray && !$boundArray) {
                 throw new NibiiException("{$key} cannot be bound to an array");
             } else if (!$isArray && $boundArray) {
                 throw new NibiiException("{$key} must be bound to an array");
@@ -114,7 +114,7 @@ 
     }
 
     public function getSorts() {
-        return count($this->sorts) ? " ORDER BY " . implode(", ", $this->sorts) : null;
+        return count($this->sorts) ? " ORDER BY ".implode(", ", $this->sorts) : null;
     }
 
     public function addFilter($field, $values = null) {

src/relationships/HasManyRelationship.php

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

@@ -26,8 +26,8 @@
 
 namespace ntentan\nibii\relationships;
 
-use ntentan\utils\Text;
 use ntentan\nibii\ORMContext;
+use ntentan\utils\Text;
 
 class HasManyRelationship extends \ntentan\nibii\Relationship {
 

Spacing +1 added lines, -1 removed lines

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

src/relationships/ManyHaveManyRelationship.php

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

@@ -26,9 +26,9 @@
 
 namespace ntentan\nibii\relationships;
 
-use ntentan\utils\Text;
-use ntentan\nibii\ORMContext;
 use ntentan\nibii\NibiiException;
+use ntentan\nibii\ORMContext;
+use ntentan\utils\Text;
 
 class ManyHaveManyRelationship extends \ntentan\nibii\Relationship {
 

Spacing +7 added lines, -7 removed lines

@@ -46,7 +46,7 @@ 
         $filter = $junctionModel->fields($this->options['junction_foreign_key'])
             ->filterBy($this->options['junction_local_key'], $data[$this->options['local_key']])
             ->fetch();
-        if($filter->count() == 0) {
+        if ($filter->count() == 0) {
             return null;
         }
         $foreignKeys = [];
@@ -56,7 +56,7 @@ 
 
         // @todo throw an exception when the data doesn't have the local key
         $query = $this->getQuery();
-        if($this->queryPrepared){
+        if ($this->queryPrepared) {
             $query->setBoundData($this->options['foreign_key'], $foreignKeys);
         } else {
             $query = $this->getQuery()
@@ -90,12 +90,12 @@ 
 
         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'])) {
             $this->options['junction_foreign_key'] = 
-                Text::singularize($foreignModel->getDBStoreInformation()['table']) . '_id';
+                Text::singularize($foreignModel->getDBStoreInformation()['table']).'_id';
         }
     }  
     
@@ -110,10 +110,10 @@ 
     
     public function postSave(&$wrapper) {
         $jointModelRecords = [];
-        foreach($this->tempdata as $relatedRecord) {
+        foreach ($this->tempdata as $relatedRecord) {
             $data = $relatedRecord->toArray();
-            if(!isset($data[$this->options['foreign_key']]) || (isset($data[$this->options['foreign_key']]) && count($data) > 1)) {
-                if(!$relatedRecord->save()) {
+            if (!isset($data[$this->options['foreign_key']]) || (isset($data[$this->options['foreign_key']]) && count($data) > 1)) {
+                if (!$relatedRecord->save()) {
                     throw new NibiiException("Failed to save related model {$this->options['model']}");
                 }
             }

src/Resolver.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use ntentan\nibii\interfaces\ModelClassResolverInterface;
 use ntentan\nibii\interfaces\ModelJoinerInterface;
 use ntentan\nibii\interfaces\TableNameResolverInterface;
-use ntentan\config\Config;
 use ntentan\utils\Text;
 
 /**

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
         }
         $classes = [$classA['class'], $classB['class']];
         sort($classes);
-        return "{$classA['namespace']}\\" . implode('', $classes);
+        return "{$classA['namespace']}\\".implode('', $classes);
     }
 
     public function getTableName($instance) {
@@ -51,7 +51,7 @@ 
 
     public static function getDriverAdapterClassName($driver = false) {
         if ($driver) {
-            return __NAMESPACE__ . '\adapters\\' . Text::ucamelize($driver) . 'Adapter';
+            return __NAMESPACE__.'\adapters\\'.Text::ucamelize($driver).'Adapter';
         }
         throw new NibiiException("Please specify a driver");
     }

src/RecordWrapper.php

Doc Comments +9 added lines, -2 removed lines

@@ -146,7 +146,7 @@ 
 
     /**
      * @method
-     * @param type $name
+     * @param string $name
      * @param type $arguments
      * @return type
      */
@@ -176,6 +176,10 @@ 
         return $this->retrieveItem($name);
     }
 
+    /**
+     * @param Relationship[] $relationships
+     * @param integer $depth
+     */
     private function expandArrayValue($array, $relationships, $depth, $index = null) {
         foreach ($relationships as $name => $relationship) {
             $array[$name] = $this->fetchRelatedFields($relationship, $index)->toArray($depth);
@@ -336,6 +340,9 @@ 
         
     }
 
+    /**
+     * @param string $id
+     */
     public function postSaveCallback($id) {
         
     }
@@ -360,7 +367,7 @@ 
     
     /**
      * 
-     * @return DataAdapter
+     * @return DriverAdapter
      */
     public function getAdapter() {
         $this->initialize();

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
     private $initialized = false;
 
     private function initialize() {
-        if($this->initialized) return;
+        if ($this->initialized) return;
         $this->context = ORMContext::getInstance();
         $this->container = $this->context->getContainer();
         $this->adapter = $this->container->resolve(DriverAdapter::class);

Braces +3 added lines, -1 removed lines

@@ -63,7 +63,9 @@
     private $initialized = false;
 
     private function initialize() {
-        if($this->initialized) return;
+        if($this->initialized) {
+            return;
+        }
         $this->context = ORMContext::getInstance();
         $this->container = $this->context->getContainer();
         $this->adapter = $this->container->resolve(DriverAdapter::class);

src/validations/UniqueValidation.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
         }
 
         return $this->evaluateResult(
-            $field, $testItem->count() === 0, "The value of " . implode(', ', $field['name']) . " must be unique"
+            $field, $testItem->count() === 0, "The value of ".implode(', ', $field['name'])." must be unique"
         );
     }
 

src/DataOperations.php

Spacing +4 added lines, -4 removed lines

@@ -61,7 +61,7 @@ 
         $invalidFields = [];
         $data = $this->wrapper->getData();
 
-        if(empty($data)) {
+        if (empty($data)) {
             return true;
         }
         
@@ -159,8 +159,8 @@ 
         $relationships = $this->wrapper->getDescription()->getRelationships();
         $presentRelationships = [];
         
-        foreach($relationships ?? [] as $model => $relationship) {
-            if(isset($record[$model])) {
+        foreach ($relationships ?? [] as $model => $relationship) {
+            if (isset($record[$model])) {
                 $relationship->preSave($record, $record[$model]);
                 $presentRelationships[$model] = $relationship;
             }
@@ -182,7 +182,7 @@ 
         
         // Reset the data so it contains any modifications made by callbacks
         $record = $this->wrapper->getData()[0];
-        foreach($presentRelationships as $model => $relationship) {
+        foreach ($presentRelationships as $model => $relationship) {
             $relationship->postSave($record);
         }        
 

src/QueryEngine.php

Spacing +6 added lines, -6 removed lines

@@ -29,8 +29,8 @@ 
             $valueFields[] = ":{$field}";
         }
 
-        return "INSERT INTO " . $table .
-            " (" . implode(", ", $quotedFields) . ") VALUES (" . implode(', ', $valueFields) . ")";
+        return "INSERT INTO ".$table.
+            " (".implode(", ", $quotedFields).") VALUES (".implode(', ', $valueFields).")";
     }
 
     public function getBulkUpdateQuery($data, $parameters) {
@@ -71,10 +71,10 @@ 
             }
         }
 
-        return "UPDATE " .
-            $model->getDBStoreInformation()['quoted_table'] .
-            " SET " . implode(', ', $valueFields) .
-            " WHERE " . implode(' AND ', $conditions);
+        return "UPDATE ".
+            $model->getDBStoreInformation()['quoted_table'].
+            " SET ".implode(', ', $valueFields).
+            " WHERE ".implode(' AND ', $conditions);
     }
 
     public function getSelectQuery($parameters) {