Don't restore the viewmodel on reload

Also added a check if the history API is avaiable
riganti · Apr 26, 2019 · f97b2e1 · f97b2e1
1 parent ccdbfb3
commit f97b2e1
Show file tree

Hide file tree

Showing 3 changed files with 84 additions and 42 deletions.
diff --git a/src/DotVVM.Framework/Resources/Scripts/DotVVM.js b/src/DotVVM.Framework/Resources/Scripts/DotVVM.js
diff --git a/src/DotVVM.Framework/Resources/Scripts/DotVVM.min.js b/src/DotVVM.Framework/Resources/Scripts/DotVVM.min.js
diff --git a/src/DotVVM.Framework/Resources/Scripts/DotVVM.ts b/src/DotVVM.Framework/Resources/Scripts/DotVVM.ts
@@ -220,11 +220,30 @@ class DotVVM {
     public useHistoryApiSpaNavigation: boolean;
     public isPostbackRunning = ko.observable(false);
 
+    private isBrowserReload(): boolean {
+        if (performance) {
+            if (performance.getEntriesByType) {
+                var entries = performance.getEntriesByType("navigation");
+
+                if (entries.length > 0) {
+                    return (<PerformanceNavigationTiming>entries[0]).type === "reload";
+                }
+            }
+
+            // deprecated in Navigation Timing Level 2 specification
+            if (performance.navigation) {
+                return performance.navigation.type === 1;
+            }
+        }
+
+        return false;
+    }
+
     public init(viewModelName: string, culture: string, thisViewModel?: IDotvvmViewModelInfo): void {
         this.addKnockoutBindingHandlers();
 
         // restore the viewmodel from the history API.
-        if (history.state && history.state.dotvvm_viewmodels && history.state.dotvvm_viewmodels[viewModelName]) {
+        if (!this.isBrowserReload() && history && history.state && history.state.dotvvm_viewmodels && history.state.dotvvm_viewmodels[viewModelName]) {
             // create a new object, otherwise the object in the history state will be changed which will result in serialize errors.
             thisViewModel = <IDotvvmViewModelInfo>{ ...history.state.dotvvm_viewmodels[viewModelName] };
         }
@@ -361,20 +380,28 @@ class DotVVM {
     }
 
     private persistViewModel(viewModelName: string) {
-        var currentState = history.state ? history.state : {};
-        var persistedViewModels = currentState.dotvvm_viewmodels ? currentState.dotvvm_viewmodels : {};
         var thisViewModel = ko.toJS(this.viewModels[viewModelName]);
 
-        // add the new viewmodel to the existing state, otherwise SPA mode will break.
-        var state = {
-            dotvvm_viewmodels: {
-                [viewModelName]: thisViewModel,
-                ...persistedViewModels
-            },
-            ...currentState
-        };
+        if (history) {
+            var currentState = history.state ? history.state : {};
+            var persistedViewModels = currentState.dotvvm_viewmodels ? currentState.dotvvm_viewmodels : {};
 
-        history.replaceState(state, document.title);
+            // add the new viewmodel to the existing state, otherwise SPA mode will break.
+            var state = {
+                dotvvm_viewmodels: {
+                    [viewModelName]: thisViewModel,
+                    ...persistedViewModels
+                },
+                ...currentState
+            };
+
+            history.replaceState(state, document.title);
+        }
+
+        var viewModelInput = <HTMLInputElement>document.getElementById("__dot_viewmodel_" + viewModelName);
+        if (viewModelInput) {
+            viewModelInput.value = JSON.stringify(thisViewModel);
+        }
     }
 
     private backUpPostBackConter(): number {