ge: include both delta qty and qty
This commit is contained in:
@@ -34,7 +34,8 @@ public class GrandExchangeTrade
|
||||
private boolean cancel;
|
||||
private boolean login;
|
||||
private int itemId;
|
||||
private int quantity;
|
||||
private int qty;
|
||||
private int dqty;
|
||||
private int total;
|
||||
private int spent;
|
||||
private int offer;
|
||||
|
||||
@@ -80,8 +80,8 @@ public class GrandExchangeController
|
||||
|
||||
// We don't keep track of pending trades in the web UI, so only add cancelled or completed trades
|
||||
if (userId != null &&
|
||||
grandExchangeTrade.getQuantity() > 0 &&
|
||||
(grandExchangeTrade.isCancel() || grandExchangeTrade.getQuantity() == grandExchangeTrade.getTotal()))
|
||||
grandExchangeTrade.getQty() > 0 &&
|
||||
(grandExchangeTrade.isCancel() || grandExchangeTrade.getQty() == grandExchangeTrade.getTotal()))
|
||||
{
|
||||
grandExchangeService.add(userId, grandExchangeTrade);
|
||||
}
|
||||
@@ -91,7 +91,8 @@ public class GrandExchangeController
|
||||
trade.setCancel(grandExchangeTrade.isCancel());
|
||||
trade.setLogin(grandExchangeTrade.isLogin());
|
||||
trade.setItemId(grandExchangeTrade.getItemId());
|
||||
trade.setQuantity(grandExchangeTrade.getQuantity());
|
||||
trade.setQty(grandExchangeTrade.getQty());
|
||||
trade.setDqty(grandExchangeTrade.getDqty());
|
||||
trade.setTotal(grandExchangeTrade.getTotal());
|
||||
trade.setSpent(grandExchangeTrade.getSpent());
|
||||
trade.setOffer(grandExchangeTrade.getOffer());
|
||||
|
||||
@@ -79,8 +79,8 @@ public class GrandExchangeService
|
||||
.addParameter("user", userId)
|
||||
.addParameter("action", grandExchangeTrade.isBuy() ? "BUY" : "SELL")
|
||||
.addParameter("item", grandExchangeTrade.getItemId())
|
||||
.addParameter("quantity", grandExchangeTrade.getQuantity())
|
||||
.addParameter("price", grandExchangeTrade.getSpent() / grandExchangeTrade.getQuantity())
|
||||
.addParameter("quantity", grandExchangeTrade.getQty())
|
||||
.addParameter("price", grandExchangeTrade.getSpent() / grandExchangeTrade.getQty())
|
||||
.executeUpdate();
|
||||
}
|
||||
}
|
||||
|
||||
@@ -34,7 +34,8 @@ class Trade
|
||||
private boolean cancel;
|
||||
private boolean login;
|
||||
private int itemId;
|
||||
private int quantity;
|
||||
private int qty;
|
||||
private int dqty;
|
||||
private int total;
|
||||
private int spent;
|
||||
private int offer;
|
||||
|
||||
@@ -412,7 +412,6 @@ public class GrandExchangePlugin extends Plugin
|
||||
GrandExchangeTrade grandExchangeTrade = new GrandExchangeTrade();
|
||||
grandExchangeTrade.setBuy(state == GrandExchangeOfferState.BUYING);
|
||||
grandExchangeTrade.setItemId(offer.getItemId());
|
||||
grandExchangeTrade.setQuantity(0);
|
||||
grandExchangeTrade.setTotal(offer.getTotalQuantity());
|
||||
grandExchangeTrade.setSpent(0);
|
||||
grandExchangeTrade.setOffer(offer.getPrice());
|
||||
@@ -443,7 +442,7 @@ public class GrandExchangePlugin extends Plugin
|
||||
grandExchangeTrade.setBuy(state == GrandExchangeOfferState.CANCELLED_BUY);
|
||||
grandExchangeTrade.setCancel(true);
|
||||
grandExchangeTrade.setItemId(offer.getItemId());
|
||||
grandExchangeTrade.setQuantity(offer.getQuantitySold());
|
||||
grandExchangeTrade.setQty(offer.getQuantitySold());
|
||||
grandExchangeTrade.setTotal(offer.getTotalQuantity());
|
||||
grandExchangeTrade.setSpent(offer.getSpent());
|
||||
grandExchangeTrade.setOffer(offer.getPrice());
|
||||
@@ -466,7 +465,8 @@ public class GrandExchangePlugin extends Plugin
|
||||
GrandExchangeTrade grandExchangeTrade = new GrandExchangeTrade();
|
||||
grandExchangeTrade.setBuy(state == GrandExchangeOfferState.BUYING);
|
||||
grandExchangeTrade.setItemId(offer.getItemId());
|
||||
grandExchangeTrade.setQuantity(qty);
|
||||
grandExchangeTrade.setQty(offer.getQuantitySold());
|
||||
grandExchangeTrade.setDqty(qty);
|
||||
grandExchangeTrade.setTotal(offer.getTotalQuantity());
|
||||
grandExchangeTrade.setSpent(dspent);
|
||||
grandExchangeTrade.setOffer(offer.getPrice());
|
||||
|
||||
@@ -157,7 +157,7 @@ public class GrandExchangePluginTest
|
||||
GrandExchangeTrade trade = captor.getValue();
|
||||
assertTrue(trade.isBuy());
|
||||
assertEquals(ItemID.ABYSSAL_WHIP, trade.getItemId());
|
||||
assertEquals(2, trade.getQuantity());
|
||||
assertEquals(2, trade.getDqty());
|
||||
assertEquals(10, trade.getTotal());
|
||||
assertEquals(20, trade.getSpent());
|
||||
}
|
||||
@@ -214,7 +214,7 @@ public class GrandExchangePluginTest
|
||||
assertTrue(trade.isBuy());
|
||||
assertTrue(trade.isCancel());
|
||||
assertEquals(ItemID.ABYSSAL_WHIP, trade.getItemId());
|
||||
assertEquals(1, trade.getQuantity());
|
||||
assertEquals(1, trade.getQty());
|
||||
assertEquals(10, trade.getTotal());
|
||||
assertEquals(25, trade.getSpent());
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user