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

src/Mapper.php

Doc Comments +9 added lines

@@ -28,6 +28,9 @@ 
         return $plugin;
     }
 
+    /**
+     * @param string $space
+     */
     public function create($space, $data)
     {
         $instance = $this->getRepository($space)->create($data);
@@ -35,11 +38,17 @@ 
         return $instance;
     }
 
+    /**
+     * @param string $space
+     */
     public function findOne($space, $params = [])
     {
         return $this->getRepository($space)->findOne($params);
     }
 
+    /**
+     * @param string $space
+     */
     public function find($space, $params = [])
     {
         return $this->getRepository($space)->find($params);

Spacing +9 added lines, -9 removed lines

@@ -19,8 +19,8 @@ 
 
     public function addPlugin($class)
     {
-        if(!is_subclass_of($class, Plugin::class)) {
-            throw new Exception("Plugin should extend " . Plugin::class . " class");
+        if (!is_subclass_of($class, Plugin::class)) {
+            throw new Exception("Plugin should extend ".Plugin::class." class");
         }
 
         $plugin = is_object($class) ? $class : new $class($this);
@@ -48,8 +48,8 @@ 
 
     public function findRepository(Entity $instance)
     {
-        foreach($this->getSchema()->getSpaces() as $space) {
-            if($space->getRepository()->knows($instance)) {
+        foreach ($this->getSchema()->getSpaces() as $space) {
+            if ($space->getRepository()->knows($instance)) {
                 return $space->getRepository();
             }
         }
@@ -59,7 +59,7 @@ 
 
     public function getBootstrap()
     {
-        if(!$this->bootstrap) {
+        if (!$this->bootstrap) {
             $this->bootstrap = new Bootstrap($this);
         }
 
@@ -73,7 +73,7 @@ 
 
     public function getPlugin($class)
     {
-        if(!array_key_exists($class, $this->plugins)) {
+        if (!array_key_exists($class, $this->plugins)) {
             throw new Exception("No plugin $class");
         }
         return $this->plugins[$class];
@@ -92,8 +92,8 @@ 
     public function getRepositories()
     {
         $repositories = [];
-        foreach($this->getSchema()->getSpaces() as $space) {
-            if($space->repositoryExists()) {
+        foreach ($this->getSchema()->getSpaces() as $space) {
+            if ($space->repositoryExists()) {
                 $repositories[] = $space->getRepository();
             }
         }
@@ -107,7 +107,7 @@ 
 
     public function remove($space, $params = [])
     {
-        if($space instanceof Entity) {
+        if ($space instanceof Entity) {
             $this->findRepository($space)->removeEntity($space);
 
         } else {

src/Plugins/Sequence.php

Doc Comments +3 added lines

@@ -19,6 +19,9 @@
         }
     }
 
+    /**
+     * @param Space $space
+     */
     private function generateValue($space)
     {
         $spaceId = $space->getId();

Spacing +4 added lines, -4 removed lines

@@ -11,9 +11,9 @@ 
     public function generateKey(Entity $instance, Space $space)
     {
         $primary = $space->getPrimaryIndex();
-        if(count($primary->parts) == 1) {
+        if (count($primary->parts) == 1) {
             $key = $space->getFormat()[$primary->parts[0][0]]['name'];
-            if(!property_exists($instance, $key)) {
+            if (!property_exists($instance, $key)) {
                 $instance->$key = $this->generateValue($space);
             }
         }
@@ -23,7 +23,7 @@ 
     {
         $spaceId = $space->getId();
 
-        if(!$this->mapper->getSchema()->hasSpace('sequence')) {
+        if (!$this->mapper->getSchema()->hasSpace('sequence')) {
 
             $sequence = $this->mapper->getSchema()->createSpace('sequence');
             $sequence->addProperty('space', 'unsigned');
@@ -37,7 +37,7 @@ 
         }
 
         $entity = $this->mapper->findOne('sequence', $space->getId());
-        if(!$entity) {
+        if (!$entity) {
             $entity = $this->mapper->create('sequence', [
                 'space' => $space->getId(),
                 'counter' => 0,

src/Plugins/UserClasses.php

Spacing +9 added lines, -9 removed lines

@@ -15,14 +15,14 @@ 
 
     public function getRepositoryClass(Space $space)
     {
-        if(array_key_exists($space->getName(), $this->repositories)) {
+        if (array_key_exists($space->getName(), $this->repositories)) {
             return $this->repositories[$space->getName()];
         }
     }
 
     public function getEntityClass(Space $space)
     {
-        if(array_key_exists($space->getName(), $this->entities)) {
+        if (array_key_exists($space->getName(), $this->entities)) {
             return $this->entities[$space->getName()];
         }
     }
@@ -31,12 +31,12 @@ 
     {
         $this->validateSpace($space);
 
-        if(!class_exists($class)) {
+        if (!class_exists($class)) {
             throw new Exception("No class $class");
         }
 
-        if(!is_subclass_of($class, Entity::class)) {
-            throw new Exception("Entity should extend " . Entity::class . " class");
+        if (!is_subclass_of($class, Entity::class)) {
+            throw new Exception("Entity should extend ".Entity::class." class");
         }
 
         $this->entities[$space] = $class;
@@ -46,12 +46,12 @@ 
     {
         $this->validateSpace($space);
 
-        if(!class_exists($class)) {
+        if (!class_exists($class)) {
             throw new Exception("No class $class");
         }
 
-        if(!is_subclass_of($class, Repository::class)) {
-            throw new Exception("Repository should extend " . Repository::class . " class");
+        if (!is_subclass_of($class, Repository::class)) {
+            throw new Exception("Repository should extend ".Repository::class." class");
         }
 
         $this->repositories[$space] = $class;
@@ -59,7 +59,7 @@ 
 
     public function validateSpace($space)
     {
-        if(!$this->mapper->getSchema()->hasSpace($space)) {
+        if (!$this->mapper->getSchema()->hasSpace($space)) {
             throw new Exception("No space $space");
         }
     }

src/Schema.php

Spacing +8 added lines, -8 removed lines

@@ -31,7 +31,7 @@ 
 
     public function formatValue($type, $value)
     {
-        switch($type) {
+        switch ($type) {
             case 'str': return (string) $value;
             case 'unsigned': return (int) $value;
             default: return $value;
@@ -40,15 +40,15 @@ 
 
     public function getSpace($id)
     {
-        if(is_string($id)) {
+        if (is_string($id)) {
             return $this->getSpace($this->getSpaceId($id));
         }
 
-        if(!$id) {
+        if (!$id) {
             throw new Exception("Space id or name not defined");
         }
 
-        if(!array_key_exists($id, $this->spaces)) {
+        if (!array_key_exists($id, $this->spaces)) {
             $this->spaces[$id] = new Space($this->mapper, $id, array_search($id, $this->names));
         }
         return $this->spaces[$id];
@@ -56,7 +56,7 @@ 
 
     public function getSpaceId($name)
     {
-        if(!$this->hasSpace($name)) {
+        if (!$this->hasSpace($name)) {
             throw new Exception("No space $id");
         }
         return $this->names[$name];
@@ -64,7 +64,7 @@ 
 
     public function getSpaces()
     {
-        foreach($this->names as $id) {
+        foreach ($this->names as $id) {
             $this->getSpace($id);
         }
         return $this->spaces;
@@ -77,10 +77,10 @@ 
 
     public function once($name, $callback)
     {
-        $key = 'once' . $name;
+        $key = 'once'.$name;
 
         $rows = $this->mapper->find('_schema', ['key' => $key]);
-        if(!count($rows)) {
+        if (!count($rows)) {
             $this->mapper->create('_schema', ['key' => $key]);
             return $callback($this->mapper);
         }

src/Bootstrap.php

Spacing +2 added lines, -2 removed lines

@@ -20,8 +20,8 @@
     public function migrate()
     {
         $schema = $this->mapper->getSchema();
-        foreach($this->migrations as $migration) {
-            if(!is_object($migration)) {
+        foreach ($this->migrations as $migration) {
+            if (!is_object($migration)) {
                 $migration = new $migration;
             }
             $schema->once(get_class($migration), function() use ($migration) {

src/Space.php

Spacing +43 added lines, -43 removed lines

@@ -27,7 +27,7 @@ 
 
     public function addProperties($config)
     {
-        foreach($config as $name => $type) {
+        foreach ($config as $name => $type) {
             $this->addProperty($name, $type);
         }
         return $this;
@@ -36,8 +36,8 @@ 
     public function addProperty($name, $type)
     {
         $format = $this->getFormat();
-        foreach($format as $field) {
-            if($field['name'] == $name) {
+        foreach ($format as $field) {
+            if ($field['name'] == $name) {
                 throw new Exception("Property $name exists");
             }
         }
@@ -54,7 +54,7 @@ 
     {
         $format = $this->getFormat();
         $last = array_pop($format);
-        if($last['name'] != $name) {
+        if ($last['name'] != $name) {
             throw new Exception("Remove only last property");
         }
         $this->mapper->getClient()->evaluate("box.space[$this->id]:format(...)", [$format]);
@@ -82,13 +82,13 @@ 
     public function createIndex($config)
     {
 
-        if(!is_array($config)) {
+        if (!is_array($config)) {
             $config = ['fields' => $config];
         }
 
 
-        if(!array_key_exists('fields', $config)) {
-            if(array_values($config) != $config) {
+        if (!array_key_exists('fields', $config)) {
+            if (array_values($config) != $config) {
                 throw new Exception("Invalid index configuration");
             }
             $config = [
@@ -96,7 +96,7 @@ 
             ];
         }
 
-        if(!is_array($config['fields'])) {
+        if (!is_array($config['fields'])) {
             $config['fields'] = [$config['fields']];
         }
 
@@ -104,17 +104,17 @@ 
             'parts' => []
         ];
 
-        foreach($config as $k => $v) {
-            if($k != 'name' && $k != 'fields') {
+        foreach ($config as $k => $v) {
+            if ($k != 'name' && $k != 'fields') {
                 $options[$k] = $v;
             }
         }
 
-        foreach($config['fields'] as $property) {
-            if(!$this->getPropertyType($property)) {
+        foreach ($config['fields'] as $property) {
+            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);
         }
 
@@ -138,15 +138,15 @@ 
 
     public function getFormat()
     {
-        if(!$this->format) {
-            if($this->isSpecial()) {
+        if (!$this->format) {
+            if ($this->isSpecial()) {
                 $this->format = $this->mapper->getClient()
                     ->getSpace(280)->select([$this->id])->getData()[0][6];
 
             } else {
                 $this->format = $this->mapper->findOne('_space', ['id' => $this->id])->format;
             }
-            if(!$this->format) {
+            if (!$this->format) {
                 $this->format = [];
             }
             $this->parseFormat();
@@ -169,7 +169,7 @@ 
     {
         $this->formatTypesHash = [];
         $this->formatNamesHash = [];
-        foreach($this->format as $key => $row) {
+        foreach ($this->format as $key => $row) {
             $this->formatTypesHash[$row['name']] = $row['type'];
             $this->formatNamesHash[$row['name']] = $key;
         }
@@ -184,7 +184,7 @@ 
 
     public function getPropertyType($name)
     {
-        if(!$this->hasProperty($name)) {
+        if (!$this->hasProperty($name)) {
             throw new Exception("No property $name");
         }
         return $this->formatTypesHash[$name];
@@ -192,7 +192,7 @@ 
 
     public function getPropertyIndex($name)
     {
-        if(!$this->hasProperty($name)) {
+        if (!$this->hasProperty($name)) {
             throw new Exception("No property $name");
         }
         return $this->formatNamesHash[$name];
@@ -200,14 +200,14 @@ 
 
     public function getIndexes()
     {
-        if(!$this->indexes) {
-            if($this->isSpecial()) {
+        if (!$this->indexes) {
+            if ($this->isSpecial()) {
                 $this->indexes = [];
                 $indexTuples = $this->mapper->getClient()->getSpace(288)->select([$this->id])->getData();
                 $indexFormat = $this->mapper->getSchema()->getSpace(288)->getFormat();
-                foreach($indexTuples as $tuple) {
+                foreach ($indexTuples as $tuple) {
                     $instance = (object) [];
-                    foreach($indexFormat as $index => $format) {
+                    foreach ($indexFormat as $index => $format) {
                         $instance->{$format['name']} = $tuple[$index];
                     }
                     $this->indexes[] = $instance;
@@ -225,44 +225,44 @@ 
         $keys = array_keys($params);
 
         $keys = [];
-        foreach($params as $name => $value) {
+        foreach ($params as $name => $value) {
             $keys[] = $this->getPropertyIndex($name);
         }
-        if($keys == [0]) {
+        if ($keys == [0]) {
             // primary index
             return 0;
         }
 
         // equals
-        foreach($this->getIndexes() as $index) {
+        foreach ($this->getIndexes() as $index) {
             $equals = false;
-            if(count($keys) == count($index->parts)) {
+            if (count($keys) == count($index->parts)) {
                 // same length
                 $equals = true;
-                foreach($index->parts as $part) {
+                foreach ($index->parts as $part) {
                     $equals = $equals && in_array($part[0], $keys);
                 }
             }
 
-            if($equals) {
+            if ($equals) {
                 return $index->iid;
             }
         }
 
         // index part
-        foreach($this->getIndexes() as $index) {
+        foreach ($this->getIndexes() as $index) {
             $partial = [];
-            foreach($index->parts as $n => $part) {
-                if(!array_key_exists($n, $keys)) {
+            foreach ($index->parts as $n => $part) {
+                if (!array_key_exists($n, $keys)) {
                     break;
                 }
-                if($keys[$n] != $part[0]) {
+                if ($keys[$n] != $part[0]) {
                     break;
                 }
                 $partial[] = $keys[$n];
             }
 
-            if(count($partial) == count($keys)) {
+            if (count($partial) == count($keys)) {
                 return $index->iid;
             }
         }
@@ -276,9 +276,9 @@ 
         $format = $this->getFormat();
 
         $values = [];
-        foreach($index->parts as $part) {
+        foreach ($index->parts as $part) {
             $name = $format[$part[0]]['name'];
-            if(!array_key_exists($name, $params)) {
+            if (!array_key_exists($name, $params)) {
                 break;
             }
             $values[] = $this->mapper->getSchema()->formatValue($part[1], $params[$name]);
@@ -289,7 +289,7 @@ 
     public function getPrimaryIndex()
     {
         $indexes = $this->getIndexes();
-        if(!count($indexes)) {
+        if (!count($indexes)) {
             throw new Exception("No primary index");
         }
         return $indexes[0];
@@ -298,7 +298,7 @@ 
     public function getTupleKey($tuple)
     {
         $key = [];
-        foreach($this->getPrimaryIndex()->parts as $part) {
+        foreach ($this->getPrimaryIndex()->parts as $part) {
             $key[] = $tuple[$part[0]];
         }
         return count($key) == 1 ? $key[0] : implode(':', $key);
@@ -309,9 +309,9 @@ 
 
         $key = [];
 
-        foreach($this->getPrimaryIndex()->parts as $part) {
+        foreach ($this->getPrimaryIndex()->parts as $part) {
             $name = $this->getFormat()[$part[0]]['name'];
-            if(!property_exists($instance, $name)) {
+            if (!property_exists($instance, $name)) {
                 throw new Exception("Field $name is undefined", 1);
             }
             $key[] = $instance->$name;
@@ -323,10 +323,10 @@ 
     public function getRepository()
     {
         $class = Repository::class;
-        foreach($this->mapper->getPlugins() as $plugin) {
+        foreach ($this->mapper->getPlugins() as $plugin) {
             $repositoryClass = $plugin->getRepositoryClass($this);
-            if($repositoryClass) {
-                if($class != Repository::class) {
+            if ($repositoryClass) {
+                if ($class != Repository::class) {
                     throw new Exception('Repository class override');
                 }
                 $class = $repositoryClass;

src/Plugins/Spy.php

Spacing +7 added lines, -7 removed lines

@@ -20,7 +20,7 @@ 
     public function beforeUpdate(Entity $instance, Space $space)
     {
         $key = $this->getKey($instance, $space);
-        if(!array_key_exists($key, $this->create)) {
+        if (!array_key_exists($key, $this->create)) {
             $this->update[$key] = $instance;
         }
     }
@@ -29,12 +29,12 @@ 
     {
         $key = $this->getKey($instance, $space);
 
-        if(array_key_exists($key, $this->create)) {
+        if (array_key_exists($key, $this->create)) {
             unset($this->create[$key]);
             return;
         }
 
-        if(array_key_exists($key, $this->update)) {
+        if (array_key_exists($key, $this->update)) {
             unset($this->update[$key]);
         }
 
@@ -53,7 +53,7 @@ 
         $key = [$space->getName()];
 
         $format = $space->getFormat();
-        foreach($space->getPrimaryIndex()->parts as $part) {
+        foreach ($space->getPrimaryIndex()->parts as $part) {
             $key[] = $instance->{$format[$part[0]]['name']};
         }
 
@@ -64,11 +64,11 @@ 
     {
         $result = (object) [];
 
-        foreach(['create', 'update', 'remove'] as $action) {
+        foreach (['create', 'update', 'remove'] as $action) {
             $data = [];
-            foreach($this->$action as $key => $row) {
+            foreach ($this->$action as $key => $row) {
                 list($space) = explode(':', $key);
-                if(!array_key_exists($space, $data)) {
+                if (!array_key_exists($space, $data)) {
                     $data[$space] = [];
                 }
                 $data[$space][] = $row;

src/Plugins/DocBlock.php

Doc Comments +3 added lines

@@ -120,6 +120,9 @@
 
     private $underscores = [];
 
+    /**
+     * @param string $input
+     */
     private function toUnderscore($input)
     {
         if(!array_key_exists($input, $this->underscores)) {

Unused Use Statements -3 removed lines

@@ -3,13 +3,10 @@
 namespace Tarantool\Mapper\Plugins;
 
 use Exception;
-
 use phpDocumentor\Reflection\DocBlockFactory;
 use ReflectionClass;
 use ReflectionProperty;
-
 use Tarantool\Mapper\Entity;
-use Tarantool\Mapper\Plugin;
 use Tarantool\Mapper\Repository;
 
 class DocBlock extends UserClasses

Spacing +24 added lines, -24 removed lines

@@ -22,19 +22,19 @@ 
         $isEntity = is_subclass_of($class, Entity::class);
         $isRepository = is_subclass_of($class, Repository::class);
 
-        if(!$isEntity && !$isRepository) {
+        if (!$isEntity && !$isRepository) {
             throw new Exception("Invalid registration");
         }
 
-        if($isEntity) {
-            if($class == Entity::class) {
+        if ($isEntity) {
+            if ($class == Entity::class) {
                 throw new Exception("Invalid entity registration");
             }
             $this->entities[] = $class;
         }
 
-        if($isRepository) {
-            if($class == Repository::class) {
+        if ($isRepository) {
+            if ($class == Repository::class) {
                 throw new Exception("Invalid repository registration");
             }
             $this->repositories[] = $class;
@@ -42,8 +42,8 @@ 
 
         $reflection = new ReflectionClass($class);
         $space = $this->toUnderscore($reflection->getShortName());
-        if($this->mapper->getSchema()->hasSpace($space)) {
-            if($isEntity) {
+        if ($this->mapper->getSchema()->hasSpace($space)) {
+            if ($isEntity) {
                 $this->mapEntity($name, $class);
             } else {
                 $this->mapRepository($name, $class);
@@ -57,7 +57,7 @@ 
 
         $schema = $this->mapper->getSchema();
 
-        foreach($this->entities as $entity) {
+        foreach ($this->entities as $entity) {
 
             $class = new ReflectionClass($entity);
             $spaceName = $this->toUnderscore($class->getShortName());
@@ -65,34 +65,34 @@ 
             $space = $schema->hasSpace($spaceName) ? $schema->getSpace($spaceName) : $schema->createSpace($spaceName);
             $this->mapEntity($spaceName, $entity);
 
-            foreach($class->getProperties(ReflectionProperty::IS_PUBLIC) as $property) {
+            foreach ($class->getProperties(ReflectionProperty::IS_PUBLIC) as $property) {
 
                 $description = $factory->create($property->getDocComment());
                 $tags = $description->getTags('var');
 
-                if(!count($tags)) {
+                if (!count($tags)) {
                     throw new Exception("No var tag for ".$entity.'::'.$property->getName());
                 }
 
-                if(count($tags) > 1) {
+                if (count($tags) > 1) {
                     throw new Exception("Invalid var tag for ".$entity.'::'.$property->getName());
                 }
 
                 $property = $this->toUnderscore($property->getName());
                 $type = $this->getTarantoolType($tags[0]->getType());
 
-                if(!$space->hasProperty($property)) {
+                if (!$space->hasProperty($property)) {
                     $space->addProperty($property, $type);
                 }
             }
         }
 
-        foreach($this->repositories as $repository) {
+        foreach ($this->repositories as $repository) {
 
             $class = new ReflectionClass($repository);
             $spaceName = $this->toUnderscore($class->getShortName());
 
-            if(!$schema->hasSpace($spaceName)) {
+            if (!$schema->hasSpace($spaceName)) {
                 throw new Exception("Repository with no entity definition");
             }
 
@@ -100,18 +100,18 @@ 
 
             $space = $schema->getSpace($spaceName);
             $properties = $class->getDefaultProperties();
-            if(array_key_exists('indexes', $properties)) {
-                foreach($properties['indexes'] as $index) {
+            if (array_key_exists('indexes', $properties)) {
+                foreach ($properties['indexes'] as $index) {
                     $space->addIndex($index);
                 }
             }
         }
 
-        foreach($schema->getSpaces() as $space) {
+        foreach ($schema->getSpaces() as $space) {
 
-            if(!count($space->getIndexes())) {
-                if(!$space->hasProperty('id')) {
-                    throw new Exception("No primary index on ". $space->getName());
+            if (!count($space->getIndexes())) {
+                if (!$space->hasProperty('id')) {
+                    throw new Exception("No primary index on ".$space->getName());
                 }
                 $space->addIndex(['id']);
             }
@@ -122,7 +122,7 @@ 
 
     private function toUnderscore($input)
     {
-        if(!array_key_exists($input, $this->underscores)) {
+        if (!array_key_exists($input, $this->underscores)) {
             preg_match_all('!([A-Z][A-Z0-9]*(?=$|[A-Z][a-z0-9])|[A-Za-z][a-z0-9]+)!', $input, $matches);
             $ret = $matches[0];
             foreach ($ret as &$match) {
@@ -137,15 +137,15 @@ 
 
     private function getTarantoolType(string $type)
     {
-        if(array_key_exists($type, $this->tarantoolTypes)) {
+        if (array_key_exists($type, $this->tarantoolTypes)) {
             return $this->tarantoolTypes[$type];
         }
 
-        if($type[0] == '\\') {
+        if ($type[0] == '\\') {
             return $this->tarantoolTypes[$type] = 'unsigned';
         }
 
-        switch($type) {
+        switch ($type) {
             case 'int':
                 return $this->tarantoolTypes[$type] = 'unsigned';