add disabled type

pyopensprinkler/const.py

@@ -47,6 +47,7 @@
 SCHEDULE_START_TIME_FIXED = "fixed_time"
 SCHEDULE_START_TIME_REPEATING = "repeating"
 
+SCHEDULE_START_TIME_OFFSET_DISABLED = "disabled"
 SCHEDULE_START_TIME_OFFSET_MIDNIGHT = "midnight"
 SCHEDULE_START_TIME_OFFSET_SUNSET = "sunset"
 SCHEDULE_START_TIME_OFFSET_SUNRISE = "sunrise"

pyopensprinkler/program.py

@@ -1,9 +1,11 @@
 """Program module handling /program/ API calls."""
 
 import json
+from tkinter import OFF
 
 from pyopensprinkler.const import (
     SCHEDULE_START_TIME_FIXED,
+    SCHEDULE_START_TIME_OFFSET_DISABLED,
     SCHEDULE_START_TIME_OFFSET_MIDNIGHT,
     SCHEDULE_START_TIME_OFFSET_SUNRISE,
     SCHEDULE_START_TIME_OFFSET_SUNSET,
@@ -92,7 +94,10 @@ def _get_offset_minutes(self, start_times, start_index):
     def _encode_offset_minutes(self, offset_type, start_time_offset):
         """Encode start time with offset minutes, sign bit, and sunset/sunrise bit"""
         new_sign_bit = 1 if start_time_offset < 0 else 0
-        if offset_type == SCHEDULE_START_TIME_OFFSET_SUNSET:
+
+        if offset_type == SCHEDULE_START_TIME_OFFSET_DISABLED:
+            return -1
+        elif offset_type == SCHEDULE_START_TIME_OFFSET_SUNSET:
             return (
                 abs(int(start_time_offset))
                 | (new_sign_bit << START_TIME_SIGN_BIT)
@@ -114,8 +119,8 @@ def _get_offset_type(self, start_times, start_index):
             self._is_set(start_times[start_index], START_TIME_SUNRISE_BIT)
         )
 
-        if start_times[start_index] == -1:  # disabled
-            return None
+        if start_times[start_index] == -1:
+            return SCHEDULE_START_TIME_OFFSET_DISABLED
         # Only start0 has offset if repeating type
         elif start_index > 0 and self.start_time_type == 0:
             return None
@@ -256,11 +261,11 @@ async def set_program_start_time_offset(self, start_index, start_time_offset):
     async def set_program_start_time_offset_type(
         self, start_index, start_time_offset_type
     ):
-        """Set program start time offset type ('midnight', 'sunset', or 'sunrise'). Resets minutes to 0."""
+        """Set program start time offset type ('disabled', 'midnight', 'sunset', or 'sunrise'). Resets minutes to 0."""
         if not 0 <= start_index <= 3:
             raise ValueError("start_index must be 0 and 3")
 
-        valid_options = ["midnight", "sunset", "sunrise"]
+        valid_options = ["disabled", "midnight", "sunset", "sunrise"]
         if start_time_offset_type not in valid_options:
             raise ValueError(
                 "start_time_offset_type must be one of {}".format(valid_options)