From 9b4dfe89db64da656efeed63ee1bf1281d0101d9 Mon Sep 17 00:00:00 2001
From: cyberarm <matthewlikesrobots@gmail.com>
Date: Thu, 8 Oct 2020 19:19:09 -0500
Subject: [PATCH] Allow signed values for variable values, added error message
 when saving an integer/long with a decimcal value

---
 .../dialogs/VariableDialog.java                      | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/app/src/main/java/org/timecrafters/TimeCraftersConfigurationTool/dialogs/VariableDialog.java b/app/src/main/java/org/timecrafters/TimeCraftersConfigurationTool/dialogs/VariableDialog.java
index 63e21d2..45951ef 100644
--- a/app/src/main/java/org/timecrafters/TimeCraftersConfigurationTool/dialogs/VariableDialog.java
+++ b/app/src/main/java/org/timecrafters/TimeCraftersConfigurationTool/dialogs/VariableDialog.java
@@ -224,6 +224,10 @@ public class VariableDialog extends TimeCraftersDialog {
             valueMessage += "Value cannot be blank for a numeric type!";
         }
 
+        if ((varType.equals("Integer") || varType.equals("Long"))  && varValue.contains(".")) {
+            valueMessage += "Integer and Long cannot have decimal value!";
+        }
+
         if (valueMessage.length() > 0) {
             valueError.setVisibility(View.VISIBLE);
             valueError.setText(valueMessage);
@@ -289,28 +293,28 @@ public class VariableDialog extends TimeCraftersDialog {
             variableValue.setVisibility(View.VISIBLE);
             variableValueBoolean.setVisibility(View.GONE);
 
-            variableValue.setInputType(InputType.TYPE_CLASS_NUMBER | InputType.TYPE_NUMBER_FLAG_DECIMAL);
+            variableValue.setInputType(InputType.TYPE_CLASS_NUMBER | InputType.TYPE_NUMBER_FLAG_DECIMAL | InputType.TYPE_NUMBER_FLAG_SIGNED);
 
         } else if (_type.equals("float")) {
             variableType.setText("Float");
             variableValue.setVisibility(View.VISIBLE);
             variableValueBoolean.setVisibility(View.GONE);
 
-            variableValue.setInputType(InputType.TYPE_CLASS_NUMBER | InputType.TYPE_NUMBER_FLAG_DECIMAL);
+            variableValue.setInputType(InputType.TYPE_CLASS_NUMBER | InputType.TYPE_NUMBER_FLAG_DECIMAL | InputType.TYPE_NUMBER_FLAG_SIGNED);
 
         } else if (_type.equals("long")){
             variableType.setText("Long");
             variableValue.setVisibility(View.VISIBLE);
             variableValueBoolean.setVisibility(View.GONE);
 
-            variableValue.setInputType(InputType.TYPE_CLASS_NUMBER | InputType.TYPE_NUMBER_VARIATION_NORMAL);
+            variableValue.setInputType(InputType.TYPE_CLASS_NUMBER | InputType.TYPE_NUMBER_VARIATION_NORMAL | InputType.TYPE_NUMBER_FLAG_SIGNED);
 
         } else if (_type.equals("integer")) {
             variableType.setText("Integer");
             variableValue.setVisibility(View.VISIBLE);
             variableValueBoolean.setVisibility(View.GONE);
 
-            variableValue.setInputType(InputType.TYPE_CLASS_NUMBER | InputType.TYPE_NUMBER_VARIATION_NORMAL);
+            variableValue.setInputType(InputType.TYPE_CLASS_NUMBER | InputType.TYPE_NUMBER_VARIATION_NORMAL | InputType.TYPE_NUMBER_FLAG_SIGNED);
 
         } else if (_type.equals("string")) {
             variableType.setText("String");