chronic 0.2.0 → 0.3.0

data/lib/chronic/ordinal.rb

@@ -9,32 +9,32 @@ module Chronic
       end
       tokens
     end
-  
+
     def self.scan_for_ordinals(token)
       if token.word =~ /^(\d*)(st|nd|rd|th)$/
         return Ordinal.new($1.to_i)
       end
       return nil
     end
-    
+
     def self.scan_for_days(token)
       if token.word =~ /^(\d*)(st|nd|rd|th)$/
-        unless $1.to_i > 31
+        unless $1.to_i > 31 || $1.to_i < 1
           return OrdinalDay.new(token.word.to_i)
         end
       end
       return nil
     end
-    
+
     def to_s
       'ordinal'
     end
   end
-  
+
   class OrdinalDay < Ordinal #:nodoc:
     def to_s
       super << '-day-' << @type.to_s
     end
   end
 
-end
+end

data/lib/chronic/pointer.rb

@@ -8,7 +8,7 @@ module Chronic
       end
       tokens
     end
-  
+
     def self.scan_for_all(token)
       scanner = {/\bpast\b/ => :past,
                  /\bfuture\b/ => :future,
@@ -18,10 +18,10 @@ module Chronic
       end
       return nil
     end
-    
+
     def to_s
       'pointer-' << @type.to_s
     end
   end
 
-end
+end

data/lib/chronic/repeater.rb

@@ -2,6 +2,7 @@ class Chronic::Repeater < Chronic::Tag #:nodoc:
   def self.scan(tokens, options)
     # for each token
     tokens.each_index do |i|
+      if t = self.scan_for_season_names(tokens[i]) then tokens[i].tag(t); next end
       if t = self.scan_for_month_names(tokens[i]) then tokens[i].tag(t); next end
       if t = self.scan_for_day_names(tokens[i]) then tokens[i].tag(t); next end
       if t = self.scan_for_day_portions(tokens[i]) then tokens[i].tag(t); next end
@@ -10,7 +11,19 @@ class Chronic::Repeater < Chronic::Tag #:nodoc:
     end
     tokens
   end
-  
+
+  def self.scan_for_season_names(token)
+    scanner = {/^springs?$/ => :spring,
+               /^summers?$/ => :summer,
+               /^(autumn)|(fall)s?$/ => :autumn,
+               /^winters?$/ => :winter}
+    scanner.keys.each do |scanner_item|
+      return Chronic::RepeaterSeasonName.new(scanner[scanner_item]) if scanner_item =~ token.word
+    end
+
+    return nil
+  end
+
   def self.scan_for_month_names(token)
     scanner = {/^jan\.?(uary)?$/ => :january,
                /^feb\.?(ruary)?$/ => :february,
@@ -29,10 +42,11 @@ class Chronic::Repeater < Chronic::Tag #:nodoc:
     end
     return nil
   end
-  
+
   def self.scan_for_day_names(token)
     scanner = {/^m[ou]n(day)?$/ => :monday,
                /^t(ue|eu|oo|u|)s(day)?$/ => :tuesday,
+               /^tue$/ => :tuesday,
                /^we(dnes|nds|nns)day$/ => :wednesday,
                /^wed$/ => :wednesday,
                /^th(urs|ers)day$/ => :thursday,
@@ -45,7 +59,7 @@ class Chronic::Repeater < Chronic::Tag #:nodoc:
     end
     return nil
   end
-  
+
   def self.scan_for_day_portions(token)
     scanner = {/^ams?$/ => :am,
                /^pms?$/ => :pm,
@@ -58,14 +72,14 @@ class Chronic::Repeater < Chronic::Tag #:nodoc:
     end
     return nil
   end
-  
+
   def self.scan_for_times(token, options)
     if token.word =~ /^\d{1,2}(:?\d{2})?([\.:]?\d{2})?$/
       return Chronic::RepeaterTime.new(token.word, options)
     end
     return nil
   end
-  
+
   def self.scan_for_units(token)
     scanner = {/^years?$/ => :year,
                /^seasons?$/ => :season,
@@ -73,6 +87,7 @@ class Chronic::Repeater < Chronic::Tag #:nodoc:
                /^fortnights?$/ => :fortnight,
                /^weeks?$/ => :week,
                /^weekends?$/ => :weekend,
+               /^(week|business)days?$/ => :weekday,
                /^days?$/ => :day,
                /^hours?$/ => :hour,
                /^minutes?$/ => :minute,
@@ -81,34 +96,34 @@ class Chronic::Repeater < Chronic::Tag #:nodoc:
       if scanner_item =~ token.word
         klass_name = 'Chronic::Repeater' + scanner[scanner_item].to_s.capitalize
         klass = eval(klass_name)
-        return klass.new(scanner[scanner_item]) 
+        return klass.new(scanner[scanner_item])
       end
     end
     return nil
   end
-  
+
   def <=>(other)
     width <=> other.width
   end
-  
+
   # returns the width (in seconds or months) of this repeatable.
   def width
     raise("Repeatable#width must be overridden in subclasses")
   end
-  
+
   # returns the next occurance of this repeatable.
   def next(pointer)
     !@now.nil? || raise("Start point must be set before calling #next")
     [:future, :none, :past].include?(pointer) || raise("First argument 'pointer' must be one of :past or :future")
     #raise("Repeatable#next must be overridden in subclasses")
   end
-  
+
   def this(pointer)
     !@now.nil? || raise("Start point must be set before calling #this")
     [:future, :past, :none].include?(pointer) || raise("First argument 'pointer' must be one of :past, :future, :none")
   end
-  
+
   def to_s
     'repeater'
   end
-end
+end

data/lib/chronic/repeaters/repeater_day.rb

@@ -1,47 +1,52 @@
 class Chronic::RepeaterDay < Chronic::Repeater #:nodoc:
   DAY_SECONDS = 86_400 # (24 * 60 * 60)
-  
+
+  def initialize(type)
+    super
+    @current_day_start = nil
+  end
+
   def next(pointer)
     super
-    
+
     if !@current_day_start
-      @current_day_start = Time.local(@now.year, @now.month, @now.day)
+      @current_day_start = Chronic.time_class.local(@now.year, @now.month, @now.day)
     end
-    
+
     direction = pointer == :future ? 1 : -1
     @current_day_start += direction * DAY_SECONDS
-    
+
     Chronic::Span.new(@current_day_start, @current_day_start + DAY_SECONDS)
   end
-  
+
   def this(pointer = :future)
     super
-    
+
     case pointer
     when :future
-      day_begin = Time.local(@now.year, @now.month, @now.day, @now.hour + 1)
-      day_end = Time.local(@now.year, @now.month, @now.day) + DAY_SECONDS
+      day_begin = Time.construct(@now.year, @now.month, @now.day, @now.hour + 1)
+      day_end = Time.construct(@now.year, @now.month, @now.day) + DAY_SECONDS
     when :past
-      day_begin = Time.local(@now.year, @now.month, @now.day)
-      day_end = Time.local(@now.year, @now.month, @now.day, @now.hour)
+      day_begin = Time.construct(@now.year, @now.month, @now.day)
+      day_end = Time.construct(@now.year, @now.month, @now.day, @now.hour)
     when :none
-      day_begin = Time.local(@now.year, @now.month, @now.day)
-      day_end = Time.local(@now.year, @now.month, @now.day) + DAY_SECONDS
+      day_begin = Time.construct(@now.year, @now.month, @now.day)
+      day_end = Time.construct(@now.year, @now.month, @now.day) + DAY_SECONDS
     end
-    
+
     Chronic::Span.new(day_begin, day_end)
   end
-  
+
   def offset(span, amount, pointer)
     direction = pointer == :future ? 1 : -1
     span + direction * amount * DAY_SECONDS
   end
-  
+
   def width
     DAY_SECONDS
   end
-  
+
   def to_s
     super << '-day'
   end
-end
+end

data/lib/chronic/repeaters/repeater_day_name.rb

@@ -1,46 +1,51 @@
 class Chronic::RepeaterDayName < Chronic::Repeater #:nodoc:
   DAY_SECONDS = 86400 # (24 * 60 * 60)
-  
+
+  def initialize(type)
+    super
+    @current_day_start = nil
+  end
+
   def next(pointer)
     super
-    
+
     direction = pointer == :future ? 1 : -1
-    
+
     if !@current_day_start
-      @current_day_start = Time.local(@now.year, @now.month, @now.day)
+      @current_day_start = Time.construct(@now.year, @now.month, @now.day)
       @current_day_start += direction * DAY_SECONDS
 
       day_num = symbol_to_number(@type)
-      
+
       while @current_day_start.wday != day_num
         @current_day_start += direction * DAY_SECONDS
       end
     else
       @current_day_start += direction * 7 * DAY_SECONDS
     end
-    
+
     Chronic::Span.new(@current_day_start, @current_day_start + DAY_SECONDS)
   end
-  
+
   def this(pointer = :future)
     super
-    
+
     pointer = :future if pointer == :none
     self.next(pointer)
   end
-  
+
   def width
     DAY_SECONDS
   end
-  
+
   def to_s
-    super << '-dayofweek-' << @type.to_s
+    super << '-dayname-' << @type.to_s
   end
-  
+
   private
-  
+
   def symbol_to_number(sym)
     lookup = {:sunday => 0, :monday => 1, :tuesday => 2, :wednesday => 3, :thursday => 4, :friday => 5, :saturday => 6}
     lookup[sym] || raise("Invalid symbol specified")
   end
-end
+end

data/lib/chronic/repeaters/repeater_day_portion.rb

@@ -3,15 +3,16 @@ class Chronic::RepeaterDayPortion < Chronic::Repeater #:nodoc:
   @@afternoon = (13 * 60 * 60)..(17 * 60 * 60) # 1pm-5pm
   @@evening = (17 * 60 * 60)..(20 * 60 * 60) # 5pm-8pm
   @@night = (20 * 60 * 60)..(24 * 60 * 60) # 8pm-12pm
-  
+
   def initialize(type)
     super
-    
+    @current_span = nil
+
     if type.kind_of? Integer
       @range = (@type * 60 * 60)..((@type + 12) * 60 * 60)
     else
-      lookup = {:am => 1..(12 * 60 * 60),
-                :pm => (12 * 60 * 60)..(24 * 60 * 60),
+      lookup = {:am => 0..(12 * 60 * 60 - 1),
+                :pm => (12 * 60 * 60)..(24 * 60 * 60 - 1),
                 :morning => @@morning,
                 :afternoon => @@afternoon,
                 :evening => @@evening,
@@ -21,37 +22,37 @@ class Chronic::RepeaterDayPortion < Chronic::Repeater #:nodoc:
     end
     @range || raise("Range should have been set by now")
   end
-  
+
   def next(pointer)
     super
-    
+
     full_day = 60 * 60 * 24
-    
+
     if !@current_span
-      now_seconds = @now - Time.local(@now.year, @now.month, @now.day)
+      now_seconds = @now - Time.construct(@now.year, @now.month, @now.day)
       if now_seconds < @range.begin
         case pointer
         when :future
-          range_start = Time.local(@now.year, @now.month, @now.day) + @range.begin
+          range_start = Time.construct(@now.year, @now.month, @now.day) + @range.begin
         when :past
-          range_start = Time.local(@now.year, @now.month, @now.day) - full_day + @range.begin
+          range_start = Time.construct(@now.year, @now.month, @now.day) - full_day + @range.begin
         end
       elsif now_seconds > @range.end
         case pointer
         when :future
-          range_start = Time.local(@now.year, @now.month, @now.day) + full_day + @range.begin
+          range_start = Time.construct(@now.year, @now.month, @now.day) + full_day + @range.begin
         when :past
-          range_start = Time.local(@now.year, @now.month, @now.day) + @range.begin
+          range_start = Time.construct(@now.year, @now.month, @now.day) + @range.begin
         end
       else
         case pointer
         when :future
-          range_start = Time.local(@now.year, @now.month, @now.day) + full_day + @range.begin
+          range_start = Time.construct(@now.year, @now.month, @now.day) + full_day + @range.begin
         when :past
-          range_start = Time.local(@now.year, @now.month, @now.day) - full_day + @range.begin
+          range_start = Time.construct(@now.year, @now.month, @now.day) - full_day + @range.begin
         end
       end
-      
+
       @current_span = Chronic::Span.new(range_start, range_start + (@range.end - @range.begin))
     else
       case pointer
@@ -62,21 +63,21 @@ class Chronic::RepeaterDayPortion < Chronic::Repeater #:nodoc:
       end
     end
   end
-  
+
   def this(context = :future)
     super
-    
-    range_start = Time.local(@now.year, @now.month, @now.day) + @range.begin
+
+    range_start = Time.construct(@now.year, @now.month, @now.day) + @range.begin
     @current_span = Chronic::Span.new(range_start, range_start + (@range.end - @range.begin))
   end
-  
+
   def offset(span, amount, pointer)
     @now = span.begin
     portion_span = self.next(pointer)
     direction = pointer == :future ? 1 : -1
     portion_span + (direction * (amount - 1) * Chronic::RepeaterDay::DAY_SECONDS)
   end
-  
+
   def width
     @range || raise("Range has not been set")
     return @current_span.width if @current_span
@@ -86,8 +87,8 @@ class Chronic::RepeaterDayPortion < Chronic::Repeater #:nodoc:
       @range.end - @range.begin
     end
   end
-  
+
   def to_s
     super << '-dayportion-' << @type.to_s
   end
-end
+end

data/lib/chronic/repeaters/repeater_fortnight.rb

@@ -1,9 +1,14 @@
 class Chronic::RepeaterFortnight < Chronic::Repeater #:nodoc:
   FORTNIGHT_SECONDS = 1_209_600 # (14 * 24 * 60 * 60)
-  
+
+  def initialize(type)
+    super
+    @current_fortnight_start = nil
+  end
+
   def next(pointer)
     super
-    
+
     if !@current_fortnight_start
       case pointer
       when :future
@@ -22,18 +27,18 @@ class Chronic::RepeaterFortnight < Chronic::Repeater #:nodoc:
       direction = pointer == :future ? 1 : -1
       @current_fortnight_start += direction * FORTNIGHT_SECONDS
     end
-    
+
     Chronic::Span.new(@current_fortnight_start, @current_fortnight_start + FORTNIGHT_SECONDS)
   end
-  
+
   def this(pointer = :future)
     super
-    
+
     pointer = :future if pointer == :none
-    
+
     case pointer
     when :future
-      this_fortnight_start = Time.local(@now.year, @now.month, @now.day, @now.hour) + Chronic::RepeaterHour::HOUR_SECONDS
+      this_fortnight_start = Time.construct(@now.year, @now.month, @now.day, @now.hour) + Chronic::RepeaterHour::HOUR_SECONDS
       sunday_repeater = Chronic::RepeaterDayName.new(:sunday)
       sunday_repeater.start = @now
       sunday_repeater.this(:future)
@@ -41,7 +46,7 @@ class Chronic::RepeaterFortnight < Chronic::Repeater #:nodoc:
       this_fortnight_end = this_sunday_span.begin
       Chronic::Span.new(this_fortnight_start, this_fortnight_end)
     when :past
-      this_fortnight_end = Time.local(@now.year, @now.month, @now.day, @now.hour)
+      this_fortnight_end = Time.construct(@now.year, @now.month, @now.day, @now.hour)
       sunday_repeater = Chronic::RepeaterDayName.new(:sunday)
       sunday_repeater.start = @now
       last_sunday_span = sunday_repeater.next(:past)
@@ -49,7 +54,7 @@ class Chronic::RepeaterFortnight < Chronic::Repeater #:nodoc:
       Chronic::Span.new(this_fortnight_start, this_fortnight_end)
     end
   end
-  
+
   def offset(span, amount, pointer)
     direction = pointer == :future ? 1 : -1
     span + direction * amount * FORTNIGHT_SECONDS
@@ -58,8 +63,8 @@ class Chronic::RepeaterFortnight < Chronic::Repeater #:nodoc:
   def width
     FORTNIGHT_SECONDS
   end
-  
+
   def to_s
     super << '-fortnight'
   end
-end
+end

data/lib/chronic/repeaters/repeater_hour.rb

@@ -1,52 +1,57 @@
 class Chronic::RepeaterHour < Chronic::Repeater #:nodoc:
   HOUR_SECONDS = 3600 # 60 * 60
-  
+
+  def initialize(type)
+    super
+    @current_hour_start = nil
+  end
+
   def next(pointer)
     super
-    
+
     if !@current_hour_start
       case pointer
       when :future
-        @current_hour_start = Time.local(@now.year, @now.month, @now.day, @now.hour + 1)
+        @current_hour_start = Time.construct(@now.year, @now.month, @now.day, @now.hour + 1)
       when :past
-        @current_hour_start = Time.local(@now.year, @now.month, @now.day, @now.hour - 1)
+        @current_hour_start = Time.construct(@now.year, @now.month, @now.day, @now.hour - 1)
       end
     else
       direction = pointer == :future ? 1 : -1
       @current_hour_start += direction * HOUR_SECONDS
     end
-    
+
     Chronic::Span.new(@current_hour_start, @current_hour_start + HOUR_SECONDS)
   end
-  
+
   def this(pointer = :future)
     super
-    
+
     case pointer
     when :future
       hour_start = Time.construct(@now.year, @now.month, @now.day, @now.hour, @now.min + 1)
       hour_end = Time.construct(@now.year, @now.month, @now.day, @now.hour + 1)
     when :past
-      hour_start = Time.local(@now.year, @now.month, @now.day, @now.hour)
-      hour_end = Time.local(@now.year, @now.month, @now.day, @now.hour, @now.min)
+      hour_start = Time.construct(@now.year, @now.month, @now.day, @now.hour)
+      hour_end = Time.construct(@now.year, @now.month, @now.day, @now.hour, @now.min)
     when :none
-      hour_start = Time.local(@now.year, @now.month, @now.day, @now.hour)
+      hour_start = Time.construct(@now.year, @now.month, @now.day, @now.hour)
       hour_end = hour_begin + HOUR_SECONDS
     end
-    
+
     Chronic::Span.new(hour_start, hour_end)
   end
-  
+
   def offset(span, amount, pointer)
     direction = pointer == :future ? 1 : -1
     span + direction * amount * HOUR_SECONDS
   end
-  
+
   def width
     HOUR_SECONDS
   end
-  
+
   def to_s
     super << '-hour'
   end
-end
+end

data/lib/chronic/repeaters/repeater_minute.rb

@@ -1,42 +1,57 @@
 class Chronic::RepeaterMinute < Chronic::Repeater #:nodoc:
   MINUTE_SECONDS = 60
-  
+
+  def initialize(type)
+    super
+    @current_minute_start = nil
+  end
+
   def next(pointer = :future)
     super
-    
-    direction = pointer == :future ? 1 : -1
-    
-    raise 'not implemented'
+
+    if !@current_minute_start
+      case pointer
+      when :future
+        @current_minute_start = Time.construct(@now.year, @now.month, @now.day, @now.hour, @now.min + 1)
+      when :past
+        @current_minute_start = Time.construct(@now.year, @now.month, @now.day, @now.hour, @now.min - 1)
+      end
+    else
+      direction = pointer == :future ? 1 : -1
+      @current_minute_start += direction * MINUTE_SECONDS
+    end
+
+    Chronic::Span.new(@current_minute_start, @current_minute_start + MINUTE_SECONDS)
   end
-  
+
   def this(pointer = :future)
     super
-    
+
     case pointer
     when :future
       minute_begin = @now
-      minute_end = Time.local(@now.year, @now.month, @now.day, @now.hour, @now.min)
+      minute_end = Time.construct(@now.year, @now.month, @now.day, @now.hour, @now.min)
     when :past
-      minute_begin = Time.local(@now.year, @now.month, @now.day, @now.hour, @now.min)
+      minute_begin = Time.construct(@now.year, @now.month, @now.day, @now.hour, @now.min)
       minute_end = @now
     when :none
-      minute_begin = Time.local(@now.year, @now.month, @now.day, @now.hour, @now.min)
-      minute_end = Time.local(@now.year, @now.month, @now.day, @now.hour, @now.min) + MINUTE_SECONDS
+      minute_begin = Time.construct(@now.year, @now.month, @now.day, @now.hour, @now.min)
+      minute_end = Time.construct(@now.year, @now.month, @now.day, @now.hour, @now.min) + MINUTE_SECONDS
     end
-    
+
     Chronic::Span.new(minute_begin, minute_end)
   end
-  
+
   def offset(span, amount, pointer)
     direction = pointer == :future ? 1 : -1
     span + direction * amount * MINUTE_SECONDS
   end
-  
+
   def width
     MINUTE_SECONDS
   end
-  
+
   def to_s
     super << '-minute'
   end
-end
+end