fix: range without end

src/js/timepicker/index.js

@@ -732,24 +732,32 @@ var TimePicker = defineClass(
      */
     setRangeMinute: function(beginHour, beginMin, endHour, endMin) {
       var disabledMinRanges = [];
+
+      if (!beginHour && !beginMin) {
+        return;
+      }
+
       disabledMinRanges.push({
         begin: START_NUMBER_OF_TIME,
         end: beginMin
       });
 
-      if (endMin) {
+      if (endHour && endMin) {
         disabledMinRanges.push({
           begin: endMin,
           end: END_NUMBER_OF_MINUTE
         });
-      }
 
-      if (disabledMinRanges.length > 1 && beginHour === endHour) {
-        this.disabledMinutes[beginHour] = util.getDisabledMinuteArr(disabledMinRanges).concat();
-      } else {
-        this.disabledMinutes[beginHour] = util.getDisabledMinuteArr([disabledMinRanges[0]]).concat();
+        if (beginHour === endHour) {
+          this.disabledMinutes[beginHour] = util.getDisabledMinuteArr(disabledMinRanges).concat();
+
+          return;
+        }
+
         this.disabledMinutes[endHour] = util.getDisabledMinuteArr([disabledMinRanges[1]]).concat();
       }
+
+      this.disabledMinutes[beginHour] = util.getDisabledMinuteArr([disabledMinRanges[0]]).concat();
     },
 
     /**

test/timepicker/timepicker.spec.js

@@ -337,6 +337,29 @@ describe('Set selectable range', function() {
     });
   });
 
+  it('should set selectable range with only begin', function() {
+    var start = makeRangeObj(9, 30);
+    var selector, hourSelect, minSelect, selectOption;
+
+    timepickerNoMeridiem.setRange(start);
+
+    selector = timepickerNoMeridiem.element.querySelectorAll('select[aria-label="Time"]');
+    hourSelect = selector[0];
+    minSelect = selector[1];
+
+    selectOption = hourSelect.querySelector('option[value="8"]');
+    expect(selectOption.disabled).toBe(true);
+    selectOption = hourSelect.querySelector('option[value="9"]');
+    expect(selectOption.disabled).toBe(false);
+
+    timepickerNoMeridiem.setTime(9, 0);
+
+    selectOption = minSelect.querySelector('option[value="30"]');
+    expect(selectOption.disabled).toBe(true);
+    selectOption = minSelect.querySelector('option[value="31"]');
+    expect(selectOption.disabled).toBe(false);
+  });
+
   it('should disable a meridiem selector when range included in the other', function() {
     var start = makeRangeObj(6, 30);
     var end = makeRangeObj(11, 30);