Projekte/monopoly/client/src/main/java/pp/monopoly/client/gui/popups/ConfirmTrade.java

@@ -80,7 +80,7 @@ public class ConfirmTrade extends Dialog {
         // Ablehnen-Button
         Button declineButton = confirmTradeContainer.addChild(new Button("Ablehnen", new ElementId("button")));
         declineButton.setFontSize(32);
-        declineButton.addClickCommands(s -> ifTopDialog(() -> {
+        declineButton.addClickCommands(s -> ifTopDialog(() -> { //TODO Buttonfunktion prüfen
             app.getGameLogic().playSound(Sound.BUTTON);
             app.getGameLogic().send(new TradeResponse(false, tradeHandler));
             close();
@@ -91,9 +91,7 @@ public class ConfirmTrade extends Dialog {
         negotiateButton.addClickCommands(s -> ifTopDialog( () -> {
             app.getGameLogic().playSound(Sound.BUTTON);
             close();
-            TradeHandler t = new TradeHandler(app.getGameLogic().getPlayerHandler().getPlayerById(tradeHandler.getSender().getId()));
-            t.setReceiver(app.getGameLogic().getPlayerHandler().getPlayerById(tradeHandler.getReceiver().getId()));
-            new TradeMenu(app, t).open();
+            new TradeMenu(app, new TradeHandler(app.getGameLogic().getPlayerHandler().getPlayerById(app.getId()))).open();
         }));
         // Confirm-Button
         Button confirmButton = confirmTradeContainer.addChild(new Button("Bestätigen", new ElementId("button")));

Projekte/monopoly/model/src/main/java/pp/monopoly/model/TradeHandler.java

@@ -55,7 +55,14 @@ public class TradeHandler {
         /**
      * Constructs a TradeHandler for a single trade instance.
      *
-     * @param sender the Player initiating the trade
+     * @param sender             the Player initiating the trade
+     * @param receiver           the Player receiving the trade offer
+     * @param offeredAmount      the amount of money offered by the sender
+     * @param offeredProperties  the properties offered by the sender
+     * @param offeredJailCards   the jail cards offered by the sender
+     * @param requestedAmount    the amount of money requested from the receiver
+     * @param requestedProperties the properties requested from the receiver
+     * @param requestedJailCards the jail cards requested from the receiver
      */
     public TradeHandler(Player sender) {
         this.sender = sender;