fix(routes): connect new burg to land route only

modules/burgs-generator.js

@@ -547,14 +547,7 @@ window.Burgs = (() => {
     cells.burg[cellId] = burgId;
 
     const newRoute = Routes.connect(cellId);
-    if (newRoute && layerIsOn("toggleRoutes")) {
+    if (newRoute && layerIsOn("toggleRoutes")) drawRoute(newRoute);
-      const path = Routes.getPath(newRoute);
-      routes
-        .select("#" + newRoute.group)
-        .append("path")
-        .attr("d", path)
-        .attr("id", "route" + newRoute.i);
-    }
 
     drawBurgIcon(burg);
     drawBurgLabel(burg);

modules/routes-generator.js

@@ -360,7 +360,8 @@ window.Routes = (function () {
   // connect cell with routes system by land
   function connect(cellId) {
     const getCost = createCostEvaluator({isWater: false, connections: new Map()});
-    const pathCells = findPath(cellId, isConnected, getCost);
+    const isExit = cellId => isLand(cellId) && isConnected(cellId);
+    const pathCells = findPath(cellId, isExit, getCost);
     if (!pathCells) return;
 
     const pointsArray = preparePointsArray();

modules/ui/layers.js

@@ -845,6 +845,14 @@ function drawRoutes() {
   TIME && console.timeEnd("drawRoutes");
 }
 
+function drawRoute(route) {
+  routes
+    .select("#" + route.group)
+    .append("path")
+    .attr("d", Routes.getPath(route))
+    .attr("id", "route" + route.i);
+}
+
 function toggleMilitary(event) {
   if (!layerIsOn("toggleMilitary")) {
     turnButtonOn("toggleMilitary");