View field editor: make sure the field value is sent to backend before the view is deleted

[teosarca]

Is this a bug or feature request?

Bug
follow-up of #1335
part of metasfresh/metasfresh-webui-api-legacy#664

What is the current behavior?

Which are the steps to reproduce?* open a draft sales order, e.g. https://w101.metasfresh.com:8443/window/143/1000096

• select the order lines and call the Create purchase order action
• edit a field value and directly press "Done" to close the modal
  => NOK: error because the field is patched after the view is deleted.

What is the expected or desired behavior?

Shall work.

If there is a chance, even if it's small, to not send back some fields then I highly suggest, on Done,
please also consider sending ALL edited fields.
It's super important to have 100% correct and up2date values.

Another thing, if, for some reason the "DELETE view" endpoint which is called on Done fails, please don't close the modal.

[metas-lc]

IT
i don't really have a clear testcase for this, but i managed to reproduce the "no view found" after trying the same thing several times
(and notice edit appears several times. Maybe you can put them together if they are for the same row? like qty and date promised in the same request?)
go to https://w101.metasfresh.com:8443/window/143/1000096
e.g. select the first line, go to actions - create purchase orders
update a qty using keyboard then press done => works
select the first line and go to actions - create purchase orders
update the qty again using keyboard then press done => error
(you might need to do this 2 or 3 times...try to update the same qty field)

[metas-lc]

IT
repeated the testcase above. Works fine: error doesn't appear in that case. Also those edits don't appear OK
still, there is a case when that error pops up: when you change a qty in "Create purchase orders" and pressing done directly. Created follow-up #1518