bug #50656 Only update autoload_runtime.php when it changed (Seldaek)

fabpot · fabpot · commit 2d2cbad1b526 · 2023-06-20T13:49:58.000+02:00
This PR was squashed before being merged into the 5.4 branch. Discussion ---------- Only update autoload_runtime.php when it changed | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Fix #...  | License | MIT | Doc PR | symfony/symfony-docs#...  The vendor dir should not be changed if not necessary as this typically gets executed during deployment processes where permissions may not be sufficient, and it can mess up docker caching layers if filemtime changes unnecessarily. Commits ------- ca53525 Only update autoload_runtime.php when it changed
diff --git a/src/Symfony/Component/Runtime/Internal/ComposerPlugin.php b/src/Symfony/Component/Runtime/Internal/ComposerPlugin.php
@@ -105,7 +105,12 @@ public function updateAutoloadFile(): void
             '%runtime_options%' => '['.substr(var_export($extra, true), 7, -1)."  'project_dir' => {$projectDir},\n]",
         ]);
 
-        file_put_contents(substr_replace($autoloadFile, '_runtime', -4, 0), $code);
+        // could use Composer\Util\Filesystem::filePutContentsIfModified once Composer 1.x support is dropped for this plugin
+        $path = substr_replace($autoloadFile, '_runtime', -4, 0);
+        $currentContent = @file_exists($path) ? @file_get_contents($path) : false;
+        if (false === $currentContent || $currentContent !== $code) {
+            file_put_contents($path, $code);
+        }
     }
 
     public static function getSubscribedEvents(): array
