thecodingmachine / tdbm

src/Utils/AbstractNamingStrategy.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
         if ($index->isUnique()) {
             return 'findOneBy'.implode('And', $methodNameComponent);
         } else {
-            return 'findBy' . implode('And', $methodNameComponent);
+            return 'findBy'.implode('And', $methodNameComponent);
         }
     }
 }

src/Utils/StringUtils.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 {
     public static function getValidVariableName(string $variableName): string
     {
-        return preg_replace_callback('/^(\d+)/', static function (array $match) {
+        return 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);

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/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);

tests/TDBMServiceTest.php

Spacing +2 added lines, -2 removed lines

@@ -447,7 +447,7 @@ 
     {
         $beans = $this->tdbmService->findObjects('person', null, [], 'person.id ASC', [], null, TDBMObject::class);
 
-        $results = $beans->map(function ($item) {
+        $results = $beans->map(function($item) {
             return $item->getProperty('id', 'person');
         })->toArray();
 
@@ -456,7 +456,7 @@ 
         // Same test with page
         $page = $beans->take(0, 2);
 
-        $results = $page->map(function ($item) {
+        $results = $page->map(function($item) {
             return $item->getProperty('id', 'person');
         })->toArray();
 

tests/MapIteratorTest.php

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
     public function testConstructorException1(): void
     {
         $this->expectException('TheCodingMachine\TDBM\TDBMException');
-        $mapIterator = new MapIterator(new \DateTime(), function ($item) {
+        $mapIterator = new MapIterator(new \DateTime(), function($item) {
             return $item;
         });
     }
@@ -36,7 +36,7 @@ 
     public function testConstructorException2(): void
     {
         $this->expectException('TheCodingMachine\TDBM\TDBMException');
-        $mapIterator = new MapIterator(array(1, 2, 3), function () {
+        $mapIterator = new MapIterator(array(1, 2, 3), function() {
             return $item;
         });
     }
@@ -44,7 +44,7 @@ 
     public function testJsonSerialize(): void
     {
         $value = array(1, 2, 3);
-        $mapIterator = new MapIterator($value, function ($item) {
+        $mapIterator = new MapIterator($value, function($item) {
             return $item;
         });
 

src/TDBMService.php

Spacing +9 added lines, -9 removed lines

@@ -416,7 +416,7 @@ 
                 }
             }
 
-            return ['(' . implode(') AND (', $sqlParts) . ')', $parameters, $counter];
+            return ['('.implode(') AND (', $sqlParts).')', $parameters, $counter];
         } elseif ($filter_bag instanceof AbstractTDBMObject) {
             $sqlParts = [];
             $parameters = [];
@@ -990,7 +990,7 @@ 
 
         return $this->fromCache(
             $this->cachePrefix.'_linkbetweeninheritedtables_'.implode('__split__', $tables),
-            function () use ($tables) {
+            function() use ($tables) {
                 return $this->_getLinkBetweenInheritedTablesWithoutCache($tables);
             }
         );
@@ -1038,7 +1038,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);
         });
     }
@@ -1233,7 +1233,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);
@@ -1276,18 +1276,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;
         }
@@ -1456,7 +1456,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/PageIterator.php

Spacing +1 added lines, -1 removed lines

@@ -262,7 +262,7 @@
      */
     public function jsonSerialize()
     {
-        return array_map(function (AbstractTDBMObject $item) {
+        return array_map(function(AbstractTDBMObject $item) {
             return $item->jsonSerialize();
         }, $this->toArray());
     }