thecodingmachine / tdbm

src/QueryFactory/FindObjectsFromSqlQueryFactory.php

Spacing +4 added lines, -4 removed lines

@@ -49,7 +49,7 @@ 
 
         // Let's compute the COUNT.
         $pkColumnNames = $this->tdbmService->getPrimaryKeyColumns($this->mainTable);
-        $pkColumnNames = array_map(function ($pkColumn) use ($mySqlPlatform) {
+        $pkColumnNames = array_map(function($pkColumn) use ($mySqlPlatform) {
             return $mySqlPlatform->quoteIdentifier($this->mainTable).'.'.$mySqlPlatform->quoteIdentifier($pkColumn);
         }, $pkColumnNames);
 
@@ -113,7 +113,7 @@ 
      */
     private function getParentRelationshipForeignKeys(string $tableName): array
     {
-        return $this->fromCache($this->cachePrefix.'_parentrelationshipfks_'.$tableName, function () use ($tableName) {
+        return $this->fromCache($this->cachePrefix.'_parentrelationshipfks_'.$tableName, function() use ($tableName) {
             return $this->getParentRelationshipForeignKeysWithoutCache($tableName);
         });
     }
@@ -142,7 +142,7 @@ 
      */
     private function getChildrenRelationshipForeignKeys(string $tableName) : array
     {
-        return $this->fromCache($this->cachePrefix.'_childrenrelationshipfks_'.$tableName, function () use ($tableName) {
+        return $this->fromCache($this->cachePrefix.'_childrenrelationshipfks_'.$tableName, function() use ($tableName) {
             return $this->getChildrenRelationshipForeignKeysWithoutCache($tableName);
         });
     }
@@ -157,7 +157,7 @@ 
         $children = $this->schemaAnalyzer->getChildrenRelationships($tableName);
 
         if (!empty($children)) {
-            $fksTables = array_map(function (ForeignKeyConstraint $fk) {
+            $fksTables = array_map(function(ForeignKeyConstraint $fk) {
                 return $this->getChildrenRelationshipForeignKeys($fk->getLocalTableName());
             }, $children);
 

src/QueryFactory/AbstractQueryFactory.php

Spacing +1 added lines, -1 removed lines

@@ -207,7 +207,7 @@
         if (strlen($alias) <= 30) { // Older oracle version had a limit of 30 characters for identifiers
             return $alias;
         }
-        return substr($columnName, 0, 20) . crc32($tableName.'____'.$columnName);
+        return substr($columnName, 0, 20).crc32($tableName.'____'.$columnName);
     }
 
     abstract protected function compute(): void;

src/ResultIterator.php

Spacing +2 added lines, -2 removed lines

@@ -84,7 +84,7 @@ 
      */
     public static function createResultIterator(QueryFactory $queryFactory, array $parameters, ObjectStorageInterface $objectStorage, ?string $className, TDBMService $tdbmService, MagicQuery $magicQuery, int $mode, LoggerInterface $logger): self
     {
-        $iterator =  new static();
+        $iterator = new static();
         if ($mode !== TDBMService::MODE_CURSOR && $mode !== TDBMService::MODE_ARRAY) {
             throw new TDBMException("Unknown fetch mode: '".$mode."'");
         }
@@ -281,7 +281,7 @@ 
      */
     public function jsonSerialize($stopRecursion = false)
     {
-        return array_map(function (AbstractTDBMObject $item) use ($stopRecursion) {
+        return array_map(function(AbstractTDBMObject $item) use ($stopRecursion) {
             return $item->jsonSerialize($stopRecursion);
         }, $this->toArray());
     }

src/TDBMSchemaAnalyzer.php

Spacing +1 added lines, -1 removed lines

@@ -142,7 +142,7 @@
     public function getIncomingForeignKeys(string $tableName): array
     {
         $junctionTables = $this->schemaAnalyzer->detectJunctionTables(true);
-        $junctionTableNames = array_map(function (Table $table) {
+        $junctionTableNames = array_map(function(Table $table) {
             return $table->getName();
         }, $junctionTables);
         $childrenRelationships = $this->schemaAnalyzer->getChildrenRelationships($tableName);

src/TDBMService.php

Spacing +12 added lines, -12 removed lines

@@ -422,7 +422,7 @@ 
                 }
             }
 
-            return ['(' . implode(') AND (', $sqlParts) . ')', $parameters, $counter];
+            return ['('.implode(') AND (', $sqlParts).')', $parameters, $counter];
         } elseif ($filter_bag instanceof ResultIterator) {
             $subQuery = $filter_bag->_getSubQuery();
             return [$subQuery, [], $counter];
@@ -1000,7 +1000,7 @@ 
 
         return $this->fromCache(
             $this->cachePrefix.'_linkbetweeninheritedtables_'.implode('__split__', $tables),
-            function () use ($tables) {
+            function() use ($tables) {
                 return $this->_getLinkBetweenInheritedTablesWithoutCache($tables);
             }
         );
@@ -1048,7 +1048,7 @@ 
      */
     public function _getRelatedTablesByInheritance(string $table): array
     {
-        return $this->fromCache($this->cachePrefix.'_relatedtables_'.$table, function () use ($table) {
+        return $this->fromCache($this->cachePrefix.'_relatedtables_'.$table, function() use ($table) {
             return $this->_getRelatedTablesByInheritanceWithoutCache($table);
         });
     }
@@ -1137,7 +1137,7 @@ 
     public function findObjects(string $mainTable, $filter = null, array $parameters = array(), $orderString = null, array $additionalTablesFetch = array(), ?int $mode = null, string $className = null, string $resultIteratorClass = ResultIterator::class): ResultIterator
     {
         if (!is_a($resultIteratorClass, ResultIterator::class, true)) {
-            throw new TDBMInvalidArgumentException('$resultIteratorClass should be a `'. ResultIterator::class. '`. `' . $resultIteratorClass . '` provided.');
+            throw new TDBMInvalidArgumentException('$resultIteratorClass should be a `'.ResultIterator::class.'`. `'.$resultIteratorClass.'` provided.');
         }
         // $mainTable is not secured in MagicJoin, let's add a bit of security to avoid SQL injection.
         if (!preg_match('/^[a-zA-Z_][a-zA-Z0-9_]*$/', $mainTable)) {
@@ -1174,7 +1174,7 @@ 
     public function findObjectsFromSql(string $mainTable, string $from, $filter = null, array $parameters = array(), $orderString = null, ?int $mode = null, string $className = null, string $resultIteratorClass = ResultIterator::class): ResultIterator
     {
         if (!is_a($resultIteratorClass, ResultIterator::class, true)) {
-            throw new TDBMInvalidArgumentException('$resultIteratorClass should be a `'. ResultIterator::class. '`. `' . $resultIteratorClass . '` provided.');
+            throw new TDBMInvalidArgumentException('$resultIteratorClass should be a `'.ResultIterator::class.'`. `'.$resultIteratorClass.'` provided.');
         }
         // $mainTable is not secured in MagicJoin, let's add a bit of security to avoid SQL injection.
         if (!preg_match('/^[a-zA-Z_][a-zA-Z0-9_]*$/', $mainTable)) {
@@ -1251,7 +1251,7 @@ 
         try {
             return $this->findObjectOrFail($table, $primaryKeys, [], $additionalTablesFetch, $className);
         } catch (NoBeanFoundException $exception) {
-            $primaryKeysStringified = implode(' and ', array_map(function ($key, $value) {
+            $primaryKeysStringified = implode(' and ', array_map(function($key, $value) {
                 return "'".$key."' = ".$value;
             }, array_keys($primaryKeys), $primaryKeys));
             throw new NoBeanFoundException("No result found for query on table '".$table."' for ".$primaryKeysStringified, 0, $exception);
@@ -1294,18 +1294,18 @@ 
         if ($count > 1) {
             $additionalErrorInfos = '';
             if (is_string($filter) && !empty($parameters)) {
-                $additionalErrorInfos = ' for filter "' . $filter.'"';
+                $additionalErrorInfos = ' for filter "'.$filter.'"';
                 foreach ($parameters as $fieldName => $parameter) {
                     if (is_array($parameter)) {
-                        $value = '(' . implode(',', $parameter) . ')';
+                        $value = '('.implode(',', $parameter).')';
                     } else {
                         $value = $parameter;
                     }
-                    $additionalErrorInfos = str_replace(':' . $fieldName, var_export($value, true), $additionalErrorInfos);
+                    $additionalErrorInfos = str_replace(':'.$fieldName, var_export($value, true), $additionalErrorInfos);
                 }
             }
             $additionalErrorInfos .= '.';
-            throw new DuplicateRowException("Error while querying an object in table '$mainTable': More than 1 row have been returned, but we should have received at most one" . $additionalErrorInfos);
+            throw new DuplicateRowException("Error while querying an object in table '$mainTable': More than 1 row have been returned, but we should have received at most one".$additionalErrorInfos);
         } elseif ($count === 0) {
             return null;
         }
@@ -1348,7 +1348,7 @@ 
     public function findObjectsFromRawSql(string $mainTable, string $sql, array $parameters = array(), ?int $mode = null, string $className = null, string $sqlCount = null, string $resultIteratorClass = ResultIterator::class): ResultIterator
     {
         if (!is_a($resultIteratorClass, ResultIterator::class, true)) {
-            throw new TDBMInvalidArgumentException('$resultIteratorClass should be a `'. ResultIterator::class. '`. `' . $resultIteratorClass . '` provided.');
+            throw new TDBMInvalidArgumentException('$resultIteratorClass should be a `'.ResultIterator::class.'`. `'.$resultIteratorClass.'` provided.');
         }
         // $mainTable is not secured in MagicJoin, let's add a bit of security to avoid SQL injection.
         if (!preg_match('/^[a-zA-Z_][a-zA-Z0-9_]*$/', $mainTable)) {
@@ -1478,7 +1478,7 @@ 
         $table1 = $fks[0]->getForeignTableName();
         $table2 = $fks[1]->getForeignTableName();
 
-        $beanTables = array_map(function (DbRow $dbRow) {
+        $beanTables = array_map(function(DbRow $dbRow) {
             return $dbRow->_getDbTableName();
         }, $bean->_getDbRows());
 

src/Commands/GenerateCommand.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 
         $fromLock = (bool) $input->getOption('from-lock');
 
-        $loggers = [ new ConsoleLogger($output) ];
+        $loggers = [new ConsoleLogger($output)];
 
         $logger = $alteredConf->getLogger();
         if ($logger) {

src/PageIterator.php

Spacing +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
      */
     public static function createResultIterator(ResultIterator $parentResult, string $magicSql, array $parameters, int $limit, int $offset, array $columnDescriptors, ObjectStorageInterface $objectStorage, ?string $className, TDBMService $tdbmService, MagicQuery $magicQuery, int $mode, LoggerInterface $logger): self
     {
-        $iterator =  new self();
+        $iterator = new self();
         $iterator->parentResult = $parentResult;
         $iterator->magicSql = $magicSql;
         $iterator->objectStorage = $objectStorage;
@@ -277,7 +277,7 @@ 
      */
     public function jsonSerialize()
     {
-        return array_map(function (AbstractTDBMObject $item) {
+        return array_map(function(AbstractTDBMObject $item) {
             return $item->jsonSerialize();
         }, $this->toArray());
     }

src/InnerResultIterator.php

Spacing +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
      */
     public static function createInnerResultIterator(string $magicSql, array $parameters, ?int $limit, ?int $offset, array $columnDescriptors, ObjectStorageInterface $objectStorage, ?string $className, TDBMService $tdbmService, MagicQuery $magicQuery, LoggerInterface $logger): self
     {
-        $iterator =  new static();
+        $iterator = new static();
         $iterator->magicSql = $magicSql;
         $iterator->objectStorage = $objectStorage;
         $iterator->className = $className;

src/Utils/StringUtils.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 {
     public static function getValidVariableName(string $variableName): string
     {
-        $valid = preg_replace_callback('/^(\d+)/', static function (array $match) {
+        $valid = preg_replace_callback('/^(\d+)/', static function(array $match) {
             $f = new \NumberFormatter('en', \NumberFormatter::SPELLOUT);
             $number = $f->format((int) $match[0]);
             return preg_replace('/[^a-z]+/i', '_', $number);