ri_cal 0.8.5 → 0.8.6

data/History.txt

@@ -1,3 +1,10 @@
+=== 0.8.6
+  Fixes an issue of RFC 2445 non-conformance - change management time attributes (dtstamp, last_modified, and created) MUST be zulu time
+  Changed VTIMEZONE export from tz_info
+     Discovered that RFC 2445 expects a timezone period onset to be local time before the change, while TZInfo gives it after the change
+       Thanks to both Michael Hale and H. Wade Minter for reporting this.
+     Changed from generating a single RDATE line for a timezone period with multiple values to multiple RDATE lines
+       to fix a problem reported by Michael Hale.
 === 0.8.5
   Fixes http://rick_denatale.lighthouseapp.com/projects/30941/tickets/26
     events with date parameters failing

data/Rakefile

@@ -1,53 +1,66 @@
-#- ©2009 Rick DeNatale
-#- All rights reserved. Refer to the file README.txt for the license
-#
-# %w[rubygems rake rake/clean fileutils newgem  rubigen].each { |f| require f }
-#require File.dirname(__FILE__) + '/lib/ri_cal'
-
 require 'rubygems'
-gem 'hoe', '>=2.1.0'
-require 'hoe'
-require 'fileutils'
-require './lib/ri_cal'
-
-Hoe.plugin :newgem
-Hoe.plugin :website
-# Hoe.plugin :cucumberfeatures
-
-# Generate all the Rake tasks
-# Run 'rake -T' to see list of generated tasks (from gem root directory)
-$hoe = Hoe.spec('ri_cal') do |p|
-  p.developer('author=Rick DeNatale', 'rick.denatale@gmail.com')
-  p.changes              = p.paragraphs_of("History.txt", 0..1).join("\n\n")
-  p.rubyforge_name       = 'ri-cal'
-  p.readme_file = "README.txt"
-  p.extra_dev_deps = [
-    ['newgem', ">= #{::Newgem::VERSION}"],
-    ['ruby-prof', ">= 0"]
-  ]
-
-  p.clean_globs |= %w[**/.DS_Store tmp *.log]
-  path = (p.rubyforge_name == p.name) ? p.rubyforge_name : "\#{p.rubyforge_name}/\#{p.name}"
-  p.remote_rdoc_dir = File.join(path.gsub(/^#{p.rubyforge_name}\/?/,''), 'rdoc')
-  p.rsync_args = '-av --delete --ignore-errors'
+require 'rake'
+
+begin
+  require 'jeweler'
+  Jeweler::Tasks.new do |gem|
+    gem.name = "ri_cal"
+    gem.summary = %Q{a new implementation of RFC2445 in Ruby}
+    gem.description = %Q{A new Ruby implementation of RFC2445 iCalendar.
+
+The existing Ruby iCalendar libraries (e.g. icalendar, vpim) provide for parsing and generating icalendar files,
+but do not support important things like enumerating occurrences of repeating events.
+
+This is a clean-slate implementation of RFC2445.
+
+A Google group for discussion of this library has been set up http://groups.google.com/group/rical_gem
+    }
+    gem.email = "rick.denatale@gmail.com"
+    gem.homepage = "http://github.com/rubyredrick/ri_cal"
+    gem.authors = ["Rick DeNatale"]
+    ['.gitignore', 'performance_data/*', 'sample_ical_files/*', 'website/*', 'config/website.yml'].each do |excl|
+      gem.files.exclude excl
+    end
+    gem.extra_rdoc_files.include %w{History.txt copyrights.txt}
+    # gem.add_development_dependency "thoughtbot-shoulda", ">= 0"
+    # gem is a Gem::Specification... see http://www.rubygems.org/read/chapter/20 for additional settings
+  end
+  Jeweler::GemcutterTasks.new
+rescue LoadError
+  puts "Jeweler (or a dependency) not available. Install it with: gem install jeweler"
+end
+
+require 'rake/testtask'
+Rake::TestTask.new(:test) do |test|
+  test.libs << 'lib' << 'test'
+  test.pattern = 'test/**/test_*.rb'
+  test.verbose = true
 end
 
-require 'newgem/tasks' # load /tasks/*.rake
-
-# It looks like newgem is already defining these
-# Rake::TaskManager.class_eval do
-#   def remove_task(task_name)
-#     @tasks.delete(task_name.to_s)
-#   end
-# end
-#  
-# def remove_task(task_name)
-#   Rake.application.remove_task(task_name)
-# end
- 
-# Override hoe's standard spec task
-remove_task :spec
+begin
+  require 'rcov/rcovtask'
+  Rcov::RcovTask.new do |test|
+    test.libs << 'test'
+    test.pattern = 'test/**/test_*.rb'
+    test.verbose = true
+  end
+rescue LoadError
+  task :rcov do
+    abort "RCov is not available. In order to run rcov, you must: sudo gem install spicycode-rcov"
+  end
+end
 
 Dir['tasks/**/*.rake'].each { |t| load t }
 
 task :default => [:"spec:with_tzinfo_gem", :"spec:with_active_support"]
+
+
+require 'rake/rdoctask'
+Rake::RDocTask.new do |rdoc|
+  version = File.exist?('VERSION') ? File.read('VERSION') : ""
+
+  rdoc.rdoc_dir = 'rdoc'
+  rdoc.title = "ri_cal #{version}"
+  rdoc.rdoc_files.include('README*')
+  rdoc.rdoc_files.include('lib/**/*.rb')
+end

data/VERSION

@@ -0,0 +1 @@
+0.8.6

data/component_attributes/component_property_defs.yml

@@ -160,16 +160,16 @@ trigger:
     rfc_ref: '4.8.6.3 p 127-129'
 created:
     purpose: This property specifies the date and time that the calendar information was created by teh calendar user agent in the calendar store.
-    type: DateTime
+    type: ZuluDateTime
     rfc_ref: '4.8.7.1 pp 129-130'   
 dtstamp:
     purpose: This property indicates the date/time that the instance of the iCalendar object was created.
-    type: DateTime
+    type: ZuluDateTime
     auto_set: to_ical
     rfc_ref: '4.8.7.2 pp 130-131'   
 last-modified:
     purpose: This property specifies the date and time that the information associated with the calendar component was last revised in teh calendar store.
-    type: DateTime
+    type: ZuluDateTime
     rfc_ref: '4.8.7.3 p 131'   
 sequence:
     purpose: This property defines the revision sequence number of the calendar component within a sequence of revisions.

data/lib/ri_cal/component/t_z_info_timezone.rb

@@ -10,10 +10,11 @@ class RiCal::Component::TZInfoTimezone < RiCal::Component::Timezone
 
     def initialize(which, this_period, prev_period)
       @which = which
-      @onset = period_local_start(this_period)
       if prev_period
+        @onset = period_local_end(prev_period)
         @offset_from = format_rfc2445_offset(prev_period.utc_total_offset)
       else
+        @onset = period_local_start(this_period)
         @offset_from = format_rfc2445_offset(this_period.utc_total_offset)
       end
       @offset_to = format_rfc2445_offset(this_period.utc_total_offset)
@@ -21,8 +22,19 @@ class RiCal::Component::TZInfoTimezone < RiCal::Component::Timezone
       @rdates = []
     end
     
+    def daylight?
+      @which == "DAYLIGHT"
+    end
+    
+    def period_local_end(period)
+      (period.local_end || DateTime.parse("99990101T000000")).strftime("%Y%m%dT%H%M%S")
+    end
+    
+    # This assumes a 1 hour shift which is why we use the previous period local end when
+    # possible
     def period_local_start(period)
-      (period.local_start || DateTime.parse("16010101T000000")).strftime("%Y%m%dT%H%M%S")
+      shift = daylight? ? Rational(-1, 24) : Rational(1, 24)
+      ((period.local_start || DateTime.parse("16010101T000000")) + shift).strftime("%Y%m%dT%H%M%S")
     end
 
     def add_period(this_period)
@@ -41,7 +53,9 @@ class RiCal::Component::TZInfoTimezone < RiCal::Component::Timezone
     def export_to(export_stream)
       export_stream.puts "BEGIN:#{@which}"
       export_stream.puts "DTSTART:#{@onset}"
-      export_stream.puts "RDATE:#{@rdates.join(",")}"
+      @rdates.each do |rdate|
+        export_stream.puts "RDATE:#{rdate}"
+      end
       export_stream.puts "TZOFFSETFROM:#{@offset_from}"
       export_stream.puts "TZOFFSETTO:#{@offset_to}"
       export_stream.puts "TZNAME:#{@abbreviation}"

data/lib/ri_cal/core_extensions/date/conversions.rb

@@ -38,20 +38,6 @@ module RiCal
             self
           end if RUBY_VERSION < '1.9'
 
-          # Converts a Date instance to a Time, where the time is set to the beginning of the day.
-          # The timezone can be either :local or :utc (default :local).
-          #
-          # ==== Examples
-          #   date = Date.new(2007, 11, 10)  # => Sat, 10 Nov 2007
-          #
-          #   date.to_time                   # => Sat Nov 10 00:00:00 0800 2007
-          #   date.to_time(:local)           # => Sat Nov 10 00:00:00 0800 2007
-          #
-          #   date.to_time(:utc)             # => Sat Nov 10 00:00:00 UTC 2007
-          def to_time(form = :local)
-            ::Time.send("#{form}_time", year, month, day)
-          end
-
           # Converts a Date instance to a DateTime, where the time is set to the beginning of the day
           # and UTC offset is set to 0.
           #

data/lib/ri_cal/core_extensions/date_time/conversions.rb

@@ -38,12 +38,6 @@ module RiCal
             ::Date.new(year, month, day)
           end
 
-          # Attempts to convert self to a Ruby Time object; returns self if out of range of Ruby Time class
-          # If self has an offset other than 0, self will just be returned unaltered, since there's no clean way to map it to a Time
-          def to_time
-            self.offset == 0 ? ::Time.utc_time(year, month, day, hour, min, sec) : self
-          end
-
           # To be able to keep Times, Dates and DateTimes interchangeable on conversions
           def to_datetime
             self

data/lib/ri_cal/fast_date_time.rb

@@ -112,6 +112,14 @@ module RiCal
     #   end
     # end
 
+    def utc
+      if offset == 0
+        self
+      else
+        advance(:seconds => -offset, :offset => 0)
+      end
+    end
+
     def hms_to_seconds(hours, minutes, seconds)
       seconds + 60 *(minutes + (60 * hours))
     end

data/lib/ri_cal/properties/alarm.rb

@@ -332,26 +332,26 @@ module RiCal
       end
 
       def export_properties_to(export_stream) #:nodoc:
-        export_prop_to(export_stream, "ACTION", @action_property)
-        export_prop_to(export_stream, "ATTENDEE", @attendee_property)
         export_prop_to(export_stream, "DURATION", @duration_property)
+        export_prop_to(export_stream, "ATTENDEE", @attendee_property)
         export_prop_to(export_stream, "TRIGGER", @trigger_property)
         export_prop_to(export_stream, "DESCRIPTION", @description_property)
         export_prop_to(export_stream, "SUMMARY", @summary_property)
         export_prop_to(export_stream, "REPEAT", @repeat_property)
         export_prop_to(export_stream, "ATTACH", @attach_property)
+        export_prop_to(export_stream, "ACTION", @action_property)
       end
 
       def ==(o) #:nodoc:
         if o.class == self.class
-        (action_property == o.action_property) &&
-        (attendee_property == o.attendee_property) &&
         (duration_property == o.duration_property) &&
+        (attendee_property == o.attendee_property) &&
         (trigger_property == o.trigger_property) &&
         (description_property == o.description_property) &&
         (summary_property == o.summary_property) &&
         (repeat_property == o.repeat_property) &&
-        (attach_property == o.attach_property)
+        (attach_property == o.attach_property) &&
+        (action_property == o.action_property)
         else
            super
         end
@@ -359,14 +359,14 @@ module RiCal
 
       def initialize_copy(o) #:nodoc:
         super
-        action_property = action_property && action_property.dup
-        attendee_property = attendee_property && attendee_property.dup
         duration_property = duration_property && duration_property.dup
+        attendee_property = attendee_property && attendee_property.dup
         trigger_property = trigger_property && trigger_property.dup
         description_property = description_property && description_property.dup
         summary_property = summary_property && summary_property.dup
         repeat_property = repeat_property && repeat_property.dup
         attach_property = attach_property && attach_property.dup
+        action_property = action_property && action_property.dup
       end
 
       def add_date_times_to(required_timezones) #:nodoc:
@@ -374,7 +374,7 @@ module RiCal
 
       module ClassMethods #:nodoc:
         def property_parser #:nodoc:
-          {"ACTION"=>:action_property_from_string, "ATTACH"=>:attach_property_from_string, "REPEAT"=>:repeat_property_from_string, "SUMMARY"=>:summary_property_from_string, "TRIGGER"=>:trigger_property_from_string, "ATTENDEE"=>:attendee_property_from_string, "DURATION"=>:duration_property_from_string, "DESCRIPTION"=>:description_property_from_string}
+          {"ACTION"=>:action_property_from_string, "REPEAT"=>:repeat_property_from_string, "ATTENDEE"=>:attendee_property_from_string, "TRIGGER"=>:trigger_property_from_string, "ATTACH"=>:attach_property_from_string, "SUMMARY"=>:summary_property_from_string, "DESCRIPTION"=>:description_property_from_string, "DURATION"=>:duration_property_from_string}
         end
       end
 

data/lib/ri_cal/properties/calendar.rb

@@ -118,16 +118,16 @@ module RiCal
 
 
       def export_properties_to(export_stream) #:nodoc:
-        export_prop_to(export_stream, "PRODID", @prodid_property)
         export_prop_to(export_stream, "CALSCALE", @calscale_property)
+        export_prop_to(export_stream, "PRODID", @prodid_property)
         export_prop_to(export_stream, "VERSION", @version_property)
         export_prop_to(export_stream, "METHOD", @method_property)
       end
 
       def ==(o) #:nodoc:
         if o.class == self.class
-        (prodid_property == o.prodid_property) &&
         (calscale_property == o.calscale_property) &&
+        (prodid_property == o.prodid_property) &&
         (version_property == o.version_property) &&
         (method_property == o.method_property)
         else
@@ -137,8 +137,8 @@ module RiCal
 
       def initialize_copy(o) #:nodoc:
         super
-        prodid_property = prodid_property && prodid_property.dup
         calscale_property = calscale_property && calscale_property.dup
+        prodid_property = prodid_property && prodid_property.dup
         version_property = version_property && version_property.dup
         method_property = method_property && method_property.dup
       end

data/lib/ri_cal/properties/event.rb

@@ -41,7 +41,7 @@ module RiCal
 
 
       # return the the CREATED property
-      # which will be an instances of RiCal::PropertyValueDateTime
+      # which will be an instances of RiCal::PropertyValueZuluDateTime
       # 
       # [purpose (from RFC 2445)]
       # This property specifies the date and time that the calendar information was created by teh calendar user agent in the calendar store.
@@ -52,24 +52,24 @@ module RiCal
       end
 
       # set the CREATED property
-      # property value should be an instance of RiCal::PropertyValueDateTime
+      # property value should be an instance of RiCal::PropertyValueZuluDateTime
       def created_property=(property_value)
-        @created_property = property_value ? property_value.for_parent(self) : nil
+        @created_property = property_value
       end
 
       # set the value of the CREATED property
       def created=(ruby_value)
-        self.created_property= RiCal::PropertyValue::DateTime.convert(self, ruby_value)
+        self.created_property= RiCal::PropertyValue::ZuluDateTime.convert(self, ruby_value)
       end
 
       # return the value of the CREATED property
-      # which will be an instance of DateTime
+      # which will be an instance of ZuluDateTime
       def created
         created_property ? created_property.ruby_value : nil
       end
 
       def created_property_from_string(line) # :nodoc:
-        @created_property = RiCal::PropertyValue::DateTime.new(self, line)
+        @created_property = RiCal::PropertyValue::ZuluDateTime.new(self, line)
       end
 
 
@@ -173,7 +173,7 @@ module RiCal
 
 
       # return the the LAST-MODIFIED property
-      # which will be an instances of RiCal::PropertyValueDateTime
+      # which will be an instances of RiCal::PropertyValueZuluDateTime
       # 
       # [purpose (from RFC 2445)]
       # This property specifies the date and time that the information associated with the calendar component was last revised in teh calendar store.
@@ -184,24 +184,24 @@ module RiCal
       end
 
       # set the LAST-MODIFIED property
-      # property value should be an instance of RiCal::PropertyValueDateTime
+      # property value should be an instance of RiCal::PropertyValueZuluDateTime
       def last_modified_property=(property_value)
-        @last_modified_property = property_value ? property_value.for_parent(self) : nil
+        @last_modified_property = property_value
       end
 
       # set the value of the LAST-MODIFIED property
       def last_modified=(ruby_value)
-        self.last_modified_property= RiCal::PropertyValue::DateTime.convert(self, ruby_value)
+        self.last_modified_property= RiCal::PropertyValue::ZuluDateTime.convert(self, ruby_value)
       end
 
       # return the value of the LAST-MODIFIED property
-      # which will be an instance of DateTime
+      # which will be an instance of ZuluDateTime
       def last_modified
         last_modified_property ? last_modified_property.ruby_value : nil
       end
 
       def last_modified_property_from_string(line) # :nodoc:
-        @last_modified_property = RiCal::PropertyValue::DateTime.new(self, line)
+        @last_modified_property = RiCal::PropertyValue::ZuluDateTime.new(self, line)
       end
 
 
@@ -305,7 +305,7 @@ module RiCal
 
 
       # return the the DTSTAMP property
-      # which will be an instances of RiCal::PropertyValueDateTime
+      # which will be an instances of RiCal::PropertyValueZuluDateTime
       # 
       # [purpose (from RFC 2445)]
       # This property indicates the date/time that the instance of the iCalendar object was created.
@@ -316,24 +316,24 @@ module RiCal
       end
 
       # set the DTSTAMP property
-      # property value should be an instance of RiCal::PropertyValueDateTime
+      # property value should be an instance of RiCal::PropertyValueZuluDateTime
       def dtstamp_property=(property_value)
-        @dtstamp_property = property_value ? property_value.for_parent(self) : nil
+        @dtstamp_property = property_value
       end
 
       # set the value of the DTSTAMP property
       def dtstamp=(ruby_value)
-        self.dtstamp_property= RiCal::PropertyValue::DateTime.convert(self, ruby_value)
+        self.dtstamp_property= RiCal::PropertyValue::ZuluDateTime.convert(self, ruby_value)
       end
 
       # return the value of the DTSTAMP property
-      # which will be an instance of DateTime
+      # which will be an instance of ZuluDateTime
       def dtstamp
         dtstamp_property ? dtstamp_property.ruby_value : nil
       end
 
       def dtstamp_property_from_string(line) # :nodoc:
-        @dtstamp_property = RiCal::PropertyValue::DateTime.new(self, line)
+        @dtstamp_property = RiCal::PropertyValue::ZuluDateTime.new(self, line)
       end
 
 
@@ -1390,72 +1390,72 @@ module RiCal
       end
 
       def export_properties_to(export_stream) #:nodoc:
-        export_prop_to(export_stream, "REQUEST-STATUS", @request_status_property)
+        export_prop_to(export_stream, "RDATE", @rdate_property)
+        export_prop_to(export_stream, "CONTACT", @contact_property)
         export_prop_to(export_stream, "EXDATE", @exdate_property)
-        export_prop_to(export_stream, "TRANSP", @transp_property)
-        export_prop_to(export_stream, "DTSTAMP", @dtstamp_property)
         export_prop_to(export_stream, "CREATED", @created_property)
-        export_prop_to(export_stream, "CONTACT", @contact_property)
-        export_prop_to(export_stream, "CATEGORIES", @categories_property)
+        export_prop_to(export_stream, "DURATION", @duration_property)
         export_prop_to(export_stream, "DTEND", @dtend_property)
         export_prop_to(export_stream, "STATUS", @status_property)
-        export_prop_to(export_stream, "LAST-MODIFIED", @last_modified_property)
         export_prop_to(export_stream, "DTSTART", @dtstart_property)
         export_prop_to(export_stream, "RECURRENCE-ID", @recurrence_id_property)
+        export_prop_to(export_stream, "TRANSP", @transp_property)
+        export_prop_to(export_stream, "DTSTAMP", @dtstamp_property)
         export_prop_to(export_stream, "RESOURCES", @resources_property)
+        export_prop_to(export_stream, "CATEGORIES", @categories_property)
+        export_prop_to(export_stream, "LAST-MODIFIED", @last_modified_property)
+        export_prop_to(export_stream, "PRIORITY", @priority_property)
+        export_prop_to(export_stream, "GEO", @geo_property)
         export_prop_to(export_stream, "ATTENDEE", @attendee_property)
-        export_prop_to(export_stream, "DURATION", @duration_property)
         export_prop_to(export_stream, "UID", @uid_property)
+        export_prop_to(export_stream, "DESCRIPTION", @description_property)
         export_prop_to(export_stream, "URL", @url_property)
-        export_prop_to(export_stream, "PRIORITY", @priority_property)
+        export_prop_to(export_stream, "SUMMARY", @summary_property)
         export_prop_to(export_stream, "ORGANIZER", @organizer_property)
         export_prop_to(export_stream, "RRULE", @rrule_property)
-        export_prop_to(export_stream, "DESCRIPTION", @description_property)
-        export_prop_to(export_stream, "CLASS", @class_property)
-        export_prop_to(export_stream, "SUMMARY", @summary_property)
-        export_prop_to(export_stream, "GEO", @geo_property)
         export_prop_to(export_stream, "ATTACH", @attach_property)
-        export_prop_to(export_stream, "SEQUENCE", @sequence_property)
+        export_prop_to(export_stream, "CLASS", @class_property)
         export_prop_to(export_stream, "RELATED-TO", @related_to_property)
+        export_prop_to(export_stream, "REQUEST-STATUS", @request_status_property)
         export_prop_to(export_stream, "EXRULE", @exrule_property)
-        export_prop_to(export_stream, "RDATE", @rdate_property)
         export_prop_to(export_stream, "LOCATION", @location_property)
         export_prop_to(export_stream, "COMMENT", @comment_property)
+        export_prop_to(export_stream, "SEQUENCE", @sequence_property)
       end
 
       def ==(o) #:nodoc:
         if o.class == self.class
-        (request_status_property == o.request_status_property) &&
+        (rdate_property == o.rdate_property) &&
+        (contact_property == o.contact_property) &&
         (exdate_property == o.exdate_property) &&
-        (transp_property == o.transp_property) &&
-        (dtstamp_property == o.dtstamp_property) &&
         (created_property == o.created_property) &&
-        (contact_property == o.contact_property) &&
-        (categories_property == o.categories_property) &&
+        (duration_property == o.duration_property) &&
         (dtend_property == o.dtend_property) &&
         (status_property == o.status_property) &&
-        (last_modified_property == o.last_modified_property) &&
         (dtstart_property == o.dtstart_property) &&
         (recurrence_id_property == o.recurrence_id_property) &&
+        (transp_property == o.transp_property) &&
+        (dtstamp_property == o.dtstamp_property) &&
         (resources_property == o.resources_property) &&
+        (categories_property == o.categories_property) &&
+        (last_modified_property == o.last_modified_property) &&
+        (priority_property == o.priority_property) &&
+        (geo_property == o.geo_property) &&
         (attendee_property == o.attendee_property) &&
-        (duration_property == o.duration_property) &&
         (uid_property == o.uid_property) &&
+        (description_property == o.description_property) &&
         (url_property == o.url_property) &&
-        (priority_property == o.priority_property) &&
+        (summary_property == o.summary_property) &&
         (organizer_property == o.organizer_property) &&
         (rrule_property == o.rrule_property) &&
-        (description_property == o.description_property) &&
-        (class_property == o.class_property) &&
-        (summary_property == o.summary_property) &&
-        (geo_property == o.geo_property) &&
         (attach_property == o.attach_property) &&
-        (sequence_property == o.sequence_property) &&
+        (class_property == o.class_property) &&
         (related_to_property == o.related_to_property) &&
+        (request_status_property == o.request_status_property) &&
         (exrule_property == o.exrule_property) &&
-        (rdate_property == o.rdate_property) &&
         (location_property == o.location_property) &&
-        (comment_property == o.comment_property)
+        (comment_property == o.comment_property) &&
+        (sequence_property == o.sequence_property)
         else
            super
         end
@@ -1463,44 +1463,41 @@ module RiCal
 
       def initialize_copy(o) #:nodoc:
         super
-        request_status_property = request_status_property && request_status_property.dup
+        rdate_property = rdate_property && rdate_property.dup
+        contact_property = contact_property && contact_property.dup
         exdate_property = exdate_property && exdate_property.dup
-        transp_property = transp_property && transp_property.dup
-        dtstamp_property = dtstamp_property && dtstamp_property.dup
         created_property = created_property && created_property.dup
-        contact_property = contact_property && contact_property.dup
-        categories_property = categories_property && categories_property.dup
+        duration_property = duration_property && duration_property.dup
         dtend_property = dtend_property && dtend_property.dup
         status_property = status_property && status_property.dup
-        last_modified_property = last_modified_property && last_modified_property.dup
         dtstart_property = dtstart_property && dtstart_property.dup
         recurrence_id_property = recurrence_id_property && recurrence_id_property.dup
+        transp_property = transp_property && transp_property.dup
+        dtstamp_property = dtstamp_property && dtstamp_property.dup
         resources_property = resources_property && resources_property.dup
+        categories_property = categories_property && categories_property.dup
+        last_modified_property = last_modified_property && last_modified_property.dup
+        priority_property = priority_property && priority_property.dup
+        geo_property = geo_property && geo_property.dup
         attendee_property = attendee_property && attendee_property.dup
-        duration_property = duration_property && duration_property.dup
         uid_property = uid_property && uid_property.dup
+        description_property = description_property && description_property.dup
         url_property = url_property && url_property.dup
-        priority_property = priority_property && priority_property.dup
+        summary_property = summary_property && summary_property.dup
         organizer_property = organizer_property && organizer_property.dup
         rrule_property = rrule_property && rrule_property.dup
-        description_property = description_property && description_property.dup
-        class_property = class_property && class_property.dup
-        summary_property = summary_property && summary_property.dup
-        geo_property = geo_property && geo_property.dup
         attach_property = attach_property && attach_property.dup
-        sequence_property = sequence_property && sequence_property.dup
+        class_property = class_property && class_property.dup
         related_to_property = related_to_property && related_to_property.dup
+        request_status_property = request_status_property && request_status_property.dup
         exrule_property = exrule_property && exrule_property.dup
-        rdate_property = rdate_property && rdate_property.dup
         location_property = location_property && location_property.dup
         comment_property = comment_property && comment_property.dup
+        sequence_property = sequence_property && sequence_property.dup
       end
 
       def add_date_times_to(required_timezones) #:nodoc:
-        add_property_date_times_to(required_timezones, created_property)
         add_property_date_times_to(required_timezones, dtstart_property)
-        add_property_date_times_to(required_timezones, last_modified_property)
-        add_property_date_times_to(required_timezones, dtstamp_property)
         add_property_date_times_to(required_timezones, recurrence_id_property)
         add_property_date_times_to(required_timezones, dtend_property)
         add_property_date_times_to(required_timezones, exdate_property)
@@ -1509,7 +1506,7 @@ module RiCal
 
       module ClassMethods #:nodoc:
         def property_parser #:nodoc:
-          {"RELATED-TO"=>:related_to_property_from_string, "RDATE"=>:rdate_property_from_string, "DTEND"=>:dtend_property_from_string, "DTSTART"=>:dtstart_property_from_string, "TRANSP"=>:transp_property_from_string, "DTSTAMP"=>:dtstamp_property_from_string, "LOCATION"=>:location_property_from_string, "EXRULE"=>:exrule_property_from_string, "CONTACT"=>:contact_property_from_string, "URL"=>:url_property_from_string, "LAST-MODIFIED"=>:last_modified_property_from_string, "RESOURCES"=>:resources_property_from_string, "EXDATE"=>:exdate_property_from_string, "ATTACH"=>:attach_property_from_string, "UID"=>:uid_property_from_string, "SEQUENCE"=>:sequence_property_from_string, "CATEGORIES"=>:categories_property_from_string, "RECURRENCE-ID"=>:recurrence_id_property_from_string, "SUMMARY"=>:summary_property_from_string, "GEO"=>:geo_property_from_string, "CLASS"=>:class_property_from_string, "RRULE"=>:rrule_property_from_string, "STATUS"=>:status_property_from_string, "ATTENDEE"=>:attendee_property_from_string, "PRIORITY"=>:priority_property_from_string, "ORGANIZER"=>:organizer_property_from_string, "CREATED"=>:created_property_from_string, "REQUEST-STATUS"=>:request_status_property_from_string, "COMMENT"=>:comment_property_from_string, "DURATION"=>:duration_property_from_string, "DESCRIPTION"=>:description_property_from_string}
+          {"RDATE"=>:rdate_property_from_string, "COMMENT"=>:comment_property_from_string, "TRANSP"=>:transp_property_from_string, "DTEND"=>:dtend_property_from_string, "LOCATION"=>:location_property_from_string, "CONTACT"=>:contact_property_from_string, "RECURRENCE-ID"=>:recurrence_id_property_from_string, "URL"=>:url_property_from_string, "LAST-MODIFIED"=>:last_modified_property_from_string, "CLASS"=>:class_property_from_string, "EXDATE"=>:exdate_property_from_string, "UID"=>:uid_property_from_string, "SEQUENCE"=>:sequence_property_from_string, "EXRULE"=>:exrule_property_from_string, "ATTENDEE"=>:attendee_property_from_string, "ORGANIZER"=>:organizer_property_from_string, "GEO"=>:geo_property_from_string, "RRULE"=>:rrule_property_from_string, "RESOURCES"=>:resources_property_from_string, "ATTACH"=>:attach_property_from_string, "CATEGORIES"=>:categories_property_from_string, "SUMMARY"=>:summary_property_from_string, "PRIORITY"=>:priority_property_from_string, "DESCRIPTION"=>:description_property_from_string, "RELATED-TO"=>:related_to_property_from_string, "REQUEST-STATUS"=>:request_status_property_from_string, "DURATION"=>:duration_property_from_string, "STATUS"=>:status_property_from_string, "DTSTART"=>:dtstart_property_from_string, "DTSTAMP"=>:dtstamp_property_from_string, "CREATED"=>:created_property_from_string}
         end
       end