Trading system flaws

Created by: Gptaqbc

It seems there is a problem with the coding of the trading system between players. I tested it this week-end and here are my observations.

I'll explain the expected process:

P1 clicks trade on P2; P1 moves next to P2 to send him the request; P2 receive and accpet the request; Trade screen opens; -- Anything happening here that isn't adding/removing items or accepting in the trade screen abort the trade process --- P1 & P2 accepts screen 1; -- Anything happening here that accepting in the trade screen abort the trade process --- P1 & P2 accepts screen 2;

Trade completed

Here's a list of the problem with our trading system:

• After the trade request is sent (click trade player) both players doesn't have to be next to each other to initiate the trading process and complete the trade.
• A player can quit the trade screen by clicking the minimap but it's not declining it for the other player. Let's call this solo trading interface.
• When a player is stuck in the solo trading interface he can still fill the trade screen with items. If the other player that initiated the trade proceed to logout, the trade screen will close but the items will stay in the trade screen until the players trade again. This is a big issues as it can expand the inventory by 28 slots for every player.

Created by: Gptaqbc

Here: https://github.com/dginovker/2006rebotted/blob/ea513131259e183017476cb3177649c804a35af9/2006Redone%20Server/src/redone/net/packets/impl/ClickingStuff.java#L34

player.getTrading().declineTrade();

What about adding oponnent.getTrading().declineTrade(); too? That should close the trade screen for both players. That should fix the issue with being able to store items in the trading interface. I'll try soon.

added Difficulty: Medium label

Created by: Gptaqbc

Almost completed. Can you assign me on the issue?

mentioned in merge request !117 (merged)

closed