tarantool-php / mapper

src/Client.php

Doc Comments +3 added lines

@@ -31,6 +31,9 @@
         return $response;
     }
 
+    /**
+     * @param double $start
+     */
     private function log($start, $class, $request = [], $response = [])
     {
         if ($this->logging) {

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
     public function disableRequest($class)
     {
         if (!is_subclass_of($class, Request::class)) {
-            throw new Exception("Invalid request type: " . $class);
+            throw new Exception("Invalid request type: ".$class);
         }
         $this->disabledRequests[] = $class;
     }
@@ -33,7 +33,7 @@ 
     public function sendRequest(Request $request)
     {
         if (in_array(get_class($request), $this->disabledRequests)) {
-            throw new Exception(get_class($request) . ' is disabled');
+            throw new Exception(get_class($request).' is disabled');
         }
         $start = microtime(1);
         $response = parent::sendRequest($request);

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;

src/Repository.php

Doc Comments +3 added lines

@@ -270,6 +270,9 @@
         $this->results = [];
     }
 
+    /**
+     * @param Entity $instance
+     */
     public function save($instance)
     {
         $key = $this->space->getInstanceKey($instance);

src/Plugin/Temporal.php

Doc Comments +3 added lines

@@ -183,6 +183,9 @@
         return $this->getData($entity, $id, $date, '_temporal_override_aggregate');
     }
 
+    /**
+     * @param string $space
+     */
     private function getData($entity, $id, $date, $space)
     {
         $entity = $this->entityNameToId($entity);

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

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/Space.php

Spacing +1 added lines, -1 removed lines

@@ -145,7 +145,7 @@
             if (!$this->getPropertyType($property)) {
                 throw new Exception("Unknown property $property", 1);
             }
-            $options['parts'][] = $this->getPropertyIndex($property)+1;
+            $options['parts'][] = $this->getPropertyIndex($property) + 1;
             $options['parts'][] = $this->getPropertyType($property);
             $this->setPropertyNullable($property, false);
         }

Doc Comments +8 added lines

@@ -285,6 +285,9 @@ 
         }
     }
 
+    /**
+     * @param string $flag
+     */
     public function getPropertyFlag($name, $flag)
     {
         $property = $this->getProperty($name);
@@ -309,6 +312,11 @@ 
         return $this->formatNamesHash[$name];
     }
 
+    /**
+     * @param string $name
+     *
+     * @return string
+     */
     public function getReference($name)
     {
         return $this->isReference($name) ? $this->formatReferences[$name] : null;