catch potential error in ImageBuilderResource.php

[Myrmod]

Just a small fix for catching a potential error that happens locally. Shouldn't break anything

[dkd-kaehm]

Please merge.

It fixes the issue file_exists(): Argument #1 ($filename) must be of type string, null given
Since $theImage var is nullable.

filefill/Classes/Resource/Handler/ImageBuilderResource.php

Lines 98 to 99 in 172e3ca

	$theImage = $gifBuilder->gifBuild()?->getFullPath();
	if (file_exists($theImage)) {

[IchHabRecht]

would you mind to give me a hint how the error you're seeing can be reproduced? hope you understand that I cannot merge any changes before evaluating and understanding the circumstances.

[Myrmod]

would you mind to give me a hint how the error you're seeing can be reproduced? hope you understand that I cannot merge any changes before evaluating and understanding the circumstances.

If I remember correctly solr stored some image data and the stored images were breaking things in combination with filefill.

I might be confusing something though. Cannot look at the code properly at the moment and it's a while ago.

Either way the change cannot negatively impact the functionality of the extension I believe. It is save to merge.
However it is obviously your choice.
I'll see if I can find the exact cause of the error for me tomorrow, in case I have time in between work. :)

[dkd-kaehm]

If I remember correctly solr stored some...

👺 Hehe just because I can reproduce it doesn’t automatically mean Solr is involved.

[Myrmod]

If I remember correctly solr stored some...

👺 Hehe just because I can reproduce it doesn’t automatically mean Solr is involved.

Hahaha, not what I meant. I remember darkly that I had an issue where something didn't work because someone stored image references in solr and the template couldn't find those files anymore ^^