wjase / ics-parser

Automatically exported from code.google.com/p/ics-parser
0 stars 0 forks source link

SUMMARY tag with LANGUAGE setting ignored completely #7

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. ICS Line: SUMMARY;LANGUAGE=en-us:Office Workgroup Focus Group

What is the expected output? What do you see instead?
Should return an array of:
('SUMMARY' => 'Office Workgroup Focus Group')

Instead returns an array of:
('SUMMARY' => '')

What version of the product are you using? On what operating system?
r13
Released: Sep 18, 2011

Please provide any additional information below.
Fixed by adding a case for the SUMMARY under function __construct($filename)

This issue might also affect timezone settings:
eg.
DTEND;TZID="Eastern Standard Time":20130502T180000
DTSTART;TZID="Eastern Standard Time":20130502T130000
etc.

But admittedly I haven't tested that yet.

Original issue reported on code.google.com by brianate...@gmail.com on 2 May 2013 at 2:13

GoogleCodeExporter commented 9 years ago
Added under the 'default' case:

        // SUMMARY with LANGUAGE setting
        case "SUMMARY;LANGUAGE=en-us":
            $type = "SUMMARY";
            break;

Original comment by brianate...@gmail.com on 2 May 2013 at 2:25

GoogleCodeExporter commented 9 years ago
This was fixed on the new version which also handles RRULE for recurring events.

Can be found here: https://github.com/johngrogg/ics-parser

Original comment by brianate...@gmail.com on 17 Jul 2013 at 7:50