I did do some testing on this...
The current code will allow stacking if you drop the item on the player and not in the trade window.
However...
The handler does need to be re-written to properly handle stack movements.
I'm not going to overly restrict the checks unless a problem arises.
If I find that directly dropping a partial stack on another partial is allowed, I'll see if I can figure out why the action isn't making it to the handler.
But, as of now, I'm just not seeing any OP_MoveItem requests making it to the 'trade' portion of Client::SwapItem(). (Ti/SoF tested)
If the later clients allow it, the re-write won't restrict it..and later testing will (hopefully) verify it.
__________________
Uleat of Bertoxxulous
Compilin' Dirty
|
Threaded Mode
