tristanpriem / qcal

Automatically exported from code.google.com/p/qcal
0 stars 0 forks source link

error on tzid for property_dtstart/dtend #20

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
adding a new property like this:
new qCal_Property_Dtend($timestamp, array('tzid'=>'GMT'))

these lines in qCal_Component produce an error:

241: $timezones = $this->getTimezones(); 
242: $tzids = array_keys($timezones);

the error occurs because the following line asks for "array_key_exists" but
the actual tzids are in the value fields of the array, not in the keys
($timezones has it the right way)

262: if (!array_key_exists($tzid, $tzids)) {

it works like this:

$timezones = $this->getTimezones(); 
$tzids = $timezones; // TODO: THIS IS A FIX!

furthermore an "strtoupper" prior to the array_key_exists check would be
nice as well, as getTimeszones() always has strtoupper keys (tzids) 

Original issue reported on code.google.com by cjaent...@googlemail.com on 11 May 2010 at 11:34