Fix an NPE with user action attachments introduced by recent refactoring. (#10503)

asvitkine · web-flow · commit aaaaf1d8060e · 2022-05-25T11:31:41.000-04:00
diff --git a/game-app/game-core/src/main/java/games/strategy/triplea/attachments/UserActionAttachment.java b/game-app/game-core/src/main/java/games/strategy/triplea/attachments/UserActionAttachment.java
@@ -160,7 +160,7 @@ public static void fireTriggers(
   public Set<GamePlayer> getOtherPlayers() {
     final Set<GamePlayer> otherPlayers = new HashSet<>();
     otherPlayers.add((GamePlayer) this.getAttachedTo());
-    otherPlayers.addAll(actionAccept);
+    otherPlayers.addAll(getActionAccept());
     return otherPlayers;
   }
 

Original file line number	Diff line number	Diff line change
`@@ -160,7 +160,7 @@ public static void fireTriggers(`
`160`	`160`	`public Set<GamePlayer> getOtherPlayers() {`
`161`	`161`	`final Set<GamePlayer> otherPlayers = new HashSet<>();`
`162`	`162`	`otherPlayers.add((GamePlayer) this.getAttachedTo());`
`163`		`- otherPlayers.addAll(actionAccept);`
	`163`	`+ otherPlayers.addAll(getActionAccept());`
`164`	`164`	`return otherPlayers;`
`165`	`165`	`}`
`166`	`166`