search

anselb / activity-planner

A simple web app for creating itineraries of locations to visit from yelp
2 stars 7 forks source link

Poor JS code that causes Call Stack issue #40

Closed Jeffchiucp closed 6 years ago

Jeffchiucp commented 6 years ago

https://github.com/anselb/activity-planner/blob/99f1288898197a7e6aaf8ced044d4ff72bb11686/public/javascripts/js-map.js#L4

https://stackoverflow.com/questions/6095530/maximum-call-stack-size-exceeded-error

Jeffchiucp commented 6 years ago

Issue persists for JS: Uncaught ReferenceError: infoWindow is not defined at js-map.js:77

screen shot 2018-01-17 at 7 33 52 pm
Jeffchiucp commented 6 years ago

error log: Uncaught RangeError: Maximum call stack size exceeded at Xc (js?key=AIzaSyA098tlCp-gUAnSgWAst8CXHWX2VYblE-o&callback=initMap:54) at Yy..m.get (js?key=AIzaSyA098tlCp-gUAnSgWAst8CXHWX2VYblE-o&callback=initMap:124) at Zy (map.js:18) at Yy..m.zoomchanged (map.js:41) at Uc (js?key=AIzaSyA098tlCp-gUAnSgWAst8CXHWX2VYblE-o&callback=initMap:53) at Yy..m.set (js?key=AIzaSyA098tlCp-gUAnSgWAst8CXHWX2VYblE-o&callback=initMap:124) at Zy (map.js:18) at Yy._.m.zoomchanged (map.js:41) at Uc (js?key=AIzaSyA098tlCp-gUAnSgWAst8CXHWX2VYblE-o&callback=initMap:53) at Yy..m.set (js?key=AIzaSyA098tlCp-gUAnSgWAst8CXHWX2VYblE-o&callback=initMap:124) js?key=AIzaSyA098tlCp-gUAnSgWAst8CXHWX2VYblE-o&callback=initMap:54 Uncaught RangeError: Maximum call stack size exceeded at Object..Wc (js?key=AIzaSyA098tlCp-gUAnSgWAst8CXHWX2VYblE-o&callback=initMap:54) at Yy..m.get (js?key=AIzaSyA098tlCp-gUAnSgWAst8CXHWX2VYblE-o&callback=initMap:124) at Zy (map.js:18) at Yy._.m.zoomchanged (map.js:41) at Uc (js?key=AIzaSyA098tlCp-gUAnSgWAst8CXHWX2VYblE-o&callback=initMap:53) at Yy..m.set (js?key=AIzaSyA098tlCp-gUAnSgWAst8CXHWX2VYblE-o&callback=initMap:124) at Zy (map.js:18) at Yy._.m.zoomchanged (map.js:41) at Uc (js?key=AIzaSyA098tlCp-gUAnSgWAst8CXHWX2VYblE-o&callback=initMap:53) at Yy..m.set (js?key=AIzaSyA098tlCp-gUAnSgWAst8CXHWX2VYblE-o&callback=initMap:124)

Jeffchiucp commented 6 years ago

correctly handled the JS by enforcing HTTPS://