cs

Author: dunglas

src/Configurator/DockerComposeConfigurator.php

@@ -156,7 +156,7 @@ private function normalizeConfig(array $config): array
             }
 
             // If the recipe still use the legacy "docker-compose.yml" names, remove the "docker-" prefix and change the extension
-            $newKey = pathinfo(substr($key, 7), PATHINFO_FILENAME).'.yaml';
+            $newKey = pathinfo(substr($key, 7), \PATHINFO_FILENAME).'.yaml';
             $config[$newKey] = $val;
             unset($config[$key]);
         }
@@ -170,12 +170,12 @@ private function normalizeConfig(array $config): array
     private function findDockerComposeFile(string $rootDir, string $file): ?string
     {
         if (isset($_SERVER['COMPOSE_FILE'])) {
-            $filenameToFind = pathinfo($file, PATHINFO_FILENAME);
+            $filenameToFind = pathinfo($file, \PATHINFO_FILENAME);
             $separator = $_SERVER['COMPOSE_PATH_SEPARATOR'] ?? ('\\' === \DIRECTORY_SEPARATOR ? ';' : ':');
 
             $files = explode($separator, $_SERVER['COMPOSE_FILE']);
             foreach ($files as $f) {
-                $filename = pathinfo($f, PATHINFO_FILENAME);
+                $filename = pathinfo($f, \PATHINFO_FILENAME);
                 if ($filename !== $filenameToFind && "docker-$filenameToFind" !== $filename) {
                     continue;
                 }

tests/Configurator/DockerComposeConfiguratorTest.php

@@ -220,7 +220,7 @@ public function testConfigure(string $fileName)
 ###< doctrine/doctrine-bundle ###
 
 YAML
-            );
+        );
 
         $this->configurator->unconfigure($this->recipeDb, self::CONFIG_DB, $this->lock);
         $this->assertEquals(self::ORIGINAL_CONTENT, file_get_contents($dockerComposeFile));
@@ -331,7 +331,7 @@ public function testConfigureFileWithExistingVolumes()
 ###< doctrine/doctrine-bundle ###
 
 YAML
-            );
+        );
 
         $this->configurator->unconfigure($this->recipeDb, self::CONFIG_DB, $this->lock);
         // Not the same original, we have an extra breaks line
@@ -426,7 +426,7 @@ public function testConfigureFileWithExistingMarks()
 ###< doctrine/doctrine-bundle ###
 
 YAML
-            );
+        );
 
         $this->configurator->unconfigure($recipe, $config, $this->lock);
         $this->assertEquals($originalContent, file_get_contents($dockerComposeFile));
@@ -541,7 +541,7 @@ public function testConfigureMultipleFiles()
 ###< doctrine/doctrine-bundle ###
 
 YAML
-                );
+            );
         }
 
         $this->configurator->unconfigure($this->recipeDb, self::CONFIG_DB_MULTIPLE_FILES, $this->lock);
@@ -586,7 +586,7 @@ public function testConfigureEnvVar()
 ###< doctrine/doctrine-bundle ###
 
 YAML
-                );
+            );
         }
 
         $this->configurator->unconfigure($this->recipeDb, self::CONFIG_DB_MULTIPLE_FILES, $this->lock);
@@ -668,7 +668,7 @@ public function testConfigureWithoutExistingDockerComposeFiles()
 ###< doctrine/doctrine-bundle ###
 
 YAML
-            );
+        );
 
         $this->configurator->unconfigure($this->recipeDb, self::CONFIG_DB, $this->lock);
         $this->assertEquals(trim($defaultContent), file_get_contents($dockerComposeFile));