Fixed issue #1689

[umpirsky]

No description provided.

[pjedrzejewski]

@umpirsky We use sylius_money for all values, I think the conversion should happen there. Thoughts?

[umpirsky]

@pjedrzejewski What class#line are you referring to?

[stloyd]

@umpirsky @Arn0d @pjedrzejewski I think this is correct fix, problem with are located in Symfony transformers (used by sylius_money) that transform into float while in Sylius we hold data as integer.

So either we should force type to be integer or replace Doctrine type into float for prices, otherwise we will endup with annoying bug #1689 always logging price change. Personally I'm more with forcing integer instead of float.

ps. I was trying to find other way to fix this, but those two are simplest ones.

[stloyd]

Additionally also OrderItem::setUnitPrice() should be changed same way as it's done in ProductVariant.

[umpirsky]

@pjedrzejewski Let's merge this.

[pjedrzejewski]

Thank you Sasha! 👍