AmericanRedCross / GeoForce

Making Salesforce spatial
BSD 3-Clause "New" or "Revised" License
13 stars 3 forks source link

Default Zoom,lat,lng layers #274

Open apollolm opened 9 years ago

apollolm commented 9 years ago

Sometimes, if local storage is cleared, it's possible to have the app start up with a State of:

http:///mapfolio/#/map@,,(),

should be: http:///mapfolio/#/map@11.178402,-58.798828,3(mozilla,gadm0),project

hallahan commented 9 years ago

Something is not right here:

https://github.com/AmericanRedCross/GeoForce/blob/master/GeoAngular/app/scripts/app.js#L101

My guess is that the second statement in the || is not being reached. We need to check to see what we have when we call:

localStorage.getItem('defaultRoute')

after empty local storage. Then, correctly check for that. I'm assuming that it is not null or undefined, thus the problem you are seeing.

apollolm commented 9 years ago

Thanks Nick. I think the problem may also have to do with bad states making it into local storage. This morning, I switched to the older version of Mapfolio to test some stuff. When I started up the older version, it initially brought me to this bad state, which may have been subsequently saved into local storage. When I switched back to the updated Mapfolio, that's what was saved in local storage. But this bad state seems to happen to me every once in awhile, and especially when I run the project from WebStorm (where I start the node process and then open a browser to localhost:3001/Mapfolio.

dalekunce commented 9 years ago

I haven't seen this bug.