icalendar 2.11.2 → 2.12.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0d1b88e3b63c6a6c5865093d45c296bfc4f0d807922b79bbd071355f5bdf72a2
-  data.tar.gz: 2b0e5e61d9dbab1ec76aab2a97bf3055da0ee42bfcb294867e1cba5d60841d17
+  metadata.gz: c581285bbacae9839202046bea20f80be28315865013f9f59e50851fbbed7a34
+  data.tar.gz: 8be871aab3ba233a56a6442783f6a32cbf138ae145fc777bed68fdf4beff0826
 SHA512:
-  metadata.gz: 070602bed48d0b45c04dc6801944b0f2d8916a67ecc39194ff93a5bc6619aca5eafad06843cb39df72910f9babbe010310dcada5bce1a27d046297586a0d1042
-  data.tar.gz: f4e43c98d659fa97b4cffadb0c79ba34ef0ba077d181272747eaf0065301d1b540a3fa5169dcd8d1701a60076f1f748ec8d28a44d1aefaf179197437ecab2958
+  metadata.gz: e847fc2a27df55033006bfc9b9d1572563a9a9bafc8cf286959e82e4a47f0c64fcda9a7b7470e37031e635a1e9b086eda769b75108a02e020782c1683fad763e
+  data.tar.gz: 3a8f8503ace1467d352d586ede3e4c8c91b7ab5d1fec5271c5add7658014a1cc61fc4b2f2ec32b09d481746ff14139fb83b7d14f63b23526056b183bb9133de6

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 ## Unreleased
 
+## 2.12.1 - 2025-10-19
+- Fix a problem with invalid ics generation for calendars with custom properties that include a `tzid` parameter.
+
+## 2.12.0 - 2025-09-26
+- Support timezone lookup by Windows names - Ronak Gothi
+
 ## 2.11.2 - 2025-06-21
 - Deal with more bad value parameter values by falling back to the property default type
 

data/lib/icalendar/downcased_hash.rb

@@ -3,32 +3,65 @@
 require 'delegate'
 
 module Icalendar
-  class DowncasedHash < ::SimpleDelegator
+  class DowncasedHash
 
     def initialize(base)
-      super Hash.new
+      @obj = Hash.new
       base.each do |key, value|
         self[key] = value
       end
     end
 
     def []=(key, value)
-      __getobj__[key.to_s.downcase] = value
+      obj[key.to_s.downcase] = value
     end
 
     def [](key)
-      __getobj__[key.to_s.downcase]
+      obj[key.to_s.downcase]
     end
 
     def has_key?(key)
-      __getobj__.has_key? key.to_s.downcase
+      obj.has_key? key.to_s.downcase
     end
     alias_method :include?, :has_key?
     alias_method :member?, :has_key?
 
     def delete(key, &block)
-      __getobj__.delete key.to_s.downcase, &block
+      obj.delete key.to_s.downcase, &block
     end
+
+    def merge(*other_hashes)
+      Icalendar::DowncasedHash.new(obj).merge!(*other_hashes)
+    end
+
+    def merge!(*other_hashes)
+      other_hashes.each do |hash|
+        hash.each do |key, value|
+          self[key] = value
+        end
+      end
+      self
+    end
+
+    def empty?
+      obj.empty?
+    end
+
+    def each(&block)
+      obj.each &block
+    end
+
+    def map(&block)
+      obj.map &block
+    end
+
+    def ==(other)
+      obj == Icalendar::DowncasedHash(other).obj
+    end
+
+    protected
+
+    attr_reader :obj
   end
 
   def self.DowncasedHash(base)

data/lib/icalendar/offset/windows_to_iana.rb

@@ -0,0 +1,144 @@
+# frozen_string_literal: true
+
+# This module contains mappings from Windows timezone identifiers to Olson timezone identifiers.
+#
+# The data is taken from the unicode consortium [0], the proposal and rationale
+# for this mapping is also available at the unicode consortium [1].
+#
+# [0] https://www.unicode.org/cldr/cldr-aux/charts/29/supplemental/zone_tzid.html
+# [1] https://cldr.unicode.org/development/development-process/design-proposals/extended-windows-olson-zid-mapping
+
+module Icalendar
+  class Offset
+    class WindowsToIana < Offset
+      WINDOWS_TO_IANA = {
+        'AUS Central Standard Time' => 'Australia/Darwin',
+        'AUS Eastern Standard Time' => 'Australia/Sydney',
+        'Afghanistan Standard Time' => 'Asia/Kabul',
+        'Alaskan Standard Time' => 'America/Anchorage',
+        'Arab Standard Time' => 'Asia/Riyadh',
+        'Arabian Standard Time' => 'Asia/Dubai',
+        'Arabic Standard Time' => 'Asia/Baghdad',
+        'Argentina Standard Time' => 'America/Argentina/Buenos_Aires',
+        'Atlantic Standard Time' => 'America/Halifax',
+        'Azerbaijan Standard Time' => 'Asia/Baku',
+        'Azores Standard Time' => 'Atlantic/Azores',
+        'Bahia Standard Time' => 'America/Bahia',
+        'Bangladesh Standard Time' => 'Asia/Dhaka',
+        'Belarus Standard Time' => 'Europe/Minsk',
+        'Canada Central Standard Time' => 'America/Regina',
+        'Cape Verde Standard Time' => 'Atlantic/Cape_Verde',
+        'Caucasus Standard Time' => 'Asia/Yerevan',
+        'Cen. Australia Standard Time' => 'Australia/Adelaide',
+        'Central America Standard Time' => 'America/Guatemala',
+        'Central Asia Standard Time' => 'Asia/Almaty',
+        'Central Brazilian Standard Time' => 'America/Cuiaba',
+        'Central Europe Standard Time' => 'Europe/Budapest',
+        'Central European Standard Time' => 'Europe/Warsaw',
+        'Central Pacific Standard Time' => 'Pacific/Guadalcanal',
+        'Central Standard Time' => 'America/Chicago',
+        'Central Standard Time (Mexico)' => 'America/Mexico_City',
+        'China Standard Time' => 'Asia/Shanghai',
+        'Dateline Standard Time' => 'Etc/GMT+12',
+        'E. Africa Standard Time' => 'Africa/Nairobi',
+        'E. Australia Standard Time' => 'Australia/Brisbane',
+        'E. Europe Standard Time' => 'Europe/Chisinau',
+        'E. South America Standard Time' => 'America/Sao_Paulo',
+        'Eastern Standard Time' => 'America/New_York',
+        'Eastern Standard Time (Mexico)' => 'America/Cancun',
+        'Egypt Standard Time' => 'Africa/Cairo',
+        'Ekaterinburg Standard Time' => 'Asia/Yekaterinburg',
+        'FLE Standard Time' => 'Europe/Kyiv',
+        'Fiji Standard Time' => 'Pacific/Fiji',
+        'GMT Standard Time' => 'Europe/London',
+        'GTB Standard Time' => 'Europe/Bucharest',
+        'Georgian Standard Time' => 'Asia/Tbilisi',
+        'Greenland Standard Time' => 'America/Nuuk',
+        'Greenwich Standard Time' => 'Atlantic/Reykjavik',
+        'Hawaiian Standard Time' => 'Pacific/Honolulu',
+        'India Standard Time' => 'Asia/Kolkata',
+        'Iran Standard Time' => 'Asia/Tehran',
+        'Israel Standard Time' => 'Asia/Jerusalem',
+        'Jordan Standard Time' => 'Asia/Amman',
+        'Kaliningrad Standard Time' => 'Europe/Kaliningrad',
+        'Korea Standard Time' => 'Asia/Seoul',
+        'Libya Standard Time' => 'Africa/Tripoli',
+        'Line Islands Standard Time' => 'Pacific/Kiritimati',
+        'Magadan Standard Time' => 'Asia/Magadan',
+        'Mauritius Standard Time' => 'Indian/Mauritius',
+        'Middle East Standard Time' => 'Asia/Beirut',
+        'Montevideo Standard Time' => 'America/Montevideo',
+        'Morocco Standard Time' => 'Africa/Casablanca',
+        'Mountain Standard Time' => 'America/Denver',
+        'Mountain Standard Time (Mexico)' => 'America/Chihuahua',
+        'Myanmar Standard Time' => 'Asia/Yangon',
+        'N. Central Asia Standard Time' => 'Asia/Novosibirsk',
+        'Namibia Standard Time' => 'Africa/Windhoek',
+        'Nepal Standard Time' => 'Asia/Kathmandu',
+        'New Zealand Standard Time' => 'Pacific/Auckland',
+        'Newfoundland Standard Time' => 'America/St_Johns',
+        'North Asia East Standard Time' => 'Asia/Irkutsk',
+        'North Asia Standard Time' => 'Asia/Krasnoyarsk',
+        'North Korea Standard Time' => 'Asia/Pyongyang',
+        'Pacific SA Standard Time' => 'America/Santiago',
+        'Pacific Standard Time' => 'America/Los_Angeles',
+        'Pakistan Standard Time' => 'Asia/Karachi',
+        'Paraguay Standard Time' => 'America/Asuncion',
+        'Romance Standard Time' => 'Europe/Paris',
+        'Russia Time Zone 10' => 'Asia/Srednekolymsk',
+        'Russia Time Zone 11' => 'Asia/Kamchatka',
+        'Russia Time Zone 3' => 'Europe/Samara',
+        'Russian Standard Time' => 'Europe/Moscow',
+        'SA Eastern Standard Time' => 'America/Cayenne',
+        'SA Pacific Standard Time' => 'America/Bogota',
+        'SA Western Standard Time' => 'America/La_Paz',
+        'SE Asia Standard Time' => 'Asia/Bangkok',
+        'Samoa Standard Time' => 'Pacific/Apia',
+        'Singapore Standard Time' => 'Asia/Singapore',
+        'South Africa Standard Time' => 'Africa/Johannesburg',
+        'Sri Lanka Standard Time' => 'Asia/Colombo',
+        'Syria Standard Time' => 'Asia/Damascus',
+        'Taipei Standard Time' => 'Asia/Taipei',
+        'Tasmania Standard Time' => 'Australia/Hobart',
+        'Tokyo Standard Time' => 'Asia/Tokyo',
+        'Tonga Standard Time' => 'Pacific/Tongatapu',
+        'Turkey Standard Time' => 'Europe/Istanbul',
+        'US Eastern Standard Time' => 'America/Indiana/Indianapolis',
+        'US Mountain Standard Time' => 'America/Phoenix',
+        'UTC' => 'Etc/GMT',
+        'UTC+12' => 'Etc/GMT-12',
+        'UTC-02' => 'Etc/GMT+2',
+        'UTC-11' => 'Etc/GMT+11',
+        'Ulaanbaatar Standard Time' => 'Asia/Ulaanbaatar',
+        'Venezuela Standard Time' => 'America/Caracas',
+        'Vladivostok Standard Time' => 'Asia/Vladivostok',
+        'W. Australia Standard Time' => 'Australia/Perth',
+        'W. Central Africa Standard Time' => 'Africa/Lagos',
+        'W. Europe Standard Time' => 'Europe/Berlin',
+        'West Asia Standard Time' => 'Asia/Tashkent',
+        'West Pacific Standard Time' => 'Pacific/Port_Moresby',
+        'Yakutsk Standard Time' => 'Asia/Yakutsk'
+      }.freeze
+
+      def valid?
+        support_classes_defined? && tz
+      end
+
+      def normalized_value
+        Icalendar.logger.debug("Plan a - parsing #{value}/#{tzid} as ActiveSupport::TimeWithZone")
+        # plan a - use ActiveSupport::TimeWithZone
+        Icalendar::Values::Helpers::ActiveSupportTimeWithZoneAdapter.new(nil, tz, value)
+      end
+
+      def normalized_tzid
+        [WINDOWS_TO_IANA[tzid]].compact
+      end
+
+      private
+
+      def tz
+        @tz ||= ActiveSupport::TimeZone[normalized_tzid.first || '']
+      end
+    end
+  end
+end

data/lib/icalendar/offset.rb

@@ -10,6 +10,7 @@ module Icalendar
       [
         Icalendar::Offset::ActiveSupportExact,
         Icalendar::Offset::TimeZoneStore,
+        Icalendar::Offset::WindowsToIana,
         Icalendar::Offset::ActiveSupportPartial,
         Icalendar::Offset::Null
       ].lazy.map { |klass| klass.new(tzid, value, timezone_store) }.detect(&:valid?)
@@ -36,7 +37,8 @@ module Icalendar
   end
 end
 
-require_relative "offset/active_support_exact"
-require_relative "offset/active_support_partial"
-require_relative "offset/null"
-require_relative "offset/time_zone_store"
+require_relative 'offset/active_support_exact'
+require_relative 'offset/active_support_partial'
+require_relative 'offset/null'
+require_relative 'offset/time_zone_store'
+require_relative 'offset/windows_to_iana'

data/lib/icalendar/parser.rb

@@ -87,9 +87,10 @@ module Icalendar
         Icalendar::Values::Helpers::Array.new fields[:value].split(WRAP_PROPERTY_VALUE_SPLIT_REGEX),
                                      klass,
                                      fields[:params],
-                                     delimiter: delimiter
+                                     delimiter: delimiter,
+                                     timezone_store: timezone_store
       else
-        klass.new fields[:value], fields[:params]
+        klass.new fields[:value], fields[:params], timezone_store: timezone_store
       end
     rescue Icalendar::Values::DateTime::FormatError => fe
       raise fe if strict?
@@ -215,9 +216,6 @@ module Icalendar
           if param_value.size > 0
             param_value = param_value.gsub(PVALUE_GSUB_REGEX, '')
             params[param_name] << param_value
-            if param_name == 'tzid'
-              params['x-tz-store'] = timezone_store
-            end
           end
         end
       end

data/lib/icalendar/value.rb

@@ -8,10 +8,11 @@ module Icalendar
 
   class Value < ::SimpleDelegator
 
-    attr_accessor :ical_params
+    attr_accessor :ical_params, :context
 
-    def initialize(value, params = {})
+    def initialize(value, params = {}, context = {})
       @ical_params = Icalendar::DowncasedHash(params)
+      @context = Icalendar::DowncasedHash(context)
       super value
     end
 

data/lib/icalendar/values/boolean.rb

@@ -5,8 +5,8 @@ module Icalendar
 
     class Boolean < Value
 
-      def initialize(value, params = {})
-        super value.to_s.downcase == 'true', params
+      def initialize(value, *args)
+        super value.to_s.downcase == 'true', *args
       end
 
       def value_ical
@@ -16,4 +16,4 @@ module Icalendar
     end
 
   end
-end
+end

data/lib/icalendar/values/date.rb

@@ -8,9 +8,8 @@ module Icalendar
     class Date < Value
       FORMAT = '%Y%m%d'
 
-      def initialize(value, params = {})
+      def initialize(value, params = {}, *args)
         params.delete 'tzid'
-        params.delete 'x-tz-store'
         if value.is_a? String
           begin
             parsed_date = ::Date.strptime(value, FORMAT)
@@ -18,9 +17,9 @@ module Icalendar
             raise FormatError.new("Failed to parse \"#{value}\" - #{e.message}")
           end
 
-          super parsed_date, params
+          super parsed_date, params, *args
         elsif value.respond_to? :to_date
-          super value.to_date, params
+          super value.to_date, params, *args
         else
           super
         end

data/lib/icalendar/values/date_time.rb

@@ -11,7 +11,7 @@ module Icalendar
 
       FORMAT = '%Y%m%dT%H%M%S'
 
-      def initialize(value, params = {})
+      def initialize(value, params = {}, *args)
         if value.is_a? String
           params['tzid'] = 'UTC' if value.end_with? 'Z'
 
@@ -21,9 +21,9 @@ module Icalendar
             raise FormatError.new("Failed to parse \"#{value}\" - #{e.message}")
           end
 
-          super parsed_date, params
+          super parsed_date, params, *args
         elsif value.respond_to? :to_datetime
-          super value.to_datetime, params
+          super value.to_datetime, params, *args
         else
           super
         end

data/lib/icalendar/values/duration.rb

@@ -7,11 +7,11 @@ module Icalendar
 
     class Duration < Value
 
-      def initialize(value, params = {})
+      def initialize(value, *args)
         if value.is_a? Icalendar::Values::Duration
-          super value.value, params
+          super value.value, *args
         else
-          super OpenStruct.new(parse_fields value), params
+          super OpenStruct.new(parse_fields value), *args
         end
       end
 

data/lib/icalendar/values/float.rb

@@ -5,8 +5,8 @@ module Icalendar
 
     class Float < Value
 
-      def initialize(value, params = {})
-        super value.to_f, params
+      def initialize(value, *args)
+        super value.to_f, *args
       end
 
       def value_ical
@@ -16,4 +16,4 @@ module Icalendar
     end
 
   end
-end
+end

data/lib/icalendar/values/helpers/array.rb

@@ -8,22 +8,23 @@ module Icalendar
 
         attr_reader :value_delimiter
 
-        def initialize(value, klass, params = {}, options = {})
-          @value_delimiter = options[:delimiter] || ','
+        def initialize(value, klass, params = {}, context = {})
+          context = Icalendar::DowncasedHash(context)
+          @value_delimiter = context['delimiter'] || ','
           mapped = if value.is_a? ::Array
                     value.map do |v|
                       if v.is_a? Icalendar::Values::Helpers::Array
-                        Icalendar::Values::Helpers::Array.new v.value, klass, v.ical_params, delimiter: v.value_delimiter
+                        Icalendar::Values::Helpers::Array.new v.value, klass, v.ical_params, context.merge(delimiter: v.value_delimiter)
                       elsif v.is_a? ::Array
-                        Icalendar::Values::Helpers::Array.new v, klass, params, delimiter: value_delimiter
+                        Icalendar::Values::Helpers::Array.new v, klass, params, context.merge(delimiter: value_delimiter)
                       elsif v.is_a? Icalendar::Value
                         v
                       else
-                        klass.new v, params
+                        klass.new v, params, context
                       end
                     end
                   else
-                    [klass.new(value, params)]
+                    [klass.new(value, params, context)]
                   end
           super mapped
         end

data/lib/icalendar/values/helpers/time_with_zone.rb

@@ -19,17 +19,20 @@ module Icalendar
       module TimeWithZone
         attr_reader :tz_utc, :timezone_store
 
-        def initialize(value, params = {})
+        def initialize(value, params = {}, context = {})
           params = Icalendar::DowncasedHash(params)
+          context = Icalendar::DowncasedHash(context)
           @tz_utc = params['tzid'] == 'UTC'
-          @timezone_store = params.delete 'x-tz-store'
+          @timezone_store = context['timezone_store']
 
           offset = Icalendar::Offset.build(value, params, timezone_store)
 
-          @offset_value = offset&.normalized_value
-          params['tzid'] = offset.normalized_tzid if offset
+          unless offset.nil?
+            @offset_value = offset.normalized_value
+            params['tzid'] = offset.normalized_tzid
+          end
 
-          super (@offset_value || value), params
+          super (@offset_value || value), params, context
         end
 
         def __getobj__

data/lib/icalendar/values/integer.rb

@@ -5,8 +5,8 @@ module Icalendar
 
     class Integer < Value
 
-      def initialize(value, params = {})
-        super value.to_i, params
+      def initialize(value, *args)
+        super value.to_i, *args
       end
 
       def value_ical
@@ -16,4 +16,4 @@ module Icalendar
     end
 
   end
-end
+end

data/lib/icalendar/values/period.rb

@@ -7,7 +7,7 @@ module Icalendar
 
       PERIOD_LAST_PART_REGEX = /\A[+-]?P.+\z/.freeze
 
-      def initialize(value, params = {})
+      def initialize(value, *args)
         parts = value.split '/'
         period_start = Icalendar::Values::DateTime.new parts.first
         if parts.last =~ PERIOD_LAST_PART_REGEX
@@ -15,7 +15,7 @@ module Icalendar
         else
           period_end = Icalendar::Values::DateTime.new parts.last
         end
-        super [period_start, period_end], params
+        super [period_start, period_end], *args
       end
 
       def value_ical
@@ -47,4 +47,4 @@ module Icalendar
       end
     end
   end
-end
+end

data/lib/icalendar/values/recur.rb

@@ -12,11 +12,11 @@ module Icalendar
       MONTHDAY = '[+-]?\d{1,2}'
       YEARDAY = '[+-]?\d{1,3}'
 
-      def initialize(value, params = {})
+      def initialize(value, *args)
         if value.is_a? Icalendar::Values::Recur
-          super value.value, params
+          super value.value, *args
         else
-          super OpenStruct.new(parse_fields value), params
+          super OpenStruct.new(parse_fields value), *args
         end
       end
 

data/lib/icalendar/values/text.rb

@@ -3,12 +3,12 @@
 module Icalendar
   module Values
     class Text < Value
-      def initialize(value, params = {})
+      def initialize(value, *args)
         value = value.gsub('\n', "\n")
         value.gsub!('\,', ',')
         value.gsub!('\;', ';')
         value.gsub!('\\\\') { '\\' }
-        super value, params
+        super value, *args
       end
 
       VALUE_ICAL_CARRIAGE_RETURN_GSUB_REGEX = /\r?\n/.freeze

data/lib/icalendar/values/time.rb

@@ -11,12 +11,12 @@ module Icalendar
 
       FORMAT = '%H%M%S'
 
-      def initialize(value, params = {})
+      def initialize(value, params = {}, *args)
         if value.is_a? String
           params['tzid'] = 'UTC' if value.end_with? 'Z'
-          super ::DateTime.strptime(value, FORMAT).to_time, params
+          super ::DateTime.strptime(value, FORMAT).to_time, params, *args
         elsif value.respond_to? :to_time
-          super value.to_time, params
+          super value.to_time, params, *args
         else
           super
         end
@@ -33,4 +33,4 @@ module Icalendar
     end
 
   end
-end
+end

data/lib/icalendar/values/uri.rb

@@ -7,9 +7,9 @@ module Icalendar
 
     class Uri < Value
 
-      def initialize(value, params = {})
+      def initialize(value, *args)
         parsed = URI.parse(value) rescue value
-        super parsed, params
+        super parsed, *args
       end
 
       def value_ical

data/lib/icalendar/values/utc_offset.rb

@@ -5,13 +5,13 @@ require 'ostruct'
 module Icalendar
   module Values
     class UtcOffset < Value
-      def initialize(value, params = {})
+      def initialize(value, *args)
         if value.is_a? Icalendar::Values::UtcOffset
           value = value.value
         else
           value = OpenStruct.new parse_fields(value)
         end
-        super value, params
+        super value, *args
       end
 
       def behind?

data/lib/icalendar/version.rb

@@ -2,6 +2,6 @@
 
 module Icalendar
 
-  VERSION = '2.11.2'
+  VERSION = '2.12.1'
 
 end

data/spec/fixtures/tz_store_param_bug.ics

@@ -0,0 +1,30 @@
+BEGIN:VCALENDAR
+METHOD:COUNTER
+PRODID:Microsoft Exchange Server 2010
+VERSION:2.0
+BEGIN:VTIMEZONE
+TZID:Pacific Standard Time
+BEGIN:STANDARD
+DTSTART:16010101T020000
+TZOFFSETFROM:-0700
+TZOFFSETTO:-0800
+RRULE:FREQ=YEARLY;INTERVAL=1;BYDAY=1SU;BYMONTH=11
+END:STANDARD
+BEGIN:DAYLIGHT
+DTSTART:16010101T020000
+TZOFFSETFROM:-0800
+TZOFFSETTO:-0700
+RRULE:FREQ=YEARLY;INTERVAL=1;BYDAY=2SU;BYMONTH=3
+END:DAYLIGHT
+END:VTIMEZONE
+BEGIN:VEVENT
+DTSTART;TZID=Pacific Standard Time:20251008T103000
+DTEND;TZID=Pacific Standard Time:20251008T113000
+X-MS-OLK-ORIGINALSTART;TZID=Pacific Standard Time:20251008T110000
+X-MS-OLK-ORIGINALEND;TZID=Pacific Standard Time:20251008T120000
+UID:1sqt3u5rn2fprt7g7u75obcgav@google.com
+SEQUENCE:0
+DTSTAMP:20251007T184328Z
+COMMENT;LANGUAGE=en-US:\n
+END:VEVENT
+END:VCALENDAR

data/spec/parser_spec.rb

@@ -126,4 +126,18 @@ describe Icalendar::Parser do
       expect(event.location.value).to eq "1000 Main St Example, State 12345"
     end
   end
+
+  describe 'custom properties with tzid' do
+    let(:fn) { 'tz_store_param_bug.ics' }
+
+    it 'parses without error' do
+      expect(subject.parse.first).to be_a Icalendar::Calendar
+    end
+
+    it 'can be output to ics and re-parsed without error' do
+      cal = subject.parse.first
+      new_cal = Icalendar::Parser.new(cal.to_ical, false).parse.first
+      expect(new_cal).to be_a Icalendar::Calendar
+    end
+  end
 end

data/spec/values/date_time_spec.rb

@@ -17,7 +17,7 @@ describe Icalendar::Values::DateTime do
         expect(subject.value_ical).to eq value
       end
 
-      context 'local time' do
+      context 'tzid in IANA format' do
         let(:value) { '20140209T160652' }
         let(:params) { {'tzid' => 'America/Denver'} }
 
@@ -28,14 +28,15 @@ describe Icalendar::Values::DateTime do
         end
       end
 
-      context 'nonstandard format tzid local time' do
+      context 'tzid with partial match' do
         let(:value) { '20230901T230404' }
-        let(:params) { {'tzid' => 'Singapore Standard Time'} }
+        let(:params) { { 'tzid' => 'Singapore Time' } }
 
         it 'parses the value as local time' do
           expect(subject.value.hour).to eq 23
-          expect(subject.value.utc_offset).to eq 28800
+          expect(subject.value.utc_offset).to eq 28_800
           expect(subject.value.utc.hour).to eq 15
+          expect(subject.value.utc.minute).to eq 04
         end
 
         it 'updates the tzid' do
@@ -43,8 +44,24 @@ describe Icalendar::Values::DateTime do
           expect(subject.ical_params['tzid']).to eq ['Asia/Singapore']
         end
       end
-    end
 
+      context 'tzid in windows format' do
+        let(:value) { '20230901T230404' }
+        let(:params) { { 'tzid' => 'India Standard Time' } }
+
+        it 'parses the value as local time' do
+          expect(subject.value.hour).to eq 23
+          expect(subject.value.utc_offset).to eq 19_800
+          expect(subject.value.utc.hour).to eq 17
+          expect(subject.value.utc.minute).to eq 34
+        end
+
+        it 'updates the tzid' do
+          # use an array because that's how output from the parser will end up
+          expect(subject.ical_params['tzid']).to eq ['Asia/Kolkata']
+        end
+      end
+    end
   else
 
     context 'without ActiveSupport' do
@@ -63,10 +80,8 @@ describe Icalendar::Values::DateTime do
         end
       end
     end
-
   end
 
-
   context 'common tests' do
     it 'does not add any tzid parameter to output' do
       expect(subject.to_ical described_class).to eq ":#{value}"

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: icalendar
 version: !ruby/object:Gem::Version
-  version: 2.11.2
+  version: 2.12.1
 platform: ruby
 authors:
 - Ryan Ahearn
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-06-21 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base64
@@ -228,6 +227,7 @@ files:
 - lib/icalendar/offset/active_support_partial.rb
 - lib/icalendar/offset/null.rb
 - lib/icalendar/offset/time_zone_store.rb
+- lib/icalendar/offset/windows_to_iana.rb
 - lib/icalendar/parser.rb
 - lib/icalendar/timezone.rb
 - lib/icalendar/timezone_store.rb
@@ -275,6 +275,7 @@ files:
 - spec/fixtures/two_day_events.ics
 - spec/fixtures/two_events.ics
 - spec/fixtures/two_time_events.ics
+- spec/fixtures/tz_store_param_bug.ics
 - spec/fixtures/tzid_search.ics
 - spec/freebusy_spec.rb
 - spec/journal_spec.rb
@@ -317,8 +318,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.27
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: A ruby implementation of the iCalendar specification (RFC-5545).
 test_files:
@@ -344,6 +344,7 @@ test_files:
 - spec/fixtures/two_day_events.ics
 - spec/fixtures/two_events.ics
 - spec/fixtures/two_time_events.ics
+- spec/fixtures/tz_store_param_bug.ics
 - spec/fixtures/tzid_search.ics
 - spec/freebusy_spec.rb
 - spec/journal_spec.rb