Skip Menu |

This queue is for tickets about the Data-ICal CPAN distribution.

Report information
The Basics
Id: 65447
Status: resolved
Priority: 0/
Queue: Data-ICal
People
Owner: Nobody in particular
Requestors: jordi.amo [...] gmail.com
njh [...] bandsman.co.uk
Cc: alexm [...] cpan.org
AdminCc:
Bug Information
Severity: Important
Broken in: 0.18
Fixed in: (no value)

History Show all quoted text
  Fri Feb 04 05:50:50 2011 jordi.amo [...] gmail.com - Ticket created
Subject: prodid and version parameters order
Date: Fri, 4 Feb 2011 11:50:15 +0100
To: bug-Data-ICal [...] rt.cpan.org
From: Jordi Amorós <jordi.amo [...] gmail.com>
Dear mantainers, I've 'discovered'* that the files generated with Data::ICal have an issue: the order of prodid and version parameters is not correct. "BEGIN:VCALENDAR PRODID:-//Smeagol Server //VERSION 2.x//EN VERSION:2.0 ... " should be: "BEGIN:VCALENDAR VERSION:2.0 PRODID:-//Smeagol Server //VERSION 2.x//EN ... " If you fix that, then the .ics files generated will become perfect. Thanks for your great job! :D *ESL, I'm not sure if discover is the appropriate word to use here.
  Fri Feb 04 06:13:18 2011 jordi.amo [...] gmail.com - Correspondence added
Subject: [rt.cpan.org #65447] Additional information
Date: Fri, 4 Feb 2011 12:12:50 +0100
To: bug-Data-ICal [...] rt.cpan.org
From: Jordi Amorós <jordi.amo [...] gmail.com>
Data::ICal version, copied and pasted from /usr/share/perl5/Data/ICal.pm: $VERSION = '0.16'; $perl -v This is perl, v5.10.1 (*) built for i686-linux-gnu-thread-multi .......@m83:~$ uname -a Linux m83 2.6.35-25-generic #44-Ubuntu SMP Fri Jan 21 17:40:48 UTC 2011 i686 GNU/Linux
  Tue Feb 15 05:46:03 2011 alexm [...] cpan.org - Cc ALEXM added
  Wed Jun 22 15:23:47 2011 njh [...] bandsman.co.uk - Ticket #69034: Ticket created
Subject: VERSION is not put first
According to http://icalvalid.cloudapp.net/Default.aspx, "The VERSION property should be the first property on the calendar. Some calendar clients cannot properly parse the calendar otherwise." However, with Data::Ical, VERSION doesn't appear first: BEGIN:VCALENDAR CALSCALE:GREGORIAN METHOD:PUBLISH PRODID:Data::ICal 0.16 VERSION:2.0 ...
  Mon Oct 15 18:29:48 2012 tsibley [...] cpan.org - Ticket #69034: Merged into ticket #65447
  Mon Oct 15 18:29:48 2012 tsibley [...] cpan.org - Merged into ticket #65447
  Mon Oct 15 18:30:19 2012 tsibley [...] cpan.org - Subject changed from 'prodid and version parameters order' to 'VERSION property must be first in output'
  Mon Oct 15 18:30:19 2012 tsibley [...] cpan.org - Severity Important added
  Mon Oct 15 18:30:19 2012 tsibley [...] cpan.org - Broken in 0.18 added
  Tue Oct 16 04:50:36 2012 nate [...] cpan.org - Correspondence added
An ugly quickfix could be replacing line 107 in ICal/Entry.pm with the following: # RFC2445 implies an order (see 4.6 Calendar Components) but does not require it, # however some applications break if VERSION is not first, see # http://icalvalid.cloudapp.net/Default.aspx for my $name ( sort { if ($a eq 'version') { return -1 }; if ($b eq 'version') { return 1 }; } keys %{ $self->properties } ) {
  Tue Oct 16 04:50:38 2012 The RT System itself - Status changed from 'new' to 'open'
  Mon Dec 03 17:43:42 2012 tsibley [...] cpan.org - Correspondence added
Fixed in 0.19, just pushed to CPAN now. Thanks for the report and suggested patch! https://metacpan.org/release/TSIBLEY/Data-ICal-0.19
  Mon Dec 03 17:43:44 2012 tsibley [...] cpan.org - Status changed from 'open' to 'resolved'