tarantool-php / 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/Aggregator.php

Doc Comments +3 added lines

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

Spacing +3 added lines, -3 removed lines

@@ -49,7 +49,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;
         });
 
@@ -80,7 +80,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;
                 }
@@ -269,7 +269,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;
         });

src/Entity.php

Unused Use Statements -1 removed lines

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

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,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 sync(): Entity

src/Plugin/Temporal.php

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

Spacing +2 added lines, -2 removed lines

@@ -505,7 +505,7 @@ 
     private function getTimestamp($string)
     {
         if (Carbon::hasTestNow() || !array_key_exists($string, $this->timestamps)) {
-            if (strlen('' . $string) == 8 && is_numeric($string)) {
+            if (strlen(''.$string) == 8 && is_numeric($string)) {
                 $value = Carbon::createFromFormat('Ymd', $string)->setTime(0, 0, 0)->timestamp;
             } else {
                 $value = Carbon::parse($string)->timestamp;
@@ -549,7 +549,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/Sequence.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 namespace Tarantool\Mapper\Plugin;
 
 use Exception;
-use Tarantool\Client\Schema\Operations;
 use Tarantool\Mapper\Entity;
 use Tarantool\Mapper\Plugin;
 use Tarantool\Mapper\Procedure\CreateSequence;

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
         $this->initializeSequence($space);
 
         $next = $this->mapper->getClient()
-            ->call('box.sequence.' . $space->getName() . ':next');
+            ->call('box.sequence.'.$space->getName().':next');
 
         return $next[0];
     }

src/Schema.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use Closure;
 use Exception;
 use Tarantool\Client\Schema\Criteria;
-use Tarantool\Client\Schema\Space as ClientSpace;
 
 class Schema
 {

Doc Comments +3 added lines

@@ -89,6 +89,9 @@
         throw new Exception("Invalid type $type");
     }
 
+    /**
+     * @return string
+     */
     public function formatValue(string $type, $value)
     {
         if ($value === null) {

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
         }
 
         if (!in_array($options['engine'], ['memtx', 'vinyl'])) {
-            throw new Exception("Invalid engine " . $options['engine']);
+            throw new Exception("Invalid engine ".$options['engine']);
         }
 
         [$id] = $this->mapper->getClient()->evaluate("
@@ -161,7 +161,7 @@ 
 
     public function once(string $name, Closure $callback)
     {
-        $key = 'mapper-once' . $name;
+        $key = 'mapper-once'.$name;
         $row = $this->mapper->findOne('_schema', ['key' => $key]);
         if (!$row) {
             $this->mapper->create('_schema', ['key' => $key]);
@@ -171,7 +171,7 @@ 
 
     public function forgetOnce(string $name)
     {
-        $key = 'mapper-once' . $name;
+        $key = 'mapper-once'.$name;
         $row = $this->mapper->findOne('_schema', ['key' => $key]);
         if ($row) {
             $this->mapper->remove($row);

src/Pool.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
             unset($this->mappers[$name]);
         }
         foreach ($this->repositories as $space => $repository) {
-            if (strpos($space, $name . '.') !== false) {
+            if (strpos($space, $name.'.') !== false) {
                 unset($this->repositories[$space]);
             }
         }

src/Space.php

Doc Comments +3 added lines

@@ -325,6 +325,9 @@
         return $this->formatNamesHash[$name];
     }
 
+    /**
+     * @return string
+     */
     public function getReference(string $name): ?string
     {
         return $this->isReference($name) ? $this->formatReferences[$name] : null;

Spacing +2 added lines, -2 removed lines

@@ -199,7 +199,7 @@ 
 
     public function isSpecial(): bool
     {
-        return in_array($this->id, [ ClientSpace::VSPACE_ID, ClientSpace::VINDEX_ID ]);
+        return in_array($this->id, [ClientSpace::VSPACE_ID, ClientSpace::VINDEX_ID]);
     }
 
     public function isSystem(): bool
@@ -418,7 +418,7 @@ 
         }
 
         if (!$suppressException) {
-            throw new Exception("No index on " . $this->name . ' for [' . implode(', ', array_keys($params)) . ']');
+            throw new Exception("No index on ".$this->name.' for ['.implode(', ', array_keys($params)).']');
         }
 
         return null;

src/Procedure/FindOrCreate.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     {
         $index = $space->castIndex($query ?: $params);
         if ($index === null) {
-            throw new Exception("No valid index for " . json_encode($params));
+            throw new Exception("No valid index for ".json_encode($params));
         }
 
         $values = $space->getIndexValues($index, $params);