[Tickets #704] NEW: iCalendar.php cannot parse its own VCALENDAR component

[bugs at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/?id=704
-----------------------------------------------------------------------
 Ticket     | 704
 Created By | ctnpublic-horde at yahoo.com
 Summary    | iCalendar.php cannot parse its own VCALENDAR component
 Queue      | Horde Framework
 State      | Unconfirmed
 Priority   | 2. Medium
 Type       | Bug
 Owners     | 
-----------------------------------------------------------------------


ctnpublic-horde at yahoo.com (2004-10-15 21:49) wrote:

The generated VCALENDAR does not end with \r\n, yet it is expected in the
itip parsing.

Thing is, this is a very prominent problem as it would prevent importing
invitations. So am I missing something, in configs or prefs?

In any case, there's a simple fix.
cvs diff -u -r HEAD framework/iCalendar/iCalendar.php
Index: framework/iCalendar/iCalendar.php
===================================================================
RCS file: /repository/framework/iCalendar/iCalendar.php,v
retrieving revision 1.53
diff -u -r1.53 iCalendar.php
--- framework/iCalendar/iCalendar.php   24 Sep 2004 03:34:43 -0000     
1.53
+++ framework/iCalendar/iCalendar.php   16 Oct 2004 04:42:28 -0000
@@ -408,7 +408,7 @@
         if ($clear) {
             $this->clear();
         }
-        if (preg_match('/(BEGIN:' . $base . '\r?\n)([\W\w]*)(END:' . $base
. '\r?\n?)/i', $text, $matches)) {
+        if (preg_match('/(BEGIN:' . $base . '\r?\n?)([\W\w]*)(END:' . $base
. '\r?\n?)/i', $text, $matches)) {
             $vCal = $matches[2];
         } else {
             // Text isn't enclosed in BEGIN:VCALENDAR