[FIX] fiscal_epos_print: new order lines not saved when changing table on Point of sale when restaurant tables are enabled

[micheledic]

This PR fixes an issue on Point of sale using tables and restaurant
It improves also the code using "some" instead of "find"
Fixes #4624

[OCA-git-bot]

Hi @eLBati,
some modules you are maintaining are being modified, check this out!

[micheledic]

Errore della build unrelated

EDIT: ho incluso un commit che fixa anche i test che falliscono su l10n_it_account_stamp

[Wvven]

LGTM

[micheledic]

rebased per applicare #4529, ora dovrebbe essere apposto ;)

[TheMule71]

LGTM

[robyf70]

LGTM

[TheMule71]

Ho solo una domanda prima di mergiare, mi sfugge in base a quale meccanismo fixa la issue #4624
(premetto che non ho familiarità col JS del POS).

Vedo che impatta un flag (has_refund) sull'ordine se almeno una riga ha una qta negativa, in che modo però il vecchio codice impediva il salvataggio delle righe come da issue? La issue non nomina refund.

Non capisco come le due cose siano collegate.

[micheledic]

Ho solo una domanda prima di mergiare, mi sfugge in base a quale meccanismo fixa la issue #4624 (premetto che non ho familiarità col JS del POS).

Vedo che impatta un flag (has_refund) sull'ordine se almeno una riga ha una qta negativa, in che modo però il vecchio codice impediva il salvataggio delle righe come da issue? La issue non nomina refund.

Non capisco come le due cose siano collegate.

E' molto strano come bug, se in quella funzione fa la find per verificare se c'è un reso , successivamente pusha al server sempre e solo le righe inserite inizialmente . E' come se lato js, dopo aver acceduto alle lines per fare la find, nel momento del push delle righe al server accedesse sempre alla cache appena inizializzata delle orderlines e non alle righe effettivamente inserite. (infatti nei logs si vedono delle pos.order.line eliminate quando si cambia tavolo)

Per essere certo che sia quella la parte di codice incriminata, inizialmente avevo provato a forzare has_refund = True cosi non facendogli fare la find e il problema non si presentava .

Con some, comunque, personalmente ritengo sia anche molto più elegante oltre che risolvere il problema

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[micheledic]

Ho solo una domanda prima di mergiare, mi sfugge in base a quale meccanismo fixa la issue #4624 (premetto che non ho familiarità col JS del POS).

Vedo che impatta un flag (has_refund) sull'ordine se almeno una riga ha una qta negativa, in che modo però il vecchio codice impediva il salvataggio delle righe come da issue? La issue non nomina refund.

Non capisco come le due cose siano collegate.

Si può mergiare ?

[francesco-ooops]

@TheMule71