Bug: Duplicate messages are added to image_conversion queue

[kottkrig]

Current behaviour

Duplicate messages are added to the image_conversion RabbitMQ queue.

How to test

1. Publish an article with media
2. Observe that the rabbitmq message queue increases with the number of renditions for the media. E.g 16 renditions results in 16 messages on the queue.
3. Publish an article without media
4. Observe that the rabbitmq message queue still increases with the same number of messages from step 2.

Possible culprit?

This listener isn't removed. Could this result in a new listener/callback each time a new post is persisted to the database?

web-publisher/src/SWP/Bundle/CoreBundle/EventListener/ImageToWebpConversionListener.php

Lines 55 to 60 in 6443cc7

	$this->eventDispatcher->addListener(PackageEvents::PACKAGE_PROCESSED, function (GenericEvent $event) use ($rendition, $tenantId) {
	$this->messageBus->dispatch(new ConvertImageMessage(
	(int) $rendition->getImage()->getId(),
	(int) $tenantId
	));
	});

[takeit]

That could be it. Can you try commenting that out when doing step 3 and see if the messages will be added to the queue? But in general, this listener should not be called when there are no renditions/images in the article you publish, unless it still references the old images.