]> code.citadel.org Git - citadel.git/blobdiff - webcit/calendar_view.c
* added gcc printf format checking to wprintf
[citadel.git] / webcit / calendar_view.c
index d0858d27e8206401e7a5cd037cb7d71e2c228132..4796713174648eb902be7ac40616ca46f5a92efb 100644 (file)
@@ -19,8 +19,7 @@ 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];
@@ -29,8 +28,7 @@ void embeddable_mini_calendar(int year, int month, char *urlformat)
 
        /* Determine what day to start.
        */
-       get_preference("weekstart", weekstart_buf, sizeof weekstart_buf);
-       weekstart = atoi(weekstart_buf);
+       get_pref_long("weekstart", &weekstart, 17);
 
        /*
        * Now back up to the 1st of the month...
@@ -144,7 +142,9 @@ void embeddable_mini_calendar(int year, int month, char *urlformat)
 
 }
 
-/* ajax embedder for the above mini calendar */
+/**
+ * \brief  ajax embedder for the above mini calendar 
+ */
 void ajax_mini_calendar(void) {
        char urlformat[256];
        int i, len;
@@ -167,7 +167,11 @@ void ajax_mini_calendar(void) {
  */
 void calendar_month_view_display_events(int year, int month, int day)
 {
-       int i;
+       long hklen;
+       char *HashKey;
+       void *vCal;
+       HashPos *Pos;
+       disp_cal *Cal;
        icalproperty *p = NULL;
        icalproperty *q = NULL;
        struct icaltimetype t;
@@ -180,15 +184,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 +214,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();
+       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 +246,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);
@@ -257,7 +264,7 @@ void calendar_month_view_display_events(int year, int month, int day)
                                        "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 +274,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:"));
@@ -302,7 +309,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,6 +348,7 @@ void calendar_month_view_display_events(int year, int month, int day)
                
                
        }
+       DeleteHashPos(&Pos);
 }
 
 
@@ -348,7 +357,10 @@ void calendar_month_view_display_events(int year, int month, int day)
  * \param thetime the month we want to see 
  */
 void calendar_month_view_brief_events(time_t thetime, const char *daycolor) {
-       int i;
+       long hklen;
+       char *HashKey;
+       void *vCal;
+       HashPos *Pos;
        time_t event_tt;
        time_t event_tts;
        time_t event_tte;
@@ -359,7 +371,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 +387,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();
+       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 +404,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 +414,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;
@@ -423,7 +435,7 @@ void calendar_month_view_brief_events(time_t thetime, const char *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 +443,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 +453,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;
@@ -469,13 +478,12 @@ void calendar_month_view(int year, int month, int day) {
        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
         */
-       get_preference("weekstart", weekstart_buf, sizeof weekstart_buf);
-       weekstart = atoi(weekstart_buf);
+       get_pref_long("weekstart", &weekstart, 17);
 
        /*
         * Now back up to the 1st of the month...
@@ -493,18 +501,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>"); 
 
@@ -531,7 +539,7 @@ void calendar_month_view(int year, int month, int day) {
 
        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,7 +554,7 @@ void calendar_month_view(int year, int month, int day) {
        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);
 
@@ -559,7 +567,7 @@ void calendar_month_view(int year, int month, int day) {
                                 chg_month = 0;
                         }
 
-                       /** Before displaying Sunday, start a new row */
+                       /* Before displaying the first day of the week, start a new row */
                        if ((i % 7) == 0) {
                                wprintf("<tr>");
                        }
@@ -589,21 +597,21 @@ void calendar_month_view(int year, int month, int day) {
 
                        wprintf("</td>");
 
-                       /** After displaying Saturday, end the row */
+                       /* After displaying the last day of the 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 */
-               "</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
@@ -620,11 +628,8 @@ void calendar_month_view(int year, int month, int day) {
        );
 }
 
-/**
- * \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 +640,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 +655,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");
@@ -689,13 +694,13 @@ void calendar_brief_month_view(int year, int month, int day) {
 
        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 +708,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 +726,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,11 +734,11 @@ 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");
                }
@@ -746,29 +751,20 @@ void calendar_brief_month_view(int year, int month, int day) {
                "</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 +774,10 @@ void calendar_day_view_display_events(time_t thetime,
        int dstart,
        int dend)
 {
-       int i;
+       long hklen;
+       char *HashKey;
+       void *vCal;
+       HashPos *Pos;
        icalproperty *p = NULL;
        icalproperty *q = NULL;
        time_t event_tt;
@@ -789,7 +788,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 +806,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 +834,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();
+       while (GetNextHashPos(WCC->disp_cal_items, Pos, &hklen, &HashKey, &vCal)) {
+               Cal = (disp_cal*)vCal;
 
                all_day_event = 0;
                ongoing_event=0;
@@ -1032,20 +1032,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 +1052,17 @@ 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);
        
-       /** 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 +1080,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;   
 
@@ -1128,7 +1123,7 @@ 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"
@@ -1156,7 +1151,7 @@ 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"
@@ -1184,12 +1179,12 @@ 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>");
@@ -1198,12 +1193,12 @@ void calendar_day_view(int year, int month, int day) {
 
        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\">",
                yesterday.year, yesterday.month, yesterday.day);
@@ -1220,7 +1215,7 @@ 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\">",
                tomorrow.year, tomorrow.month, tomorrow.day);
@@ -1229,9 +1224,9 @@ void calendar_day_view(int year, int month, int day) {
        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");
 
@@ -1249,11 +1244,15 @@ void calendar_day_view(int year, int month, int day) {
 }
 
 
-/**
- * \brief Display today's events.
+/*
+ * Display today's events.
  */
 void calendar_summary_view(void) {
-       int i;
+       long hklen;
+       char *HashKey;
+       void *vCal;
+       HashPos *Pos;
+       disp_cal *Cal;
        icalproperty *p;
        struct icaltimetype t;
        time_t event_tt;
@@ -1262,16 +1261,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();
+       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 +1300,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,29 +1310,13 @@ void calendar_summary_view(void) {
                        }
                }
        }
-       free_calendar_buffer();
-}
-
-
-/**
- * \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;
+       DeleteHashPos(&Pos);
+       DeleteHash(&WC->disp_cal_items);
 }
 
 
-
-/**
- * \brief do the whole calendar page
+/*
+ * do the whole calendar page
  * view any part of the calender. decide which way, etc.
  */
 void do_calendar_view(void) {
@@ -1339,19 +1325,19 @@ void do_calendar_view(void) {
        int year, month, day;
        char calview[SIZ];
 
-       /** 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;
 
-       /** Now see if a date was specified */
+       /* 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");
 
-       /** How would you like that cooked? */
+       /* How would you like that cooked? */
        if (havebstr("calview")) {
                strcpy(calview, bstr("calview"));
        }
@@ -1359,7 +1345,7 @@ 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);
        }
@@ -1375,17 +1361,13 @@ void do_calendar_view(void) {
                }
        }
 
-       /** Free the calendar stuff */
-       free_calendar_buffer();
-
+       /* Free the in-memory list of calendar items */
+       DeleteHash(&WC->disp_cal_items);
 }
 
 
-/**
- * \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.
+/*
+ * 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 +1376,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 +1399,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 +1433,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;
+       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 +1461,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();
+       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);
+               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 +1500,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 +1511,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 +1522,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);
 }