Fix incorrect money format being used

[kayue]

Only format accept currency as argument.
Fix #11941.

Q	A
Branch?	1.8
Bug fix?	yes
New feature?	no
BC breaks?	no
Deprecations?	no
Related tickets	fixes #11941
License	MIT

A "Cart" menu/collector is added in Sylius 1.8, but this feature requires Channel to exist in the debugger/profiler's route, otherwise user will get the following error:

An exception has been thrown during the rendering of a template ("Channel could not be found! Tip: You can use the Web Debug Toolbar to switch between channels in development.").

The error is triggered by convertAndFormat() in the Twig file. Change it to format() fix the problem. I think it was meant to use format since method signature was incorrect also.

[kayue]

Anything blocking this? Just want to make sure this will get merged in the next release. Thanks.

[kayue]

@pamil Could you include this into the next release? Clearly a bug and simple fix. Thanks.

[Zales0123]

👍 Shouldn't it go to the 1.8 branch? It's clearly a bugfix :)

[kayue]

@Zales0123 do you want me to change the base to 1.8?

[kayue]

This didn't get merged in the last release 🤦

Ping @pamil @lchrusciel

[kayue]

Ping @pamil again. Think this is an easy bug fix.

[AdamKasp]

The base of this pull-request was changed, you need fetch and reset your local branch
if you want to add new commits to this pull request. Reset before you pull, else commits
may become messed-up.

Unless you added new commits (to this branch) locally that you did not push yet,
execute git fetch origin && git reset "patch-5" to update your local branch.

Feel free to ask for assistance when you get stuck 👍

[kayue]

Not sure what you mean, do you still want this branch to merge against 1.8?
There was no conflict in this PR so should be ready to merge anytime.

Anyway I have rebased the commit against the latest 1.8 branch.

[AdamKasp]

Thanks, @kayue! 🥇