AverageRatingUpdater is called twice when a review is accepted

[martijnc]

Sylius version affected: 1.13

Description
The average product rating is updates through AverageRatingUpdater, this service is called by;

• ReviewChangeListener;
• the sylius_update_rating callback on the sylius_product_review state machine.

This means that the score is calculated twice for the accept transition; once through the callback and once after flushing the review (which you have to do in order to persist the new state value on the entity).

Steps to reproduce
Accept a review.

Possible Solution
Remove the sylius_update_rating callback from the sylius_product_review state machine. Don't think this is required anymore now that the ReviewChangeListener handles this (was added later).

[Rafikooo]

Hello @martijnc, your reported issue is valid. We will fix it later, in Sylius 2.0 release. However, if you'd like to try your hand at proposing the solution, it would be great. 🕺