ntentan / nibii

src/RecordWrapper.php

Unused Use Statements -1 removed lines

@@ -25,7 +25,6 @@
 
 namespace ntentan\nibii;
 
-use ntentan\utils\Utils;
 use ntentan\kaikai\Cache;
 
 class RecordWrapper implements \ArrayAccess, \Countable, \Iterator

Spacing +16 added lines, -16 removed lines

@@ -49,7 +49,7 @@ 
     {
         $this->table = Nibii::getModelTable($this);
         $this->adapter = $adapter;
-        foreach($this->behaviours as $behaviour) {
+        foreach ($this->behaviours as $behaviour) {
             $behaviourInstance = $this->getComponentInstance($behaviour);
             $behaviourInstance->setModel($this);
         }
@@ -76,7 +76,7 @@ 
     public function getDescription()
     {
         return Cache::read(
-            (new \ReflectionClass($this))->getName() . '::desc',
+            (new \ReflectionClass($this))->getName().'::desc',
             function() 
             {
                 return new ModelDescription($this);
@@ -96,7 +96,7 @@ 
     private function retrieveItem($key)
     {
         $relationships = $this->getDescription()->getRelationships();
-        if(isset($relationships[$key])) {
+        if (isset($relationships[$key])) {
             return $this->fetchRelatedFields($relationships[$key]);
         } else {
             return isset($this->modelData[$key]) ? $this->modelData[$key] : null;
@@ -121,7 +121,7 @@ 
      */
     public function __call($name, $arguments)
     {
-        if($this->dynamicOperations === null) {
+        if ($this->dynamicOperations === null) {
             $this->dynamicOperations = new Operations($this, InjectionContainer::singleton(DriverAdapter::class));
         }
         return $this->dynamicOperations->perform($name, $arguments);
@@ -150,7 +150,7 @@ 
     
     private function expandArrayValue($array, $relationships, $depth, $index = null)
     {
-        foreach($relationships as $name => $relationship) {
+        foreach ($relationships as $name => $relationship) {
             $array[$name] = $this->fetchRelatedFields($relationship, $index)->toArray($depth);
         }
         return $array;
@@ -160,9 +160,9 @@ 
     {
         $relationships = $this->getDescription()->getRelationships();
         $array = $this->modelData;
-        if($depth > 0) {
-            if($this->hasMultipleData()) {
-                foreach($array as $i => $value) {
+        if ($depth > 0) {
+            if ($this->hasMultipleData()) {
+                foreach ($array as $i => $value) {
                     $array[$i] = $this->expandArrayValue($value, $relationships, $depth - 1, $i);
                 }
             } else {
@@ -186,7 +186,7 @@ 
 
     private function hasMultipleData()
     {
-        if(count($this->modelData) > 0) {
+        if (count($this->modelData) > 0) {
             return is_numeric(array_keys($this->modelData)[0]);
         } else {
             return false;
@@ -197,11 +197,11 @@ 
     {
         $data = [];
                 
-        if(count($this->modelData) == 0) {
+        if (count($this->modelData) == 0) {
             $data = $this->modelData;
-        } else if($this->hasMultipleData()) {
+        } else if ($this->hasMultipleData()) {
             $data = $this->modelData;
-        } else if(count($this->modelData) > 0) {
+        } else if (count($this->modelData) > 0) {
             $data[] = $this->modelData;
         }
         
@@ -216,7 +216,7 @@ 
     
     public function mergeData($data)
     {
-        foreach($data as $key => $value) {
+        foreach ($data as $key => $value) {
             $this->modelData[$key] = $value;
         }
         $this->dataSet = true;
@@ -249,7 +249,7 @@ 
 
     private function wrap($offset)
     {
-        if(isset($this->modelData[$offset])) {
+        if (isset($this->modelData[$offset])) {
             $newInstance = $this->createNew();
             $newInstance->setData($this->modelData[$offset]);
             return $newInstance;
@@ -300,13 +300,13 @@ 
 
     private function fetchRelatedFields($relationship, $index = null)
     {
-        if($index === null) {
+        if ($index === null) {
             $data = $this->modelData;
         } else {
             $data = $this->modelData[$index];
         }
         $model = $relationship->getModelInstance();
-        if(empty($data)) {
+        if (empty($data)) {
             return $model;
         } else {
             return $model->fetch($relationship->getQuery($data));

Doc Comments +5 added lines, -1 removed lines

@@ -101,7 +101,7 @@ 
 
     /**
      * @method
-     * @param type $name
+     * @param string $name
      * @param type $arguments
      * @return type
      */
@@ -136,6 +136,10 @@ 
         return $this->table;
     }
     
+    /**
+     * @param Relationship[] $relationships
+     * @param integer $depth
+     */
     private function expandArrayValue($array, $relationships, $depth, $index = null)
     {
         foreach($relationships as $name => $relationship) {

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\ClassResolverInterface;
 use ntentan\nibii\interfaces\ModelJoinerInterface;
 use ntentan\nibii\interfaces\TableNameResolverInterface;
-use ntentan\config\Config;
 use ntentan\utils\Text;
 
 /**

Spacing +5 added lines, -5 removed lines

@@ -31,7 +31,7 @@ 
     {
         $arrayed = explode('\\', $className);
         $class = array_pop($arrayed);
-        if($arrayed[0] == '') {
+        if ($arrayed[0] == '') {
             array_shift($arrayed);
         }
         return ['class' => $class, 'namespace' => implode('\\', $arrayed)];
@@ -41,7 +41,7 @@ 
     {
         $classA = $this->getClassFileDetails($classA);
         $classB = $this->getClassFileDetails($classB);
-        if($classA['namespace'] != $classB['namespace']) {
+        if ($classA['namespace'] != $classB['namespace']) {
             throw new NibiiException(
                 "Cannot automatically join two classes of different "
                     . "namespaces. Please provide a model joiner or "
@@ -50,7 +50,7 @@ 
         }
         $classes = [$classA['class'], $classB['class']];
         sort($classes);
-        return "{$classA['namespace']}\\" . implode('', $classes);        
+        return "{$classA['namespace']}\\".implode('', $classes);        
     }
 
     public function getTableName($instance)
@@ -63,8 +63,8 @@ 
     public static function getDriverAdapterClassName()
     {
         $driver = Config::get('ntentan:db.driver', false);
-        if($driver) {
-            return __NAMESPACE__ . '\adapters\\' . Text::ucamelize(Config::get('ntentan:db.driver')) . 'Adapter';
+        if ($driver) {
+            return __NAMESPACE__.'\adapters\\'.Text::ucamelize(Config::get('ntentan:db.driver')).'Adapter';
         } 
         throw new NibiiException("Please specify a driver");
     }