Tickets :: [#6563] iCalendar can't parse GEO-vCard-Tags

6.0.0-beta1

7/13/25

Summary	iCalendar can't parse GEO-vCard-Tags
Queue	Synchronization
Type	Bug
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	horde (at) ssn (dot) at
Created	04/03/2008 (6310 days ago)
Due
Updated	08/20/2008 (6171 days ago)
Assigned	04/04/2008 (6309 days ago)
Resolved	08/20/2008 (6171 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

08/20/2008 07:26:24 PM	CVS Commit	Comment #8	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/framework/iCalendar/iCalendar.php?r1=1.57.4.65&r2=1.57.4.66&ty=u http://cvs.horde.org/co.php/framework/iCalendar/tests/geo.phpt?r=1.1.2.1 http://cvs.horde.org/diff.php/horde/docs/CHANGES?r1=1.515.2.445&r2=1.515.2.446&ty=u

08/20/2008 06:57:16 PM	Jan Schneider	Comment #7 State ⇒ Resolved	Reply to this comment
The phone was sending correct data. vCard 2.1 specifies GEO tags completely different than vCard 3.0.

08/20/2008 06:56:20 PM	CVS Commit	Comment #6	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/framework/iCalendar/iCalendar.php?r1=1.145&r2=1.146&ty=u http://cvs.horde.org/co.php/framework/iCalendar/tests/geo.phpt?r=1.1 http://cvs.horde.org/diff.php/horde/docs/CHANGES?r1=1.1143&r2=1.1144&ty=u

04/09/2008 11:02:28 PM	Jan Schneider	Milestone ⇒

04/04/2008 06:50:53 PM	horde (at) ssn (dot) at	Comment #5 New Attachment: Nokia_GEO_E90entry.vcf	Reply to this comment
Attached a vCard extracted from the wbxml-file in /tmp/sync. This vCard is a bit long, but it has all fields the N90 offers filled. The GEO-Tags are not vCard-RFC-compliant, as the are in wrong order and with the wrong delimiter. Additionally, the N90 doesn't understand a correctly formattet GEO-Tag when sending it from the PC to the E90. What we should do in any case is taking care of a wrong formatted GEO-Tag in iCalendar.php. In case of a wrong format (as in my case) the explode-function will return an array with just one element, so 'longitude' => floatval($floats[1])); will fail, because there is no $floats[1].

04/04/2008 04:47:29 PM	Jan Schneider	Comment #4	Reply to this comment
Can you upload an example for such a vCard (from the sync logs)?

04/04/2008 11:57:06 AM	Jan Schneider	Comment #3 State ⇒ Assigned Assigned to Jan Schneider Milestone ⇒ 3.2	Reply to this comment
Perhaps a bug to address in SyncML/Device/Nokia.php... Yes, sounds like a good place.

04/03/2008 02:18:19 PM	horde (at) ssn (dot) at	Comment #2	Reply to this comment
Just checked against RFC2426: The GEO-Implementation of Nokia seems to be buggy as they send longitude before latitude using a comma as separator - the RFC says LAT;LON... Perhaps a bug to address in SyncML/Device/Nokia.php... regards Lukas

04/03/2008 01:04:10 AM	horde (at) ssn (dot) at	Comment #1 Milestone ⇒ State ⇒ Unconfirmed Patch ⇒ No Queue ⇒ Synchronization Summary ⇒ iCalendar can't parse GEO-vCard-Tags Type ⇒ Bug Priority ⇒ 1. Low	Reply to this comment
The GEO-vCard-Tag (at least the one submitted by my E90) is not parsed correctly. It expects a semicolon between latitude and longitude, but there is a comma. So in iCalendar.php i suggest in line 724 (v1.31 2008-01-24): if (!strpos($value, ';')) { $floats = explode(',', $value); } else { $floats = explode(';', $value); } regards Lukas