File.create:after hook generates error at file upload (video thumbnail creation attempt)

The CMS Questions
1

Hi everyone,

Sometimes I don’t really understand how PHP handles its queue jobs…

I have a hook that creates a video thumbnail when I upload a video file :

'file.create:after' => function (Kirby\Cms\File $file) {
          if (option('cloudinary', false) === true && $file->type() == 'video') {
            $url = str_replace('q_auto:best,f_mp4', 'c_scale,f_jpg,q_100,w_400,so_0', $file->url());
            $imagedata = file_get_contents($url);
            F::write($file->parent()->root() . '/' . $file->filename() . '.jpg', $imagedata);
          }
},

It works but I ran into an error inside the panel and the file is not selected in the file field where I uploaded the file.

array_key_exists(): Argument #2 ($array) must be of type array, Kirby\Cms\Field given

Capture d’écran 2022-01-11 à 11.50.07
Capture d’écran 2022-01-11 à 11.50.073634×1842 239 KB

I have the same problem if I try to upload it in a files section.

Capture d’écran 2022-01-11 à 11.55.33
Capture d’écran 2022-01-11 à 11.55.333596×1976 223 KB

Any clues ?
Thanks

2

Is there more information in the browser console or in your php error logs?

3

Hi @texnixe !

I got this :

{"status":"error","message":"array_key_exists(): Argument #2 ($array) must be of type array, Kirby\\Cms\\Field given","code":500,"key":null,"details":[]}

Capture d’écran 2022-01-11 à 13.38.30
Capture d’écran 2022-01-11 à 13.38.302382×1064 214 KB

No Apache error on the server… :frowning:

4

Is it because my video file name is :
video.mp4

And my thumbnail file name is :
video.mp4.jpg

???

Maybe Kirby thinks the file already exists at upload ?

5

Well, the Apache error log is not interesting, but the PHP error log to get a stack trace of the error.

6

This doesn’t work either (to prevent same filename issue) :

if (option('cloudinary', false) === true && $file->type() == 'video') {
            $url = str_replace('q_auto:best,f_mp4', 'c_scale,f_jpg,q_100,w_400,so_0', $file->url());
            $imagedata = file_get_contents($url);
            F::write($file->parent()->root() . '/' . F::name($file->filename()) . '-thumbnail.jpg', $imagedata);
}
7

Found the problem 1 year later.
It was due to a plugin with a hook on uploaded files that couldn’t handle video files and was missing a condition to filter files only of type “image” !