[openstreetmap/openstreetmap-website] Introduce "reload" js controller method (PR #3774)

Anton Khorev notifications at github.com
Mon Oct 31 13:05:24 UTC 2022


@AntonKhorev commented on this pull request.



> @@ -179,8 +187,20 @@ OSM.Router = function (map, rts) {
   };
 
   router.load = function () {
-    var loadState = currentRoute.run("load", currentPath);
+    var loadState = loadOrReload();

Are you asking why it's nested or why it's defined when there's only one caller?

The short answer is to initialize var loadState when it's declared.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/3774#discussion_r1009396726
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/3774/review/1161845109 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20221031/81d23277/attachment.htm>


More information about the rails-dev mailing list