thumbnailCreated signal is emitted even if thumbnail creation failed

[kdambekalns]

Description

see

neos-development-collection/Neos.Media/Classes/Domain/Service/ThumbnailService.php

Lines 140 to 147 in 780fffa

	$thumbnail = new Thumbnail($asset, $configuration);
	$this->emitThumbnailCreated($thumbnail);
	// If the thumbnail strategy failed to generate a valid thumbnail
	if ($async === false && $thumbnail->getResource() === null && $thumbnail->getStaticResource() === null) {
	$this->thumbnailRepository->remove($thumbnail);
	return null;
	}

Expected behavior

Signal is emitted after successful thumbnail creation

Actual behavior

Signal is emitted unconditionally

Affected Versions

Neos: 5.3+