[V3] Removing upload from FileUpload does not trigger `afterStateUpdated`

Hiya!

Testing our project on V3 and found this bug(?) or at least unexpected behaviour.

I have this FileUpload field in a form with multiple files allowed. Whenever you upload a file I need to execute some code, which I do using

afterStateUpdated

afterStateUpdated

. This works perfectly fine for adding files, but it's not called when you remove a file.

FileUpload::make('uploads')
    ->multiple()
    ->storeFiles(false)
    ->afterStateUpdated(function (array $state, Get $get, Set $set) {
        collect($state)->map(fn (TemporaryUploadedFile $file, $key) => do_something());
    }),

FileUpload::make('uploads')
    ->multiple()
    ->storeFiles(false)
    ->afterStateUpdated(function (array $state, Get $get, Set $set) {
        collect($state)->map(fn (TemporaryUploadedFile $file, $key) => do_something());
    }),

Dan Harrin•6/15/23, 1:46 PM

i dont think this has changed in v3

Dan Harrin•6/15/23, 1:46 PM

its probably also in v2

Dan Harrin•6/15/23, 1:46 PM

you can submit a PR to fix it if you want, preferably to v2 since we dont provide v3 support

Dan Harrin•6/15/23, 1:47 PM

in this case, what are you trying to do?

Dennis Koch•6/15/23, 1:52 PM

I guess: Uploading is handled on the server side. Removing only on client side until the form is saved.

ThandruilOP•6/15/23, 1:52 PM

No it's actually sending a message to server.

ThandruilOP•6/15/23, 1:53 PM

So upload uses Livewire's native which calls (and thus

afterStateUpdated

afterStateUpdated

)

ThandruilOP•6/15/23, 1:53 PM

Remove calls Filament's own which only sets the state using

Dan Harrin•6/15/23, 1:53 PM

there is a way to trigger the afterstateupdated if we want to

Dan Harrin•6/15/23, 1:53 PM

which we can do in removeFormUploadedFile or the equivalent method in the fileupload

ThandruilOP•6/15/23, 1:54 PM

Yeah that's the fix I was thinking about as well

Dan Harrin•6/15/23, 1:54 PM

or just call syncinput

Dan Harrin•6/15/23, 1:54 PM

but i'd rather not do that as i know for sure that will break lw3 lol

ThandruilOP•6/15/23, 1:54 PM

yeah fair

ThandruilOP•6/15/23, 1:54 PM

is there a way in user land to trigger the afterStateUpdated somehow?

ThandruilOP•6/15/23, 1:55 PM

I tried listening to but to no avail

Dan Harrin•6/15/23, 1:55 PM

im guessing callAfterStateUpdated() but i might be wrong

ivankuraev•1/27/25, 9:00 PM

@erikgaal Hi. Have you found a solution? I have the same problem.