ntentan / nibii

src/QueryEngine.php

Spacing +6 added lines, -6 removed lines

@@ -34,8 +34,8 @@ 
         }
 
         return $this->filter(
-            "INSERT INTO " . $table .
-            " (" . implode(", ", $quotedFields) . ") VALUES (" . implode(', ', $valueFields) . ")"
+            "INSERT INTO ".$table.
+            " (".implode(", ", $quotedFields).") VALUES (".implode(', ', $valueFields).")"
         );
     }
 
@@ -77,10 +77,10 @@ 
             }
         }
 
-        return $this->filter("UPDATE " .
-            $model->getDBStoreInformation()['quoted_table'] .
-            " SET " . implode(', ', $valueFields) .
-            " WHERE " . implode(' AND ', $conditions)
+        return $this->filter("UPDATE ".
+            $model->getDBStoreInformation()['quoted_table'].
+            " SET ".implode(', ', $valueFields).
+            " WHERE ".implode(' AND ', $conditions)
         );
     }
 

Doc Comments +26 added lines

@@ -6,10 +6,16 @@ 
 
     private $db;
 
+    /**
+     * @param \ntentan\atiaa\Driver $driver
+     */
     public function setDriver($driver) {
         $this->db = $driver;
     }
  
+    /**
+     * @param string $query
+     */
     private function filter($query) {
         return $query;
     }
@@ -39,6 +45,11 @@ 
         );
     }
 
+    /**
+     * @param QueryParameters $parameters
+     *
+     * @return string
+     */
     public function getBulkUpdateQuery($data, $parameters) {
         $updateData = [];
         foreach ($data as $field => $value) {
@@ -84,6 +95,11 @@ 
         );
     }
 
+    /**
+     * @param QueryParameters $parameters
+     *
+     * @return string
+     */
     public function getSelectQuery($parameters) {
         return $this->filter(sprintf(
                 "SELECT %s FROM %s%s%s%s%s", 
@@ -97,6 +113,11 @@ 
         );
     }
 
+    /**
+     * @param QueryParameters $parameters
+     *
+     * @return string
+     */
     public function getCountQuery($parameters) {
         return $this->filter(sprintf(
                 "SELECT count(*) as count FROM %s%s", 
@@ -106,6 +127,11 @@ 
         );
     }
 
+    /**
+     * @param QueryParameters $parameters
+     *
+     * @return string
+     */
     public function getDeleteQuery($parameters) {
         return $this->filter(sprintf(
                 "DELETE FROM %s%s", 

src/ORMContext.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
     }
     
     public static function getInstance() {
-        if(self::$instance === null) throw new NibiiException("A context has not yet been initialized");
+        if (self::$instance === null) throw new NibiiException("A context has not yet been initialized");
         return self::$instance;
     }
     

Braces +3 added lines, -1 removed lines

@@ -67,7 +67,9 @@
     }
     
     public static function getInstance() {
-        if(self::$instance === null) throw new NibiiException("A context has not yet been initialized");
+        if(self::$instance === null) {
+            throw new NibiiException("A context has not yet been initialized");
+        }
         return self::$instance;
     }
     

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/RecordWrapper.php

Doc Comments +8 added lines, -1 removed lines

@@ -131,7 +131,7 @@ 
 
     /**
      * @method
-     * @param type $name
+     * @param string $name
      * @param type $arguments
      * @return type
      */
@@ -160,6 +160,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);
@@ -319,6 +323,9 @@ 
         
     }
 
+    /**
+     * @param string $id
+     */
     public function postSaveCallback($id) {
         
     }

src/relationships/BelongsToRelationship.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 BelongsToRelationship extends \ntentan\nibii\Relationship {
 

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     public function prepareQuery($data) {
         // @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'], $data[$this->options['local_key']]);
         } else {
             $query->setTable($this->getModelInstance()->getDBStoreInformation()['quoted_table'])

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

Spacing +4 added lines, -4 removed lines

@@ -43,7 +43,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'], $data[$this->options['local_key']]);
         } else {
             $query->setTable($this->getModelInstance()->getDBStoreInformation()['quoted_table'])
@@ -55,10 +55,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];
         }
     }

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
 {

src/relationships/ManyHaveManyRelationship.php

Spacing +4 added lines, -4 removed lines

@@ -44,7 +44,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 = [];
@@ -54,7 +54,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()
@@ -88,12 +88,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';
         }
     }
 

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 ManyHaveManyRelationship extends \ntentan\nibii\Relationship {