ntentan / nibii

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

Unused Use Statements -2 removed lines

@@ -2,8 +2,6 @@
 
 namespace ntentan\nibii;
 
-use ntentan\atiaa\Db;
-
 class ModelDescription {
 
     private $fields = [];

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 -2 removed lines

@@ -26,9 +26,7 @@
 
 namespace ntentan\nibii;
 
-use ntentan\kaikai\Cache;
 use ntentan\utils\Text;
-use ntentan\atiaa\Db;
 
 /**
  * 

Spacing +3 added lines, -3 removed lines

@@ -64,8 +64,8 @@ 
             $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'];
         }
@@ -83,7 +83,7 @@ 
      */
     public function getDescription() {
         return $this->context->getCache()->read(
-            (new \ReflectionClass($this))->getName() . '::desc', function() {
+            (new \ReflectionClass($this))->getName().'::desc', function() {
                 return $this->container->resolve(ModelDescription::class, ['model' => $this]);
             }
         );

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 +3 added lines

@@ -10,6 +10,9 @@
         $this->db = $driver;
     }
  
+    /**
+     * @param string $query
+     */
     private function filter($query) {
         return $query;
     }

src/QueryParameters.php

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 = ' ,';

Doc Comments +7 added lines, -1 removed lines

@@ -22,7 +22,7 @@ 
 
     /**
      *
-     * @param \ $model
+     * @param string $table
      */
     public function __construct($table) {
         $this->table = $table;
@@ -104,6 +104,9 @@ 
         $this->boundData += $values;
     }
 
+    /**
+     * @param boolean $firstOnly
+     */
     public function setFirstOnly($firstOnly) {
         $this->firstOnly = $firstOnly;
         return $this;
@@ -121,6 +124,9 @@ 
         $this->offset = $offset;
     }
 
+    /**
+     * @param string $field
+     */
     public function addSort($field, $direction = 'ASC') {
         $this->sorts[] = "$field $direction";
     }

src/relationships/ManyHaveManyRelationship.php

Spacing +2 added lines, -2 removed lines

@@ -70,11 +70,11 @@
         }
 
         if (!isset($this->options['junction_local_key'])) {
-            $this->options['junction_local_key'] = Text::singularize($this->setupTable) . '_id';
+            $this->options['junction_local_key'] = Text::singularize($this->setupTable).'_id';
         }
 
         if (!isset($this->options['junction_foreign_key'])) {
-            $this->options['junction_foreign_key'] = Text::singularize($foreignModel->getDBStoreInformation()['table']) . '_id';
+            $this->options['junction_foreign_key'] = Text::singularize($foreignModel->getDBStoreInformation()['table']).'_id';
         }
     }
 

src/Context.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;
     }
     

Doc Comments +6 added lines

@@ -56,6 +56,9 @@ 
             ->getJunctionClassName($classA, $classB);
     }
 
+    /**
+     * @param RecordWrapper $instance
+     */
     public function getModelTable($instance) {
         return$this->container->singleton(interfaces\TableNameResolverInterface::class)
             ->getTableName($instance);
@@ -66,6 +69,9 @@ 
             ->getModelClassName($model, $context);
     }
 
+    /**
+     * @param string $class
+     */
     public function getModelName($class) {
         return $class;
     }

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

@@ -2,9 +2,9 @@
 
 namespace ntentan\nibii;
 
-use ntentan\panie\Container;
-use ntentan\kaikai\Cache;
 use ntentan\atiaa\DbContext;
+use ntentan\kaikai\Cache;
+use ntentan\panie\Container;
 
 /**
  * A collection of utility methods used as helpers for loading

Indentation +3 added lines, -3 removed lines

@@ -21,11 +21,11 @@
         $this->container = $container;
         $this->dbContext = $container->resolve(DbContext::class);
         $this->container->bind(interfaces\ModelJoinerInterface::class)
-             ->to(Resolver::class);
+                ->to(Resolver::class);
         $this->container->bind(interfaces\TableNameResolverInterface::class)
-             ->to(Resolver::class);
+                ->to(Resolver::class);
         $this->container->bind(interfaces\ModelClassResolverInterface::class)
-             ->to(Resolver::class);
+                ->to(Resolver::class);
         $this->cache = $this->container->resolve(Cache::class);
         self::$instance = $this;
     }

src/DriverAdapter.php

Doc Comments +13 added lines, -1 removed lines

@@ -41,7 +41,7 @@ 
     /**
      * 
      * 
-     * @param type $parameters
+     * @param QueryParameters $parameters
      * @return type
      */
     public function select($parameters) {
@@ -57,6 +57,9 @@ 
         return $result;
     }
 
+    /**
+     * @param QueryParameters $parameters
+     */
     public function count($parameters) {
         $result = $this->driver->query(
             $this->getQueryEngine()->getCountQuery($parameters), $parameters->getBoundData()
@@ -87,12 +90,18 @@ 
         return $this->driver->query($this->updateQuery, $record);
     }
 
+    /**
+     * @param QueryParameters $parameters
+     */
     public function bulkUpdate($data, $parameters) {
         return $this->driver->query(
             $this->getQueryEngine()->getBulkUpdateQuery($data, $parameters), array_merge($data, $parameters->getBoundData())
         );
     }
 
+    /**
+     * @param QueryParameters $parameters
+     */
     public function delete($parameters) {
         return $this->driver->query(
             $this->getQueryEngine()->getDeleteQuery($parameters), $parameters->getBoundData()
@@ -119,6 +128,9 @@ 
         return $this->queryEngine;
     }
 
+    /**
+     * @param RecordWrapper $model
+     */
     public function setModel($model) {
         $this->modelInstance = $model;
     }

Unused Use Statements -3 removed lines

@@ -2,9 +2,6 @@
 
 namespace ntentan\nibii;
 
-use ntentan\utils\Text;
-use ntentan\atiaa\Db;
-
 /**
  * A DriverAdaptr is a generic database adapter.
  * This adapter implements a lot of its operations through the atiaa library.

src/QueryOperations.php

Doc Comments +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
     /**
      * 
      * @param RecordWrapper $wrapper
-     * @param DataAdapter $adapter
+     * @param DriverAdapter $adapter
      * @param DataOperations $dataOperations
      */
     public function __construct(Context $context, RecordWrapper $wrapper, DriverAdapter $adapter, $dataOperations) {

Unused Use Statements -2 removed lines

@@ -27,8 +27,6 @@
 namespace ntentan\nibii;
 
 use ntentan\utils\Text;
-use ntentan\atiaa\Db;
-use ntentan\panie\Container;
 
 class QueryOperations {