chronic 0.4.1 → 0.6.5

data/lib/chronic/repeaters/repeater_season.rb

@@ -1,22 +1,4 @@
 module Chronic
-  class Season
-    attr_reader :start, :end
-
-    def initialize(myStart, myEnd)
-      @start = myStart
-      @end = myEnd
-    end
-
-    def self.find_next_season(season, pointer)
-      lookup = {:spring => 0, :summer => 1, :autumn => 2, :winter => 3}
-      next_season_num = (lookup[season]+1*pointer) % 4
-      lookup.invert[next_season_num]
-    end
-
-    def self.season_after(season); find_next_season(season, +1); end
-    def self.season_before(season); find_next_season(season, -1); end
-  end
-
   class RepeaterSeason < Repeater #:nodoc:
     SEASON_SECONDS = 7_862_400 # 91 * 24 * 60 * 60
     SEASONS = {
@@ -28,14 +10,13 @@ module Chronic
 
     def initialize(type)
       super
-      @next_season_start = nil
     end
 
     def next(pointer)
       super
 
       direction = pointer == :future ? 1 : -1
-      next_season = Season.find_next_season(find_current_season(@now.to_minidate), direction)
+      next_season = Season.find_next_season(find_current_season(MiniDate.from_time(@now)), direction)
 
       find_next_season_span(direction, next_season)
     end
@@ -45,8 +26,8 @@ module Chronic
 
       direction = pointer == :future ? 1 : -1
 
-      today = Time.construct(@now.year, @now.month, @now.day)
-      this_ssn = find_current_season(@now.to_minidate)
+      today = Chronic.construct(@now.year, @now.month, @now.day)
+      this_ssn = find_current_season(MiniDate.from_time(@now))
       case pointer
       when :past
         this_ssn_start = today + direction * num_seconds_til_start(this_ssn, direction)
@@ -82,9 +63,9 @@ module Chronic
     private
 
     def find_next_season_span(direction, next_season)
-      if !@next_season_start or !@next_season_end
-        @next_season_start = Time.construct(@now.year, @now.month, @now.day)
-        @next_season_end = Time.construct(@now.year, @now.month, @now.day)
+      unless @next_season_start or @next_season_end
+        @next_season_start = Chronic.construct(@now.year, @now.month, @now.day)
+        @next_season_end = Chronic.construct(@now.year, @now.month, @now.day)
       end
 
       @next_season_start += direction * num_seconds_til_start(next_season, direction)
@@ -100,10 +81,10 @@ module Chronic
     end
 
     def num_seconds_til(goal, direction)
-      start = Time.construct(@now.year, @now.month, @now.day)
+      start = Chronic.construct(@now.year, @now.month, @now.day)
       seconds = 0
 
-      until (start + direction * seconds).to_minidate.equals?(goal)
+      until MiniDate.from_time(start + direction * seconds).equals?(goal)
         seconds += RepeaterDay::DAY_SECONDS
       end
 
@@ -120,8 +101,8 @@ module Chronic
 
     def construct_season(start, finish)
       Span.new(
-        Time.construct(start.year, start.month, start.day),
-        Time.construct(finish.year, finish.month, finish.day)
+        Chronic.construct(start.year, start.month, start.day),
+        Chronic.construct(finish.year, finish.month, finish.day)
       )
     end
   end

data/lib/chronic/repeaters/repeater_season_name.rb

@@ -9,14 +9,12 @@ module Chronic
     end
 
     def this(pointer = :future)
-      # super
-
       direction = pointer == :future ? 1 : -1
 
-      today = Time.construct(@now.year, @now.month, @now.day)
+      today = Chronic.construct(@now.year, @now.month, @now.day)
       goal_ssn_start = today + direction * num_seconds_til_start(@type, direction)
       goal_ssn_end = today + direction * num_seconds_til_end(@type, direction)
-      curr_ssn = find_current_season(@now.to_minidate)
+      curr_ssn = find_current_season(MiniDate.from_time(@now))
       case pointer
       when :past
         this_ssn_start = goal_ssn_start

data/lib/chronic/repeaters/repeater_second.rb

@@ -4,7 +4,6 @@ module Chronic
 
     def initialize(type)
       super
-      @second_start = nil
     end
 
     def next(pointer = :future)

data/lib/chronic/repeaters/repeater_time.rb

@@ -27,30 +27,29 @@ module Chronic
     end
 
     def initialize(time)
-      @current_time = nil
       t = time.gsub(/\:/, '')
 
       @type =
       case t.size
-        when 1..2
-          hours = t.to_i
-          hours == 12 ? Tick.new(0 * 60 * 60, true) : Tick.new(hours * 60 * 60, true)
-        when 3
-          hours = t[0..0].to_i
-          ambiguous = hours > 1
-          Tick.new((hours * 60 * 60) + (t[1..2].to_i * 60), ambiguous)
-        when 4
-          ambiguous = time =~ /:/ && t[0..0].to_i != 0 && t[0..1].to_i <= 12
-          hours = t[0..1].to_i
-          hours == 12 ? Tick.new(0 * 60 * 60 + t[2..3].to_i * 60, ambiguous) : Tick.new(hours * 60 * 60 + t[2..3].to_i * 60, ambiguous)
-        when 5
-          Tick.new(t[0..0].to_i * 60 * 60 + t[1..2].to_i * 60 + t[3..4].to_i, true)
-        when 6
-          ambiguous = time =~ /:/ && t[0..0].to_i != 0 && t[0..1].to_i <= 12
-          hours = t[0..1].to_i
-          hours == 12 ? Tick.new(0 * 60 * 60 + t[2..3].to_i * 60 + t[4..5].to_i, ambiguous) : Tick.new(hours * 60 * 60 + t[2..3].to_i * 60 + t[4..5].to_i, ambiguous)
-        else
-          raise("Time cannot exceed six digits")
+      when 1..2
+        hours = t.to_i
+        Tick.new((hours == 12 ? 0 : hours) * 60 * 60, true)
+      when 3
+        hours = t[0..0].to_i
+        ambiguous = hours > 0
+        Tick.new((hours * 60 * 60) + (t[1..2].to_i * 60), ambiguous)
+      when 4
+        ambiguous = time =~ /:/ && t[0..0].to_i != 0 && t[0..1].to_i <= 12
+        hours = t[0..1].to_i
+        hours == 12 ? Tick.new(0 * 60 * 60 + t[2..3].to_i * 60, ambiguous) : Tick.new(hours * 60 * 60 + t[2..3].to_i * 60, ambiguous)
+      when 5
+        Tick.new(t[0..0].to_i * 60 * 60 + t[1..2].to_i * 60 + t[3..4].to_i, true)
+      when 6
+        ambiguous = time =~ /:/ && t[0..0].to_i != 0 && t[0..1].to_i <= 12
+        hours = t[0..1].to_i
+        hours == 12 ? Tick.new(0 * 60 * 60 + t[2..3].to_i * 60 + t[4..5].to_i, ambiguous) : Tick.new(hours * 60 * 60 + t[2..3].to_i * 60 + t[4..5].to_i, ambiguous)
+      else
+        raise("Time cannot exceed six digits")
       end
     end
 

data/lib/chronic/repeaters/repeater_week.rb

@@ -4,7 +4,6 @@ module Chronic
 
     def initialize(type)
       super
-      @current_week_start = nil
     end
 
     def next(pointer)

data/lib/chronic/repeaters/repeater_weekday.rb

@@ -13,7 +13,6 @@ module Chronic
 
     def initialize(type)
       super
-      @current_weekday_start = nil
     end
 
     def next(pointer)
@@ -22,7 +21,7 @@ module Chronic
       direction = pointer == :future ? 1 : -1
 
       if !@current_weekday_start
-        @current_weekday_start = Time.construct(@now.year, @now.month, @now.day)
+        @current_weekday_start = Chronic.construct(@now.year, @now.month, @now.day)
         @current_weekday_start += direction * DAY_SECONDS
 
         until is_weekday?(@current_weekday_start.wday)

data/lib/chronic/repeaters/repeater_weekend.rb

@@ -4,7 +4,6 @@ module Chronic
 
     def initialize(type)
       super
-      @current_week_start = nil
     end
 
     def next(pointer)

data/lib/chronic/repeaters/repeater_year.rb

@@ -4,7 +4,6 @@ module Chronic
 
     def initialize(type)
       super
-      @current_year_start = nil
     end
 
     def next(pointer)
@@ -13,16 +12,16 @@ module Chronic
       if !@current_year_start
         case pointer
         when :future
-          @current_year_start = Time.construct(@now.year + 1)
+          @current_year_start = Chronic.construct(@now.year + 1)
         when :past
-          @current_year_start = Time.construct(@now.year - 1)
+          @current_year_start = Chronic.construct(@now.year - 1)
         end
       else
         diff = pointer == :future ? 1 : -1
-        @current_year_start = Time.construct(@current_year_start.year + diff)
+        @current_year_start = Chronic.construct(@current_year_start.year + diff)
       end
 
-      Span.new(@current_year_start, Time.construct(@current_year_start.year + 1))
+      Span.new(@current_year_start, Chronic.construct(@current_year_start.year + 1))
     end
 
     def this(pointer = :future)
@@ -30,14 +29,14 @@ module Chronic
 
       case pointer
       when :future
-        this_year_start = Time.construct(@now.year, @now.month, @now.day) + RepeaterDay::DAY_SECONDS
-        this_year_end = Time.construct(@now.year + 1, 1, 1)
+        this_year_start = Chronic.construct(@now.year, @now.month, @now.day + 1)
+        this_year_end = Chronic.construct(@now.year + 1, 1, 1)
       when :past
-        this_year_start = Time.construct(@now.year, 1, 1)
-        this_year_end = Time.construct(@now.year, @now.month, @now.day)
+        this_year_start = Chronic.construct(@now.year, 1, 1)
+        this_year_end = Chronic.construct(@now.year, @now.month, @now.day)
       when :none
-        this_year_start = Time.construct(@now.year, 1, 1)
-        this_year_end = Time.construct(@now.year + 1, 1, 1)
+        this_year_start = Chronic.construct(@now.year, 1, 1)
+        this_year_end = Chronic.construct(@now.year + 1, 1, 1)
       end
 
       Span.new(this_year_start, this_year_end)
@@ -45,13 +44,8 @@ module Chronic
 
     def offset(span, amount, pointer)
       direction = pointer == :future ? 1 : -1
-
-      sb = span.begin
-      new_begin = Time.construct(sb.year + (amount * direction), sb.month, sb.day, sb.hour, sb.min, sb.sec)
-
-      se = span.end
-      new_end = Time.construct(se.year + (amount * direction), se.month, se.day, se.hour, se.min, se.sec)
-
+      new_begin = build_offset_time(span.begin, amount, direction)
+      new_end   = build_offset_time(span.end, amount, direction)
       Span.new(new_begin, new_end)
     end
 
@@ -62,5 +56,22 @@ module Chronic
     def to_s
       super << '-year'
     end
+
+    private
+
+    def build_offset_time(time, amount, direction)
+      year = time.year + (amount * direction)
+      days = month_days(year, time.month)
+      day = time.day > days ? days : time.day
+      Chronic.construct(year, time.month, day, time.hour, time.min, time.sec)
+    end
+
+    def month_days(year, month)
+      if Date.leap?(year)
+        RepeaterMonth::MONTH_DAYS_LEAP[month - 1]
+      else
+        RepeaterMonth::MONTH_DAYS[month - 1]
+      end
+    end
   end
 end

data/lib/chronic/scalar.rb

@@ -1,7 +1,13 @@
 module Chronic
-  class Scalar < Tag #:nodoc:
+  class Scalar < Tag
     DAY_PORTIONS = %w( am pm morning afternoon evening night )
 
+    # Scan an Array of {Token}s and apply any necessary Scalar tags to
+    # each token
+    #
+    # @param [Array<Token>] tokens Array of tokens to scan
+    # @param [Hash] options Options specified in {Chronic.parse}
+    # @return [Array] list of tokens
     def self.scan(tokens, options)
       tokens.each_index do |i|
         if t = scan_for_scalars(tokens[i], tokens[i + 1]) then tokens[i].tag(t) end
@@ -11,6 +17,9 @@ module Chronic
       end
     end
 
+    # @param [Token] token
+    # @param [Token] post_token
+    # @return [Scalar, nil]
     def self.scan_for_scalars(token, post_token)
       if token.word =~ /^\d*$/
         unless post_token && DAY_PORTIONS.include?(post_token.word)
@@ -19,6 +28,9 @@ module Chronic
       end
     end
 
+    # @param [Token] token
+    # @param [Token] post_token
+    # @return [ScalarDay, nil]
     def self.scan_for_days(token, post_token)
       if token.word =~ /^\d\d?$/
         toi = token.word.to_i
@@ -28,6 +40,9 @@ module Chronic
       end
     end
 
+    # @param [Token] token
+    # @param [Token] post_token
+    # @return [ScalarMonth, nil]
     def self.scan_for_months(token, post_token)
       if token.word =~ /^\d\d?$/
         toi = token.word.to_i
@@ -37,6 +52,10 @@ module Chronic
       end
     end
 
+    # @param [Token] token
+    # @param [Token] post_token
+    # @param [Hash] options Options specified in {Chronic.parse}
+    # @return [ScalarYear, nil]
     def self.scan_for_years(token, post_token, options)
       if token.word =~ /^([1-9]\d)?\d\d?$/
         unless post_token && DAY_PORTIONS.include?(post_token.word)
@@ -47,6 +66,15 @@ module Chronic
     end
 
     # Build a year from a 2 digit suffix
+    #
+    # @example
+    #   make_year(96, 50) #=> 1996
+    #   make_year(79, 20) #=> 2079
+    #   make_year(00, 50) #=> 2000
+    #
+    # @param [Integer] year The two digit year to build from
+    # @param [Integer] bias The amount of future years to bias
+    # @return [Integer] The 4 digit year
     def self.make_year(year, bias)
       return year if year.to_s.size > 2
       start_year = Chronic.time_class.now.year - bias

data/lib/chronic/season.rb

@@ -0,0 +1,37 @@
+module Chronic
+  class Season
+    # @return [MiniDate]
+    attr_reader :start
+
+    # @return [MiniDate]
+    attr_reader :end
+
+    # @param [MiniDate] start_date
+    # @param [MiniDate] end_date
+    def initialize(start_date, end_date)
+      @start = start_date
+      @end = end_date
+    end
+
+    # @param [Symbol]  season  The season name
+    # @param [Integer] pointer The direction (-1 for past, 1 for future)
+    # @return [Symbol] The new season name
+    def self.find_next_season(season, pointer)
+      lookup = [:spring, :summer, :autumn, :winter]
+      next_season_num = (lookup.index(season) + 1 * pointer) % 4
+      lookup[next_season_num]
+    end
+
+    # @param [Symbol] season The season name
+    # @return [Symbol] The new season name
+    def self.season_after(season)
+      find_next_season(season, +1)
+    end
+
+    # @param [Symbol] season The season name
+    # @return [Symbol] The new season name
+    def self.season_before(season)
+      find_next_season(season, -1)
+    end
+  end
+end

data/lib/chronic/separator.rb

@@ -1,19 +1,30 @@
 module Chronic
-  class Separator < Tag #:nodoc:
+  class Separator < Tag
+
+    # Scan an Array of {Token}s and apply any necessary Separator tags to
+    # each token
+    #
+    # @param [Array<Token>] tokens Array of tokens to scan
+    # @param [Hash] options Options specified in {Chronic.parse}
+    # @return [Array] list of tokens
     def self.scan(tokens, options)
-      tokens.each_index do |i|
-        if t = scan_for_commas(tokens[i]) then tokens[i].tag(t); next end
-        if t = scan_for_slash_or_dash(tokens[i]) then tokens[i].tag(t); next end
-        if t = scan_for_at(tokens[i]) then tokens[i].tag(t); next end
-        if t = scan_for_in(tokens[i]) then tokens[i].tag(t); next end
-        if t = scan_for_on(tokens[i]) then tokens[i].tag(t); next end
+      tokens.each do |token|
+        if t = scan_for_commas(token) then token.tag(t); next end
+        if t = scan_for_slash_or_dash(token) then token.tag(t); next end
+        if t = scan_for_at(token) then token.tag(t); next end
+        if t = scan_for_in(token) then token.tag(t); next end
+        if t = scan_for_on(token) then token.tag(t); next end
       end
     end
 
+    # @param [Token] token
+    # @return [SeparatorComma, nil]
     def self.scan_for_commas(token)
       scan_for token, SeparatorComma, { /^,$/ => :comma }
     end
 
+    # @param [Token] token
+    # @return [SeparatorSlashOrDash, nil]
     def self.scan_for_slash_or_dash(token)
       scan_for token, SeparatorSlashOrDash,
       {
@@ -22,14 +33,20 @@ module Chronic
       }
     end
 
+    # @param [Token] token
+    # @return [SeparatorAt, nil]
     def self.scan_for_at(token)
       scan_for token, SeparatorAt, { /^(at|@)$/ => :at }
     end
 
+    # @param [Token] token
+    # @return [SeparatorIn, nil]
     def self.scan_for_in(token)
       scan_for token, SeparatorIn, { /^in$/ => :in }
     end
 
+    # @param [Token] token
+    # @return [SeparatorOn, nil]
     def self.scan_for_on(token)
       scan_for token, SeparatorOn, { /^on$/ => :on }
     end

data/lib/chronic/tag.rb

@@ -1,13 +1,17 @@
 module Chronic
   # Tokens are tagged with subclassed instances of this class when
   # they match specific criteria
-  class Tag #:nodoc:
+  class Tag
+
+    # @return [Symbol]
     attr_accessor :type
 
+    # @param [Symbol] type
     def initialize(type)
       @type = type
     end
 
+    # @param [Time] s Set the start timestamp for this Tag
     def start=(s)
       @now = s
     end
@@ -15,6 +19,11 @@ module Chronic
     class << self
       private
 
+      # @param [Token] token
+      # @param [Class] klass The class instance to create
+      # @param [Regexp, Hash] items
+      # @return [Object, nil] either a new instance of `klass` or `nil` if
+      #   no match is found
       def scan_for(token, klass, items={})
         case items
         when Regexp
@@ -26,6 +35,8 @@ module Chronic
         end
         nil
       end
+
     end
+
   end
 end

data/lib/chronic/time_zone.rb

@@ -1,16 +1,25 @@
 module Chronic
-  class TimeZone < Tag #:nodoc:
+  class TimeZone < Tag
+
+    # Scan an Array of {Token}s and apply any necessary TimeZone tags to
+    # each token
+    #
+    # @param [Array<Token>] tokens Array of tokens to scan
+    # @param [Hash] options Options specified in {Chronic.parse}
+    # @return [Array] list of tokens
     def self.scan(tokens, options)
-      tokens.each_index do |i|
-        if t = scan_for_all(tokens[i]) then tokens[i].tag(t); next end
+      tokens.each do |token|
+        if t = scan_for_all(token) then token.tag(t); next end
       end
     end
 
+    # @param [Token] token
+    # @return [TimeZone, nil]
     def self.scan_for_all(token)
       scan_for token, self,
       {
-        /[PMCE][DS]T/i => :tz,
-        /(tzminus)?\d{4}/ => :tz
+        /[PMCE][DS]T|UTC/i => :tz,
+        /(tzminus)?\d{2}:?\d{2}/ => :tz
       }
     end
 

data/lib/chronic/token.rb

@@ -1,6 +1,11 @@
 module Chronic
-  class Token #:nodoc:
-    attr_accessor :word, :tags
+  class Token
+
+    # @return [String] The word this Token represents
+    attr_accessor :word
+
+    # @return [Array] A list of tag associated with this Token
+    attr_accessor :tags
 
     def initialize(word)
       @word = word
@@ -8,21 +13,26 @@ module Chronic
     end
 
     # Tag this token with the specified tag
+    #
+    # @param [Tag] new_tag An instance of {Tag} or one of its subclasses
     def tag(new_tag)
       @tags << new_tag
     end
 
     # Remove all tags of the given class
+    #
+    # @param [Class] The tag class to remove
     def untag(tag_class)
       @tags.delete_if { |m| m.kind_of? tag_class }
     end
 
-    # Return true if this token has any tags
+    # @return [Boolean] true if this token has any tags
     def tagged?
       @tags.size > 0
     end
 
-    # Return the Tag that matches the given class
+    # @param [Class] tag_class The tag class to search for
+    # @return [Tag] The first Tag that matches the given class
     def get_tag(tag_class)
       @tags.find { |m| m.kind_of? tag_class }
     end