Merge pull request #11 from php-etl/fix/composer-require

Fixed the composer require command : escape shell arg

Author: gplanchat

src/Dockerfile/Copy.php

@@ -6,7 +6,9 @@
 
 final readonly class Copy implements LayerInterface, \Stringable
 {
-    public function __construct(private string $source, private string $destination) {}
+    public function __construct(private string $source, private string $destination)
+    {
+    }
 
     /** @return \Iterator|self[] */
     public static function directory(string $sourcePath, string $destinationPath): \Iterator

src/Dockerfile/From.php

@@ -6,7 +6,9 @@
 
 final readonly class From implements LayerInterface, \Stringable
 {
-    public function __construct(private string $source) {}
+    public function __construct(private string $source)
+    {
+    }
 
     public function __toString(): string
     {

src/Dockerfile/Label.php

@@ -6,7 +6,9 @@
 
 final readonly class Label implements LayerInterface, \Stringable
 {
-    public function __construct(private string $key, private string $value) {}
+    public function __construct(private string $key, private string $value)
+    {
+    }
 
     public function __toString(): string
     {

src/Dockerfile/Maintainer.php

@@ -6,7 +6,9 @@
 
 final readonly class Maintainer implements LayerInterface, \Stringable
 {
-    public function __construct(private string $name, private string $email) {}
+    public function __construct(private string $name, private string $email)
+    {
+    }
 
     public function __toString(): string
     {

src/Dockerfile/Run.php

@@ -6,7 +6,9 @@
 
 final readonly class Run implements LayerInterface, \Stringable
 {
-    public function __construct(private string $command) {}
+    public function __construct(private string $command)
+    {
+    }
 
     public function __toString(): string
     {

src/Dockerfile/Workdir.php

@@ -6,7 +6,9 @@
 
 final readonly class Workdir implements LayerInterface, \Stringable
 {
-    public function __construct(private string $path) {}
+    public function __construct(private string $path)
+    {
+    }
 
     public function __toString(): string
     {

src/PHP/ComposerAddComposerRepository.php

@@ -11,7 +11,8 @@
     public function __construct(
         private string $name,
         private string $url
-    ) {}
+    ) {
+    }
 
     public function __toString(): string
     {

src/PHP/ComposerAddGithubRepository.php

@@ -11,7 +11,8 @@
     public function __construct(
         private string $name,
         private string $url
-    ) {}
+    ) {
+    }
 
     public function __toString(): string
     {

src/PHP/ComposerAddVcsRepository.php

@@ -11,7 +11,8 @@
     public function __construct(
         private string $name,
         private string $url
-    ) {}
+    ) {
+    }
 
     public function __toString(): string
     {

src/PHP/ComposerAuthenticationToken.php

@@ -11,7 +11,8 @@
     public function __construct(
         private string $url,
         private string $token,
-    ) {}
+    ) {
+    }
 
     public function __toString(): string
     {

src/PHP/ComposerAutoload.php

@@ -13,7 +13,8 @@ public function __construct(
          * @param array<string, array<string, string|list<string>>> $autoloads
          */
         private array $autoloads
-    ) {}
+    ) {
+    }
 
     private static function command(string ...$command): string
     {

src/PHP/ComposerInit.php

@@ -8,7 +8,9 @@
 
 final readonly class ComposerInit implements Dockerfile\LayerInterface, \Stringable
 {
-    public function __construct(private string $name) {}
+    public function __construct(private string $name)
+    {
+    }
 
     public function __toString(): string
     {

src/PHP/ComposerMinimumStability.php

@@ -8,7 +8,9 @@
 
 final readonly class ComposerMinimumStability implements Dockerfile\LayerInterface, \Stringable
 {
-    public function __construct(private string $minimumStability) {}
+    public function __construct(private string $minimumStability)
+    {
+    }
 
     public function __toString(): string
     {

src/PHP/ComposerRequire.php

@@ -20,6 +20,6 @@ public function __toString(): string
         return (string) new Dockerfile\Run(sprintf(<<<'RUN'
             set -ex \
                 && composer require --prefer-dist --no-progress --prefer-stable --sort-packages --optimize-autoloader --with-dependencies %s
-            RUN, implode(' ', $this->packages)));
+            RUN, implode(' ', array_map(fn (string $package) => escapeshellarg($package), $this->packages))));
     }
 }