fix waypoint colors not updating on config change

thebuildcraft · Jun 27, 2024 · 8c431fc · 8c431fc
1 parent e994527
commit 8c431fc
Showing 1 changed file with 19 additions and 0 deletions.
diff --git a/...src/main/java/de/the_build_craft/remote_player_waypoints_for_xaero/common/UpdateTask.java b/...src/main/java/de/the_build_craft/remote_player_waypoints_for_xaero/common/UpdateTask.java
@@ -83,6 +83,9 @@ public UpdateTask() {
     private ArrayList<Waypoint> playerWaypointList = null;
     private ArrayList<Waypoint> markerWaypointList = null;
 
+    private int previousPlayerWaypointColor = 0;
+    private int previousMarkerWaypointColor = 0;
+
     @Override
     public void run() {
         try{
@@ -338,6 +341,14 @@ public void runUpdate() {
                     }
                     // Remove any waypoints for players not shown on the map anymore
                     playerWaypointList.removeIf(waypoint -> !currentPlayerWaypointNames.contains(waypoint.getName()));
+
+                    int newPlayerWaypointColor = CommonModConfig.Instance.playerWaypointColor();
+                    if (previousPlayerWaypointColor != newPlayerWaypointColor){
+                        previousPlayerWaypointColor = newPlayerWaypointColor;
+                        for (Waypoint waypoint : playerWaypointList){
+                            waypoint.setColor(newPlayerWaypointColor);
+                        }
+                    }
                 }
                 else {
                     playerWaypointList.clear();
@@ -395,6 +406,14 @@ public void runUpdate() {
                     // Remove any waypoints for markers not shown on the map anymore
                     markerWaypointList.removeIf(waypoint -> !currentMarkerWaypointNames.contains(waypoint.getName()));
 
+                    int newMarkerWaypointColor = CommonModConfig.Instance.markerWaypointColor();
+                    if (previousMarkerWaypointColor != newMarkerWaypointColor){
+                        previousMarkerWaypointColor = newMarkerWaypointColor;
+                        for (Waypoint waypoint : markerWaypointList){
+                            waypoint.setColor(newMarkerWaypointColor);
+                        }
+                    }
+
                     if (!markerMessageWasShown && currentMarkerWaypointNames.size() > maxMarkerCountBeforeWarning && !CommonModConfig.Instance.ignoreMarkerMessage()) {
                         markerMessageWasShown = true;
                         Utils.sendToClientChat(Text.literal("[" + AbstractModInitializer.MOD_NAME + "]: " +