[5.x] Pass original upload filename into AssetUploaded event

[daun]

Suggested change

Pass the original filename into the AssetUploaded and AssetReuploaded events.

Use case

This will allow saving the original filename into the asset metadata from a custom listener. Mostly applicable to download buttons or zip features that benefit from keeping the original name including capitalization and any © marks.

{{ assets }}
  <a href="{{ url }}" download="{{ original_filename }}">Download</a>
{{ /assets }}

<a href="image--photographer.jpg" download="Image © Photographer.jpg">Download</a>

class OriginalFilenameSubscriber extends Subscriber
{
    protected $listeners = [
        AssetUploaded::class => 'saveOriginalFilename',
    ];

    public function saveOriginalFilename(AssetUploaded $event)
    {
        $event->asset->set('original_filename', $event->originalFilename);
        $event->asset->saveQuietly();
    }
}

Semver

• Technically, this is only adding an argument, so it should be non-breaking
• However, any addons/consumers "faking" those Events will get errors from the now-missing required argument

Alternatives considered

• Just save the original filename into the asset meta on creation? Wouldn't require updates to the event signatures
• Make filename sanitization hookable and somehow match the change to the asset? Sounds a bit hacky

Closes

Closes statamic/ideas#1282

[jasonvarga]

Can't you already do $asset->getOriginal('path')? Sorry I misunderstood what this PR was actually doing. 🤐

[daun]

The asset is created with the filename already sanitized, so unfortunately no :)