]> code.citadel.org Git - citadel.git/blobdiff - webcit/calendar_view.c
* follow api-change in several places
[citadel.git] / webcit / calendar_view.c
index 0cb0273d19a429edaf1094482cc4854e688f7fd1..14112d0932918217dd91821ace59899a7f90276a 100644 (file)
@@ -19,18 +19,17 @@ void embeddable_mini_calendar(int year, int month, char *urlformat)
        time_t colheader_time;
        struct tm colheader_tm;
        char colheader_label[32];
-       int weekstart = 0;
-       char weekstart_buf[16];
+       long weekstart = 0;
        char url[256];
        char div_id[256];
        char escaped_urlformat[256];
 
        snprintf(div_id, sizeof div_id, "mini_calendar_%d", rand() );
 
-       /* Determine what day to start.
+       /* Determine what day to start.  If an impossible value is found, start on Sunday.
        */
-       get_preference("weekstart", weekstart_buf, sizeof weekstart_buf);
-       weekstart = atoi(weekstart_buf);
+       get_pref_long("weekstart", &weekstart, 17);
+       if (weekstart > 6) weekstart = 0;
 
        /*
        * Now back up to the 1st of the month...
@@ -48,11 +47,11 @@ void embeddable_mini_calendar(int year, int month, char *urlformat)
                localtime_r(&thetime, &tm);
        }
 
-       /** Determine previous and next months ... for links */
+       /* Determine previous and next months ... for links */
        previous_month = thetime - (time_t)864000L;     /* back 10 days */
        next_month = thetime + (time_t)(31L * 86400L);  /* ahead 31 days */
 
-       /** Now back up until we're on the user's preferred start day */
+       /* Now back up until we're on the user's preferred start day */
        localtime_r(&thetime, &tm);
        while (tm.tm_wday != weekstart) {
                thetime = thetime - (time_t)86400;      /* go back 24 hours */
@@ -91,13 +90,13 @@ void embeddable_mini_calendar(int year, int month, char *urlformat)
        wprintf("</tr>\n");
 
 
-        /** Now do 35 or 42 days */
+        /* Now do 35 or 42 days */
         for (i = 0; i < 42; ++i) {
                 localtime_r(&thetime, &tm);
 
                 if (i < 35) {
 
-                       /** Before displaying Sunday, start a new row */
+                       /* Before displaying Sunday, start a new row */
                        if ((i % 7) == 0) {
                                wprintf("<tr>");
                        }
@@ -111,17 +110,17 @@ void embeddable_mini_calendar(int year, int month, char *urlformat)
                                wprintf("<td> </td>");
                        }
 
-                       /** After displaying one week, end the row */
+                       /* After displaying one week, end the row */
                        if ((i % 7) == 6) {
                                wprintf("</tr>\n");
                        }
 
                }
 
-               thetime += (time_t)86400;               /** ahead 24 hours */
+               thetime += (time_t)86400;               /* ahead 24 hours */
        }
 
-       wprintf("</table>"                      /** end of inner table */
+       wprintf("</table>"                      /* end of inner table */
                "</div>\n");
 
        /* javascript for previous and next month */
@@ -130,21 +129,22 @@ void embeddable_mini_calendar(int year, int month, char *urlformat)
                sprintf(&escaped_urlformat[i*2], "%02X", urlformat[i]);
        }
 
-       wprintf("<script type=\"text/javascript\">                                                      "
-               "       function minical_change_month(year, month) {                                    "
-               "               p = 'year=' + year + '&month=' + month                                  "
-               "                       + '&urlformat=%s&r=' + CtdlRandomString();                      "
-               "               new Ajax.Updater('%s', 'mini_calendar',                                 "
-               "                       { method: 'get', parameters: p, evalScripts: true } );          "
-               "       }                                                                               "
-               "</script>\n"
-               ,
+       StrBufAppendPrintf(WC->trailing_javascript,
+               "       function minical_change_month(year, month) {                                    \n"
+               "               p = 'year=' + year + '&month=' + month                                  \n"
+               "                       + '&urlformat=%s&r=' + CtdlRandomString();                      \n"
+               "               new Ajax.Updater('%s', 'mini_calendar',                                 \n"
+               "                       { method: 'get', parameters: p, evalScripts: true } );          \n"
+               "       }                                                                               \n"
+               "",
                escaped_urlformat, div_id
-               );
+       );
 
 }
 
-/* ajax embedder for the above mini calendar */
+/*
+ * ajax embedder for the above mini calendar 
+ */
 void ajax_mini_calendar(void) {
        char urlformat[256];
        int i, len;
@@ -161,13 +161,16 @@ void ajax_mini_calendar(void) {
 }
 
 
-/**
- * \brief Display one day of a whole month view of a calendar
- * \param thetime the month we want to see 
+/*
+ * Display one day of a whole month view of a calendar
  */
 void calendar_month_view_display_events(int year, int month, int day)
 {
-       int i;
+       long hklen;
+       const char *HashKey;
+       void *vCal;
+       HashPos *Pos;
+       disp_cal *Cal;
        icalproperty *p = NULL;
        icalproperty *q = NULL;
        struct icaltimetype t;
@@ -180,15 +183,15 @@ void calendar_month_view_display_events(int year, int month, int day)
        int show_event = 0;
        char buf[256];
        struct wcsession *WCC = WC;     /* This is done to make it run faster; WC is a function */
-       struct disp_cal *Cal;
        time_t tt;
 
-       if (WCC->num_cal == 0) {
+       if (GetCount(WCC->disp_cal_items) == 0) {
                wprintf("<br /><br /><br />\n");
                return;
        }
 
-       /* Create an imaginary event which spans the 24 hours of today.  Any events which
+       /*
+        * Create an imaginary event which spans the 24 hours of today.  Any events which
         * overlap with this one take place at least partially in this day.  We have to
         * convert it from a struct tm in order to make it UTC.
         */
@@ -210,10 +213,12 @@ void calendar_month_view_display_events(int year, int month, int day)
        today_end_t = icaltime_from_timet_with_zone(mktime(&ending_tm), 0, icaltimezone_get_utc_timezone());
        today_end_t.is_utc = 1;
 
-       /* Now loop through our list of events to see which ones occur today.
+       /*
+        * Now loop through our list of events to see which ones occur today.
         */
-       for (i=0; i<(WCC->num_cal); ++i) {
-               Cal = &WCC->disp_cal[i];
+       Pos = GetNewHashPos(WCC->disp_cal_items, 0);
+       while (GetNextHashPos(WCC->disp_cal_items, Pos, &hklen, &HashKey, &vCal)) {
+               Cal = (disp_cal*)vCal;
                all_day_event = 0;
                q = icalcomponent_get_first_property(Cal->cal, ICAL_DTSTART_PROPERTY);
                if (q != NULL) {
@@ -240,7 +245,8 @@ void calendar_month_view_display_events(int year, int month, int day)
                        show_event = ical_ctdl_is_overlap(t, end_t, today_start_t, today_end_t);
                }
 
-               /* If we determined that this event occurs today, then display it.
+               /*
+                * If we determined that this event occurs today, then display it.
                 */
                if (show_event) {
                        p = icalcomponent_get_first_property(Cal->cal, ICAL_SUMMARY_PROPERTY);
@@ -252,12 +258,12 @@ void calendar_month_view_display_events(int year, int month, int day)
                                                );
                                }
 
-                               wprintf("<font size=-1>"
+                               wprintf("<font size=\"-1\">"
                                        "<a class=\"event%s\" href=\"display_edit_event?"
-                                       "msgnum=%ld&calview=month&year=%d&month=%d&day=%d\""
+                                       "msgnum=%ld?calview=month?year=%d?month=%d?day=%d\""
                                        " btt_tooltext=\"",
                                        (Cal->unread)?"_unread":"_read",
-                                       WC->disp_cal[i].cal_msgnum,
+                                       Cal->cal_msgnum,
                                        year, month, day
                                        );
 
@@ -267,7 +273,7 @@ void calendar_month_view_display_events(int year, int month, int day)
                                wprintf("<br />");
                                
                                q = icalcomponent_get_first_property(
-                                       WC->disp_cal[i].cal,
+                                       Cal->cal,
                                        ICAL_LOCATION_PROPERTY);
                                if (q) {
                                        wprintf("<i>%s</i> ", _("Location:"));
@@ -275,7 +281,7 @@ void calendar_month_view_display_events(int year, int month, int day)
                                        wprintf("<br />");
                                }
                                
-                               /**
+                               /*
                                 * Only show start/end times if we're actually looking at the VEVENT
                                 * component.  Otherwise it shows bogus dates for e.g. timezones
                                 */
@@ -302,7 +308,8 @@ void calendar_month_view_display_events(int year, int month, int day)
                                                        wprintf("<i>%s</i> %s<br>",
                                                                _("Starting date/time:"), buf);
                                                        
-                                                       /* Embed the 'show end date/time' loop inside here so it
+                                                       /*
+                                                        * Embed the 'show end date/time' loop inside here so it
                                                         * only executes if this is NOT an all day event.
                                                         */
                                                        q = icalcomponent_get_first_property(Cal->cal, ICAL_DTEND_PROPERTY);
@@ -340,15 +347,18 @@ void calendar_month_view_display_events(int year, int month, int day)
                
                
        }
+       DeleteHashPos(&Pos);
 }
 
 
-/**
- * \brief Display one day of a whole month view of a calendar
- * \param thetime the month we want to see 
+/*
+ * Display one day of a whole month view of a calendar
  */
 void calendar_month_view_brief_events(time_t thetime, const char *daycolor) {
-       int i;
+       long hklen;
+       const char *HashKey;
+       void *vCal;
+       HashPos *Pos;
        time_t event_tt;
        time_t event_tts;
        time_t event_tte;
@@ -359,7 +369,7 @@ void calendar_month_view_brief_events(time_t thetime, const char *daycolor) {
        icalproperty *p;
        icalproperty *e;
        struct icaltimetype t;
-       struct disp_cal *Cal;
+       disp_cal *Cal;
        int month, day, year;
        int all_day_event = 0;
        char *timeformat;
@@ -375,10 +385,10 @@ void calendar_month_view_brief_events(time_t thetime, const char *daycolor) {
        day = today_tm.tm_mday;
        year = today_tm.tm_year + 1900;
 
-       for (i=0; i<(WC->num_cal); ++i) {
-               Cal = &WCC->disp_cal[i];
-               p = icalcomponent_get_first_property(Cal->cal,
-                       ICAL_DTSTART_PROPERTY);
+       Pos = GetNewHashPos(WCC->disp_cal_items, 0);
+       while (GetNextHashPos(WCC->disp_cal_items, Pos, &hklen, &HashKey, &vCal)) {
+               Cal = (disp_cal*)vCal;
+               p = icalcomponent_get_first_property(Cal->cal, ICAL_DTSTART_PROPERTY);
                if (p != NULL) {
                        t = icalproperty_get_dtstart(p);
                        event_tt = icaltime_as_timet(t);
@@ -392,7 +402,7 @@ void calendar_month_view_brief_events(time_t thetime, const char *daycolor) {
                        else {
                                localtime_r(&event_tts, &event_tms);
                        }
-                       /** \todo epoch &! daymask */
+                       /* \todo epoch &! daymask */
                        if ((event_tms.tm_year == today_tm.tm_year)
                                && (event_tms.tm_mon == today_tm.tm_mon)
                        && (event_tms.tm_mday == today_tm.tm_mday)) {
@@ -402,10 +412,10 @@ void calendar_month_view_brief_events(time_t thetime, const char *daycolor) {
                        char ebuf[255];
                        
                        p = icalcomponent_get_first_property(
-                               WC->disp_cal[i].cal,
+                               Cal->cal,
                                ICAL_SUMMARY_PROPERTY);
                        e = icalcomponent_get_first_property(
-                               WC->disp_cal[i].cal, 
+                               Cal->cal, 
                                ICAL_DTEND_PROPERTY);
                        if ((p != NULL) && (e != NULL)) {
                                time_t difftime;
@@ -417,13 +427,13 @@ void calendar_month_view_brief_events(time_t thetime, const char *daycolor) {
                                hours=(int)(difftime / 60);
                                minutes=difftime % 60;
                                wprintf("<tr><td bgcolor='%s'>%i:%2i</td><td bgcolor='%s'>"
-                                       "<font size=-1>"
-                                       "<a class=\"event%s\" href=\"display_edit_event?msgnum=%ld&calview=calbrief&year=%s&month=%s&day=%s\">",
+                                       "<font size=\"-1\">"
+                                       "<a class=\"event%s\" href=\"display_edit_event?msgnum=%ld?calview=calbrief?year=%s?month=%s?day=%s\">",
                                        daycolor,
                                        hours, minutes,
                                        (Cal->unread)?"_unread":"_read",                                                
                                        daycolor,
-                                       WC->disp_cal[i].cal_msgnum,
+                                       Cal->cal_msgnum,
                                        bstr("year"),
                                        bstr("month"),
                                        bstr("day")
@@ -431,7 +441,7 @@ void calendar_month_view_brief_events(time_t thetime, const char *daycolor) {
                                
                                escputs((char *)
                                        icalproperty_get_comment(p));
-                               /** \todo: allso ammitime format */
+                               /* \todo: allso ammitime format */
                                wc_strftime(&sbuf[0], sizeof(sbuf), timeformat, &event_tms);
                                wc_strftime(&ebuf[0], sizeof(sbuf), timeformat, &event_tme);
                                
@@ -441,22 +451,19 @@ void calendar_month_view_brief_events(time_t thetime, const char *daycolor) {
                                        sbuf,
                                        daycolor,
                                        ebuf);
-                               
-                       }
+                               }
                        
                        }
                        
                        
                }
        }
+       DeleteHashPos(&Pos);
 }
 
 
-/**
- * \brief view one month. pretty view
- * \param year the year
- * \param month the month
- * \param day the actual day we want to see
+/*
+ * view one month. pretty view
  */
 void calendar_month_view(int year, int month, int day) {
        struct tm starting_tm;
@@ -468,14 +475,13 @@ void calendar_month_view(int year, int month, int day) {
        time_t colheader_time;
        struct tm colheader_tm;
        char colheader_label[32];
-       int chg_month = 0;
-       int weekstart = 0;
-       char weekstart_buf[16];
+       long weekstart = 0;
 
-       /* Determine what day to start.
+       /*
+        * Determine what day to start.  If an impossible value is found, start on Sunday.
         */
-       get_preference("weekstart", weekstart_buf, sizeof weekstart_buf);
-       weekstart = atoi(weekstart_buf);
+       get_pref_long("weekstart", &weekstart, 17);
+       if (weekstart > 6) weekstart = 0;
 
        /*
         * Now back up to the 1st of the month...
@@ -493,18 +499,18 @@ void calendar_month_view(int year, int month, int day) {
                localtime_r(&thetime, &tm);
        }
 
-       /** Determine previous and next months ... for links */
+       /* Determine previous and next months ... for links */
        previous_month = thetime - (time_t)864000L;     /* back 10 days */
        next_month = thetime + (time_t)(31L * 86400L);  /* ahead 31 days */
 
-       /** Now back up until we're on the user's preferred start day */
+       /* Now back up until we're on the user's preferred start day */
        localtime_r(&thetime, &tm);
        while (tm.tm_wday != weekstart) {
                thetime = thetime - (time_t)86400;      /* go back 24 hours */
                localtime_r(&thetime, &tm);
        }
 
-       /** Outer table (to get the background color) */
+       /* Outer table (to get the background color) */
        wprintf("<div class=\"fix_scrollbar_bug\">"
                "<table class=\"calendar\"> \n <tr><td>"); 
 
@@ -513,7 +519,7 @@ void calendar_month_view(int year, int month, int day) {
        wprintf("<td align=center>");
 
        localtime_r(&previous_month, &tm);
-       wprintf("<a href=\"readfwd?calview=month&year=%d&month=%d&day=1\">",
+       wprintf("<a href=\"readfwd?calview=month?year=%d?month=%d?day=1\">",
                (int)(tm.tm_year)+1900, tm.tm_mon + 1);
        wprintf("<img align=middle src=\"static/prevdate_32x.gif\" border=0></A>\n");
 
@@ -525,13 +531,13 @@ void calendar_month_view(int year, int month, int day) {
                "&nbsp;&nbsp;", colheader_label, year);
 
        localtime_r(&next_month, &tm);
-       wprintf("<a href=\"readfwd?calview=month&year=%d&month=%d&day=1\">",
+       wprintf("<a href=\"readfwd?calview=month?year=%d?month=%d?day=1\">",
                (int)(tm.tm_year)+1900, tm.tm_mon + 1);
        wprintf("<img align=middle src=\"static/nextdate_32x.gif\" border=0></A>\n");
 
        wprintf("</td></tr></table>\n");
 
-       /** Inner table (the real one) */
+       /* Inner table (the real one) */
        wprintf("<table width=100%% border=0 cellpadding=1 cellspacing=1 "
                "bgcolor=#204B78 id=\"inner_month\"><tr>");
        colheader_time = thetime;
@@ -546,64 +552,58 @@ void calendar_month_view(int year, int month, int day) {
        wprintf("</tr>\n");
 
 
-        /** Now do 35 or 42 days */
-        for (i = 0; i < 42; ++i) {
-                localtime_r(&thetime, &tm);
-
-                if ((i < 35) || (chg_month == 0)) {
+        /* Now do 35 or 42 days */
+       localtime_r(&thetime, &tm);
+        for (i = 0; i<42; ++i) {
 
-                        if ((i > 27) && ((tm.tm_mday == 1) || (tm.tm_mday == 31))) {
-                                chg_month = 1;
-                        }
-                        if (i > 35) {
-                                chg_month = 0;
-                        }
+               /* Before displaying the first day of the week, start a new row */
+               if ((i % 7) == 0) {
+                       wprintf("<tr>");
+               }
 
-                       /** Before displaying Sunday, start a new row */
-                       if ((i % 7) == 0) {
-                               wprintf("<tr>");
-                       }
+               wprintf("<td class=\"cal%s\"><div class=\"day\">",
+                       ((tm.tm_mon != month-1) ? "out" :
+                               ((tm.tm_wday==0 || tm.tm_wday==6) ? "weekend" :
+                                       "day"))
+                       );
+               if ((i==0) || (tm.tm_mday == 1)) {
+                       wc_strftime(colheader_label, sizeof colheader_label, "%B", &tm);
+                       wprintf("%s ", colheader_label);
+               }
+               wprintf("<a href=\"readfwd?calview=day?year=%d?month=%d?day=%d\">"
+                       "%d</a></div>",
+                       tm.tm_year + 1900,
+                       tm.tm_mon + 1,
+                       tm.tm_mday,
+                       tm.tm_mday);
+
+               /* put the data here, stupid */
+               calendar_month_view_display_events(
+                       tm.tm_year + 1900,
+                       tm.tm_mon + 1,
+                       tm.tm_mday
+                       );
 
-                       wprintf("<td class=\"cal%s\"><div class=\"day\">",
-                               ((tm.tm_mon != month-1) ? "out" :
-                                       ((tm.tm_wday==0 || tm.tm_wday==6) ? "weekend" :
-                                               "day"))
-                               );
-                       if ((i==0) || (tm.tm_mday == 1)) {
-                               wc_strftime(colheader_label, sizeof colheader_label, "%B", &tm);
-                               wprintf("%s ", colheader_label);
-                       }
-                       wprintf("<a href=\"readfwd?calview=day&year=%d&month=%d&day=%d\">"
-                               "%d</a></div>",
-                               tm.tm_year + 1900,
-                               tm.tm_mon + 1,
-                               tm.tm_mday,
-                               tm.tm_mday);
-
-                       /** put the data here, stupid */
-                       calendar_month_view_display_events(
-                               tm.tm_year + 1900,
-                               tm.tm_mon + 1,
-                               tm.tm_mday
-                               );
+               wprintf("</td>");
 
-                       wprintf("</td>");
+               /* After displaying the last day of the week, end the row */
+               if ((i % 7) == 6) {
+                       wprintf("</tr>\n");
+               }
 
-                       /** After displaying Saturday, end the row */
-                       if ((i % 7) == 6) {
-                               wprintf("</tr>\n");
-                       }
+               thetime += (time_t)86400;               /* ahead 24 hours */
+               localtime_r(&thetime, &tm);
 
+               if ( ((i % 7) == 6) && (tm.tm_mon != month-1) && (tm.tm_mday < 15) ) {
+                       i = 100;        /* break out of the loop */
                }
-
-               thetime += (time_t)86400;               /** ahead 24 hours */
        }
 
-       wprintf("</table>"                      /** end of inner table */
-               "</td></tr></table>"            /** end of outer table */
+       wprintf("</table>"                      /* end of inner table */
+               "</td></tr></table>"            /* end of outer table */
                "</div>\n");
 
-       /**
+       /*
         * Initialize the bubble tooltips.
         *
         * Yes, this is as stupid as it looks.  Instead of just making the call
@@ -614,17 +614,13 @@ void calendar_month_view(int year, int month, int day) {
         * being rendered.  See http://www.shaftek.org/blog/archives/000212.html
         * for more information.
         */ 
-       wprintf("<script type=\"text/javascript\">"
-               " setTimeout(\"btt_enableTooltips('inner_month')\", 1); "
-               "</script>\n"
+       StrBufAppendPrintf(WC->trailing_javascript,
+               " setTimeout(\"btt_enableTooltips('inner_month')\", 1); \n"
        );
 }
 
-/**
- * \brief view one month. brief view
- * \param year the year
- * \param month the month
- * \param day the actual day we want to see
+/*
+ * view one month. brief view
  */
 void calendar_brief_month_view(int year, int month, int day) {
        struct tm starting_tm;
@@ -635,7 +631,7 @@ void calendar_brief_month_view(int year, int month, int day) {
        time_t next_month;
        char month_label[32];
 
-       /** Determine what day to start.
+       /* Determine what day to start.
         * First, back up to the 1st of the month...
         */
        memset(&starting_tm, 0, sizeof(struct tm));
@@ -650,18 +646,18 @@ void calendar_brief_month_view(int year, int month, int day) {
                localtime_r(&thetime, &tm);
        }
 
-       /** Determine previous and next months ... for links */
+       /* Determine previous and next months ... for links */
        previous_month = thetime - (time_t)864000L;     /* back 10 days */
        next_month = thetime + (time_t)(31L * 86400L);  /* ahead 31 days */
 
-       /** Now back up until we're on a Sunday */
+       /* Now back up until we're on a Sunday */
        localtime_r(&thetime, &tm);
        while (tm.tm_wday != 0) {
                thetime = thetime - (time_t)86400;      /* go back 24 hours */
                localtime_r(&thetime, &tm);
        }
 
-       /** Outer table (to get the background color) */
+       /* Outer table (to get the background color) */
        wprintf("<div class=\"fix_scrollbar_bug\">"
                "<table width=100%% border=0 cellpadding=0 cellspacing=0 "
                "bgcolor=#204B78><TR><TD>\n");
@@ -671,7 +667,7 @@ void calendar_brief_month_view(int year, int month, int day) {
        wprintf("<td align=center>");
 
        localtime_r(&previous_month, &tm);
-       wprintf("<a href=\"readfwd?calview=month&year=%d&month=%d&day=1\">",
+       wprintf("<a href=\"readfwd?calview=month?year=%d?month=%d?day=1\">",
                (int)(tm.tm_year)+1900, tm.tm_mon + 1);
        wprintf("<img align=middle src=\"static/prevdate_32x.gif\" border=0></A>\n");
 
@@ -683,19 +679,19 @@ void calendar_brief_month_view(int year, int month, int day) {
                "&nbsp;&nbsp;", month_label, year);
 
        localtime_r(&next_month, &tm);
-       wprintf("<a href=\"readfwd?calview=month&year=%d&month=%d&day=1\">",
+       wprintf("<a href=\"readfwd?calview=month?year=%d?month=%d?day=1\">",
                (int)(tm.tm_year)+1900, tm.tm_mon + 1);
        wprintf("<img align=middle src=\"static/nextdate_32x.gif\" border=0></A>\n");
 
        wprintf("</td></tr></table>\n");
 
-       /** Inner table (the real one) */
+       /* Inner table (the real one) */
        wprintf("<table width=100%% border=0 cellpadding=1 cellspacing=1 "
                "bgcolor=#EEEECC><TR>");
        wprintf("</tr>\n");
-       wprintf("<tr><td colspan=\"100%\">\n");
+       wprintf("<tr><td colspan=\"100%%\">\n");
 
-       /** Now do 35 days */
+       /* Now do 35 days */
        for (i = 0; i < 35; ++i) {
                char weeknumber[255];
                char weekday_name[32];
@@ -703,10 +699,10 @@ void calendar_brief_month_view(int year, int month, int day) {
                localtime_r(&thetime, &tm);
 
 
-               /** Before displaying Sunday, start a new CELL */
+               /* Before displaying Sunday, start a new CELL */
                if ((i % 7) == 0) {
                        wc_strftime(&weeknumber[0], sizeof(weeknumber), "%U", &tm);
-                       wprintf("<table border='0' bgcolor=\"#EEEECC\" width='100%'> <tr><th colspan='4'>%s %s</th></tr>"
+                       wprintf("<table border='0' bgcolor=\"#EEEECC\" width='100%%'> <tr><th colspan='4'>%s %s</th></tr>"
                                "   <tr><td>%s</td><td width=70%%>%s</td><td>%s</td><td>%s</td></tr>\n",
                                _("Week"), 
                                weeknumber,
@@ -721,7 +717,7 @@ void calendar_brief_month_view(int year, int month, int day) {
                        ((tm.tm_wday==0 || tm.tm_wday==6) ? "EEEECC" :
                                "FFFFFF"));
                
-               /** Day Header */
+               /* Day Header */
                wc_strftime(weekday_name, sizeof weekday_name, "%A", &tm);
                wprintf("<tr><td bgcolor='%s' colspan='1' align='left'> %s,%i."
                        "</td><td bgcolor='%s' colspan='3'><hr></td></tr>\n",
@@ -729,46 +725,37 @@ void calendar_brief_month_view(int year, int month, int day) {
                        weekday_name,tm.tm_mday,
                        daycolor);
 
-               /** put the data of one day  here, stupid */
+               /* put the data of one day  here, stupid */
                calendar_month_view_brief_events(thetime, daycolor);
 
 
-               /** After displaying Saturday, end the row */
+               /* After displaying Saturday, end the row */
                if ((i % 7) == 6) {
                        wprintf("</td></tr></table>\n");
                }
 
-               thetime += (time_t)86400;               /** ahead 24 hours */
+               thetime += (time_t)86400;               /* ahead 24 hours */
        }
 
-       wprintf("</table>"                      /** end of inner table */
-               "</td></tr></table>"            /** end of outer table */
+       wprintf("</table>"                      /* end of inner table */
+               "</td></tr></table>"            /* end of outer table */
                "</div>\n");
 }
 
-/** 
- * \brief view one week
- * this should view just one week, but it's not here yet.
- * \todo ny implemented
- * \param year the year
- * \param month the month
- * \param day the day which we want to see the week around
+/*
+ * Calendar week view -- not implemented yet, this is a stub function
  */
 void calendar_week_view(int year, int month, int day) {
        wprintf("<center><i>week view FIXME</i></center><br />\n");
 }
 
 
-/**
- * \brief display one day
+/*
+ * display one day
  * Display events for a particular hour of a particular day.
  * (Specify hour < 0 to show "all day" events)
- * \param year the year
- * \param month the month
- * \param day the day
- * \param hour the hour we want to start displaying
- * \param dstart daystart 
- * \param dend dayend 
+ *
+ * dstart and dend indicate which hours our "daytime" begins and end
  */
 void calendar_day_view_display_events(time_t thetime,
        int year,
@@ -778,7 +765,10 @@ void calendar_day_view_display_events(time_t thetime,
        int dstart,
        int dend)
 {
-       int i;
+       long hklen;
+       const char *HashKey;
+       void *vCal;
+       HashPos *Pos;
        icalproperty *p = NULL;
        icalproperty *q = NULL;
        time_t event_tt;
@@ -789,7 +779,7 @@ void calendar_day_view_display_events(time_t thetime,
        int all_day_event = 0;
        int ongoing_event = 0;
        struct wcsession *WCC = WC;     /* This is done to make it run faster; WC is a function */
-       struct disp_cal *Cal;
+       disp_cal *Cal;
        struct icaltimetype t;
        struct icaltimetype end_t;
        struct icaltimetype today_start_t;
@@ -807,7 +797,7 @@ void calendar_day_view_display_events(time_t thetime,
         struct tm d_tm;
         char d_str[32];
 
-       if (WCC->num_cal == 0) {
+       if (GetCount(WCC->disp_cal_items) == 0) {
                /* nothing to display */
                return;
        }
@@ -835,8 +825,9 @@ void calendar_day_view_display_events(time_t thetime,
 
        /* Now loop through our list of events to see which ones occur today.
         */
-       for (i=0; i<(WCC->num_cal); ++i) {
-               Cal = &WCC->disp_cal[i];
+       Pos = GetNewHashPos(WCC->disp_cal_items, 0);
+       while (GetNextHashPos(WCC->disp_cal_items, Pos, &hklen, &HashKey, &vCal)) {
+               Cal = (disp_cal*)vCal;
 
                all_day_event = 0;
                ongoing_event=0;
@@ -882,7 +873,7 @@ void calendar_day_view_display_events(time_t thetime,
                        {
                                wprintf("<li class=\"event_framed%s\"> "
                                        "<a href=\"display_edit_event?"
-                                       "msgnum=%ld&calview=day&year=%d&month=%d&day=%d\" "
+                                       "msgnum=%ld?calview=day?year=%d?month=%d?day=%d\" "
                                        " class=\"event_title\" "
                                        " btt_tooltext=\"",
                                        (Cal->unread)?"_unread":"_read",
@@ -920,7 +911,7 @@ void calendar_day_view_display_events(time_t thetime,
                        {
                                wprintf("<li class=\"event_framed%s\"> "
                                        "<a href=\"display_edit_event?"
-                                       "msgnum=%ld&calview=day&year=%d&month=%d&day=%d\" "
+                                       "msgnum=%ld&calview=day?year=%d?month=%d?day=%d\" "
                                        " class=\"event_title\" " 
                                        "btt_tooltext=\"",
                                        (Cal->unread)?"_unread":"_read",
@@ -1001,7 +992,7 @@ void calendar_day_view_display_events(time_t thetime,
                                        top, (gap * 40), (bottom-top)
                                        );
                                wprintf("<a href=\"display_edit_event?"
-                                       "msgnum=%ld&calview=day&year=%d&month=%d&day=%d&hour=%d\" "
+                                       "msgnum=%ld?calview=day?year=%d?month=%d?day=%d?hour=%d\" "
                                        "class=\"event_title\" "
                                                "btt_tooltext=\"",
                                        Cal->cal_msgnum, year, month, day, t.hour);
@@ -1032,20 +1023,17 @@ void calendar_day_view_display_events(time_t thetime,
                        }
                }
        }
+       DeleteHashPos(&Pos);
 }
 
-/**
- * \brief view one day
- * \param year the year
- * \param month the month 
- * \param day the day we want to display
+/*
+ * view one day
  */
 void calendar_day_view(int year, int month, int day) {
        int hour;
        struct icaltimetype today, yesterday, tomorrow;
-       int daystart = 8;
-       int dayend = 17;
-       char daystart_str[16], dayend_str[16];
+       long daystart;
+       long dayend;
        struct tm d_tm;
        char d_str[128];
        int time_format;
@@ -1055,19 +1043,23 @@ void calendar_day_view(int year, int month, int day) {
        int gap = 0;
 
        time_format = get_time_format_cached ();
-       get_preference("daystart", daystart_str, sizeof daystart_str);
-       if (!IsEmptyStr(daystart_str)) daystart = atoi(daystart_str);
-       get_preference("dayend", dayend_str, sizeof dayend_str);
-       if (!IsEmptyStr(dayend_str)) dayend = atoi(dayend_str);
+       get_pref_long("daystart", &daystart, 8);
+       get_pref_long("dayend", &dayend, 17);
+
+       /* when loading daystart/dayend, replace missing, corrupt, or impossible values with defaults */
+       if ((daystart < 0) || (dayend < 2) || (daystart >= 23) || (dayend > 23) || (dayend <= daystart)) {
+               daystart = 9;
+               dayend = 17;
+       }
        
-       /** Today's date */
+       /* Today's date */
        memset(&d_tm, 0, sizeof d_tm);
        d_tm.tm_year = year - 1900;
        d_tm.tm_mon = month - 1;
        d_tm.tm_mday = day;
        today_t = mktime(&d_tm); 
 
-       /** Figure out the dates for "yesterday" and "tomorrow" links */
+       /* Figure out the dates for "yesterday" and "tomorrow" links */
 
        memset(&today, 0, sizeof(struct icaltimetype));
        today.year = year;
@@ -1085,14 +1077,14 @@ void calendar_day_view(int year, int month, int day) {
 
        wprintf("<div class=\"fix_scrollbar_bug\">");
 
-       /** Inner table (the real one) */
+       /* Inner table (the real one) */
        wprintf("<table class=\"calendar\" id=\"inner_day\"><tr> \n");
 
-       /** Innermost cell (contains hours etc.) */
+       /* Innermost cell (contains hours etc.) */
        wprintf("<td class=\"events_of_the_day\" >");
                wprintf("<dl class=\"events\" >");
 
-       /** Now the middle of the day... */
+       /* Now the middle of the day... */
 
        extratimeline = timeline / 3;   
 
@@ -1104,7 +1096,7 @@ void calendar_day_view(int year, int month, int day) {
                        "height: %dpx;          "       
                        "\" >                   "
                        "<a href=\"display_edit_event?msgnum=0"
-                       "&calview=day&year=%d&month=%d&day=%d&hour=%d&minute=0\">",
+                       "?calview=day?year=%d?month=%d?day=%d?hour=%d?minute=0\">",
                        (hour * extratimeline ), extratimeline, 
                        year, month, day, hour
                        );
@@ -1128,11 +1120,11 @@ void calendar_day_view(int year, int month, int day) {
                 wprintf("<dt class=\"hour\"     "
                         "style=\"               "
                         "position: absolute;    "
-                        "top: %dpx; left: 0px;  "
+                        "top: %ldpx; left: 0px;  "
                         "height: %dpx;          "
                         "\" >                   "
-                        "<a href=\"display_edit_event?msgnum=0&calview=day"
-                        "&year=%d&month=%d&day=%d&hour=%d&minute=0\">",
+                        "<a href=\"display_edit_event?msgnum=0?calview=day"
+                        "?year=%d?month=%d?day=%d?hour=%d?minute=0\">",
                         gap + ((hour - daystart) * timeline ), timeline,
                         year, month, day, hour
                        );
@@ -1156,11 +1148,11 @@ void calendar_day_view(int year, int month, int day) {
                 wprintf("<dt class=\"extrahour\"     "
                         "style=\"               "
                         "position: absolute;    "
-                        "top: %dpx; left: 0px;  "
+                        "top: %ldpx; left: 0px; "
                         "height: %dpx;          "
                         "\" >                   "
-                        "<a href=\"display_edit_event?msgnum=0&calview=day"
-                        "&year=%d&month=%d&day=%d&hour=%d&minute=0\">",
+                        "<a href=\"display_edit_event?msgnum=0?calview=day"
+                        "?year=%d?month=%d?day=%d?hour=%d?minute=0\">",
                         gap + ((hour - dayend - 1) * extratimeline ), extratimeline,
                         year, month, day, hour
                 );
@@ -1184,28 +1176,28 @@ void calendar_day_view(int year, int month, int day) {
                wprintf("</dl>");
        wprintf("</td>");                       /* end of innermost table */
 
-       /** Display extra events (start/end times not present or not today) in the middle column */
+       /* Display extra events (start/end times not present or not today) in the middle column */
         wprintf("<td class=\"extra_events\">");
 
         wprintf("<ul>");
 
-        /** Display all-day events */
+        /* Display all-day events */
        calendar_day_view_display_events(today_t, year, month, day, 1, daystart, dayend);
 
         wprintf("</ul>");
 
-       wprintf("</td>");       /** end extra on the middle */
+       wprintf("</td>");       /* end extra on the middle */
 
-       wprintf("<td width=20%% align=center valign=top>");     /** begin stuff-on-the-right */
+       wprintf("<td width=20%% align=center valign=top>");     /* begin stuff-on-the-right */
 
-       /** Begin todays-date-with-left-and-right-arrows */
+       /* Begin todays-date-with-left-and-right-arrows */
        wprintf("<table border=0 width=100%% "
                "cellspacing=0 cellpadding=0 bgcolor=\"#FFFFFF\">\n");
        wprintf("<tr>");
 
-       /** Left arrow */       
+       /* Left arrow */        
        wprintf("<td align=center>");
-       wprintf("<a href=\"readfwd?calview=day&year=%d&month=%d&day=%d\">",
+       wprintf("<a href=\"readfwd?calview=day?year=%d?month=%d?day=%d\">",
                yesterday.year, yesterday.month, yesterday.day);
        wprintf("<img align=middle src=\"static/prevdate_32x.gif\" border=0></A>");
        wprintf("</td>");
@@ -1220,40 +1212,43 @@ void calendar_day_view(int year, int month, int day) {
                );
        wprintf("%s", d_str);
 
-       /** Right arrow */
+       /* Right arrow */
        wprintf("<td align=center>");
-       wprintf("<a href=\"readfwd?calview=day&year=%d&month=%d&day=%d\">",
+       wprintf("<a href=\"readfwd?calview=day?year=%d?month=%d?day=%d\">",
                tomorrow.year, tomorrow.month, tomorrow.day);
        wprintf("<img align=middle src=\"static/nextdate_32x.gif\""
                " border=0></a>\n");
        wprintf("</td>");
 
        wprintf("</tr></table>\n");
-       /** End todays-date-with-left-and-right-arrows */
+       /* End todays-date-with-left-and-right-arrows */
 
-       /** Embed a mini month calendar in this space */
+       /* Embed a mini month calendar in this space */
        wprintf("<br />\n");
-       embeddable_mini_calendar(year, month, "readfwd?calview=day&year=%d&month=%d&day=%d");
+       embeddable_mini_calendar(year, month, "readfwd?calview=day?year=%d?month=%d?day=%d");
 
        wprintf("</font></center>\n");
 
-       wprintf("</td></tr>");                  /** end stuff-on-the-right */
+       wprintf("</td></tr>");                  /* end stuff-on-the-right */
 
-       wprintf("</table>"                      /** end of inner table */
+       wprintf("</table>"                      /* end of inner table */
                "</div>");
 
-        wprintf("<script type=\"text/javascript\">"
-                " setTimeout(\"btt_enableTooltips('inner_day')\", 1); "
-                "</script>\n"
+       StrBufAppendPrintf(WC->trailing_javascript,
+                " setTimeout(\"btt_enableTooltips('inner_day')\", 1);  \n"
         );
 }
 
 
-/**
- * \brief Display today's events.
+/*
+ * Display today's events.
  */
 void calendar_summary_view(void) {
-       int i;
+       long hklen;
+       const char *HashKey;
+       void *vCal;
+       HashPos *Pos;
+       disp_cal *Cal;
        icalproperty *p;
        struct icaltimetype t;
        time_t event_tt;
@@ -1262,16 +1257,19 @@ void calendar_summary_view(void) {
        time_t now;
        int all_day_event = 0;
        char timestring[SIZ];
+       struct wcsession *WCC = WC;     /* This is done to make it run faster; WC is a function */
 
-       if (WC->num_cal == 0) {
+       if (GetCount(WC->disp_cal_items) == 0) {
                return;
        }
 
        now = time(NULL);
        localtime_r(&now, &today_tm);
 
-       for (i=0; i<(WC->num_cal); ++i) {
-               p = icalcomponent_get_first_property(WC->disp_cal[i].cal,
+       Pos = GetNewHashPos(WCC->disp_cal_items, 0);
+       while (GetNextHashPos(WCC->disp_cal_items, Pos, &hklen, &HashKey, &vCal)) {
+               Cal = (disp_cal*)vCal;
+               p = icalcomponent_get_first_property(Cal->cal,
                                                ICAL_DTSTART_PROPERTY);
                if (p != NULL) {
                        t = icalproperty_get_dtstart(p);
@@ -1298,7 +1296,7 @@ void calendar_summary_view(void) {
 
 
                        p = icalcomponent_get_first_property(
-                               WC->disp_cal[i].cal,
+                               Cal->cal,
                                ICAL_SUMMARY_PROPERTY);
                        if (p != NULL) {
                                escputs((char *)
@@ -1308,50 +1306,32 @@ void calendar_summary_view(void) {
                        }
                }
        }
-       free_calendar_buffer();
+       DeleteHashPos(&Pos);
+       DeleteHash(&WC->disp_cal_items);
 }
 
 
-/**
- * \brief clean up ical memory
- * \todo this could get troubel with future ical versions
- */
-void free_calendar_buffer(void) {
-       int i;
-       if (WC->num_cal) for (i=0; i<(WC->num_cal); ++i) {
-               icalcomponent_free(WC->disp_cal[i].cal);
-               free(WC->disp_cal[i].from);
-       }
-       WC->num_cal = 0;
-       free(WC->disp_cal);
-       WC->disp_cal = NULL;
-}
-
-
-
-/**
- * \brief do the whole calendar page
- * view any part of the calender. decide which way, etc.
+/*
+ * Parse the URL variables in order to determine the scope and display of a calendar view
  */
-void do_calendar_view(void) {
+void parse_calendar_view_request(struct calview *c) {
        time_t now;
        struct tm tm;
-       int year, month, day;
-       char calview[SIZ];
+       char calview[32];
 
-       /** In case no date was specified, go with today */
+       /* In case no date was specified, go with today */
        now = time(NULL);
        localtime_r(&now, &tm);
-       year = tm.tm_year + 1900;
-       month = tm.tm_mon + 1;
-       day = tm.tm_mday;
+       c->year = tm.tm_year + 1900;
+       c->month = tm.tm_mon + 1;
+       c->day = tm.tm_mday;
 
-       /** Now see if a date was specified */
-       if (havebstr("year")) year = ibstr("year");
-       if (havebstr("month")) month = ibstr("month");
-       if (havebstr("day")) day = ibstr("day");
+       /* Now see if a date was specified */
+       if (havebstr("year")) c->year = ibstr("year");
+       if (havebstr("month")) c->month = ibstr("month");
+       if (havebstr("day")) c->day = ibstr("day");
 
-       /** How would you like that cooked? */
+       /* How would you like that cooked? */
        if (havebstr("calview")) {
                strcpy(calview, bstr("calview"));
        }
@@ -1359,33 +1339,70 @@ void do_calendar_view(void) {
                strcpy(calview, "month");
        }
 
-       /** Display the selected view */
+       /* Display the selected view */
        if (!strcasecmp(calview, "day")) {
-               calendar_day_view(year, month, day);
+               c->view = calview_day;
        }
        else if (!strcasecmp(calview, "week")) {
-               calendar_week_view(year, month, day);
+               c->view = calview_week;
        }
        else {
                if (WC->wc_view == VIEW_CALBRIEF) {
-                       calendar_brief_month_view(year, month, day);
+                       c->view = calview_brief;
                }
                else {
-                       calendar_month_view(year, month, day);
+                       c->view = calview_month;
                }
        }
 
-       /** Free the calendar stuff */
-       free_calendar_buffer();
+       /* Now try and set the lower and upper bounds so that we don't
+        * burn too many cpu cycles parsing data way in the past or future
+        */
+
+       tm.tm_year = c->year - 1900;
+       tm.tm_mon = c->month - 1;
+       tm.tm_mday = c->day;
+       now = mktime(&tm);
+
+       int span = 3888000;
+       if (c->view == calview_month)   span = 3888000;
+       if (c->view == calview_brief)   span = 3888000;
+       if (c->view == calview_week)    span = 604800;
+       if (c->view == calview_day)     span = 86400;
 
+       c->lower_bound = now - span;
+       c->upper_bound = now + span;
 }
 
 
-/**
- * \brief get task due date
- * Helper function for do_tasks_view().  
- * \param vtodo a task to get the due date
- * \return the date/time due.
+
+/*
+ * Render a calendar view from data previously loaded into memory
+ */
+void render_calendar_view(struct calview *c)
+{
+       if (c->view == calview_day) {
+               calendar_day_view(c->year, c->month, c->day);
+       }
+       else if (c->view == calview_week) {
+               calendar_week_view(c->year, c->month, c->day);
+       }
+       else {
+               if (WC->wc_view == VIEW_CALBRIEF) {
+                       calendar_brief_month_view(c->year, c->month, c->day);
+               }
+               else {
+                       calendar_month_view(c->year, c->month, c->day);
+               }
+       }
+
+       /* Free the in-memory list of calendar items */
+       DeleteHash(&WC->disp_cal_items);
+}
+
+
+/*
+ * Helper function for do_tasks_view().  Returns the due date/time of a vtodo.
  */
 time_t get_task_due_date(icalcomponent *vtodo) {
        icalproperty *p;
@@ -1394,7 +1411,7 @@ time_t get_task_due_date(icalcomponent *vtodo) {
                return(0L);
        }
 
-       /**
+       /*
         * If we're looking at a fully encapsulated VCALENDAR
         * rather than a VTODO component, recurse into the data
         * structure until we get a VTODO.
@@ -1417,27 +1434,31 @@ time_t get_task_due_date(icalcomponent *vtodo) {
 }
 
 
-/**
- * \brief Compare the due dates of two tasks (this is for sorting)
- * \param task1 first task to compare
- * \param task2 second task to compare
+/*
+ * Compare the due dates of two tasks (this is for sorting)
  */
-int task_due_cmp(const void *task1, const void *task2) {
+int task_due_cmp(const void *vtask1, const void *vtask2) {
+       disp_cal * Task1 = (disp_cal *)GetSearchPayload(vtask1);
+       disp_cal * Task2 = (disp_cal *)GetSearchPayload(vtask2);
+
        time_t t1;
        time_t t2;
 
-       t1 =  get_task_due_date(((struct disp_cal *)task1)->cal);
-       t2 =  get_task_due_date(((struct disp_cal *)task2)->cal);
+       t1 =  get_task_due_date(Task1->cal);
+       t2 =  get_task_due_date(Task2->cal);
        if (t1 < t2) return(-1);
        if (t1 > t2) return(1);
        return(0);
 }
 
-/**
- * \brief qsort filter to move completed tasks to bottom of task list
+/*
+ * qsort filter to move completed tasks to bottom of task list
  */
-int task_completed_cmp(const void *task1, const void *task2) {
-       icalproperty_status t1 = icalcomponent_get_status(((struct disp_cal *)task1)->cal);
+int task_completed_cmp(const void *vtask1, const void *vtask2) {
+       disp_cal * Task1 = (disp_cal *)GetSearchPayload(vtask1);
+//     disp_cal * Task2 = (disp_cal *)GetSearchPayload(vtask2);
+
+       icalproperty_status t1 = icalcomponent_get_status((Task1)->cal);
        // icalproperty_status t2 = icalcomponent_get_status(((struct disp_cal *)task2)->cal);
        
        if (t1 == ICAL_STATUS_COMPLETED) 
@@ -1447,14 +1468,21 @@ int task_completed_cmp(const void *task1, const void *task2) {
 
 
 
-/**
- * \brief do the whole task view stuff
+/*
+ * do the whole task view stuff
  */
 void do_tasks_view(void) {
-       int i;
+       long hklen;
+       const char *HashKey;
+       void *vCal;
+       disp_cal *Cal;
+       HashPos *Pos;
+       int nItems;
        time_t due;
        char buf[SIZ];
        icalproperty *p;
+       struct wcsession *WCC = WC;     /* This is done to make it run faster; WC is a function */
+
        wprintf("<div class=\"fix_scrollbar_bug\">"
                "<table class=\"calendar_view_background\"><tbody id=\"taskview\">\n<tr>\n"
                "<th>");
@@ -1468,35 +1496,35 @@ void do_tasks_view(void) {
        wprintf(" (<select id=\"selectcategory\"><option value=\"showall\">%s</option></select>)</th></tr>\n",
                _("Show All"));
 
-       /** Sort them if necessary */
-       if (WC->num_cal > 1) {
-               qsort(WC->disp_cal,
-                       WC->num_cal,
-                       sizeof(struct disp_cal),
-                       task_due_cmp
-                       );
+       nItems = GetCount(WC->disp_cal_items);
+
+       /* Sort them if necessary
+       if (nItems > 1) {
+               SortByPayload(WC->disp_cal_items, task_due_cmp);
        }
-       /** And then again, by completed */
-       if (WC->num_cal > 1) {
-               qsort(WC->disp_cal,
-                       WC->num_cal,
-                       sizeof(struct disp_cal),
-                       task_completed_cmp
-                       );
+       * this shouldn't be neccessary, since we sort by the start time.
+       */
+
+       /* And then again, by completed */
+       if (nItems > 1) {
+               SortByPayload(WC->disp_cal_items, 
+                             task_completed_cmp);
        }
 
-       if (WC->num_cal) for (i=0; i<(WC->num_cal); ++i) {
+       Pos = GetNewHashPos(WCC->disp_cal_items, 0);
+       while (GetNextHashPos(WCC->disp_cal_items, Pos, &hklen, &HashKey, &vCal)) {
+               Cal = (disp_cal*)vCal;
                wprintf("<tr><td>");
-               icalproperty_status todoStatus = icalcomponent_get_status(WC->disp_cal[i].cal);
-               wprintf("<input style=\"text-align: center;\" type=\"checkbox\" name=\"completed\" value=\"completed\" ");
+               icalproperty_status todoStatus = icalcomponent_get_status(Cal->cal);
+               wprintf("<input type=\"checkbox\" name=\"completed\" value=\"completed\" ");
                if (todoStatus == ICAL_STATUS_COMPLETED) {
                        wprintf("checked=\"checked\" ");
                }
                wprintf("disabled=\"disabled\">\n</td><td>");
-               p = icalcomponent_get_first_property(WC->disp_cal[i].cal,
+               p = icalcomponent_get_first_property(Cal->cal,
                        ICAL_SUMMARY_PROPERTY);
                wprintf("<a href=\"display_edit_task?msgnum=%ld&amp;taskrm=",
-                       WC->disp_cal[i].cal_msgnum );
+                       Cal->cal_msgnum );
                urlescputs(WC->wc_roomname);
                wprintf("\">");
                /* wprintf("<img align=middle "
@@ -1507,7 +1535,7 @@ void do_tasks_view(void) {
                wprintf("</a>\n");
                wprintf("</td>\n");
 
-               due = get_task_due_date(WC->disp_cal[i].cal);
+               due = get_task_due_date(Cal->cal);
                wprintf("<td><span");
                if (due > 0) {
                        webcit_fmt_date(buf, due, 0);
@@ -1518,7 +1546,7 @@ void do_tasks_view(void) {
                }
                wprintf("</span></td>");
                wprintf("<td>");
-               p = icalcomponent_get_first_property(WC->disp_cal[i].cal,
+               p = icalcomponent_get_first_property(Cal->cal,
                        ICAL_CATEGORIES_PROPERTY);
                if (p != NULL) {
                        escputs((char *)icalproperty_get_categories(p));
@@ -1529,8 +1557,8 @@ void do_tasks_view(void) {
 
        wprintf("</tbody></table></div>\n");
 
-       /** Free the list */
-       free_calendar_buffer();
-
+       /* Free the list */
+       DeleteHash(&WC->disp_cal_items);
+       DeleteHashPos(&Pos);
 }