still pretty print cmodel

Author: karussell

src/NavBar.ts

@@ -20,6 +20,7 @@ import { getQueryStore } from '@/stores/Stores'
 import { getBBoxFromCoord, getBBoxPoints } from '@/utils'
 import { decodeCoords, encodeCoords } from '@/util/flexPolyline'
 import { canCompress, deflateB64url, inflateB64url } from '@/util/urlCompress'
+import { customModel2prettyString } from '@/sidebar/CustomModelExamples'
 
 // Minimum number of (initialized) waypoints before we switch from the legible
 // `point=lat,lng` format to the compact `fpolyline=` representation. For 1-3
@@ -260,7 +261,13 @@ export default class NavBar {
 
         const customModelStr = await NavBar.parseCustomModel(url)
         if (customModelStr) {
-            Dispatcher.dispatch(new SetCustomModel(customModelStr, true))
+            // Pretty-print so the editor shows readable JSON (URL form is minified).
+            // Mirrors QueryStore.getInitialState behaviour for the initial-load case.
+            let prettyStr = customModelStr
+            try {
+                prettyStr = customModel2prettyString(JSON.parse(customModelStr))
+            } catch {}
+            Dispatcher.dispatch(new SetCustomModel(prettyStr, true))
             Dispatcher.dispatch(new SetCustomModelEnabled(true))
         }
 

test/NavBar.test.ts

@@ -292,7 +292,8 @@ describe('NavBar', function () {
             await navBar.updateStateFromUrl()
 
             expect(queryStore.state.customModelEnabled).toEqual(true)
-            expect(queryStore.state.customModelStr).toEqual(cm)
+            // stored value is pretty-printed, so compare JSON semantically
+            expect(JSON.parse(queryStore.state.customModelStr)).toEqual(JSON.parse(cm))
         })
 
         it('parses legacy custom_model= URL', async () => {
@@ -308,7 +309,7 @@ describe('NavBar', function () {
             await navBar.updateStateFromUrl()
 
             expect(queryStore.state.customModelEnabled).toEqual(true)
-            expect(queryStore.state.customModelStr).toEqual(cm)
+            expect(JSON.parse(queryStore.state.customModelStr)).toEqual(JSON.parse(cm))
         })
 
         it('sets no points when no points are in URL', async () => {