* add contexttype, so the subst can precheck the context pointer for us, and bad...
[citadel.git] / webcit / iconbar.c
index c05bce888b8d9c78e5b93d6b0088516cf70326ce..99350602b47902378fcbcb91a73a42729b677b96 100644 (file)
@@ -1,11 +1,9 @@
 /*
  * $Id$
+ *
+ * Displays and customizes the iconbar.
  */
-/**
- * \defgroup IconBar Displays and customizes the iconbar.
- * \ingroup MenuInfrastructure
- */
-/*@{*/
+
 #include "webcit.h"
 
 
@@ -15,6 +13,9 @@
 #define IB_TEXTONLY    2 /**< just text */
 
 
+void do_iconbar(void);
+void do_iconbar_roomlist(void);
+
 /**
  * \brief draw the icon bar?????
  */
@@ -27,61 +28,80 @@ void do_selected_iconbar(void) {
        }
 }
 
-/**
- * \brief draw the icon bar???
- */
-void do_iconbar(void) {
-       char iconbar[SIZ];
-       char buf[SIZ];
-       char key[SIZ], value[SIZ];
-       int i;
+void DontDeleteThis(void *Data){};
+
+#define IconbarIsEnabled(a, b) IconbarIsENABLED(a, sizeof(a) - 1, b)
 
-       WC->current_iconbar = current_iconbar_menu;
+long IconbarIsENABLED(const char *key, size_t keylen, long defval)
+{
+       void *Data;
+       if (GetHash(WC->IconBarSetttings, key, keylen,
+                   &Data))
+               return (long) Data;
+       else 
+               return defval;
+}
 
+#ifdef DBG_ICONBAR_HASH
+static char nbuf[32];
+inline const char *PrintInt(void *Prefstr)
+{
+       snprintf(nbuf, sizeof(nbuf), "%ld", (long)Prefstr);
+       return nbuf;
+}
+#endif
+
+void LoadIconSettings(void)
+{
+       struct wcsession *WCC = WC;
+       StrBuf *iconbar = NULL;
+       StrBuf *buf = NewStrBuf();;
+       StrBuf *key = NewStrBuf();
+       long val;
+       int i, nTokens;
+
+       WCC->current_iconbar = current_iconbar_menu;
+       if (WCC->IconBarSetttings == NULL)
+               WCC->IconBarSetttings = NewHash(1, NULL);
        /**
         * The initialized values of these variables also happen to
         * specify the default values for users who haven't customized
         * their iconbars.  These should probably be set in a master
         * configuration somewhere.
         */
-       int ib_displayas = 0;   /**< pictures and text, pictures, text */
-       int ib_logo = 0;        /**< Site logo */
-       int ib_summary = 1;     /**< Summary page icon */
-       int ib_inbox = 1;       /**< Inbox icon */
-       int ib_calendar = 1;    /**< Calendar icon */
-       int ib_contacts = 1;    /**< Contacts icon */
-       int ib_notes = 1;       /**< Notes icon */
-       int ib_tasks = 1;       /**< Tasks icon */
-       int ib_rooms = 1;       /**< Rooms icon */
-       int ib_users = 1;       /**< Users icon */
-       int ib_chat = 1;        /**< Chat icon */
-       int ib_advanced = 1;    /**< Advanced Options icon */
-       int ib_citadel = 1;     /**< 'Powered by Citadel' logo */
-       /*
-        */
 
-       get_preference("iconbar", iconbar, sizeof iconbar);
-       for (i=0; i<num_tokens(iconbar, ','); ++i) {
-               extract_token(buf, iconbar, i, ',', sizeof buf);
-               extract_token(key, buf, 0, '=', sizeof key);
-               extract_token(value, buf, 1, '=', sizeof value);
-
-               if (!strcasecmp(key, "ib_displayas")) ib_displayas = atoi(value);
-               if (!strcasecmp(key, "ib_logo")) ib_logo = atoi(value);
-               if (!strcasecmp(key, "ib_summary")) ib_summary = atoi(value);
-               if (!strcasecmp(key, "ib_inbox")) ib_inbox = atoi(value);
-               if (!strcasecmp(key, "ib_calendar")) ib_calendar = atoi(value);
-               if (!strcasecmp(key, "ib_contacts")) ib_contacts = atoi(value);
-               if (!strcasecmp(key, "ib_notes")) ib_notes = atoi(value);
-               if (!strcasecmp(key, "ib_tasks")) ib_tasks = atoi(value);
-               if (!strcasecmp(key, "ib_rooms")) ib_rooms = atoi(value);
-               if (!strcasecmp(key, "ib_users")) ib_users = atoi(value);
-               if (!strcasecmp(key, "ib_chat")) ib_chat = atoi(value);
-               if (!strcasecmp(key, "ib_advanced")) ib_advanced = atoi(value);
-               if (!strcasecmp(key, "ib_citadel")) ib_citadel = atoi(value);
+       if (get_preference("iconbar", &iconbar)) {
+               nTokens = StrBufNum_tokens(iconbar, ',');
+               for (i=0; i<nTokens; ++i) {
+                       StrBufExtract_token(buf, iconbar, i, ',');
+                       StrBufExtract_token(key, buf, 0, '=');
+                       val = StrBufExtract_long(buf, 1, '=');
+                       Put(WCC->IconBarSetttings, 
+                           ChrPtr(key), StrLength(key),
+                           (void*)val, DontDeleteThis);
+               }
        }
 
-        if (ib_logo) {
+#ifdef DBG_ICONBAR_HASH
+       dbg_PrintHash(WCC->IconBarSetttings, PrintInt, NULL);
+#endif
+
+       FreeStrBuf(&key);
+       FreeStrBuf(&buf);
+}
+
+
+/**
+ * \brief draw the icon bar???
+ */
+void do_iconbar(void) {
+       int ib_displayas = IB_PICTEXT;  /**< pictures and text, pictures, text */
+
+       LoadIconSettings();
+       ib_displayas = IconbarIsEnabled("ib_displayas", IB_PICTEXT);
+
+/** Site logo */
+       if (IconbarIsEnabled("ib_logo", 0)) {
                 if (ib_displayas != IB_TEXTONLY) {
                         wprintf("<div class=\"logo\"> <img "
                                 "src=\"image&name=hello\" alt=\"&nbsp;\"> "
@@ -91,10 +111,11 @@ void do_iconbar(void) {
                 wprintf("\n");
         }
 
-        if (ib_citadel) if (ib_displayas != IB_TEXTONLY) wprintf(
+/** 'Powered by Citadel' logo */
+        if (IconbarIsEnabled("ib_citadel", 1) && (ib_displayas != IB_TEXTONLY)) wprintf(
                 "<div class=\"logo_citadel\"> "
                 "<a href=\"http://www.citadel.org\" "
-                "title=\"%s\" target=\"aboutcit\"> "
+                "title=\"%s\"> "
                 "<img "
                 "src=\"static/citadel-logo.gif\" alt=\"%s\"></a> "
                 "</div>\n",
@@ -108,13 +129,14 @@ void do_iconbar(void) {
        wprintf(_("switch to room list"));
        wprintf("</a></li>");
 
-       if (ib_summary) {
+/** Summary page icon */
+       if (IconbarIsEnabled("ib_summary", 1)) {
                wprintf("<li><a href=\"summary\" "
                        "title=\"%s\" "
                        ">", _("Your summary page")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<img border=\"0\" width=\"32\" height=\"32\" "
+                       wprintf("<img alt=\"\" "
                                "src=\"static/summscreen_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
@@ -123,7 +145,8 @@ void do_iconbar(void) {
                wprintf("</a></li>\n");
        }
 
-       if (ib_inbox) {
+/** Inbox icon */
+       if (IconbarIsEnabled("ib_inbox", 1)) {
                wprintf("<li>"
                        "<a href=\"dotgoto?room=_MAIL_\" "
                        "title=\"%s\" "
@@ -131,7 +154,7 @@ void do_iconbar(void) {
                        _("Go to your email inbox")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<img border=\"0\" width=\"32\" height=\"32\" "
+                       wprintf("<img alt=\"\" "
                                "src=\"static/privatemess_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
@@ -148,7 +171,8 @@ void do_iconbar(void) {
                wprintf("</a></li>\n");
        }
 
-       if (ib_calendar) {
+/** Calendar icon */
+       if (IconbarIsEnabled("ib_calendar", 1)) {
                wprintf("<li>"
                        "<a href=\"dotgoto?room=_CALENDAR_\" "
                        "title=\"%s\" "
@@ -156,7 +180,7 @@ void do_iconbar(void) {
                        _("Go to your personal calendar")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<img border=\"0\" width=\"32\" height=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/calarea_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
@@ -165,7 +189,8 @@ void do_iconbar(void) {
                wprintf("</a></li>\n");
        }
 
-       if (ib_contacts) {
+/** Contacts icon */
+       if (IconbarIsEnabled("ib_contacts", 1)) {
                wprintf("<li>"
                        "<a href=\"dotgoto?room=_CONTACTS_\" "
                        "title=\"%s\" "
@@ -173,7 +198,7 @@ void do_iconbar(void) {
                        _("Go to your personal address book")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<img border=\"0\" width=\"32\" height=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/viewcontacts_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
@@ -182,7 +207,8 @@ void do_iconbar(void) {
                wprintf("</a></li>\n");
        }
 
-       if (ib_notes) {
+/** Notes icon */
+       if (IconbarIsEnabled("ib_notes", 1)) {
                wprintf("<li>"
                        "<a href=\"dotgoto?room=_NOTES_\" "
                        "title=\"%s\" "
@@ -190,7 +216,7 @@ void do_iconbar(void) {
                        _("Go to your personal notes")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<img border=\"0\" width=\"32\" height=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/storenotes_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
@@ -199,7 +225,8 @@ void do_iconbar(void) {
                wprintf("</a></li>\n");
        }
 
-       if (ib_tasks)  {
+/** Tasks icon */
+       if (IconbarIsEnabled("ib_tasks", 1))  {
                wprintf("<li>"
                        "<a href=\"dotgoto?room=_TASKS_\" "
                        "title=\"%s\" "
@@ -207,7 +234,7 @@ void do_iconbar(void) {
                        _("Go to your personal task list")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<img border=\"0\" width=\"32\" height=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/taskmanag_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
@@ -216,13 +243,14 @@ void do_iconbar(void) {
                wprintf("</a></li>\n");
        }
 
-       if (ib_rooms) {
+/** Rooms icon */
+       if (IconbarIsEnabled("ib_rooms", 1)) {
                wprintf("<li>"
                        "<a href=\"knrooms\" title=\"%s\" >",
                        _("List all of your accessible rooms")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<img border=\"0\" width=\"32\" height=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/chatrooms_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
@@ -231,14 +259,15 @@ void do_iconbar(void) {
                wprintf("</a></li>\n");
        }
 
-       if (ib_users) {
+/** Users icon */
+       if (IconbarIsEnabled("ib_users", 1)) {
                wprintf("<li>"
-                       "<a href=\"who\" title=\"%s\" "
+                       "<a href=\"do_template?template=who\" title=\"%s\" "
                        ">",
                        _("See who is online right now")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<img border=\"0\" width=\"32\" height=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/usermanag_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
@@ -247,13 +276,14 @@ void do_iconbar(void) {
                 
                wprintf("</a>\n");
 
-               if (ib_users > 1) {
+               if (IconbarIsEnabled("ib_users", 0)) {
                        wprintf("<ul id=\"wholist\">");
                        wprintf("</ul></li>\n");
                }
        }
 
-       if (ib_chat) {
+/** Chat icon */
+       if (IconbarIsEnabled("ib_chat", 1)) {
                wprintf("<li>"
                        "<a href=\"#\" onClick=\"window.open('chat', "
                        "'ctdl_chat_window', "
@@ -262,7 +292,7 @@ void do_iconbar(void) {
                        ">"
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<img border=\"0\" width=\"32\" height=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/citadelchat_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
@@ -271,15 +301,16 @@ void do_iconbar(void) {
                wprintf("</a></li>\n");
        }
 
-       if (ib_advanced) {
+/** Advanced Options icon */
+       if (IconbarIsEnabled("ib_advanced", 1)) {
                wprintf("<li>"
-                       "<a href=\"display_main_menu\" "
+                       "<a href=\"do_template?template=display_main_menu\" "
                        "title=\"%s\" "
                        ">",
                        _("Advanced Options Menu: Advanced Room commands, Account Info, and Chat")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<img border=\"0\" width=\"32\" height=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/advanpage2_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
@@ -290,13 +321,13 @@ void do_iconbar(void) {
 
        if ((WC->axlevel >= 6) || (WC->is_room_aide)) {
                wprintf("<li>"
-                       "<a href=\"display_aide_menu\" "
+                       "<a href=\"do_template?template=display_aide_menu\" "
                        "title=\"%s\" "
                        ">",
                        _("Room and system administration functions")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<img border=\"0\" width=\"32\" height=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/advanpage2_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
@@ -313,7 +344,7 @@ void do_iconbar(void) {
                
        );
        if (ib_displayas != IB_TEXTONLY) {
-       wprintf("<img border=\"0\" width=\"32\" height=\"32\" "
+       wprintf("<img alt=\"\" "
                "src=\"static/logoff_32x.gif\">");
        }
        if (ib_displayas != IB_PICONLY) {
@@ -333,14 +364,13 @@ void do_iconbar(void) {
 
        wprintf("</ul>\n");
 
-       if (ib_users > 1) {
+       if (IconbarIsEnabled("ib_users", 0)) {
                wprintf(
                        "<script type=\"text/javascript\"> "
-                       " new Ajax.PeriodicalUpdater('wholist', 'wholist_section', { method: 'get', frequency: 30 } );"
-                "</script> \n"
-               );
+                       " new Ajax.PeriodicalUpdater('wholist', 'do_template?template=wholist_section', { method: 'get', frequency: 30 } );"
+                       "</script> \n"
+                       );
        }
-
 }
 
 
@@ -350,11 +380,7 @@ void do_iconbar(void) {
  * we generate its innerHTML...
  */
 void do_iconbar_roomlist(void) {
-       char iconbar[SIZ];
-       char buf[SIZ];
-       char key[SIZ], value[SIZ];
-       int i;
-
+                               
        WC->current_iconbar = current_iconbar_roomlist;
 
        /**
@@ -363,24 +389,14 @@ void do_iconbar_roomlist(void) {
         * their iconbars.  These should probably be set in a master
         * configuration somewhere.
         */
-       int ib_displayas = 0;   /* pictures and text, pictures, text */
-       int ib_logo = 0;        /* Site logo */
-       int ib_citadel = 1;     /* 'Powered by Citadel' logo */
-       /*
-        */
+       int ib_displayas;
 
-       get_preference("iconbar", iconbar, sizeof iconbar);
-       for (i=0; i<num_tokens(iconbar, ','); ++i) {
-               extract_token(buf, iconbar, i, ',', sizeof buf);
-               extract_token(key, buf, 0, '=', sizeof key);
-               extract_token(value, buf, 1, '=', sizeof value);
+       LoadIconSettings();
 
-               if (!strcasecmp(key, "ib_displayas")) ib_displayas = atoi(value);
-               if (!strcasecmp(key, "ib_logo")) ib_logo = atoi(value);
-               if (!strcasecmp(key, "ib_citadel")) ib_citadel = atoi(value);
-       }
+       ib_displayas = IconbarIsEnabled("ib_displayas", IB_PICTEXT);    /* pictures and text, pictures, text */
 
-       if (ib_logo) {
+/** Site logo */
+       if (IconbarIsEnabled("ib_logo", 0)) {
                if (ib_displayas != IB_TEXTONLY) {
                         wprintf("<div class=\"logo\"> <img "
                                 "src=\"image&name=hello\" alt=\"&nbsp;\"> "
@@ -389,10 +405,11 @@ void do_iconbar_roomlist(void) {
                }
        }
 
-        if (ib_citadel) if (ib_displayas != IB_TEXTONLY) wprintf(
+/** 'Powered by Citadel' logo */
+        if (IconbarIsEnabled("ib_citadel", 1) && (ib_displayas != IB_TEXTONLY)) wprintf(
                 "<div class=\"logo_citadel\"> "
                 "<a href=\"http://www.citadel.org\" "
-                "title=\"%s\" target=\"aboutcit\"> "
+                "title=\"%s\"> "
                 "<img "
                 "src=\"static/citadel-logo.gif\" alt=\"%s\"></a> "
                 "</div>\n",
@@ -414,7 +431,7 @@ void do_iconbar_roomlist(void) {
                
        );
        if (ib_displayas != IB_TEXTONLY) {
-       wprintf("<img border=\"0\" width=\"32\" height=\"32\" "
+       wprintf("<img alt=\"\" "
                "src=\"static/logoff_32x.gif\">");
        }
        if (ib_displayas != IB_PICONLY) {
@@ -435,54 +452,13 @@ void do_iconbar_roomlist(void) {
  * \brief display a customized version of the iconbar
  */
 void display_customize_iconbar(void) {
-       char iconbar[SIZ];
-       char buf[SIZ];
-       char key[SIZ], value[SIZ];
        int i;
        int bar = 0;
+       long val;
 
-       /**
-        * The initialized values of these variables also happen to
-        * specify the default values for users who haven't customized
-        * their iconbars.  These should probably be set in a master
-        * configuration somewhere.
-        */
-       int ib_displayas = IB_PICTEXT;  /**< pictures and text, pictures, text */
-       int ib_logo = 0;        /**< Site logo */
-       int ib_summary = 1;     /**< Summary page icon */
-       int ib_inbox = 1;       /**< Inbox icon */
-       int ib_calendar = 1;    /**< Calendar icon */
-       int ib_contacts = 1;    /**< Contacts icon */
-       int ib_notes = 1;       /**< Notes icon */
-       int ib_tasks = 1;       /**< Tasks icon */
-       int ib_rooms = 1;       /**< Rooms icon */
-       int ib_users = 1;       /**< Users icon */
-       int ib_chat = 1;        /**< Chat icon */
-       int ib_advanced = 1;    /**< Advanced Options icon */
-       int ib_citadel = 1;     /**< 'Powered by Citadel' logo */
-       /*
-        */
+       int ib_displayas;
 
-       get_preference("iconbar", iconbar, sizeof iconbar);
-       for (i=0; i<num_tokens(iconbar, ','); ++i) {
-               extract_token(buf, iconbar, i, ',', sizeof buf);
-               extract_token(key, buf, 0, '=', sizeof key);
-               extract_token(value, buf, 1, '=', sizeof value);
-
-               if (!strcasecmp(key, "ib_displayas")) ib_displayas = atoi(value);
-               if (!strcasecmp(key, "ib_logo")) ib_logo = atoi(value);
-               if (!strcasecmp(key, "ib_summary")) ib_summary = atoi(value);
-               if (!strcasecmp(key, "ib_inbox")) ib_inbox = atoi(value);
-               if (!strcasecmp(key, "ib_calendar")) ib_calendar = atoi(value);
-               if (!strcasecmp(key, "ib_contacts")) ib_contacts = atoi(value);
-               if (!strcasecmp(key, "ib_notes")) ib_notes = atoi(value);
-               if (!strcasecmp(key, "ib_tasks")) ib_tasks = atoi(value);
-               if (!strcasecmp(key, "ib_rooms")) ib_rooms = atoi(value);
-               if (!strcasecmp(key, "ib_users")) ib_users = atoi(value);
-               if (!strcasecmp(key, "ib_chat")) ib_chat = atoi(value);
-               if (!strcasecmp(key, "ib_advanced")) ib_advanced = atoi(value);
-               if (!strcasecmp(key, "ib_citadel")) ib_citadel = atoi(value);
-       }
+       LoadIconSettings();
 
        output_headers(1, 1, 2, 0, 0, 0);
        wprintf("<div id=\"banner\">");
@@ -495,12 +471,13 @@ void display_customize_iconbar(void) {
        wprintf("<div class=\"fix_scrollbar_bug\">");
 
        wprintf("<form method=\"post\" action=\"commit_iconbar\">\n");
-       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
 
        wprintf("<table class=\"altern\" >\n");
        wprintf("<tr><td></td><td colspan=\"2\"><b>");
        wprintf(_("Display icons as:"));
        wprintf("</b>");
+       ib_displayas = IconbarIsEnabled("ib_displayas",IB_PICTEXT);
        for (i=0; i<=2; ++i) {
                wprintf("<input type=\"radio\" name=\"ib_displayas\" value=\"%d\"", i);
                if (ib_displayas == i) wprintf(" CHECKED");
@@ -518,6 +495,7 @@ void display_customize_iconbar(void) {
        wprintf("</td></tr>\n");
 
        bar = 1 - bar;
+       val = IconbarIsEnabled("ib_logo", 0);
        wprintf("<tr class=\"%s\"><td>"
                "<input type=\"radio\" name=\"ib_logo\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
                "<input type=\"radio\" name=\"ib_logo\" value=\"no\" %s> %s <br />"
@@ -528,13 +506,14 @@ void display_customize_iconbar(void) {
                "%s"
                "</td></tr>\n",
                (bar ? "even" : "odd"),
-               (ib_logo ? "CHECKED" : ""),_("Yes"),
-               (!ib_logo ? "CHECKED" : ""),_("No"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Site logo"),
                _("An icon describing this site")
        );
 
        bar = 1 - bar;
+       val = IconbarIsEnabled("ib_summary", 1);
        wprintf("<tr class=\"%s\"><td>"
                "<input type=\"radio\" name=\"ib_summary\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
                "<input type=\"radio\" name=\"ib_summary\" value=\"no\" %s> %s <br />"
@@ -545,13 +524,14 @@ void display_customize_iconbar(void) {
                "%s"
                "</td></tr>\n",
                (bar ? "even" : "odd"),
-               (ib_summary ? "CHECKED" : ""),_("Yes"),
-               (!ib_summary ? "CHECKED" : ""),_("No"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Summary"),
                _("Your summary page")
        );
 
        bar = 1 - bar;
+       val = IconbarIsEnabled("ib_inbox", 1);
        wprintf("<tr class=\"%s\"><td>"
                "<input type=\"radio\" name=\"ib_inbox\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
                "<input type=\"radio\" name=\"ib_inbox\" value=\"no\" %s> %s <br />"
@@ -562,13 +542,14 @@ void display_customize_iconbar(void) {
                "%s"
                "</td></tr>\n",
                (bar ? "even" : "odd"),
-               (ib_inbox ? "CHECKED" : ""),_("Yes"),
-               (!ib_inbox ? "CHECKED" : ""),_("No"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Mail (inbox)"),
                _("A shortcut to your email Inbox")
        );
 
        bar = 1 - bar;
+       val = IconbarIsEnabled("ib_contacts", 1);
        wprintf("<tr class=\"%s\"><td>"
                "<input type=\"radio\" name=\"ib_contacts\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
                "<input type=\"radio\" name=\"ib_contacts\" value=\"no\" %s> %s <br />"
@@ -579,13 +560,14 @@ void display_customize_iconbar(void) {
                "%s"
                "</td></tr>\n",
                (bar ? "even" : "odd"),
-               (ib_contacts ? "CHECKED" : ""),_("Yes"),
-               (!ib_contacts ? "CHECKED" : ""),_("No"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Contacts"),
                _("Your personal address book")
        );
 
        bar = 1 - bar;
+       val = IconbarIsEnabled("ib_notes", 1);
        wprintf("<tr class=\"%s\"><td>"
                "<input type=\"radio\" name=\"ib_notes\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
                "<input type=\"radio\" name=\"ib_notes\" value=\"no\" %s> %s <br />"
@@ -596,14 +578,14 @@ void display_customize_iconbar(void) {
                "%s"
                "</td></tr>\n",
                (bar ? "even" : "odd"),
-               (ib_notes ? "CHECKED" : ""),_("Yes"),
-               (!ib_notes ? "CHECKED" : ""),_("No"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Notes"),
                _("Your personal notes")
        );
 
-#ifdef WEBCIT_WITH_CALENDAR_SERVICE
        bar = 1 - bar;
+       val = IconbarIsEnabled("ib_calendar", 1);
        wprintf("<tr class=\"%s\"><td>"
                "<input type=\"radio\" name=\"ib_calendar\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
                "<input type=\"radio\" name=\"ib_calendar\" value=\"no\" %s> %s <br />"
@@ -614,13 +596,14 @@ void display_customize_iconbar(void) {
                "%s"
                "</td></tr>\n",
                (bar ? "even" : "odd"),
-               (ib_calendar ? "CHECKED" : ""),_("Yes"),
-               (!ib_calendar ? "CHECKED" : ""),_("No"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Calendar"),
                _("A shortcut to your personal calendar")
        );
 
        bar = 1 - bar;
+       val = IconbarIsEnabled("ib_tasks", 1);
        wprintf("<tr class=\"%s\"><td>"
                "<input type=\"radio\" name=\"ib_tasks\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
                "<input type=\"radio\" name=\"ib_tasks\" value=\"no\" %s> %s <br />"
@@ -631,14 +614,14 @@ void display_customize_iconbar(void) {
                "%s"
                "</td></tr>\n",
                (bar ? "even" : "odd"),
-               (ib_tasks ? "CHECKED" : ""),_("Yes"),
-               (!ib_tasks ? "CHECKED" : ""),_("No"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Tasks"),
                _("A shortcut to your personal task list")
        );
-#endif /* WEBCIT_WITH_CALENDAR_SERVICE */
 
        bar = 1 - bar;
+       val = IconbarIsEnabled("ib_rooms", 1);
        wprintf("<tr class=\"%s\"><td>"
                "<input type=\"radio\" name=\"ib_rooms\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
                "<input type=\"radio\" name=\"ib_rooms\" value=\"no\" %s> %s <br />"
@@ -649,14 +632,15 @@ void display_customize_iconbar(void) {
                "%s"
                "</td></tr>\n",
                (bar ? "even" : "odd"),
-               (ib_rooms ? "CHECKED" : ""),_("Yes"),
-               (!ib_rooms ? "CHECKED" : ""),_("No"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Rooms"),
                _("Clicking this icon displays a list of all accessible "
                "rooms (or folders) available.")
        );
 
        bar = 1 - bar;
+       val = IconbarIsEnabled("ib_users", 1);
        wprintf("<tr class=\"%s\"><td>"
                "<input type=\"radio\" name=\"ib_users\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
                "<input type=\"radio\" name=\"ib_users\" value=\"no\" %s> %s <br />"
@@ -668,15 +652,16 @@ void display_customize_iconbar(void) {
                "<br />%s"
                "</td></tr>\n",
                (bar ? "even" : "odd"),
-               (ib_users ? "CHECKED" : ""),_("Yes"),
-               (!ib_users ? "CHECKED" : ""),_("No"),
-               ((ib_users > 1) ? "CHECKED" : ""),_("Yes with users list"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
+               ((val > 1) ? "CHECKED" : ""),_("Yes with users list"),
                _("Who is online?"),
                _("Clicking this icon displays a list of all users "
                "currently logged in.")
        );
 
        bar = 1 - bar;
+       val = IconbarIsEnabled("ib_chat", 1);
        wprintf("<tr class=\"%s\"><td>"
                "<input type=\"radio\" name=\"ib_chat\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
                "<input type=\"radio\" name=\"ib_chat\" value=\"no\" %s> %s <br />"
@@ -687,8 +672,8 @@ void display_customize_iconbar(void) {
                "%s"
                "</td></tr>\n",
                (bar ? "even" : "odd"),
-               (ib_chat ? "CHECKED" : ""),_("Yes"),
-               (!ib_chat ? "CHECKED" : ""),_("No"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Chat"),
                _("Clicking this icon enters real-time chat mode "
                "with other users in the same room.")
@@ -696,6 +681,7 @@ void display_customize_iconbar(void) {
        );
 
        bar = 1 - bar;
+       val = IconbarIsEnabled("ib_advanced", 1);
        wprintf("<tr class=\"%s\"><td>"
                "<input type=\"radio\" name=\"ib_advanced\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
                "<input type=\"radio\" name=\"ib_advanced\" value=\"no\" %s> %s <br />"
@@ -706,14 +692,15 @@ void display_customize_iconbar(void) {
                "%s"
                "</td></tr>\n",
                (bar ? "even" : "odd"),
-               (ib_advanced ? "CHECKED" : ""),_("Yes"),
-               (!ib_advanced ? "CHECKED" : ""),_("No"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Advanced options"),
                _("Access to the complete menu of Citadel functions.")
 
        );
 
        bar = 1 - bar;
+       val = IconbarIsEnabled("ib_citadel", 1);
        wprintf("<tr class=\"%s\"><td>"
                "<input type=\"radio\" name=\"ib_citadel\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
                "<input type=\"radio\" name=\"ib_citadel\" value=\"no\" %s> %s <br />"
@@ -725,8 +712,8 @@ void display_customize_iconbar(void) {
                "%s"
                "</td></tr>\n",
                (bar ? "even" : "odd"),
-               (ib_citadel ? "CHECKED" : ""),_("Yes"),
-               (!ib_citadel ? "CHECKED" : ""),_("No"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Citadel logo"),
                _("Displays the 'Powered by Citadel' icon")
        );
@@ -749,7 +736,8 @@ void display_customize_iconbar(void) {
  * \brief commit the changes of an edited iconbar ????
  */
 void commit_iconbar(void) {
-       char iconbar[SIZ];
+       StrBuf *iconbar;
+       StrBuf *buf;
        int i;
 
        char *boxen[] = {
@@ -768,27 +756,30 @@ void commit_iconbar(void) {
                "ib_citadel"
        };
 
-       if (IsEmptyStr(bstr("ok_button"))) {
+       if (!havebstr("ok_button")) {
                display_main_menu();
                return;
        }
 
-       sprintf(iconbar, "ib_displayas=%d", atoi(bstr("ib_displayas")));
-
+       iconbar = NewStrBuf();
+       buf = NewStrBuf();
+       StrBufPrintf(iconbar, "ib_displayas=%d", ibstr("ib_displayas"));
        for (i=0; i<(sizeof(boxen)/sizeof(char *)); ++i) {
                char *Val;
-               if (!strcasecmp(bstr(boxen[i]), "yes")) {
+               if (!strcasecmp(BSTR(boxen[i]), "yes")) {
                        Val = "1";
                }
-               else if (!strcasecmp(bstr(boxen[i]), "yeslist")) {
+               else if (!strcasecmp(BSTR(boxen[i]), "yeslist")) {
                        Val = "2";
                }
                else {
                        Val = "0";
                }
-               sprintf(&iconbar[strlen(iconbar)], ",%s=%s", boxen[i], Val);
-       }
+               StrBufPrintf(buf, ",%s=%s", boxen[i], Val);
+               StrBufAppendBuf(iconbar, buf, 0);
 
+       }
+       FreeStrBuf(&buf);
        set_preference("iconbar", iconbar, 1);
 
        output_headers(1, 1, 2, 0, 0, 0);
@@ -806,8 +797,36 @@ void commit_iconbar(void) {
                "choices to continue."));
        wprintf("</td></tr></table>\n");
        wDumpContent(2);
+#ifdef DBG_ICONBAR_HASH
+       dbg_PrintHash(WC->IconBarSetttings, PrintInt, NULL);
+#endif
 }
 
 
+void tmplput_iconbar(StrBuf *Target, int nArgs, WCTemplateToken *Tokens, void *Context, int ContextType)
+{
+       struct wcsession *WCC = WC;
+       
+       if ((WCC != NULL) && (WCC->logged_in)) {
+               wprintf("<div id=\"iconbar\">");
+               do_selected_iconbar();
+               /** check for instant messages (these display in a new window) */
+               page_popup();
+               wprintf("</div>");
+       }
+}
+
+void 
+InitModule_ICONBAR
+(void)
+{
+       WebcitAddUrlHandler(HKEY("iconbar_ajax_menu"), do_iconbar, AJAX);
+       WebcitAddUrlHandler(HKEY("iconbar_ajax_rooms"), do_iconbar_roomlist, AJAX);
+       WebcitAddUrlHandler(HKEY("display_customize_iconbar"), display_customize_iconbar, 0);
+       WebcitAddUrlHandler(HKEY("commit_iconbar"), commit_iconbar, 0);
+       RegisterNamespace("ICONBAR", 0, 0, tmplput_iconbar, 0);
+
+}
+
 
 /*@}*/