[Tickets #6539] Handling missing ENCODING parameter in iCal date

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/6539
-----------------------------------------------------------------------
 Ticket             | 6539
 Created By         | Gunnar Wrobel <p at rdus.de>
 Summary            | Handling missing ENCODING parameter in iCal date
 Queue              | Horde Framework Packages
 Version            | HEAD
 Type               | Bug
 State              | Unconfirmed
 Priority           | 1. Low
 Milestone          | 
 Patch              | 
 Owners             | 
+New Attachment     | HK-GW-iCal_no_ENCODING.patch
-----------------------------------------------------------------------


Gunnar Wrobel <p at rdus.de> (2008-03-29 06:01) wrote:

iCalendar.php unfolds QUOTED-PRINTABLE encoded lines before parsing them.

The parameter ENCODING=QUOTED-PRINTABLE is expected in these lines.

A blackberry with SyncJe delivers lines like this during a SyncML run:

N;QUOTED-PRINTABLE;CHARSET=UTF-8:Euro=E2=82=AC;Lars=E2=82=AC

ENCODING= is missing here. From the format specs I guess the client is
broken.

But iCalendar.php seems to support such broken clients:

                if ((isset($params['ENCODING'])
                     && String::upper($params['ENCODING']) ==
'QUOTED-PRINTABLE')
                    || isset($params['QUOTED-PRINTABLE'])) {

                    $value = quoted_printable_decode($value);

isset($params['QUOTED-PRINTABLE']) allows to specify QUOTED-PRINTABLE as
parameter without the leading ENCODING.

If iCalendar.php should support this then I'd suggest to apply the
attached patch to correct unfolding.