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

@@ -28,7 +28,6 @@
 
 use ntentan\kaikai\Cache;
 use ntentan\utils\Text;
-use ntentan\atiaa\Db;
 
 /**
  * 

Spacing +3 added lines, -3 removed lines

@@ -61,8 +61,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'];
         }
@@ -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);
             }
         );

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 +6 added lines, -1 removed lines

@@ -23,7 +23,6 @@ 
 
     /**
      *
-     * @param \ $model
      */
     public function __construct(DriverAdapter $db, $table) {
         $this->db = $db;
@@ -106,6 +105,9 @@ 
         $this->boundData += $values;
     }
 
+    /**
+     * @param boolean $firstOnly
+     */
     public function setFirstOnly($firstOnly) {
         $this->firstOnly = $firstOnly;
         return $this;
@@ -123,6 +125,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';
         }
     }
 

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

@@ -27,8 +27,8 @@
 namespace ntentan\nibii\relationships;
 
 use ntentan\nibii\Nibii;
-use ntentan\utils\Text;
 use ntentan\panie\InjectionContainer;
+use ntentan\utils\Text;
 
 class ManyHaveManyRelationship extends \ntentan\nibii\Relationship {
 

src/relationships/BelongsToRelationship.php

Spacing +3 added lines, -3 removed lines

@@ -47,11 +47,11 @@
     {
         $model = InjectionContainer::resolve(Nibii::getClassName($this->options['model'], self::BELONGS_TO));
         $table = $model->getDBStoreInformation()['table'];
-        if($this->options['foreign_key'] == null) {
+        if ($this->options['foreign_key'] == null) {
             $this->options['foreign_key'] = $model->getDescription()->getPrimaryKey()[0];
         }
-        if($this->options['local_key'] == null) {
-            $this->options['local_key'] = Text::singularize($table) . '_id';
+        if ($this->options['local_key'] == null) {
+            $this->options['local_key'] = Text::singularize($table).'_id';
         }
     }
 }

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

@@ -26,10 +26,10 @@
 
 namespace ntentan\nibii\relationships;
 
-use ntentan\nibii\QueryParameters;
-use ntentan\utils\Text;
 use ntentan\nibii\Nibii;
+use ntentan\nibii\QueryParameters;
 use ntentan\panie\InjectionContainer;
+use ntentan\utils\Text;
 
 class BelongsToRelationship extends \ntentan\nibii\Relationship
 {

src/Context.php

Doc Comments +6 added lines

@@ -52,11 +52,17 @@
             ->getJunctionClassName($classA, $classB);
     }
 
+    /**
+     * @param RecordWrapper $instance
+     */
     public function getModelTable($instance) {
         return$this->container->singleton(interfaces\TableNameResolverInterface::class)
             ->getTableName($instance);
     }
 
+    /**
+     * @param string $model
+     */
     public function getClassName($model, $context = null) {
         return$this->container->singleton(interfaces\ModelClassResolverInterface::class)
             ->getModelClassName($model, $context);

Indentation +3 added lines, -3 removed lines

@@ -18,11 +18,11 @@
         $this->container = $container;
         $this->dbContext = $container->resolve(\ntentan\atiaa\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);
         self::$instance = $this;
     }
 

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