pos_order_to_sale_order Fiscal Position applied at Header but not at OrderLines about pos

[legalsylvain]

Hi. At first glance, i think the problem is not in pos order to sale order module. The module just use export to JSON function that looks to return bad résult

https://github.com/OCA/pos/blob/16.0/pos_order_to_sale_order/static/src/js/CreateOrderPopup.js#L49

Investigation should be done over there.

Thanks !

[legalsylvain]

Hi. @xmorejon. As far as I understand, the taxes of the sale order lines, are the taxes on the pos order lines.
See : https://github.com/OCA/pos/blob/16.0/pos_order_to_sale_order/models/sale_order_line.py#L19

Are the pos order line correctly set ?

thanks !

[xmorejon]

Hi,

The POS Order Line are not correctly set:

• Product Price=4,5€
• VAT (21%)= 0,945€
• Recarco Equivalencia (5,2%) = 0,234€
• Total taxes should be: 1,18€

At Product Info / line Level it is not displaying the 0,234€ in the POS Order Line:

But on the Total it is correctly displayed:

Also if instead of generating the Sales Order, we do the Payment on the POS, the receipt will be correctly built:

Could it be that the Point of Sale does a recomputing of the taxes in a later stage, so while preparing Sales Order Lines from the POS Order Lines this Fiscal Position is wrong?
https://github.com/OCA/pos/blob/16.0/pos_order_to_sale_order/models/sale_order_line.py#L19

Thank you so much for looking into it!

[legalsylvain]

Well. Could add a screenshot of the pos order taxes in the back-office ?

[xmorejon]

Apologies not sure if I follow you:

• When I do a regular payment in the POS (instead of generating the Sales Order with "pos_order_to_sale_order" module) there is indeed a POS Order in the back-office:

This one corresponds to the receipt of the screenshot Order 00172-001-001. Note that the Taxes are CORRECT.

• But when we generate a Sales order from the POS, there is NO "POS Order" so I cannot make a screenshot:

Sales Order screenshot:

No available POS Order: Order 00167-001-0001

But corresponding Sales Order in the back-office do miss the Fiscal Position items:

Should I look somewhere else?
Merci Sylvain!

[xmorejon]

Hi again!

Forcing a recompute of Taxes, did the trick and seems to fix the issue! :)

And now the Sales Order has the correct Fiscal Position included:

As mentioned, I am pretty new to ODOO, and not sure if this way of fixing the issue is correct.
Therefore, I would wait for feedback and not suggest this pull request.

Thank you,

Xavier Morejon

[legalsylvain]

could you add a print(order_data) here : https://github.com/OCA/pos/blob/16.0/pos_order_to_sale_order/models/sale_order.py#L13 ?
and put her the result.
thanks !

[xmorejon]

Hi,

Here you go. (Order with 2 lines):

{'name': 'Order 00188-001-0001', 'amount_paid': 0, 'amount_total': 11.36, 'amount_tax': 2.36, 'amount_return': 0, 'lines': [[0, 0, {'qty': 1, 'price_unit': 4.5, 'price_subtotal': 4.5, 'price_subtotal_incl': 5.68, 'discount': 0, 'product_id': 62, 'tax_ids': [[6, False, [1]]], 'id': 1, 'pack_lot_ids': [], 'description': '', 'full_product_name': 'B-0001', 'price_extra': 0, 'price_manually_set': False, 'price_automatically_set': False}], [0, 0, {'qty': 1, 'price_unit': 4.5, 'price_subtotal': 4.5, 'price_subtotal_incl': 5.68, 'discount': 0, 'product_id': 78, 'tax_ids': [[6, False, [1]]], 'id': 2, 'pack_lot_ids': [], 'description': '', 'full_product_name': 'BC-0002', 'price_extra': 0, 'price_manually_set': False, 'price_automatically_set': False}]], 'statement_ids': [], 'pos_session_id': 190, 'pricelist_id': 3, 'partner_id': 11157, 'user_id': 2, 'uid': '00188-001-0001', 'sequence_number': 1, 'creation_date': '2023-12-10T08:50:06.942Z', 'fiscal_position_id': 6, 'server_id': False, 'to_invoice': False, 'to_ship': False, 'is_tipped': False, 'tip_amount': 0, 'access_token': '3eb2f19d-8a81-4299-a0ff-e41ee1dce4df'}

So the Subtotal is right -> 'price_subtotal_incl': 5.68
But there is only one of the TaxID's (corresponding to the 21%) -> 'tax_ids': [[6, False, [1]]]

Thank you.
Xavier

[xmorejon]

Thank you Sylvain!
I believe you are right. As the issue looks to be out of this module, I will close the reported issue.

Will look further into:

But might also apply the "dirty" fix of recomputing the taxes in alternate version of the module. I really need the Fiscal positions to give expected outcome!

Thank you very much. Great job!
Xavier