From bc5169fbf0f5cc21aefdb55a2125b4d52bed9d0e Mon Sep 17 00:00:00 2001
From: Silver Kuusik <silver.kuusik@gmail.com>
Date: Mon, 15 Jan 2018 15:09:54 +0100
Subject: [PATCH] fix disabling control_if block

---
 index.html   | 2 +-
 sumorobot.js | 8 +++-----
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/index.html b/index.html
index 8198981..cf32b91 100755
--- a/index.html
+++ b/index.html
@@ -75,7 +75,7 @@
     </xml>
 
     <xml id="toolbox_no_if" style="display: none">
-        <block type="controls_if"></block>
+        <block type="controls_if" disabled="true"></block>
         <block type="sumorobot_move"></block>
         <block type="sumorobot_delay"></block>
         <block type="sumorobot_enemy"></block>
diff --git a/sumorobot.js b/sumorobot.js
index be09058..bebd61a 100755
--- a/sumorobot.js
+++ b/sumorobot.js
@@ -295,6 +295,8 @@ window.onload = function() {
         } else if (event.type == Blockly.Events.DELETE && event.oldXml.getAttributeNode("type").nodeValue == "controls_if") {
             /* remove the control_if block id */
             controlBlockId = "";
+            /* enable the if condition block */
+            workspace.updateToolbox(document.getElementById("toolbox"));
         }
 
         /* only process change and move commands */
@@ -309,14 +311,10 @@ window.onload = function() {
         var xml = Blockly.Xml.workspaceToDom(workspace);
         localStorage.setItem("sumorobot.currentProgram", Blockly.Xml.domToText(xml));
 
-        /* if control_if is used*/
+        /* if control_if block is used */
         if (controlBlockId != "") {
             /* disable the if condition block */
             workspace.updateToolbox(document.getElementById("toolbox_no_if"));
-        /* if no control_if is used */
-        } else {
-            /* enable the if condition block */
-            workspace.updateToolbox(document.getElementById("toolbox"));
         }
     }