ntentan / nibii

src/Resolver.php

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;
 
 /**

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) {
@@ -52,7 +52,7 @@ 
     public static function getDriverAdapterClassName() {
         $driver = Config::get('ntentan:db.driver', false);
         if ($driver) {
-            return __NAMESPACE__ . '\adapters\\' . Text::ucamelize(Config::get('ntentan:db.driver')) . 'Adapter';
+            return __NAMESPACE__.'\adapters\\'.Text::ucamelize(Config::get('ntentan:db.driver')).'Adapter';
         }
         throw new NibiiException("Please specify a driver");
     }

src/Nibii.php

Doc Comments +6 added lines

@@ -36,6 +36,9 @@ 
             ->getJunctionClassName($classA, $classB);
     }
 
+    /**
+     * @param RecordWrapper $instance
+     */
     public static function getModelTable($instance) {
         return InjectionContainer::singleton(interfaces\TableNameResolverInterface::class)
             ->getTableName($instance);
@@ -46,6 +49,9 @@ 
             ->getModelClassName($model, $context);
     }
 
+    /**
+     * @param string $class
+     */
     public static function getModelName($class) {
         return $class;
     }

src/FilterCompiler.php

Doc Comments +4 added lines

@@ -190,6 +190,10 @@
         return $return;
     }
 
+    /**
+     * @param integer $level
+     * @param string $opr
+     */
     private function parseRightExpression($level, $opr) {
         switch ($opr) {
             case 'between': return $this->parseBetween();

Spacing +6 added lines, -6 removed lines

@@ -57,7 +57,7 @@ 
         $this->getToken();
         $expression = $this->parseExpression();
         if ($this->token !== false) {
-            throw new FilterCompilerException("Unexpected '" . $this->token . "' in filter [$filter]");
+            throw new FilterCompilerException("Unexpected '".$this->token."' in filter [$filter]");
         }
         $parsed = $this->renderExpression($expression);
         return $parsed;
@@ -65,7 +65,7 @@ 
 
     private function renderExpression($expression) {
         if (is_array($expression)) {
-            $expression = $this->renderExpression($expression['left']) . " {$expression['opr']} " . $this->renderExpression($expression['right']);
+            $expression = $this->renderExpression($expression['left'])." {$expression['opr']} ".$this->renderExpression($expression['right']);
         }
         return $expression;
     }
@@ -75,7 +75,7 @@ 
             $tokens = [$tokens];
         }
         if (array_search($this->lookahead, $tokens) === false) {
-            throw new FilterCompilerException("Expected " . implode(' or ', $tokens) . " but found " . $this->lookahead);
+            throw new FilterCompilerException("Expected ".implode(' or ', $tokens)." but found ".$this->lookahead);
         }
     }
 
@@ -160,7 +160,7 @@ 
     }
 
     private function returnPositionTag() {
-        return ":filter_bind_" . ( ++$this->numPositions);
+        return ":filter_bind_".( ++$this->numPositions);
     }
 
     private function parseObracket() {
@@ -237,7 +237,7 @@ 
         }
 
         if ($this->token === false && strlen($this->filter) > 0) {
-            throw new FilterCompilerException("Unexpected character [" . $this->filter[0] . "] begining " . $this->filter . ".");
+            throw new FilterCompilerException("Unexpected character [".$this->filter[0]."] begining ".$this->filter.".");
         }
     }
 
@@ -251,7 +251,7 @@ 
         $rewritten = [];
         foreach ($data as $key => $value) {
             if (is_numeric($key)) {
-                $rewritten["filter_bind_" . ($key + 1)] = $value;
+                $rewritten["filter_bind_".($key + 1)] = $value;
             } else {
                 $rewritten[$key] = $value;
             }

src/ModelDescription.php

Doc Comments +3 added lines

@@ -90,6 +90,9 @@
         return $relationshipDetails;
     }
 
+    /**
+     * @param string $type
+     */
     private function createRelationships($type, $relationships) {
         foreach ($relationships as $relationship) {
             $relationship = $this->getRelationshipDetails($relationship);

src/Operations.php

Doc Comments +5 added lines

@@ -22,6 +22,11 @@
         'save'
     ];
 
+    /**
+     * @param RecordWrapper $wrapper
+     * @param DriverAdapter $adapter
+     * @param string $table
+     */
     public function __construct($wrapper, $adapter, $table) {
         $this->wrapper = $wrapper;
         $this->adapter = $adapter;

src/QueryEngine.php

Doc Comments +18 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,9 @@ 
         );
     }
 
+    /**
+     * @return string
+     */
     public function getBulkUpdateQuery($data, $parameters) {
         $updateData = [];
         foreach ($data as $field => $value) {
@@ -84,6 +93,9 @@ 
         );
     }
 
+    /**
+     * @return string
+     */
     public function getSelectQuery($parameters) {
         return $this->filter(sprintf(
                 "SELECT %s FROM %s%s%s%s%s", 
@@ -97,6 +109,9 @@ 
         );
     }
 
+    /**
+     * @return string
+     */
     public function getCountQuery($parameters) {
         return $this->filter(sprintf(
                 "SELECT count(*) as count FROM %s%s", 
@@ -106,6 +121,9 @@ 
         );
     }
 
+    /**
+     * @return string
+     */
     public function getDeleteQuery($parameters) {
         return $this->filter(sprintf(
                 "DELETE FROM %s%s", 

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)
         );
     }
 

src/QueryOperations.php

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

@@ -26,8 +26,8 @@
 
 namespace ntentan\nibii;
 
-use ntentan\utils\Text;
 use ntentan\atiaa\Db;
+use ntentan\utils\Text;
 
 class QueryOperations {
 

src/QueryParameters.php

Doc Comments +7 added lines, -1 removed lines

@@ -23,7 +23,7 @@ 
 
     /**
      *
-     * @param \ $model
+     * @param string $table
      */
     public function __construct($table) {
         $this->db = DriverAdapter::getDefaultInstance();
@@ -106,6 +106,9 @@ 
         $this->boundData += $values;
     }
 
+    /**
+     * @param boolean $firstOnly
+     */
     public function setFirstOnly($firstOnly) {
         $this->firstOnly = $firstOnly;
         return $this;
@@ -123,6 +126,9 @@ 
         $this->offset = $offset;
     }
 
+    /**
+     * @param string $field
+     */
     public function addSort($field, $direction = 'ASC') {
         $this->sorts[] = "$field $direction";
     }

Spacing +2 added lines, -2 removed lines

@@ -70,7 +70,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 = []) {
@@ -90,7 +90,7 @@ 
             $this->whereClause .= "{$field} IN (";
             $comma = '';
             for ($i = 0; $i < $numValues; $i++) {
-                $key = "filter_" . ($startIndex + $i);
+                $key = "filter_".($startIndex + $i);
                 $this->whereClause .= "$comma:$key";
                 $this->boundData[$key] = $values[$i];
                 $comma = ' ,';

src/RecordWrapper.php

Doc Comments +5 added lines, -1 removed lines

@@ -122,7 +122,7 @@ 
 
     /**
      * @method
-     * @param type $name
+     * @param string $name
      * @param type $arguments
      * @return type
      */
@@ -148,6 +148,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);

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

@@ -26,11 +26,10 @@
 
 namespace ntentan\nibii;
 
-use ntentan\utils\Utils;
+use ntentan\atiaa\Db;
 use ntentan\kaikai\Cache;
 use ntentan\panie\InjectionContainer;
 use ntentan\utils\Text;
-use ntentan\atiaa\Db;
 
 /**
  * 

Spacing +4 added lines, -4 removed lines

@@ -57,12 +57,12 @@ 
     public function __construct(DriverAdapter $adapter) {
         $table = Nibii::getModelTable($this);
         $driver = Db::getDriver();
-        if(is_string($table)) {
+        if (is_string($table)) {
             $this->quotedTable = $driver->quoteIdentifier($table);
             $this->table = $this->unquotedTable = $table;
         } else {
-            $this->quotedTable = (isset($table['schema']) ? "{$driver->quoteIdentifier($table["schema"])}." : ""). $driver->quoteIdentifier($table["table"]);
-            $this->unquotedTable = (isset($table['schema']) ? "{$table['schema']}." : "") . $table['table'];
+            $this->quotedTable = (isset($table['schema']) ? "{$driver->quoteIdentifier($table["schema"])}." : "").$driver->quoteIdentifier($table["table"]);
+            $this->unquotedTable = (isset($table['schema']) ? "{$table['schema']}." : "").$table['table'];
             $this->table = $table['table'];
             $this->schema = $table['schema'];
         }
@@ -80,7 +80,7 @@ 
      */
     public function getDescription() {
         return Cache::read(
-            (new \ReflectionClass($this))->getName() . '::desc', function() {
+            (new \ReflectionClass($this))->getName().'::desc', function() {
                 return new ModelDescription($this);
             }
         );