make it clear @next_time is not set to false

as surfaced in discussion at gh-129

CREDITS.txt

@@ -31,6 +31,7 @@
 
 == Feedback
 
+- ecin - https://github.com/ecin - set_next_time() ambiguity
 - junhanamaki - https://github.com/junhanamaki - #next_time and dst ambiguities
 - kreynolds (tossrock) - inspiration for #occurrences
 - Matteo - https://github.com/m4ce - dst and cron issue

lib/rufus/scheduler/jobs.rb

@@ -117,7 +117,7 @@ def initialize(scheduler, original, opts, block)
 
       def trigger(time)
 
-        set_next_time(false, time)
+        set_next_time(time)
 
         return if opts[:overlap] == false && running?
 
@@ -272,7 +272,7 @@ def do_trigger(time)
 
       def post_trigger(time)
 
-        set_next_time(true, time)
+        set_next_time(time, true)
 
         callback(:on_post_trigger, time)
       end
@@ -356,7 +356,7 @@ def determine_id
 
       # There is no next_time for one time jobs, hence the false.
       #
-      def set_next_time(is_post, trigger_time)
+      def set_next_time(trigger_time, is_post=false)
 
         @next_time = is_post ? nil : false
       end
@@ -527,12 +527,12 @@ def initialize(scheduler, duration, opts, block)
           "of #{@frequency.inspect} (#{@original.inspect})"
         ) if @frequency <= 0
 
-        set_next_time(false, nil)
+        set_next_time(nil)
       end
 
       protected
 
-      def set_next_time(is_post, trigger_time)
+      def set_next_time(trigger_time, is_post=false)
 
         return if is_post
 
@@ -567,12 +567,12 @@ def initialize(scheduler, interval, opts, block)
           "of #{@interval.inspect} (#{@original.inspect})"
         ) if @interval <= 0
 
-        set_next_time(false, nil)
+        set_next_time(nil)
       end
 
       protected
 
-      def set_next_time(is_post, trigger_time)
+      def set_next_time(trigger_time, is_post=false)
 
         @next_time =
           if is_post
@@ -616,7 +616,7 @@ def brute_frequency
 
       protected
 
-      def set_next_time(is_post, trigger_time)
+      def set_next_time(trigger_time, is_post=false)
 
         @next_time = @cron_line.next_time
       end