'Smart' Trade Transfers

[Uleat]

Trade transfers are now a little more intelligent on how items are placed into the inventory.


Items are parsed in this order:

- All bags are transferred first

- Any stackable items are moved to fill up existing stacks..including to those in the aforementioned bags

- All remaining items


QueryServ logging properly indicates the number of items in the 'Event Entry' table and charge movements are recorded in the
'Event Details' table.

Note: There may detail entries that show a movement from one trade slot to another on the same character. This indicates a
'stack biasing' used to eliminate partial stack overwrites when the remaining items are processed.


Please post any discrepancies as bugs.

[Drajor]

Isn't this how live works? I've never paid attention to whether bags go first or not.

[Uleat]

Yes..but, it's not how the Emu worked..until now :)

Not 100% sure on the bags..but, this will to keep from losing bag items when mutiple bags are sent to the cursor.

[joligario]

Update to your change: http://www.peqtgc.com/phpBB3/viewtopic.php?f=17&t=15050

[Uleat]

That's an intentional action.

It's an attempt to avoid having trade items..especially loaded bags..going to the cursor buffer and losing them.


I didn't have a model to go by, other than items should stack when traded, so I took the liberty..if it needs to be tweaked, I can do that.

[joligario]

If there is a partial stack, it should first fill that. If not (or after the partial stack is full) it (or the remainder) should go in the first main slot available. If no main slots available, then the next available bag slot.

[Uleat]

So, I just need to change it to a 2-pass solution?

[Uleat]

Stacking and final pass item transfers will now return all available main slots before iterating into the sub-slot range.

Let me know if this still needs tweaking :)