thecodingmachine / tdbm

tests/WeakrefObjectStorageTest.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
         $objectStorage = new WeakrefObjectStorage();
         $dbRow = $this->createMock(DbRow::class);
 
-        for ($i=0; $i<10001; $i++) {
+        for ($i = 0; $i < 10001; $i++) {
             $objectStorage->set('foo', $i, clone $dbRow);
         }
         $this->assertNull($objectStorage->get('foo', 42));

tests/Utils/DbalUtilsTest.php

Spacing +2 added lines, -2 removed lines

@@ -12,8 +12,8 @@
     {
         $params = [
             'key1' => 'foo',
-            'key2' => [1,2,3],
-            'key3' => [1,2,'baz'],
+            'key2' => [1, 2, 3],
+            'key3' => [1, 2, 'baz'],
             'key4' => 1,
         ];
 

tests/AlterableResultIteratorTest.php

Spacing +1 added lines, -1 removed lines

@@ -128,7 +128,7 @@
 
         $alterableResultIterator = new AlterableResultIterator($iterator);
 
-        $map = $alterableResultIterator->map(function ($item) {
+        $map = $alterableResultIterator->map(function($item) {
             return $item->foo;
         });
 

src/Schema/ForeignKey.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
     public function getCacheKey(): string
     {
         if ($this->cacheKey === null) {
-            $this->cacheKey = 'from__' . implode(',', $this->getUnquotedLocalColumns()) . '__to__table__' . $this->getForeignTableName() . '__columns__' . implode(',', $this->getUnquotedForeignColumns());
+            $this->cacheKey = 'from__'.implode(',', $this->getUnquotedLocalColumns()).'__to__table__'.$this->getForeignTableName().'__columns__'.implode(',', $this->getUnquotedForeignColumns());
         }
         return $this->cacheKey;
     }

src/Utils/PivotTableMethodsDescriptor.php

Spacing +9 added lines, -9 removed lines

@@ -201,7 +201,7 @@ 
 
         $getter = new MethodGenerator($this->getName());
         $getterDocBlock = new DocBlockGenerator(sprintf('Returns the list of %s associated to this bean via the %s pivot table.', $remoteBeanName, $this->pivotTable->getName()));
-        $getterDocBlock->setTag(new ReturnTag([ $fqcnRemoteBeanName.'[]' ]));
+        $getterDocBlock->setTag(new ReturnTag([$fqcnRemoteBeanName.'[]']));
         $getterDocBlock->setWordWrap(false);
         $getter->setDocBlock($getterDocBlock);
         $getter->setReturnType('array');
@@ -210,7 +210,7 @@ 
 
         $adder = new MethodGenerator('add'.$singularName);
         $adderDocBlock = new DocBlockGenerator(sprintf('Adds a relationship with %s associated to this bean via the %s pivot table.', $remoteBeanName, $this->pivotTable->getName()));
-        $adderDocBlock->setTag(new ParamTag($variableName, [ $fqcnRemoteBeanName ]));
+        $adderDocBlock->setTag(new ParamTag($variableName, [$fqcnRemoteBeanName]));
         $adderDocBlock->setWordWrap(false);
         $adder->setDocBlock($adderDocBlock);
         $adder->setReturnType('void');
@@ -219,7 +219,7 @@ 
 
         $remover = new MethodGenerator('remove'.$singularName);
         $removerDocBlock = new DocBlockGenerator(sprintf('Deletes the relationship with %s associated to this bean via the %s pivot table.', $remoteBeanName, $this->pivotTable->getName()));
-        $removerDocBlock->setTag(new ParamTag($variableName, [ $fqcnRemoteBeanName ]));
+        $removerDocBlock->setTag(new ParamTag($variableName, [$fqcnRemoteBeanName]));
         $removerDocBlock->setWordWrap(false);
         $remover->setDocBlock($removerDocBlock);
         $remover->setReturnType('void');
@@ -228,8 +228,8 @@ 
 
         $has = new MethodGenerator('has'.$singularName);
         $hasDocBlock = new DocBlockGenerator(sprintf('Returns whether this bean is associated with %s via the %s pivot table.', $remoteBeanName, $this->pivotTable->getName()));
-        $hasDocBlock->setTag(new ParamTag($variableName, [ $fqcnRemoteBeanName ]));
-        $hasDocBlock->setTag(new ReturnTag([ 'bool' ]));
+        $hasDocBlock->setTag(new ParamTag($variableName, [$fqcnRemoteBeanName]));
+        $hasDocBlock->setTag(new ReturnTag(['bool']));
         $hasDocBlock->setWordWrap(false);
         $has->setDocBlock($hasDocBlock);
         $has->setReturnType('bool');
@@ -239,15 +239,15 @@ 
         $setter = new MethodGenerator('set'.$pluralName);
         $setterDocBlock = new DocBlockGenerator(sprintf('Sets all relationships with %s associated to this bean via the %s pivot table.
 Exiting relationships will be removed and replaced by the provided relationships.', $remoteBeanName, $this->pivotTable->getName()));
-        $setterDocBlock->setTag(new ParamTag($pluralVariableName, [ $fqcnRemoteBeanName.'[]' ]));
-        $setterDocBlock->setTag(new ReturnTag([ 'void' ]));
+        $setterDocBlock->setTag(new ParamTag($pluralVariableName, [$fqcnRemoteBeanName.'[]']));
+        $setterDocBlock->setTag(new ReturnTag(['void']));
         $setterDocBlock->setWordWrap(false);
         $setter->setDocBlock($setterDocBlock);
         $setter->setReturnType('void');
         $setter->setParameter(new ParameterGenerator($pluralVariableName, 'array'));
         $setter->setBody(sprintf('$this->setRelationships(%s, $%s);', $pathKey, $pluralVariableName));
 
-        return [ $getter, $adder, $remover, $has, $setter ];
+        return [$getter, $adder, $remover, $has, $setter];
     }
 
     /**
@@ -276,7 +276,7 @@ 
         /** @var Annotation\JsonFormat|null $jsonFormat */
         $jsonFormat = $this->findRemoteAnnotation(Annotation\JsonFormat::class);
         if ($jsonFormat !== null) {
-            $method = $jsonFormat->method ?? 'get' . ucfirst($jsonFormat->property);
+            $method = $jsonFormat->method ?? 'get'.ucfirst($jsonFormat->property);
             $format = "$method()";
         } else {
             $stopRecursion = $this->findRemoteAnnotation(Annotation\JsonRecursive::class) ? '' : 'true';

src/Utils/Annotation/Annotations.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
      */
     public function findAnnotations(string $annotationType): array
     {
-        return array_values(array_filter($this->annotations, function ($annotation) use ($annotationType) {
+        return array_values(array_filter($this->annotations, function($annotation) use ($annotationType) {
             return is_a($annotation, $annotationType);
         }));
     }

src/Utils/Annotation/AnnotationParser.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
 
         // Let's add * in front of the line (otherwise, parsing is failing)
         $lines = explode("\n", $comment);
-        $lines = array_map(function (string $line) {
+        $lines = array_map(function(string $line) {
             return '* '.$line;
         }, $lines);
         $comment = implode("\n", $lines);

src/Utils/DirectForeignKeyMethodDescriptor.php

Spacing +9 added lines, -9 removed lines

@@ -71,9 +71,9 @@ 
     public function getName() : string
     {
         if (!$this->useAlternateName) {
-            return 'get' . $this->getPropertyName();
+            return 'get'.$this->getPropertyName();
         } else {
-            $methodName = 'get' . $this->getPropertyName() . 'By';
+            $methodName = 'get'.$this->getPropertyName().'By';
 
             $camelizedColumns = array_map([TDBMDaoGenerator::class, 'toCamelCase'], $this->foreignKey->getUnquotedLocalColumns());
 
@@ -128,12 +128,12 @@ 
         $getter = new MethodGenerator($this->getName());
 
         if ($this->hasLocalUniqueIndex()) {
-            $classType = '\\' . $this->beanNamespace . '\\' . $beanClass;
+            $classType = '\\'.$this->beanNamespace.'\\'.$beanClass;
             $getterDocBlock = new DocBlockGenerator(sprintf('Returns the %s pointing to this bean via the %s column.', $beanClass, implode(', ', $this->foreignKey->getUnquotedLocalColumns())));
-            $getterDocBlock->setTag([new ReturnTag([$classType . '|null'])]);
+            $getterDocBlock->setTag([new ReturnTag([$classType.'|null'])]);
             $getterDocBlock->setWordWrap(false);
             $getter->setDocBlock($getterDocBlock);
-            $getter->setReturnType('?' . $classType);
+            $getter->setReturnType('?'.$classType);
 
             $code = sprintf(
                 'return $this->retrieveManyToOneRelationshipsStorage(%s, %s, %s)->first();',
@@ -143,7 +143,7 @@ 
             );
         } else {
             $getterDocBlock = new DocBlockGenerator(sprintf('Returns the list of %s pointing to this bean via the %s column.', $beanClass, implode(', ', $this->foreignKey->getUnquotedLocalColumns())));
-            $getterDocBlock->setTag(new ReturnTag([$beanClass . '[]', '\\' . AlterableResultIterator::class]));
+            $getterDocBlock->setTag(new ReturnTag([$beanClass.'[]', '\\'.AlterableResultIterator::class]));
             $getterDocBlock->setWordWrap(false);
             $getter->setDocBlock($getterDocBlock);
             $getter->setReturnType(AlterableResultIterator::class);
@@ -162,7 +162,7 @@ 
             $getter->setVisibility(AbstractMemberGenerator::VISIBILITY_PROTECTED);
         }
 
-        return [ $getter ];
+        return [$getter];
     }
 
     private function getFilters(ForeignKeyConstraint $fk) : string
@@ -233,7 +233,7 @@ 
         /** @var Annotation\JsonFormat|null $jsonFormat */
         $jsonFormat = $this->findAnnotation(Annotation\JsonFormat::class);
         if ($jsonFormat !== null) {
-            $method = $jsonFormat->method ?? 'get' . ucfirst($jsonFormat->property);
+            $method = $jsonFormat->method ?? 'get'.ucfirst($jsonFormat->property);
             $format = "$method()";
         } else {
             $stopRecursion = $this->findAnnotation(Annotation\JsonRecursive::class) ? '' : 'true';
@@ -242,7 +242,7 @@ 
         $isIncluded = $this->findAnnotation(Annotation\JsonInclude::class) !== null;
         $index = $jsonCollection->key ?: lcfirst($this->getPropertyName());
         $class = $this->getBeanClassName();
-        $variableName = '$' . TDBMDaoGenerator::toVariableName($class);
+        $variableName = '$'.TDBMDaoGenerator::toVariableName($class);
         $getter = $this->getName();
         if ($this->hasLocalUniqueIndex()) {
             $code = "\$array['$index'] = (\$object = \$this->$getter()) ? \$object->$format : null;";

src/Utils/ScalarBeanPropertyDescriptor.php

Spacing +7 added lines, -7 removed lines

@@ -162,7 +162,7 @@ 
                 'time',
                 'time_immutable',
             ], true)) {
-                if ($default !== null && in_array(strtoupper($default), ['CURRENT_TIMESTAMP' /* MySQL */, 'NOW()' /* PostgreSQL */, 'SYSDATE' /* Oracle */ , 'CURRENT_TIMESTAMP()' /* MariaDB 10.3 */], true)) {
+                if ($default !== null && in_array(strtoupper($default), ['CURRENT_TIMESTAMP' /* MySQL */, 'NOW()' /* PostgreSQL */, 'SYSDATE' /* Oracle */, 'CURRENT_TIMESTAMP()' /* MariaDB 10.3 */], true)) {
                     $defaultCode = 'new \DateTimeImmutable()';
                 } else {
                     throw new TDBMException('Unable to set default value for date in "'.$this->table->getName().'.'.$this->column->getName().'". Database passed this default value: "'.$default.'"');
@@ -186,7 +186,7 @@ 
             case 'v4':
                 return 'Uuid::uuid4()->toString()';
             default:
-                throw new TDBMException('@UUID annotation accepts either "v1" or "v4" parameter. Unexpected parameter: ' . $comment);
+                throw new TDBMException('@UUID annotation accepts either "v1" or "v4" parameter. Unexpected parameter: '.$comment);
         }
     }
 
@@ -234,14 +234,14 @@ 
             $resourceTypeCheck = sprintf($resourceTypeCheck, $checkNullable, $variableName, $variableName);
         }
 
-        $types = [ $normalizedType ];
+        $types = [$normalizedType];
         if ($isNullable) {
             $types[] = 'null';
         }
 
         $paramType = null;
         if ($this->isTypeHintable()) {
-            $paramType = ($isNullable?'?':'').$normalizedType;
+            $paramType = ($isNullable ? '?' : '').$normalizedType;
         }
 
         $getter = new MethodGenerator($columnGetterName);
@@ -337,11 +337,11 @@ 
                             break;
                         }
                     }
-                    $args = array_map(function ($v) {
+                    $args = array_map(function($v) {
                         return var_export($v, true);
                     }, $args);
-                    $args = empty($args) ? '' : ', ' . implode(', ', $args);
-                    $unit = $jsonFormat->unit ? ' . ' . var_export($jsonFormat->unit, true) : '';
+                    $args = empty($args) ? '' : ', '.implode(', ', $args);
+                    $unit = $jsonFormat->unit ? ' . '.var_export($jsonFormat->unit, true) : '';
                     if ($this->column->getNotnull()) {
                         return "\$array['$index'] = number_format(\$this->$getter()$args)$unit;";
                     } else {