QAbstractItemView handles drop event action inconsistently

When QAbstractItemView handles drop events (e.g. in the methods dragEnterEvent, dragMoveEvent, dropEvent) it has to evaluate the action of the event. This is done correctly by quering the dropAction method of the event.

However, when the event is later accepted by those methods, this is sometimes done via the event's acceptProposedAction method instead of accept. This is wrong, as the event's action is then overwritten by the proposed action - but all the processing and checking before used the event's action.

Normally, this does not lead to problems as the event's action is set to its proposed action. As soon as somebody overrides the drop handling methods and sets a different action (which is a totally reasonable thing to do if you want to select the action based on the drop target), then things fall apart. First all checking is done on the replacement action, but in the end the event will be forced back to the preferred action leading to accepting/declining the drops based on the wrong action.

The right thing to do here is to definitely stay away from acceptProposedAction, and just use accept().