Data layer also checks to see if it's legal before allowing updating.…

… Part of #10.
jkomoros · Nov 21, 2021 · 2827e51 · 2827e51
1 parent 72c7c55
commit 2827e51
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/src/actions/data.js b/src/actions/data.js
@@ -10,7 +10,8 @@ import {
 import {
 	selectFrameIndex,
 	selectRunIndex,
-	selectCurrentSimulationMaxRunIndex
+	selectCurrentSimulationMaxRunIndex,
+	selectCurrentSimulationRun
 } from '../selectors.js';
 
 export const loadData = (data) => {
@@ -41,8 +42,14 @@ export const updateSimulationIndex = (index) => (dispatch) => {
 	dispatch(canonicalizePath());
 };
 
-export const updateFrameIndex = (index) => (dispatch) => {
+export const updateFrameIndex = (index) => (dispatch, getState) => {
 	if (typeof index == 'string') index = parseInt(index);
+	const state = getState();
+	const run = selectCurrentSimulationRun(state);
+	const maxFrameIndex = run ? run.maxFrameIndex : Number.MAX_SAFE_INTEGER;
+	if (index > maxFrameIndex) index = maxFrameIndex;
+	//no op
+	if (index == selectFrameIndex(state)) return;
 	dispatch({
 		type: UPDATE_FRAME_INDEX,
 		index,