Fix bugs with connection edges.

lang/en/home.php

@@ -27,7 +27,7 @@
     "<select id=\"enumVertexsText\">" .
     "</select></span>";
     $g_lang["select_first_vertext_to_connect"] = "Select first vertex of edge";
-    $g_lang["select_second_vertext_to_connect"] = "Select second vertext of edge";
+    $g_lang["select_second_vertext_to_connect"] = "Select second vertex of edge";
     $g_lang["select_start_short_path_vertex"] = "Select the initial vertex of the shortest path";
     $g_lang["select_finish_short_path_vertex"] = "Select the end vertex of the shortest path";
     $g_lang["short_path_result"] = "Shortest path length is %d";

script/EventHandlers.js

@@ -1058,60 +1058,15 @@ ConnectionGraphHandler.prototype.SelectFirst = function()
 {
 	this.firstObject = null;
     
-    let hasEdges           = this.app.graph.hasEdges();
-    let hasDirectedEdges   = this.app.graph.hasDirectEdge();
-    let hasUndirectedEdges = this.app.graph.hasUndirectEdge();
-
 	this.message     = g_selectFirstVertexToConnect + this.GetSelect2VertexMenu();
-
+    this.message = this.AppendSpecialSctionsButton(this.message);
-    if (!hasEdges) {
-        return;
-    }
-
-    this.message =
-        ". <div class=\"btn-group\" style=\"float:right; position: relative; margin-left: 8px\">"
-        +  "<button type=\"button\" class=\"btn btn-default btn-sm dropdown-toggle\" data-toggle=\"dropdown\" aria-haspopup=\"true\" aria-expanded=\"false\">"
-        +  g_additionalActions + " <span class=\"caret\"></span>"
-        +  " </button> "
-        +  "<ul class=\"dropdown-menu dropdown-menu-right\" style=\"z-index:15; position: absolute;\">"
-        +  (hasDirectedEdges ? " <li><a href=\"#\" id=\"reverseAll\">" + g_reverseAllEdges + "</a></li>" : "")
-        +  (hasDirectedEdges ? " <li><a href=\"#\" id=\"makeAllUndirected\">" + g_makeAllUndirected + "</a></li>" : "")
-        +  (hasUndirectedEdges ? " <li><a href=\"#\" id=\"makeAllDirected\">" + g_makeAllDirected + "</a></li>" : "")
-        +  "</ul>"
-        +  "</div> " + this.message;
-
-    let handler = this;
-
-    $('#message').on('click', '#reverseAll', function() {
-            handler.app.PushToStack("ReverseAllEdges");
-
-            handler.app.graph.reverseAllEdges();
-            handler.app.redrawGraph();
-
-            userAction("ReverseAllEdges");
-        });
-    $('#message').on('click', '#makeAllUndirected', function(){        
-        handler.app.PushToStack("makeAllEdgesUndirected");
-
-        handler.app.graph.makeAllEdgesUndirected();
-        handler.app.redrawGraph();
-
-        userAction("makeAllEdgesUndirected");
-    });
-    $('#message').on('click', '#makeAllDirected', function(){        
-        handler.app.PushToStack("makeAllEdgesDirected");
-
-        handler.app.graph.makeAllEdgesDirected();
-        handler.app.redrawGraph();
-
-        userAction("makeAllEdgesDirected");
-    });        
 }
 
 ConnectionGraphHandler.prototype.SelectSecond = function(selectedObject)
 {
 	this.firstObject = selectedObject;
-	this.message     = g_selectSecondVertexToConnect + this.GetSelect2VertexMenu();						
+	this.message     = g_selectSecondVertexToConnect + this.GetSelect2VertexMenu();		
+    this.message = this.AppendSpecialSctionsButton(this.message);				
 }
 
 ConnectionGraphHandler.prototype.SelectFirstVertexMenu = function(vertex1Text, vertex)
@@ -1141,6 +1096,55 @@ ConnectionGraphHandler.prototype.UpdateSecondVertexMenu = function(vertex2Text)
     }   
 }
 
+ConnectionGraphHandler.prototype.AppendSpecialSctionsButton = function(baseMessage)
+{
+    let hasEdges           = this.app.graph.hasEdges();
+
+    if (!hasEdges) return baseMessage;
+
+    let hasDirectedEdges   = this.app.graph.hasDirectEdge();
+    let hasUndirectedEdges = this.app.graph.hasUndirectEdge();
+
+    let handler = this;
+
+    $('#message').on('click', '#reverseAll', function() {
+            handler.app.PushToStack("ReverseAllEdges");
+
+            handler.app.graph.reverseAllEdges();
+            handler.app.redrawGraph();
+
+            userAction("ReverseAllEdges");
+        });
+    $('#message').on('click', '#makeAllUndirected', function(){        
+        handler.app.PushToStack("makeAllEdgesUndirected");
+
+        handler.app.graph.makeAllEdgesUndirected();
+        handler.app.redrawGraph();
+
+        userAction("makeAllEdgesUndirected");
+    });
+    $('#message').on('click', '#makeAllDirected', function(){        
+        handler.app.PushToStack("makeAllEdgesDirected");
+
+        handler.app.graph.makeAllEdgesDirected();
+        handler.app.redrawGraph();
+
+        userAction("makeAllEdgesDirected");
+    });     
+        
+    return "<div class=\"btn-group\" style=\"float:right; position: relative; margin-left: 8px\">"
+    +  "<button type=\"button\" class=\"btn btn-default btn-sm dropdown-toggle\" data-toggle=\"dropdown\" aria-haspopup=\"true\" aria-expanded=\"false\">"
+    +  g_additionalActions + " <span class=\"caret\"></span>"
+    +  " </button> "
+    +  "<ul class=\"dropdown-menu dropdown-menu-right\" style=\"z-index:15; position: absolute;\">"
+    +  (hasDirectedEdges ? " <li><a href=\"#\" id=\"reverseAll\">" + g_reverseAllEdges + "</a></li>" : "")
+    +  (hasDirectedEdges ? " <li><a href=\"#\" id=\"makeAllUndirected\">" + g_makeAllUndirected + "</a></li>" : "")
+    +  (hasUndirectedEdges ? " <li><a href=\"#\" id=\"makeAllDirected\">" + g_makeAllDirected + "</a></li>" : "")
+    +  "</ul>"
+    +  "</div> " + baseMessage;
+}
+
+
 /**
  * Delete Graph handler.
  *

tpl/home.php

@@ -11,7 +11,7 @@
         <script src="<?= Root('i/js/dev/jquery-ui.js')?>"></script>
 	    <script src="<?= Root('i/js/dev/jquery.feedback_me.js')?>"></script>
       <script src="<?= Root("script/canvas2svg.js")?>" ></script>
-        <script src="<?= Root("script/example.js?v=69")?>" ></script>
+        <script src="<?= Root("script/example.js?v=70")?>" ></script>
 
 <!-- Yandex.RTB -->
 <script>window.yaContextCb=window.yaContextCb||[]</script>