[PHPiCalendar-DEV] WebCal display name patch and some comments

[Jamin W. C. <jco...@as...>]

[Sent this a few days ago but it doesn't appear to have made it to the
list so I'm sending it again.  Sorry if this results in a duplicate]

I've been looking at using PHP iCalendar to display our iCal files for
a short while now.  One of the things I noticed is that the code relies
on X-WR-CALNAME to provide the calendar name for the index.  Without
this, it looks like the code strips the .ics suffix and /'s from the
webcal url and uses that for the name.  This can lead to rather long
and hideous calendar names and at this time Sunbird doesn't appear to
set the X-WR-CALNAME attribute for the calendars it creates.  In an
effort to make the resulting calendar names more readable I've created
the attached and referenced[1] (in case the list removes it) patch that
instead uses the iCal file's base filename (without the .ics suffix).
The patch could probably use a little cleaning up but it's worked well
for us so far.

Also, I've been looking at the recent CVS version and like the addition
of the search option.  However, it looks like the new CVS code
introduces some display problems.  For instance, events that span
multiple days aren't displayed on any day but the day that they start.
An example, we have a calendar listing the on call representative for
the week.  This representative starts their tour of duty on Monday at
7am and is relieved by the next representative the following Monday at
7am.  Under the previous 2.0 beta release this was properly displayed
(though some details were missing in the pop-up, see below).  The
current CVS appears to display this the same way the 1.1 release did,
that is to show it only on the day that it started (Monday in this
case).

The 2.0 beta release while displaying it properly in the calendar view
did not display useful information in the pop-up details.  In the
detailed window it listed the task duration as "(7:00 - 7:00 AM)".  As
this event spans multiple days it would be very useful to include the
date for each of these times.

[1] - http://asgardsrealm.net/~jcollins/patches/ical-displayname.patch

-- 
Jamin W. Collins

This is the typical unix way of doing things: you string together lots
of very specific tools to accomplish larger tasks. -- Vineet Kumar