Fix order cycle mail Report grouping by product and price.

[lufafajoshua]

What? Why?

• Closes Display error in mail reports when product price change during order cycle #13413

In the Producer Order Cycle Mail Report, products from different orders were grouped solely by product name. As a result, when the same product appeared in multiple orders at different unit prices, the quantity multiplied by unit price did not match the displayed total price because the price differences were not reflected.
This solution groups line items by both product name and unit price. In the report, same products with identical prices continue to be integrated into a single row, while products with different prices are displayed in separate rows, ensuring that unit prices and totals are reported accurately.

What should we test?

1. Open an order cycle
2. Make an order with product A
3. Modify price of product A
4. Make another order with product A
5. Wait for end of cycle and check mail received by producer

Release notes

• User facing changes

[sigmundpetersen]

Hi @lufafajoshua there's a linter error that maybe you could have a look at?

[lufafajoshua]

I think i forgot to run rubocop, let me update and push again.

[lufafajoshua]

I personally think failing tests in ./spec/system/admin/order_cycles/simple_spec.rb are not caused by changes in this PR, though i will continue to investigate.

[sigmundpetersen]

You are right @lufafajoshua it's a flaky spec. Trying to get Copilot fix it in #14435 🙂

[sigmundpetersen]

Copilot did not manage to fix it 😄 We can move on with this PR, there will be someone reviewing it.
I'll try rebuilding to get it green.

[rioug]

Looks good, thanks for your help 🙏

[dacook]

Nice work, thank you very much!
Awaiting a manual test before it will be merged and included in a release.

[AMEA-LYON]

Hi all, it's ok, products are correctly grouped by price... 🎉