Skip to content

Commit e455de3

Browse files
committed
DBC22-4796: fixed context switching
1 parent d281418 commit e455de3

2 files changed

Lines changed: 22 additions & 3 deletions

File tree

src/frontend/src/Components/map/panels/RouteDetailsPanel.js

Lines changed: 22 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ export default function RouteDetailsPanel(props) {
2828
feeds: {
2929
events: { filteredList: filteredEvents },
3030
ferries: { filteredList: filteredFerries },
31+
wildfires: { filteredList: filteredWildfires },
3132
},
3233
advisories: { filteredList: filteredAdvisories },
3334
routes: { selectedRoute, searchedRoutes },
@@ -38,6 +39,7 @@ export default function RouteDetailsPanel(props) {
3839
feeds: {
3940
events: state.feeds.events,
4041
ferries: state.feeds.ferries,
42+
wildfires: state.feeds.wildfires,
4143
},
4244
advisories: state.cms.advisories,
4345
routes: state.routes,
@@ -50,30 +52,48 @@ export default function RouteDetailsPanel(props) {
5052
const [pendingAdvisories, setPendingAdvisories] = useState(false);
5153
const [pendingEvents, setPendingEvents] = useState(false);
5254
const [pendingFerries, setPendingFerries] = useState(false);
55+
const [pendingWildfires, setPendingWildfires] = useState(false);
5356

5457
// Effects
5558
// Mark data as not updating when they finish filtering
5659
useEffect(() => {
5760
setPendingAdvisories(true);
61+
if (!routeSwitched) {
62+
setRouteSwitched(true);
63+
}
5864
}, [filteredAdvisories]);
5965

6066
useEffect(() => {
6167
setPendingEvents(true);
68+
if (!routeSwitched) {
69+
setRouteSwitched(true);
70+
}
6271
}, [filteredEvents]);
6372

6473
useEffect(() => {
6574
setPendingFerries(true);
75+
if (!routeSwitched) {
76+
setRouteSwitched(true);
77+
}
6678
}, [filteredFerries]);
6779

80+
useEffect(() => {
81+
setPendingWildfires(true);
82+
if (!routeSwitched) {
83+
setRouteSwitched(true);
84+
}
85+
}, [filteredWildfires]);
86+
6887
// When all data is filtered, rank the object list again
6988
useEffect(() => {
70-
if (pendingAdvisories && pendingEvents && pendingFerries) {
89+
if (pendingAdvisories && pendingEvents && pendingFerries && pendingWildfires) {
7190
setPendingAdvisories(false);
7291
setPendingEvents(false);
7392
setPendingFerries(false);
93+
setPendingWildfires(false);
7494
setRouteSwitched(false);
7595
}
76-
}, [pendingAdvisories, pendingEvents, pendingFerries]);
96+
}, [pendingAdvisories, pendingEvents, pendingFerries, pendingWildfires]);
7797

7898
/* Rendering */
7999
// Sub components

src/frontend/src/Components/routing/RouteDetails.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -322,7 +322,6 @@ export default function RouteDetails(props) {
322322

323323
const switchRouteHandler = () => {
324324
if (!compareRoutes(route, selectedRoute)){
325-
setRouteSwitched(true);
326325
dispatch(updateSelectedRoute(route));
327326
}
328327
}

0 commit comments

Comments
 (0)