From 5188b7e12848deafaaa1e5959b34dc4eaec6b8b6 Mon Sep 17 00:00:00 2001
From: Phillip Hsu <philliphsu@outlook.com>
Date: Thu, 25 Aug 2016 17:35:29 -0700
Subject: [PATCH] Fix 24 hour numpad not disabling FAB for invalid 3-digit
 times upon backspacing

---
 .../com/philliphsu/clock2/editalarm/NumpadTimePicker.java | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/app/src/main/java/com/philliphsu/clock2/editalarm/NumpadTimePicker.java b/app/src/main/java/com/philliphsu/clock2/editalarm/NumpadTimePicker.java
index 2db8214..fc9949e 100644
--- a/app/src/main/java/com/philliphsu/clock2/editalarm/NumpadTimePicker.java
+++ b/app/src/main/java/com/philliphsu/clock2/editalarm/NumpadTimePicker.java
@@ -408,6 +408,7 @@ public class NumpadTimePicker extends GridLayoutNumpad {
         int len = mFormattedInput.length();
         mFormattedInput.delete(len - 1, len);
         if (count() == 3) {
+            int value = getInput();
             // Move the colon from its 4-digit position to its 3-digit position,
             // unless doing so gives an invalid time.
             // e.g. 17:55 becomes 1:75, which is invalid.
@@ -417,14 +418,19 @@ public class NumpadTimePicker extends GridLayoutNumpad {
             // in the 24-hour clock can only go up to 15:5[0-9] or be within the range
             // [20:00, 23:59] if they are to remain valid when they become three digits.
             // The is24HourFormat() check is therefore unnecessary.
-            int value = getInput();
             if (value <= 155 || value >= 200 && value <= 235) {
                 mFormattedInput.deleteCharAt(mFormattedInput.indexOf(":"));
                 mFormattedInput.insert(1, ":");
+            } else {
+                // previously [16:00, 19:59]
+                mAmPmState = UNSPECIFIED;
             }
         } else if (count() == 2) {
             // Remove the colon
             mFormattedInput.deleteCharAt(mFormattedInput.indexOf(":"));
+            // No time can be valid with only 2 digits in either system.
+            // I don't think we actually need this, but it can't hurt?
+            mAmPmState = UNSPECIFIED;
         }
     }