[14.0][FIX] pos_order_mgmt: Return with positive amounts

[zamberjo]

When making a return, the POS user can change the return quantity into positive. If the user does not realise this, the error message "Please specify at least one non-zero quantity" is displayed at the end of the order. leaving the order to be synchronised, always giving the same error.

I propose that when making the return from the POS, this line should always be forced to a negative amount.

In addition, a ConfirmPopup has been added to make sure that the user is aware that he is making a return of more than the amount sold.

[belenaures]

I've checked it and I've noticed that when trying to delete a line an error appears:
-> what i do is to is press the delete button in the POS, first it remains as qty 0 and then it should delete the line, instead, the error shows.
Rastreo de error: Error: 'remove' no es un decimal correcto at Object.parseFloat [as float

[zamberjo]

I've checked it and I've noticed that when trying to delete a line an error appears: -> what i do is to is press the delete button in the POS, first it remains as qty 0 and then it should delete the line, instead, the error shows. Rastreo de error: Error: 'remove' no es un decimal correcto at Object.parseFloat [as float

Fixed

[belenaures]

Functionality checked, all ok

[github-actions]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

[chienandalu]

Looks good 👍

[pedrobaeza]

/ocabot merge patch

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-1067-by-pedrobaeza-bump-patch, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 07c91e0. Thanks a lot for contributing to OCA. ❤️