tarantool-php / mapper

src/Bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
             if (!is_object($migration)) {
                 $migration = new $migration;
             }
-            $schema->once(get_class($migration), function () use ($migration) {
+            $schema->once(get_class($migration), function() use ($migration) {
                 $migration->migrate($this->mapper);
             });
         }

src/Plugin/UserClasses.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
         }
 
         if (!is_subclass_of($class, Entity::class)) {
-            throw new Exception("Entity should extend " . Entity::class . " class");
+            throw new Exception("Entity should extend ".Entity::class." class");
         }
 
         $this->entityMapping[$space] = $class;
@@ -51,7 +51,7 @@ 
         }
 
         if (!is_subclass_of($class, Repository::class)) {
-            throw new Exception("Repository should extend " . Repository::class . " class");
+            throw new Exception("Repository should extend ".Repository::class." class");
         }
 
         $this->repositoryMapping[$space] = $class;

Doc Comments +3 added lines

@@ -33,6 +33,9 @@
         return null;
     }
 
+    /**
+     * @param string $space
+     */
     public function mapEntity($space, $class) : self
     {
         $this->validateSpace($space);

src/Plugin/Temporal/Schema.php

Spacing +5 added lines, -5 removed lines

@@ -31,7 +31,7 @@ 
 
     private function initLink()
     {
-        $this->mapper->getSchema()->once(__CLASS__.'@link', function (Mapper $mapper) {
+        $this->mapper->getSchema()->once(__CLASS__.'@link', function(Mapper $mapper) {
             $mapper->getSchema()
                 ->createSpace('_temporal_link', [
                     'id'        => 'unsigned',
@@ -62,14 +62,14 @@ 
                 ->addIndex(['entity', 'id', 'begin']);
         });
 
-        $this->mapper->getSchema()->once(__CLASS__.'@link-idle', function (Mapper $mapper) {
+        $this->mapper->getSchema()->once(__CLASS__.'@link-idle', function(Mapper $mapper) {
             $mapper->getSchema()->getSpace('_temporal_link')->addProperty('idle', 'unsigned');
         });
     }
 
     private function initOverride()
     {
-        $this->mapper->getSchema()->once(__CLASS__.'@states', function (Mapper $mapper) {
+        $this->mapper->getSchema()->once(__CLASS__.'@states', function(Mapper $mapper) {
             $mapper->getSchema()
                 ->createSpace('_temporal_override', [
                     'entity'     => 'unsigned',
@@ -93,14 +93,14 @@ 
                 ->addIndex(['entity', 'id', 'begin']);
         });
 
-        $this->mapper->getSchema()->once(__CLASS__.'@override-idle', function (Mapper $mapper) {
+        $this->mapper->getSchema()->once(__CLASS__.'@override-idle', function(Mapper $mapper) {
             $mapper->getSchema()->getSpace('_temporal_override')->addProperty('idle', 'unsigned');
         });
     }
 
     private function initReference()
     {
-        $this->mapper->getSchema()->once(__CLASS__.'@reference', function (Mapper $mapper) {
+        $this->mapper->getSchema()->once(__CLASS__.'@reference', function(Mapper $mapper) {
             $mapper->getSchema()
                 ->createSpace('_temporal_reference', [
                     'idle'       => 'unsigned',

src/Plugin/Temporal/Aggregator.php

Doc Comments +3 added lines

@@ -258,6 +258,9 @@
         }
     }
 
+    /**
+     * @param \Closure $callback
+     */
     private function generateStates($changes, $callback)
     {
         $slices = [];

Spacing +9 added lines, -9 removed lines

@@ -46,7 +46,7 @@ 
         ];
 
         $changes = $mapper->find('_temporal_reference', $params);
-        $states = $this->generateStates($changes, function ($state, $change) {
+        $states = $this->generateStates($changes, function($state, $change) {
             $state->data = $change->targetId;
         });
 
@@ -77,7 +77,7 @@ 
                 'targetId' => $entityId,
                 'entity' => $params['entity'],
             ]);
-            $aggregates = $this->generateStates($changes, function ($state, $change) {
+            $aggregates = $this->generateStates($changes, function($state, $change) {
                 if (!in_array($change->id, $state->data)) {
                     $state->data[] = $change->id;
                 }
@@ -232,11 +232,11 @@ 
             while (!$clean) {
                 $clean = true;
                 foreach ($states as $i => $state) {
-                    if (array_key_exists($i+1, $states)) {
-                        $next = $states[$i+1];
+                    if (array_key_exists($i + 1, $states)) {
+                        $next = $states[$i + 1];
                         if (json_encode($state->data) == json_encode($next->data)) {
                             $states[$i]->end = $next->end;
-                            unset($states[$i+1]);
+                            unset($states[$i + 1]);
                             $states = array_values($states);
                             $clean = false;
                             break;
@@ -264,7 +264,7 @@ 
         ];
 
         $changes = $mapper->find('_temporal_override', $params);
-        $states = $this->generateStates($changes, function ($state, $change) {
+        $states = $this->generateStates($changes, function($state, $change) {
             $state->data = array_merge($state->data, $change->data);
             $state->exists = false;
         });
@@ -344,15 +344,15 @@ 
         while (!$clean) {
             $clean = true;
             foreach ($states as $i => $state) {
-                if (array_key_exists($i+1, $states)) {
-                    $next = $states[$i+1];
+                if (array_key_exists($i + 1, $states)) {
+                    $next = $states[$i + 1];
                     if ($state->end && $state->end < $next->begin) {
                         // unmergable
                         continue;
                     }
                     if (json_encode($state->data) == json_encode($next->data)) {
                         $state->end = $next->end;
-                        unset($states[$i+1]);
+                        unset($states[$i + 1]);
                         $states = array_values($states);
                         $clean = false;
                         break;

src/Plugin/Procedure.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 
     private function initSchema()
     {
-        $this->mapper->getSchema()->once(__CLASS__, function ($mapper) {
+        $this->mapper->getSchema()->once(__CLASS__, function($mapper) {
             $mapper->getSchema()
                 ->createSpace('_procedure')
                 ->addProperties([

src/Mapper.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     public function getPlugin($mixed)
     {
         if (!is_subclass_of($mixed, Plugin::class)) {
-            throw new Exception("Plugin should extend " . Plugin::class . " class");
+            throw new Exception("Plugin should extend ".Plugin::class." class");
         }
 
         $plugin = is_object($mixed) ? $mixed : new $mixed($this);

src/Plugin/Temporal.php

Spacing +2 added lines, -2 removed lines

@@ -382,7 +382,7 @@ 
                 'entityId' => $id,
                 'parent'   => $node ? $node->id : 0,
             ];
-            if (count($config) == $i+1) {
+            if (count($config) == $i + 1) {
                 $params['begin'] = $link['begin'];
                 $params['timestamp'] = 0;
             }
@@ -461,7 +461,7 @@ 
             $this->mapper->getPlugin(Sequence::class);
         }
 
-        $this->mapper->getSchema()->once(__CLASS__.'@entity', function (Mapper $mapper) {
+        $this->mapper->getSchema()->once(__CLASS__.'@entity', function(Mapper $mapper) {
             $this->mapper->getSchema()
                 ->createSpace('_temporal_entity', [
                     'id'   => 'unsigned',

Doc Comments +6 added lines

@@ -250,6 +250,9 @@ 
         return $this->getData($entity, $id, $date, '_temporal_override_aggregate');
     }
 
+    /**
+     * @param string $space
+     */
     private function getData($entity, $id, $date, $space)
     {
         $entity = $this->entityNameToId($entity);
@@ -468,6 +471,9 @@ 
         return $this;
     }
 
+    /**
+     * @return string
+     */
     private function getTimestamp($string)
     {
         if (Carbon::hasTestNow() || !array_key_exists($string, $this->timestamps)) {

src/Entity.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
                         'id' => $this->$property,
                     ]);
                 }
-            } else if(strpos($property, 'Collection') !== false) {
+            } else if (strpos($property, 'Collection') !== false) {
                 $property = substr($property, 0, -10);
                 $targetSpace = $mapper->getSchema()->toUnderscore($property);
                 if ($mapper->getSchema()->hasSpace($targetSpace)) {
@@ -58,7 +58,7 @@ 
                 }
             }
         }
-        throw new BadMethodCallException("Call to undefined method ". get_class($this).'::'.$name);
+        throw new BadMethodCallException("Call to undefined method ".get_class($this).'::'.$name);
     }
 
     public function __debugInfo()

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use BadMethodCallException;
 use Exception;
-use Tarantool\Mapper\Plugin\Annotation;
 
 class Entity
 {

src/Plugin/Annotation.php

Spacing +2 added lines, -2 removed lines

@@ -193,7 +193,7 @@ 
                         $space->addIndex($index);
                     } catch (Exception $e) {
                         $presentation = json_encode($properties['indexes'][$i]);
-                        throw new Exception("Failed to add index $presentation. ". $e->getMessage(), 0, $e);
+                        throw new Exception("Failed to add index $presentation. ".$e->getMessage(), 0, $e);
                     }
                 }
             }
@@ -204,7 +204,7 @@ 
             }
             if (!count($space->getIndexes())) {
                 if (!$space->hasProperty('id')) {
-                    throw new Exception("No primary index on ". $space->getName());
+                    throw new Exception("No primary index on ".$space->getName());
                 }
                 $space->addIndex(['id']);
             }