Remove unused code and unnecessary else branches

Author: alexandre-daubois

Dumper/PhpDumper.php

@@ -1940,11 +1940,11 @@ private function dumpValue(mixed $value, bool $interpolate = true): string
                 // we do this to deal with non string values (Boolean, integer, ...)
                 // the preg_replace_callback converts them to strings
                 return $this->dumpParameter($match[1]);
-            } else {
-                $replaceParameters = fn ($match) => "'.".$this->dumpParameter($match[2]).".'";
-
-                return str_replace('%%', '%', preg_replace_callback('/(?<!%)(%)([^%]+)\1/', $replaceParameters, $this->export($value)));
             }
+
+            $replaceParameters = fn($match) => "'." . $this->dumpParameter($match[2]) . ".'";
+
+            return str_replace('%%', '%', preg_replace_callback('/(?<!%)(%)([^%]+)\1/', $replaceParameters, $this->export($value)));
         } elseif ($value instanceof \UnitEnum) {
             return \sprintf('\%s::%s', $value::class, $value->name);
         } elseif ($value instanceof AbstractArgument) {

EnvVarProcessor.php

@@ -151,9 +151,9 @@ public function getEnv(string $prefix, string $name, \Closure $getEnv): mixed
 
             if ('file' === $prefix) {
                 return file_get_contents($file);
-            } else {
-                return require $file;
             }
+
+            return require $file;
         }
 
         $returnNull = false;