Add Timer.isCancelled() method

Signed-off-by: Jimmy Tanagra <[email protected]>

bundles/org.openhab.core.model.script/src.moved/test/java/org/openhab/core/model/script/internal/actions/TimerImplTest.java

@@ -56,6 +56,7 @@ public void testTimerIsActiveAndCancel() {
         subject.cancel();
         assertThat(subject.isActive(), is(false));
         assertThat(subject.hasTerminated(), is(true));
+        assertThat(subject.isCancelled(), is(true));
 
         subject.reschedule(ZonedDateTime.now().plusSeconds(DEFAULT_TIMEOUT_SECONDS));
         assertThat(subject.isActive(), is(true));

bundles/org.openhab.core.model.script/src/org/openhab/core/model/script/actions/Timer.java

@@ -44,6 +44,13 @@ public interface Timer {
      */
     public boolean isActive();
 
+    /**
+     * Determines whether the timer has been cancelled
+     *
+     * @return true, if the timer has been cancelled, false otherwise
+     */
+    public boolean isCancelled();
+
     /**
      * Determines whether the scheduled code is currently executed.
      *

bundles/org.openhab.core.model.script/src/org/openhab/core/model/script/internal/actions/TimerImpl.java

@@ -36,8 +36,6 @@ public class TimerImpl implements Timer {
     private final SchedulerRunnable runnable;
     private ScheduledCompletableFuture<Object> future;
 
-    private boolean cancelled;
-
     public TimerImpl(Scheduler scheduler, ZonedDateTime startTime, SchedulerRunnable runnable) {
         this.scheduler = scheduler;
         this.startTime = startTime;
@@ -48,26 +46,29 @@ public TimerImpl(Scheduler scheduler, ZonedDateTime startTime, SchedulerRunnable
 
     @Override
     public boolean cancel() {
-        cancelled = true;
         return future.cancel(true);
     }
 
     @Override
     public boolean reschedule(ZonedDateTime newTime) {
         future.cancel(false);
-        cancelled = false;
         future = scheduler.schedule(runnable, newTime.toInstant());
         return true;
     }
 
     @Override
     public ZonedDateTime getExecutionTime() {
-        return cancelled ? null : ZonedDateTime.now().plusNanos(future.getDelay(TimeUnit.NANOSECONDS));
+        return future.isCancelled() ? null : ZonedDateTime.now().plusNanos(future.getDelay(TimeUnit.NANOSECONDS));
     }
 
     @Override
     public boolean isActive() {
-        return !future.isDone() && !cancelled;
+        return !future.isDone();
+    }
+
+    @Override
+    public boolean isCancelled() {
+        return future.isCancelled();
     }
 
     @Override