Reduce complexity of recursive selection

Mark Robinson · Mark Robinson · commit f6da31493982 · 2017-03-08T14:36:13.000Z
diff --git a/src/main/resources/static/js/workflow.js b/src/main/resources/static/js/workflow.js
@@ -377,9 +377,11 @@ require(['jquery', 'jquery.svg', 'jquery.svgdom'],
             var rootID = root.attr("id");
             for (var i = 0; i < graphModel[rootID][listName].length; i++) {
                 var next = $("#" + graphModel[rootID][listName][i]);
-                next.find("polygon").addClass("selected");
-                getTableRow(next).addClass("selected");
-                expandSelection(next, listName);
+                if (!next.find("polygon").hasClass("selected")) {
+                    next.find("polygon").addClass("selected");
+                    getTableRow(next).addClass("selected");
+                    expandSelection(next, listName);
+                }
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -377,9 +377,11 @@ require(['jquery', 'jquery.svg', 'jquery.svgdom'],`
`377`	`377`	`var rootID = root.attr("id");`
`378`	`378`	`for (var i = 0; i < graphModel[rootID][listName].length; i++) {`
`379`	`379`	`var next = $("#" + graphModel[rootID][listName][i]);`
`380`		`- next.find("polygon").addClass("selected");`
`381`		`- getTableRow(next).addClass("selected");`
`382`		`- expandSelection(next, listName);`
	`380`	`+ if (!next.find("polygon").hasClass("selected")) {`
	`381`	`+ next.find("polygon").addClass("selected");`
	`382`	`+ getTableRow(next).addClass("selected");`
	`383`	`+ expandSelection(next, listName);`
	`384`	`+ }`
`383`	`385`	`}`
`384`	`386`	`}`
`385`	`387`