Map: engagement is ok for solo and AI but not for network

1 files changed, 2 insertions, 3 deletions

diff --git a/core/src/ch/asynk/tankontank/game/Map.java b/core/src/ch/asynk/tankontank/game/Map.java
index 7d4bf94..7d22dd8 100644
--- a/core/src/ch/asynk/tankontank/game/Map.java
+++ b/core/src/ch/asynk/tankontank/game/Map.java
@@ -342,6 +342,8 @@ public abstract class Map extends Board implements MoveToAnimationCb, ObjectiveS
                 r = promoteUnit(cmd.unit, cmd.player);
                 break;
             case ENGAGE:
+                // FIXME having the dice roll here does not fit networking game
+                resolveEngagement(cmd.engagement);
                 r = doEngagement(cmd.engagement);
                 break;
             default:
@@ -438,9 +440,6 @@ public abstract class Map extends Board implements MoveToAnimationCb, ObjectiveS
 
         Command cmd = Command.get(ctrl.player);
         cmd.setEngage(unit, target);
-
-        // FIXME resolve the engagement here does not work for the AI
-        resolveEngagement(cmd.engagement);
         return (process(cmd) == 1);
     }