Comments on: A very simple unassuming ICS parser

By: Michael

Michael — Wed, 27 Jan 2010 18:30:04 +0000

And, thanks, Aidan! This works like a charm, once the class name is corrected.

Michael — Wed, 27 Jan 2010 18:24:28 +0000

This won’t work! The name of the class needs to be iCalReader

file, class name, and constructor all should have the same name!!!

Will — Mon, 16 Nov 2009 04:56:22 +0000

Please note that there is now a “\” before the s in the preg_split line mentioned above.

Will — Mon, 16 Nov 2009 04:55:08 +0000

I agree with Andrew. I also changed the preg_split line to include two lines:

$source = preg_replace(‘/(\r\n|\n) /m’,”,$source);
$source = preg_split(‘#\n(?!\s)#’, $source);

I changed line 161 ( list($key, $value) = explode(‘:’, $line, 2); ) to this:

$temp = explode(‘:’, $line, 2);
$key = !empty($temp[0])?$temp[0]:”;
$value = !empty($temp[1])?$temp[1]:”;

Andrew — Mon, 22 Jun 2009 20:16:37 +0000

Hi Aidan -

Thanks for this class! Two things:

1) Shouldn’t function iCalReader($source) be function ICSReader($source)?

2) Why are you replacing commas and semi-colons (without actually replacing them) on lines 165-168?