From 70a09659ead5f7ca2c050c957b43b85c039861b8 Mon Sep 17 00:00:00 2001
From: Dustin Carlino <dabreegster@gmail.com>
Date: Mon, 27 Sep 2021 12:35:22 -0700
Subject: [PATCH] Recalculate mode shift when map is edited #448

---
 game/src/app.rs           |  3 ++-
 game/src/ungap/predict.rs | 11 +++++------
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/game/src/app.rs b/game/src/app.rs
index 88a7cd5aba..07379943e0 100644
--- a/game/src/app.rs
+++ b/game/src/app.rs
@@ -733,7 +733,8 @@ pub struct SessionState {
     // Specific to the ungap tool
     pub elevation_contours: Cached<MapName, (FindClosest<Distance>, Drawable)>,
     pub routing_params: RoutingParams,
-    pub mode_shift: Cached<MapName, crate::ungap::ModeShiftData>,
+    // Map and edit change key
+    pub mode_shift: Cached<(MapName, usize), crate::ungap::ModeShiftData>,
 }
 
 impl SessionState {
diff --git a/game/src/ungap/predict.rs b/game/src/ungap/predict.rs
index dfa30b177d..ab62050d5c 100644
--- a/game/src/ungap/predict.rs
+++ b/game/src/ungap/predict.rs
@@ -30,9 +30,8 @@ impl TakeLayers for ShowGaps {
 impl ShowGaps {
     pub fn new_state(ctx: &mut EventCtx, app: &mut App, layers: Layers) -> Box<dyn State<App>> {
         let map_name = app.primary.map.get_name().clone();
-        if app.session.mode_shift.key().as_ref() == Some(&map_name) {
-            // TODO If the map's been edited, recalculate_gaps
-
+        let change_key = app.primary.map.get_edits_change_key();
+        if app.session.mode_shift.key().as_ref() == Some(&(map_name.clone(), change_key)) {
             return Box::new(ShowGaps {
                 top_panel: make_top_panel(ctx, app),
                 layers,
@@ -46,19 +45,19 @@ impl ShowGaps {
             // entirely?
             app.session
                 .mode_shift
-                .set(map_name, ModeShiftData::empty(ctx));
+                .set((map_name, change_key), ModeShiftData::empty(ctx));
             ShowGaps::new_state(ctx, app, layers)
         } else {
             FileLoader::<App, Scenario>::new_state(
                 ctx,
                 abstio::path_scenario(&map_name, &scenario_name),
-                Box::new(|ctx, app, _, maybe_scenario| {
+                Box::new(move |ctx, app, _, maybe_scenario| {
                     // TODO Handle corrupt files
                     let scenario = maybe_scenario.unwrap();
                     let data = ctx.loading_screen("predict mode shift", |ctx, timer| {
                         ModeShiftData::from_scenario(ctx, app, scenario, timer)
                     });
-                    app.session.mode_shift.set(map_name, data);
+                    app.session.mode_shift.set((map_name, change_key), data);
                     Transition::Replace(ShowGaps::new_state(ctx, app, layers))
                 }),
             )