[Tickets #12501] Icalendar.php vCard import case sensitive

[noreply at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/12501
------------------------------------------------------------------------------
  Ticket             | 12501
  Created By         | carlos.velasco at nimastelecom.com
  Summary            | Icalendar.php vCard import case sensitive
  Queue              | Horde Framework Packages
  Version            | Git master
  Type               | Bug
  State              | Unconfirmed
  Priority           | 3. High
  Milestone          |
  Patch              | 1
  Owners             |
------------------------------------------------------------------------------


carlos.velasco at nimastelecom.com (2013-07-25 11:24) wrote:

Trying to import a vCard in turba like this example:
===
BEGIN:VCARD
VERSION:3.0
N:;KKKK2
FN:KKKK2
TEL;TYPE=WORK:1111
TEL;TYPE=home:2222
X-MOZILLA-HTML:FALSE
END:VCARD
===

The "WORK" telephone is imported ok. But the "home" one is not imported.
This is due to case sensitive.

RFC2426 states these should not be case sensitive:
===
    tel-type     = "HOME" / "WORK" / "PREF" / "VOICE" / "FAX" / "MSG"
                 / "CELL" / "PAGER" / "BBS" / "MODEM" / "CAR" / "ISDN"
                 / "VIDEO" / "PCS" / iana-token / x-name
         ; Values are case insensitive
===

Attached is a patch to Icalendar,php to "upper" all param values to  
fix the problem. Although I don't know if this is the right place to  
fix this, it works.

Note: I really found this problem when debugging Sogo connector sync  
problems with Cardav in turba, but problem arise also when using  
manual import of vcard files.