Merge pull request #4213 from benface/replaced-assets-filename

Allow plugins and modules to change the filename of replaced assets

CHANGELOG-v3.md

@@ -4,6 +4,7 @@
 
 ### Changed
 - Craft now stores the Default User Group’s UID in the project config, in case the group’s ID is different across environments.
+- `craft\services\Assets::EVENT_BEFORE_REPLACE_ASSET` event handlers can now change the filename of the replaced asset before it is saved.
 - Improved the performance of background jobs. ([#4219](https://github.com/craftcms/cms/pull/4219))
 - Improved the Plugin Store’s screenshots with arrows for navigation and pinch-to-zoom capability for touch devices.
 

src/services/Assets.php

@@ -152,11 +152,13 @@ public function replaceAssetFile(Asset $asset, string $pathOnServer, string $fil
     {
         // Fire a 'beforeReplaceFile' event
         if ($this->hasEventHandlers(self::EVENT_BEFORE_REPLACE_ASSET)) {
-            $this->trigger(self::EVENT_BEFORE_REPLACE_ASSET, new ReplaceAssetEvent([
+            $event = new ReplaceAssetEvent([
                 'asset' => $asset,
                 'replaceWith' => $pathOnServer,
                 'filename' => $filename
-            ]));
+            ]);
+            $this->trigger(self::EVENT_BEFORE_REPLACE_ASSET, $event);
+            $filename = $event->filename;
         }
 
         $asset->tempFilePath = $pathOnServer;