handling of items and aborting actions

[ghost]

Expected behaviour

When executing an action involving a vItem resulting in either it getting replaced by another one, removed, or some new added to the inventory, the inventory stuff should only happen after the action is completed. If it's interrupted no inventory action should happen at all.

Actual behaviour

Handling of vItems happens before executing the action resulting in items gets lost or duped when the action is interrupted.

Steps to reproduce the behaviour

1. By a full fuel canister from any store selling it.
2. Drive a car until its fule tank starts to drain.
3. Refuel the car by using the full fuel canister.
4. While refueling move usng WASD.
5. The refueling action is interrupted.
6. Look at your virtual inventory: The fuel canister is lost.

RPT and/or extDB3 logs (if applicable)

not applicable

Mission version: 6.0.0
Launcher version: 1.5.146373
Game version: 1.98.146373
Branch: main / beta branch not specified

I spotted this problem while I was testing #685 and tried to figure out the interruption. As I had to first somehow get an empty canister I drove a car and refueld it. I wanted to see if this is a general issue with how vItems are handled so I moved during the refueling - which ended up in the full fuel canister got lost. As this seem to not only affect the full and empty fuel canister while refuling but maybe any vItems and any action involving them this seem like an overall issue about the order of execution.