From 35fa1ff22415e3c941adde61f47ec8331e294c3d Mon Sep 17 00:00:00 2001
From: Jonathan Redpath <legoboyvdlp@gmail.com>
Date: Tue, 25 Jan 2022 09:42:09 +0000
Subject: [PATCH] Revert "Better long pressed CLR and check numeric format on
 plus/minus"

This reverts commit 871c645fbaa0f689cb3a2f753fc2bc94eed366e1.
---
 Models/Instruments/MCDU/MCDU1.xml | 31 ++++------------------
 Models/Instruments/MCDU/MCDU2.xml | 33 +++++-------------------
 Nasal/MCDU/MCDU.nas               | 43 +++++++++++++++++++------------
 3 files changed, 38 insertions(+), 69 deletions(-)

diff --git a/Models/Instruments/MCDU/MCDU1.xml b/Models/Instruments/MCDU/MCDU1.xml
index 54116e98..2133e61a 100644
--- a/Models/Instruments/MCDU/MCDU1.xml
+++ b/Models/Instruments/MCDU/MCDU1.xml
@@ -1678,7 +1678,7 @@
 		<object-name>overfly</object-name>
 		<action>
 			<button>0</button>
-			<repeatable>false</repeatable>
+			<repeatable>true</repeatable>
 			<binding>
 				<command>nasal</command>
 				<script>mcdu.button("OVFY", 0);</script>
@@ -1703,23 +1703,11 @@
 		<action>
 			<button>0</button>
 			<repeatable>true</repeatable>
-			<binding>
-				<command>property-adjust</command>
-				<property>/MCDU[0]/clrbtn-timeout</property>
-				<step>1</step>
-				<min>0</min>
-				<max>6</max>
-				<wrap>false</wrap>
-			</binding>
 			<binding>
 				<command>nasal</command>
-				<script>mcdu.button("LONGCLR", 0);</script>
+				<script>mcdu.button("CLR", 0, "down");</script>
 				<condition>
 					<and>
-						<equals>
-							<property>/MCDU[0]/clrbtn-timeout</property>
-							<value>5</value>
-						</equals>
 						<greater-than-equals>
 							<property>systems/electrical/bus/ac-1</property>
 							<value>110</value>
@@ -1730,21 +1718,17 @@
 						</greater-than>
 					</and>
 				</condition>
-			</binding>					
+			</binding>
 			<mod-up>
 				<binding>
 					<command>nasal</command>
-					<script>mcdu.button("CLR", 0);</script>
+					<script>mcdu.button("CLR", 0, "up");</script>
 					<condition>
 						<and>
-							<less-than>
-								<property>/MCDU[0]/clrbtn-timeout</property>
-								<value>5</value>
-							</less-than>
 							<greater-than-equals>
 								<property>systems/electrical/bus/ac-1</property>
 								<value>110</value>
-							</greater-than-equals>						
+							</greater-than-equals>
 							<greater-than>
 								<property>controls/lighting/DU/mcdu1</property>
 								<value>0.01</value>
@@ -1752,11 +1736,6 @@
 						</and>
 					</condition>
 				</binding>
-				<binding>
-					<command>property-assign</command>
-					<property>/MCDU[0]/clrbtn-timeout</property>
-					<value>0</value>
-				</binding>
 			</mod-up>
 		</action>
 	</animation>
diff --git a/Models/Instruments/MCDU/MCDU2.xml b/Models/Instruments/MCDU/MCDU2.xml
index 6da38c5c..d49c0a8d 100644
--- a/Models/Instruments/MCDU/MCDU2.xml
+++ b/Models/Instruments/MCDU/MCDU2.xml
@@ -1678,7 +1678,7 @@
 		<object-name>overfly</object-name>
 		<action>
 			<button>0</button>
-			<repeatable>false</repeatable>
+			<repeatable>true</repeatable>
 			<binding>
 				<command>nasal</command>
 				<script>mcdu.button("OVFY", 1);</script>
@@ -1703,48 +1703,32 @@
 		<action>
 			<button>0</button>
 			<repeatable>true</repeatable>
-			<binding>
-				<command>property-adjust</command>
-				<property>/MCDU[1]/clrbtn-timeout</property>
-				<step>1</step>
-				<min>0</min>
-				<max>6</max>
-				<wrap>false</wrap>
-			</binding>
 			<binding>
 				<command>nasal</command>
-				<script>mcdu.button("LONGCLR", 1);</script>
+				<script>mcdu.button("CLR", 1, "down");</script>
 				<condition>
 					<and>
-						<equals>
-							<property>/MCDU[1]/clrbtn-timeout</property>
-							<value>5</value>
-						</equals>
 						<greater-than-equals>
 							<property>systems/electrical/bus/ac-2</property>
 							<value>110</value>
-						</greater-than-equals>					
+						</greater-than-equals>
 						<greater-than>
 							<property>controls/lighting/DU/mcdu2</property>
 							<value>0.01</value>
 						</greater-than>
 					</and>
 				</condition>
-			</binding>			
+			</binding>
 			<mod-up>
 				<binding>
 					<command>nasal</command>
-					<script>mcdu.button("CLR", 1);</script>
+					<script>mcdu.button("CLR", 1, "up");</script>
 					<condition>
 						<and>
-							<less-than>
-								<property>/MCDU[1]/clrbtn-timeout</property>
-								<value>5</value>
-							</less-than>
 							<greater-than-equals>
 								<property>systems/electrical/bus/ac-2</property>
 								<value>110</value>
-							</greater-than-equals>									
+							</greater-than-equals>
 							<greater-than>
 								<property>controls/lighting/DU/mcdu2</property>
 								<value>0.01</value>
@@ -1752,11 +1736,6 @@
 						</and>
 					</condition>
 				</binding>
-				<binding>
-					<command>property-assign</command>
-					<property>/MCDU[1]/clrbtn-timeout</property>
-					<value>0</value>
-				</binding>
 			</mod-up>
 		</action>
 	</animation>
diff --git a/Nasal/MCDU/MCDU.nas b/Nasal/MCDU/MCDU.nas
index 160b8831..2ecd3c93 100644
--- a/Nasal/MCDU/MCDU.nas
+++ b/Nasal/MCDU/MCDU.nas
@@ -1499,6 +1499,8 @@ var pagebutton = func(btn, i) {
 	}
 }
 
+var buttonCLRDown = [0,0]; # counter for down event
+
 var button = func(btn, i, event = "") {
 	page = pageNode[i].getValue();
 	if (page != "MCDU") {
@@ -1508,32 +1510,41 @@ var button = func(btn, i, event = "") {
 		} else if (btn == "SP") {
 			mcdu_scratchpad.scratchpads[i].addChar(" ");
 		} else if (btn == "CLR") {
-			if (size(scratchpad) == 0) {
-				mcdu_scratchpad.scratchpads[i].addChar("CLR");
-			} else {
-				mcdu_scratchpad.scratchpads[i].clear();
+			if (event == "down") {
+				if (size(scratchpad) > 0) {
+					if (buttonCLRDown[i] > 4) {
+						mcdu_scratchpad.scratchpads[i].empty();
+					}
+					buttonCLRDown[i] = buttonCLRDown[i] + 1;
+				}
+			}
+			else if (event == "" or buttonCLRDown[i]<=4) {
+				buttonCLRDown[i] = 0;
+				#var scratchpad = mcdu_scratchpad.scratchpads[i].scratchpad;  <- useless??
+				if (size(scratchpad) == 0) {
+					mcdu_scratchpad.scratchpads[i].addChar("CLR");
+				} else {
+					mcdu_scratchpad.scratchpads[i].clear();
+				}
+			} else {  # up with buttonCLRDown[i]>4
+				buttonCLRDown[i] = 0;
 			}
 		} else if (btn == "LONGCLR") {
 			mcdu_scratchpad.scratchpads[i].empty();
 		} else if (btn == "DOT") {
 			mcdu_scratchpad.scratchpads[i].addChar(".");
 		} else if (btn == "PLUSMINUS") {
-			if (size(scratchpad)==0) {
-				mcdu_message(i, "NOT ALLOWED");
-			} else if (isint(scratchpad)==1) {
+			if (right(mcdu_scratchpad.scratchpads[i].scratchpad, 1) == "-") {
+				mcdu_scratchpad.scratchpads[i].clear();
+				mcdu_scratchpad.scratchpads[i].addChar("+");
+			} else if (right(mcdu_scratchpad.scratchpads[i].scratchpad, 1) == "+") {
+				mcdu_scratchpad.scratchpads[i].clear();
 				mcdu_scratchpad.scratchpads[i].addChar("-");
 			} else {
-				var _toggle = right(scratchpad,1);
-				if (find(_toggle,"-+")!=-1) {
-					_toggle = (_toggle == "-") ? "+" : "-";
-					mcdu_scratchpad.scratchpads[i].clear();
-					mcdu_scratchpad.scratchpads[i].addChar(_toggle);
-				} else {
-					mcdu_message(i, "NOT ALLOWED");
-				}
+				mcdu_scratchpad.scratchpads[i].addChar("-");
 			}
 		} else if (btn == "OVFY") {
-			if (size(scratchpad)==0) {
+			if (mcdu_scratchpad.scratchpads[i].scratchpad == "") {
 				mcdu_scratchpad.scratchpads[i].addChar("@");
 			} else {
 				mcdu_message(i, "NOT ALLOWED");