Revert "Use quote id just for debugging (#3054)"

[sunce86]

This reverts commit cea22f6.
Then implements #3100

Description

Reverts the change introduced by the PR #3054

Apparently the current implementation is not good at recovering the right quote as it sometimes finds the quote that is too old, while the order was intended to be created with the most recent quote.

A different fix is implemented to actually continue using quote_id, but compute the fresh quote if quote_id cant be found.

[squadgazzz]

Didn't actually get from the description why this revert is even required.

[MartinquaXD]

Didn't actually get from the description why this revert is even required.

The problem is a bit subtle. Under most circumstances we only use the quote to see if an order was created out of market to classify it either as a limit or a market order.
For Ethflow orders we do something extra. To avoid people spamming ethflow orders and draining the automatic refund service we require them to create the ethflow order based off a reasonable quote. If the quote we recover or compute just in time suggest the order is out-of-market at the time of creation the order is not eligible for automatic refunding.
If we don't look up the quote exactly by ID and instead do a fuzzy match we frequently end up finding the wrong quote which leads to us having to manually refund the order which is annoying for us and the user.

@sunce86 could you implement the change within this PR or merge the revert and new implementation together? The reason is that some wallets already rely on the fact that they can create orders and have a quote generated if none can be found.