feat: add equipment

* chore: add simple equipment system for the player

Collected collectibles are now displayed if we click on a non-collectible item

* feat: Improve equipment system

* add InventoryPopup

* Update src/main/java/io/rpg/controller/PopupController.java

Co-authored-by: Marcin Hawryluk <70582973+mhawryluk@users.noreply.github.com>

* Update src/main/java/io/rpg/model/data/Inventory.java

Co-authored-by: Marcin Hawryluk <70582973+mhawryluk@users.noreply.github.com>

* Update src/main/java/io/rpg/model/object/CollectibleGameObject.java

Co-authored-by: Marcin Hawryluk <70582973+mhawryluk@users.noreply.github.com>

* Update src/main/java/io/rpg/view/InventoryPopup.java

Co-authored-by: Marcin Hawryluk <70582973+mhawryluk@users.noreply.github.com>

* Update src/main/java/io/rpg/viewmodel/InventoryPopupViewModel.java

Co-authored-by: Marcin Hawryluk <70582973+mhawryluk@users.noreply.github.com>

* Update src/main/java/io/rpg/view/InventoryGameObjectView.java

Co-authored-by: Marcin Hawryluk <70582973+mhawryluk@users.noreply.github.com>

* fix: make equipment work on current arch

Co-authored-by: Marcin Hawryluk <70582973+mhawryluk@users.noreply.github.com>
Co-authored-by: Kacper Kafara <kacperkafara@gmail.com>

Author: 3 people

configurations/demo-config-1/locations/location-1/objects/object-2.json

@@ -1,4 +1,4 @@
 {
   "tag": "object-2",
-  "onLeftClick": {"tag": "dialogue-action", "type": "dialogue", "statements": ["A key.,A, keyA, key.A, key.A, key.A, key.A, key.A, key.A key."], "assetPath": "assets/key.png"}
+  "onLeftClick": {"tag": "dialogue-action", "type": "collect"}
 }

src/main/java/io/rpg/config/model/ActionConfigBundle.java

@@ -252,6 +252,10 @@ Result<Void, Exception> validateForBattle() {
     return Result.ok();
   }
 
+  Result<Void, Exception> validateForCollectAction() {
+    return Result.ok();
+  }
+
   Result<Void, Exception> validateBasic() {
     ErrorMessageBuilder builder = new ErrorMessageBuilder();
     if (tag == null) {

src/main/java/io/rpg/controller/Controller.java

@@ -5,23 +5,30 @@
 import io.rpg.model.data.MouseClickedEvent;
 import io.rpg.model.data.Position;
 import io.rpg.model.location.LocationModel;
+import io.rpg.model.object.CollectibleGameObject;
 import io.rpg.model.object.GameObject;
 import io.rpg.model.object.Player;
 import io.rpg.model.object.Question;
 import io.rpg.util.BattleResult;
 import io.rpg.util.Result;
 import io.rpg.view.GameEndView;
 import io.rpg.view.GameObjectView;
+import io.rpg.view.InventoryPopup;
 import io.rpg.view.LocationView;
+
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
 import java.util.LinkedHashMap;
 import java.util.List;
+
 import javafx.geometry.Point2D;
 import javafx.scene.Scene;
+import javafx.scene.image.Image;
 import javafx.scene.input.KeyEvent;
 import javafx.scene.input.MouseButton;
 import javafx.stage.Stage;
+import javafx.stage.StageStyle;
+import io.rpg.view.popups.TextPopup;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.jetbrains.annotations.NotNull;
@@ -36,10 +43,8 @@ public class Controller implements KeyboardEvent.Observer, MouseClickedEvent.Obs
   private PlayerController playerController;
   private Stage mainStage;
 
-
   public Controller() {
     logger = LogManager.getLogger(Controller.class);
-
     tagToLocationModelMap = new LinkedHashMap<>();
     tagToLocationViewMap = new LinkedHashMap<>();
   }
@@ -168,7 +173,13 @@ private void onAction(BattleAction action) {
     }
     popupController.openTextPopup(result.getMessage(), getWindowCenterX(), getWindowCenterY());
   }
-  
+
+  private void onAction(CollectAction action) {
+    popupController.openTextImagePopup("Picked up an item!", new Image("file:assets/key.png"), getWindowCenterX(), getWindowCenterY());
+//     objectView.setVisible(false);
+    playerController.getPlayer().getInventory().add(action.getOwner());
+  }
+
   public Scene getView() {
     return currentView;
   }
@@ -215,6 +226,7 @@ public void onKeyboardEvent(KeyboardEvent event) {
         case Q -> popupController.openQuestionPopup(new Question("How many bits are there in one byte?", new String[]{"1/8", "1024", "8", "256"}, 'C'), getWindowCenterX(), getWindowCenterY());
         case L -> consumeAction(new LocationChangeAction("location-2", new Position(1, 2)));
         case U -> consumeAction(new GameEndAction("You have pressed the forbidden button"));
+        case E -> popupController.openInventoryPopup(playerController.getPlayer().getInventory(), getWindowCenterX(), getWindowCenterY());
       }
     }
     // } else if (payload.getEventType() == KeyEvent.KEY_RELEASED) {
@@ -236,7 +248,7 @@ public void onMouseClickedEvent(MouseClickedEvent event) {
     GameObjectView objectView = event.source();
     Position position = new Position(objectView.getPosition());
     GameObject object = currentModel.getObject(position)
-                                    .orElseThrow(() -> new RuntimeException("No object present at position " + position));
+        .orElseThrow(() -> new RuntimeException("No object present at position " + position));
 
     double distance = playerPos.distance(objectView.getPosition());
 

src/main/java/io/rpg/controller/PopupController.java

@@ -3,6 +3,8 @@
 import io.rpg.model.object.Question;
 import io.rpg.view.popups.DialoguePopup;
 import io.rpg.view.popups.QuestionPopup;
+import io.rpg.model.data.Inventory;
+import io.rpg.view.InventoryPopup;
 import io.rpg.view.popups.TextImagePopup;
 import io.rpg.view.popups.TextPopup;
 import javafx.scene.image.Image;
@@ -50,6 +52,19 @@ public void openTextImagePopup(String text, Image image, int x, int y){
   public void openPointsPopup(int pointsCount, int x, int y) {
     openTextImagePopup("You earned " + pointsCount + " points!", coinImage, x, y);
   }
+  public void openInventoryPopup(Inventory inventory,int x, int y) {
+//    InventoryPopup inventoryPopup=new InventoryPopup();
+//    final Stage popupStage = new Stage(StageStyle.TRANSPARENT);
+
+    InventoryPopup popupScene = new InventoryPopup(inventory);
+    popupStage.setScene(popupScene);
+
+    popupStage.show();
+
+    popupStage.setX(x - popupScene.getWidth() / 2);
+    popupStage.setY(y - popupScene.getHeight() / 2);
+
+  }
 
   public void openQuestionPopup(Question question, int x, int y, Runnable successCallback, Runnable failureCallback) {
     QuestionPopup popupScene = new QuestionPopup(question);

src/main/java/io/rpg/model/actions/ActionType.java

@@ -21,7 +21,10 @@ public enum ActionType {
   Dialogue("dialogue"),
 
   @SerializedName("battle")
-  Battle("battle");
+  Battle("battle"),
+
+  @SerializedName("collect")
+  Collect("collect");
 
   private final String asString;
 
@@ -41,6 +44,7 @@ public static Optional<ActionType> fromString(String action) {
       case "show-description" -> { return Optional.of(ShowDescription); }
       case "dialogue" -> { return Optional.of(Dialogue); }
       case "battle" -> { return Optional.of(Battle); }
+      case "collect" -> { return Optional.of(Collect); }
       default -> { return Optional.empty(); }
     }
   }

src/main/java/io/rpg/model/actions/CollectAction.java

@@ -0,0 +1,16 @@
+package io.rpg.model.actions;
+
+import io.rpg.model.object.GameObject;
+
+public class CollectAction implements Action {
+  private GameObject owner;
+
+  public void setOwner(GameObject owner) {
+    this.owner = owner;
+  }
+
+  public GameObject getOwner() {
+    return owner;
+  }
+}
+

src/main/java/io/rpg/model/data/Inventory.java

@@ -0,0 +1,19 @@
+package io.rpg.model.data;
+
+import io.rpg.model.object.GameObject;
+
+import java.util.ArrayList;
+import java.util.List;
+
+public class Inventory {
+
+  public List<GameObject> items;
+
+  public Inventory() {
+    items = new ArrayList<>();
+  }
+
+  public void add(GameObject object) {
+    items.add(object);
+  }
+}

src/main/java/io/rpg/model/object/CollectibleGameObject.java

@@ -4,8 +4,8 @@
 import org.jetbrains.annotations.NotNull;
 
 public final class CollectibleGameObject extends InteractiveGameObject {
-  public CollectibleGameObject(@NotNull String tag, @NotNull Position position) {
-    super(tag, position);
+  public CollectibleGameObject(@NotNull String tag, @NotNull Position position, String assetPath) {
+    super(tag, position, assetPath);
   }
 
   @Override

src/main/java/io/rpg/model/object/GameObject.java

@@ -130,4 +130,5 @@ public void onLeftClick() {
   public int getStrength() {
     return strength;
   }
+
 }

src/main/java/io/rpg/model/object/InteractiveGameObject.java

@@ -4,6 +4,9 @@
 import org.jetbrains.annotations.NotNull;
 
 public abstract class InteractiveGameObject extends GameObject {
+  public InteractiveGameObject(@NotNull String tag, @NotNull Position position,String assetPath) {
+    super(tag, position);
+  }
   public InteractiveGameObject(@NotNull String tag, @NotNull Position position) {
     super(tag, position);
   }