* shrinked the rest of these shutdown stuff into the module-handler.
[citadel.git] / webcit / roomops.c
index 8a03c30bb39e9fb3fbb8b148f6dfeeefd468e259..442d167ac5b7e086d4a35f6e301110a2bb293330 100644 (file)
@@ -4,11 +4,30 @@
  */
 
 #include "webcit.h"
-
-char floorlist[128][SIZ]; /**< list of our floor names */
+#include "webserver.h"
+#define MAX_FLOORS 128
+char floorlist[MAX_FLOORS][SIZ]; /**< list of our floor names */
 
 char *viewdefs[9]; /**< the different kinds of available views */
 
+/** See GetFloorListHash and GetRoomListHash for info on these. Basically we pull LFLR/LKRA etc. and set up a room HashList with these keys. */
+const char FLOOR_PARAM_NAMES[(FLOOR_PARAM_LEN + 1)][15] = {"ID",
+                                                          "NAME", 
+                                                          "ROOMS"};
+const char ROOM_PARAM_NAMES[(ROOM_PARAM_LEN + 1)][20] = {"NAME",
+                                                        "FLAG",
+                                                        "FLOOR",
+                                                        "LISTORDER",
+                                                        "ACL",
+                                                        "CURVIEW",
+                                                        "DEFVIEW",
+                                                        "LASTCHANGE"};
+/* Because avoiding strlen at run time is a Good Thing(TM) */
+const int FLOOR_PARAM_NAMELEN[(FLOOR_PARAM_LEN +1)] = {2, 4, 5};
+const int ROOM_PARAM_NAMELEN[(ROOM_PARAM_LEN +1)] = {4, 4, 5, 9, 3, 7, 7, 8};
+
+void display_whok(void);
+
 /*
  * Initialize the viewdefs with localized strings
  */
@@ -30,16 +49,22 @@ void initialize_viewdefs(void) {
 int is_view_allowed_as_default(int which_view)
 {
        switch(which_view) {
-               case VIEW_BBS:          return(1);
-               case VIEW_MAILBOX:      return(1);
-               case VIEW_ADDRESSBOOK:  return(1);
-               case VIEW_CALENDAR:     return(1);
-               case VIEW_TASKS:        return(1);
-               case VIEW_NOTES:        return(1);
-               case VIEW_WIKI:         return(0);      /**< because it isn't finished yet */
-               case VIEW_CALBRIEF:     return(0);
-               case VIEW_JOURNAL:      return(0);
-               default:                return(0);      /**< should never get here */
+       case VIEW_BBS:          return(1);
+       case VIEW_MAILBOX:      return(1);
+       case VIEW_ADDRESSBOOK:  return(1);
+       case VIEW_CALENDAR:     return(1);
+       case VIEW_TASKS:        return(1);
+       case VIEW_NOTES:        return(1);
+
+#ifdef TECH_PREVIEW
+       case VIEW_WIKI:         return(1);
+#else /* TECH_PREVIEW */
+       case VIEW_WIKI:         return(0);      /* because it isn't finished yet */
+#endif /* TECH_PREVIEW */
+
+       case VIEW_CALBRIEF:     return(0);
+       case VIEW_JOURNAL:      return(0);
+       default:                return(0);      /* should never get here */
        }
 }
 
@@ -47,22 +72,27 @@ int is_view_allowed_as_default(int which_view)
 /*
  * load the list of floors
  */
-void load_floorlist(void)
+void load_floorlist(StrBuf *Buf)
 {
        int a;
-       char buf[SIZ];
+       int Done = 0;
 
-       for (a = 0; a < 128; ++a)
+       for (a = 0; a < MAX_FLOORS; ++a)
                floorlist[a][0] = 0;
 
        serv_puts("LFLR");
-       serv_getln(buf, sizeof buf);
-       if (buf[0] != '1') {
+       StrBuf_ServGetln(Buf);
+       if (GetServerStatus(Buf, NULL) != 1) {
                strcpy(floorlist[0], "Main Floor");
                return;
        }
-       while (serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
-               extract_token(floorlist[extract_int(buf, 0)], buf, 1, '|', sizeof floorlist[0]);
+       while (!Done && (StrBuf_ServGetln(Buf)>=0)) {
+               if ( (StrLength(Buf)==3) && 
+                    !strcmp(ChrPtr(Buf), "000")) {
+                       Done = 1;
+                       break;
+               }
+               extract_token(floorlist[StrBufExtract_int(Buf, 0, '|')], ChrPtr(Buf), 1, '|', sizeof floorlist[0]);
        }
 }
 
@@ -70,7 +100,7 @@ void load_floorlist(void)
 /*
  * Free a session's march list
  */
-void free_march_list(struct wcsession *wcf)
+void free_march_list(wcsession *wcf)
 {
        struct march *mptr;
 
@@ -87,14 +117,14 @@ void free_march_list(struct wcsession *wcf)
 /*
  * remove a room from the march list
  */
-void remove_march(char *aaa)
+void remove_march(const StrBuf *aaa)
 {
        struct march *mptr, *mptr2;
 
        if (WC->march == NULL)
                return;
 
-       if (!strcasecmp(WC->march->march_name, aaa)) {
+       if (!strcasecmp(WC->march->march_name, ChrPtr(aaa))) {
                mptr = WC->march->next;
                free(WC->march);
                WC->march = mptr;
@@ -102,7 +132,7 @@ void remove_march(char *aaa)
        }
        mptr2 = WC->march;
        for (mptr = WC->march; mptr != NULL; mptr = mptr->next) {
-               if (!strcasecmp(mptr->march_name, aaa)) {
+               if (!strcasecmp(mptr->march_name, ChrPtr(aaa))) {
                        mptr2->next = mptr->next;
                        free(mptr);
                        mptr = mptr2;
@@ -247,16 +277,23 @@ void listrms(char *variety)
  */
 void zapped_list(void)
 {
+       WCTemplputParams SubTP;
+       StrBuf *Buf;
+
        output_headers(1, 1, 1, 0, 0, 0);
+       memset(&SubTP, 0, sizeof(WCTemplputParams));
+       Buf = NewStrBufPlain(_("Zapped (forgotten) rooms"), -1);
+       SubTP.Filter.ContextType = CTX_STRBUF;
+       SubTP.Context = Buf;
+       DoTemplate(HKEY("beginbox"), NULL, &SubTP);
 
-       svprintf("BOXTITLE", WCS_STRING, _("Zapped (forgotten) rooms"));
-       do_template("beginbox");
+       FreeStrBuf(&Buf);
 
        listrms("LZRM -1");
 
        wprintf("<br /><br />\n");
        wprintf(_("Click on any room to un-zap it and goto that room.\n"));
-       do_template("endbox");
+       do_template("endbox", NULL);
        wDumpContent(1);
 }
 
@@ -264,7 +301,7 @@ void zapped_list(void)
 /**
  * \brief read this room's info file (set v to 1 for verbose mode)
  */
-void readinfo(void)
+void readinfo(StrBuf *Target, WCTemplputParams *TP)
 {
        char buf[256];
        char briefinfo[128];
@@ -286,15 +323,15 @@ void readinfo(void)
                strcpy(&briefinfo[50], "...");
 
                 wprintf("<div class=\"infos\" "
-                "onclick=\"javascript:Effect.Appear('room_infos', { duration: 0.5 });\" "
-                ">");
+                       "onclick=\"javascript:Effect.Appear('room_infos', { duration: 0.5 });\" "
+                       ">");
                escputs(briefinfo);
                 wprintf("</div><div id=\"room_infos\" style=\"display:none;\">");
                wprintf("<img class=\"close_infos\" "
                        "onclick=\"javascript:Effect.Fade('room_infos', { duration: 0.5 });\" "
                        "src=\"static/closewindow.gif\" alt=\"%s\">",
                        _("Close window")
-               );
+                       );
                escputs(fullinfo);
                 wprintf("</div>");
        }
@@ -313,53 +350,54 @@ void readinfo(void)
  * keep the browser from using a cached icon from 
  * another room.
  */
-void embed_room_graphic(void) {
+void embed_room_graphic(StrBuf *Target, WCTemplputParams *TP)
+{
        char buf[SIZ];
 
        serv_puts("OIMG _roompic_");
        serv_getln(buf, sizeof buf);
        if (buf[0] == '2') {
                wprintf("<img height=\"64px\" src=\"image&name=_roompic_&room=");
-               urlescputs(WC->wc_roomname);
+               urlescputs(ChrPtr(WC->wc_roomname));
                wprintf("\">");
                serv_puts("CLOS");
                serv_getln(buf, sizeof buf);
        }
        else if (WC->wc_view == VIEW_ADDRESSBOOK) {
-               wprintf("<img height=48 width=48 src=\""
+               wprintf("<img class=\"roompic\" alt=\"\" src=\""
                        "static/viewcontacts_48x.gif"
                        "\">"
-               );
+                       );
        }
        else if ( (WC->wc_view == VIEW_CALENDAR) || (WC->wc_view == VIEW_CALBRIEF) ) {
-               wprintf("<img height=48 width=48 src=\""
+               wprintf("<img class=\"roompic\" alt=\"\" src=\""
                        "static/calarea_48x.gif"
                        "\">"
-               );
+                       );
        }
        else if (WC->wc_view == VIEW_TASKS) {
-               wprintf("<img height=48 width=48 src=\""
+               wprintf("<img class=\"roompic\" alt=\"\" src=\""
                        "static/taskmanag_48x.gif"
                        "\">"
-               );
+                       );
        }
        else if (WC->wc_view == VIEW_NOTES) {
-               wprintf("<img height=48 width=48 src=\""
+               wprintf("<img class=\"roompic\" alt=\"\" src=\""
                        "static/storenotes_48x.gif"
                        "\">"
-               );
+                       );
        }
        else if (WC->wc_view == VIEW_MAILBOX) {
-               wprintf("<img height=48 width=48 src=\""
+               wprintf("<img class=\"roompic\" alt=\"\" src=\""
                        "static/privatemess_48x.gif"
                        "\">"
-               );
+                       );
        }
        else {
-               wprintf("<img height=48 width=48 src=\""
+               wprintf("<img class=\"roompic\" alt=\"\" src=\""
                        "static/chatrooms_48x.gif"
                        "\">"
-               );
+                       );
        }
 
 }
@@ -369,11 +407,12 @@ void embed_room_graphic(void) {
 /**
  * \brief Display the current view and offer an option to change it
  */
-void embed_view_o_matic(void) {
+void embed_view_o_matic(StrBuf *Target, WCTemplputParams *TP)
+{
        int i;
 
        wprintf("<form name=\"viewomatic\" action=\"changeview\">\n");
-       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+       wprintf("\t<div style=\"display: inline;\">\n\t<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
        wprintf("<label for=\"view_name\">");
        wprintf(_("View as:"));
        wprintf("</label> "
@@ -394,7 +433,7 @@ void embed_view_o_matic(void) {
                        ||      ( (i == 0) && (WC->wc_default_view == 1) )      /**< mail or bulletin */
                        ||      ( (i == 1) && (WC->wc_default_view == 0) )      /**< mail or bulletin */
                        /** ||  ( (i == 7) && (WC->wc_default_view == 3) )      (calendar list temporarily disabled) */
-               ) {
+                       ) {
 
                        wprintf("<option %s value=\"changeview?view=%d\">",
                                ((i == WC->wc_view) ? "selected" : ""),
@@ -403,23 +442,24 @@ void embed_view_o_matic(void) {
                        wprintf("</option>\n");
                }
        }
-       wprintf("</select></form>\n");
+       wprintf("</select></div></form>\n");
 }
 
 
 /**
  * \brief Display a search box
  */
-void embed_search_o_matic(void) {
+void embed_search_o_matic(StrBuf *Target, WCTemplputParams *TP)
+{
        wprintf("<form name=\"searchomatic\" action=\"do_search\">\n");
-       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
-       wprintf("<label for=\"search_name\">");
+       wprintf("<div style=\"display: inline;\"><input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
+       wprintf("<label for=\"srchquery\">");
        wprintf(_("Search: "));
-       wprintf("</label> <input "
-               "type=\"text\" name=\"query\" size=\"15\" maxlength=\"128\" "
-               "id=\"search_name\" class=\"inputbox\">\n"
-       );
-       wprintf("</form>\n");
+       wprintf("</label><input ");
+       wprintf("%s", WC->serv_info->serv_fulltext_enabled ? "" : "disabled ");
+       wprintf("type=\"text\" name=\"query\" id=\"srchquery\" size=\"15\" maxlength=\"128\" class=\"inputbox\">\n"
+               );
+       wprintf("</div></form>\n");
 }
 
 
@@ -434,7 +474,6 @@ void embed_search_o_matic(void) {
 void embed_room_banner(char *got, int navbar_style) {
        char buf[256];
        char buf2[1024];
-       char sanitized_roomname[256];
        char with_files[256];
        int file_count=0;
        
@@ -443,7 +482,9 @@ void embed_room_banner(char *got, int navbar_style) {
         * If it isn't supplied, we fake it by issuing our own GOTO.
         */
        if (got == NULL) {
-               serv_printf("GOTO %s", WC->wc_roomname);
+               memset(buf, '0', 20);
+               buf[20] = '\0';
+               serv_printf("GOTO %s", ChrPtr(WC->wc_roomname));
                serv_getln(buf, sizeof buf);
                got = buf;
        }
@@ -453,15 +494,18 @@ void embed_room_banner(char *got, int navbar_style) {
                "       room_is_trash = %d;             \n"
                "</script>\n",
                WC->wc_is_trash
-       );
+               );
 
        /**
         * If the user happens to select the "make this my start page" link,
         * we want it to remember the URL as a "/dotskip" one instead of
         * a "skip" or "gotonext" or something like that.
         */
-       snprintf(WC->this_page, sizeof(WC->this_page), "dotskip&room=%s",
-               WC->wc_roomname);
+       if (WC->this_page == NULL)
+               WC->this_page = NewStrBuf();
+       StrBufPrintf(WC->this_page, 
+                    "dotskip&room=%s",
+                    ChrPtr(WC->wc_roomname));
 
        /** Check for new mail. */
        WC->new_mail = extract_int(&got[4], 9);
@@ -473,231 +517,238 @@ void embed_room_banner(char *got, int navbar_style) {
                serv_puts("RDIR");
                serv_getln(buf2, sizeof buf2);
                if (buf2[0] == '1') while (serv_getln(buf2, sizeof buf2), strcmp(buf2, "000"))
-                       file_count++;
+                                           file_count++;
                snprintf (with_files, sizeof with_files, 
-                         "; <a href=\"display_room_directory\"> %d %s </a>", 
+                         "; <a href=\"do_template?template=files\"> %d %s </a>", 
                          file_count, 
                          ((file_count>1) || (file_count == 0)  ? _("files") : _("file")));
        }
        else
                strcpy (with_files, "");
-               
-       stresc(sanitized_roomname, 256, WC->wc_roomname, 1, 1);
-       svprintf("ROOMNAME", WCS_STRING, "%s", sanitized_roomname);
-       svprintf("NUMMSGS", WCS_STRING,
-               _("%d new of %d messages%s"),
-               extract_int(&got[4], 1),
-               extract_int(&got[4], 2),
-               with_files
-       );
+       
+       svprintf(HKEY("NUMMSGS"), WCS_STRING,
+                _("%d new of %d messages%s"),
+                extract_int(&got[4], 1),
+                extract_int(&got[4], 2),
+                with_files
+               );
        svcallback("ROOMPIC", embed_room_graphic);
        svcallback("ROOMINFO", readinfo);
-       svcallback("VIEWOMATIC", embed_view_o_matic);
+       svcallback("VIEWOMATIC", embed_view_o_matic); 
        svcallback("SEARCHOMATIC", embed_search_o_matic);
-       svcallback("START", offer_start_page);
-
-       do_template("roombanner");
-       if (navbar_style != navbar_none) {
+       svcallback("START", offer_start_page); 
+       do_template("roombanner", NULL);
+       /* roombanner contains this for mobile */
+       if (navbar_style != navbar_none && !WC->is_mobile) { 
 
                wprintf("<div id=\"navbar\"><ul>");
 
                if (navbar_style == navbar_default) wprintf(
                        "<li class=\"ungoto\">"
                        "<a href=\"ungoto\">"
-                       "<img align=\"middle\" src=\"static/ungoto2_24x.gif\" border=\"0\">"
+                       "<img src=\"static/ungoto2_24x.gif\" alt=\"\">"
                        "<span class=\"navbar_link\">%s</span></A>"
                        "</li>\n", _("Ungoto")
-               );
+                       );
 
                if ( (navbar_style == navbar_default) && (WC->wc_view == VIEW_BBS) ) {
                        wprintf(
                                "<li class=\"newmess\">"
                                "<a href=\"readnew\">"
-                               "<img align=\"middle\" src=\"static/newmess2_24x.gif\" border=\"0\">"
+                               "<img src=\"static/newmess2_24x.gif\" alt=\"\">"
                                "<span class=\"navbar_link\">%s</span></A>"
                                "</li>\n", _("Read new messages")
-                       );
+                               );
                }
 
                if (navbar_style == navbar_default) {
                        switch(WC->wc_view) {
-                               case VIEW_ADDRESSBOOK:
-                                       wprintf(
-                                               "<li class=\"viewcontacts\">"
-                                               "<a href=\"readfwd\">"
-                                               "<img align=\"middle\" src=\"static/viewcontacts_24x.gif\" "
-                                               "border=\"0\">"
-                                               "<span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("View contacts")
+                       case VIEW_ADDRESSBOOK:
+                               wprintf(
+                                       "<li class=\"viewcontacts\">"
+                                       "<a href=\"readfwd\">"
+                                       "<img src=\"static/viewcontacts_24x.gif\" "
+                                       "alt=\"\">"
+                                       "<span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("View contacts")
                                        );
-                                       break;
-                               case VIEW_CALENDAR:
-                                       wprintf(
-                                               "<li class=\"staskday\">"
-                                               "<a href=\"readfwd?calview=day\">"
-                                               "<img align=\"middle\" src=\"static/taskday2_24x.gif\" "
-                                               "border=\"0\">"
-                                               "<span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("Day view")
+                               break;
+                       case VIEW_CALENDAR:
+                               wprintf(
+                                       "<li class=\"staskday\">"
+                                       "<a href=\"readfwd?calview=day\">"
+                                       "<img src=\"static/taskday2_24x.gif\" "
+                                       "alt=\"\">"
+                                       "<span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("Day view")
                                        );
-                                       wprintf(
-                                               "<li class=\"monthview\">"
-                                               "<a href=\"readfwd?calview=month\">"
-                                               "<img align=\"middle\" src=\"static/monthview2_24x.gif\" "
-                                               "border=\"0\">"
-                                               "<span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("Month view")
+                               wprintf(
+                                       "<li class=\"monthview\">"
+                                       "<a href=\"readfwd?calview=month\">"
+                                       "<img src=\"static/monthview2_24x.gif\" "
+                                       "alt=\"\">"
+                                       "<span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("Month view")
                                        );
-                                       break;
-                               case VIEW_CALBRIEF:
-                                       wprintf(
-                                               "<li class=\"monthview\">"
-                                               "<a href=\"readfwd?calview=month\">"
-                                               "<img align=\"middle\" src=\"static/monthview2_24x.gif\" "
-                                               "border=\"0\">"
-                                               "<span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("Calendar list")
+                               break;
+                       case VIEW_CALBRIEF:
+                               wprintf(
+                                       "<li class=\"monthview\">"
+                                       "<a href=\"readfwd?calview=month\">"
+                                       "<img src=\"static/monthview2_24x.gif\" "
+                                       "alt=\"\">"
+                                       "<span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("Calendar list")
                                        );
-                                       break;
-                               case VIEW_TASKS:
-                                       wprintf(
-                                               "<li class=\"taskmanag\">"
-                                               "<a href=\"readfwd\">"
-                                               "<img align=\"middle\" src=\"static/taskmanag_24x.gif\" "
-                                               "border=\"0\">"
-                                               "<span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("View tasks")
+                               break;
+                       case VIEW_TASKS:
+                               wprintf(
+                                       "<li class=\"taskmanag\">"
+                                       "<a href=\"readfwd\">"
+                                       "<img src=\"static/taskmanag_24x.gif\" "
+                                       "alt=\"\">"
+                                       "<span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("View tasks")
                                        );
-                                       break;
-                               case VIEW_NOTES:
-                                       wprintf(
-                                               "<li class=\"viewnotes\">"
-                                               "<a href=\"readfwd\">"
-                                               "<img align=\"middle\" src=\"static/viewnotes_24x.gif\" "
-                                               "border=\"0\">"
-                                               "<span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("View notes")
+                               break;
+                       case VIEW_NOTES:
+                               wprintf(
+                                       "<li class=\"viewnotes\">"
+                                       "<a href=\"readfwd\">"
+                                       "<img src=\"static/viewnotes_24x.gif\" "
+                                       "alt=\"\">"
+                                       "<span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("View notes")
                                        );
-                                       break;
-                               case VIEW_MAILBOX:
-                                       wprintf(
-                                               "<li class=\"readallmess\">"
-                                               "<a href=\"readfwd\">"
-                                               "<img align=\"middle\" src=\"static/readallmess3_24x.gif\" "
-                                               "border=\"0\">"
-                                               "<span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("View message list")
+                               break;
+                       case VIEW_MAILBOX:
+                               wprintf(
+                                       "<li class=\"readallmess\">"
+                                       "<a id=\"m_refresh\" href=\"readfwd\">"
+                                       "<img src=\"static/readallmess3_24x.gif\" "
+                                       "alt=\"\">"
+                                       "<span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("Refresh message list")
                                        );
-                                       break;
-                               case VIEW_WIKI:
-                                       wprintf(
-                                               "<li class=\"readallmess\">"
-                                               "<a href=\"readfwd\">"
-                                               "<img align=\"middle\" src=\"static/readallmess3_24x.gif\" "
-                                               "border=\"0\">"
-                                               "<span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("Wiki home")
+                               break;
+                       case VIEW_WIKI:
+                               wprintf(
+                                       "<li class=\"readallmess\">"
+                                       "<a href=\"readfwd\">"
+                                       "<img src=\"static/readallmess3_24x.gif\" "
+                                       "alt=\"\">"
+                                       "<span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("Wiki home")
                                        );
-                                       break;
-                               default:
-                                       wprintf(
-                                               "<li class=\"readallmess\">"
-                                               "<a href=\"readfwd\">"
-                                               "<img align=\"middle\" src=\"static/readallmess3_24x.gif\" "
-                                               "border=\"0\">"
-                                               "<span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("Read all messages")
+                               break;
+                       default:
+                               wprintf(
+                                       "<li class=\"readallmess\">"
+                                       "<a href=\"readfwd\">"
+                                       "<img src=\"static/readallmess3_24x.gif\" "
+                                       "alt=\"\">"
+                                       "<span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("Read all messages")
                                        );
-                                       break;
+                               break;
                        }
                }
 
                if (navbar_style == navbar_default) {
                        switch(WC->wc_view) {
-                               case VIEW_ADDRESSBOOK:
-                                       wprintf(
-                                               "<li class=\"addnewcontact\">"
-                                               "<a href=\"display_enter\">"
-                                               "<img align=\"middle\" src=\"static/addnewcontact_24x.gif\" "
-                                               "border=\"0\"><span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("Add new contact")
+                       case VIEW_ADDRESSBOOK:
+                               wprintf(
+                                       "<li class=\"addnewcontact\">"
+                                       "<a href=\"display_enter\">"
+                                       "<img src=\"static/addnewcontact_24x.gif\" "
+                                       "alt=\"\"><span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("Add new contact")
+                                       );
+                               break;
+                       case VIEW_CALENDAR:
+                       case VIEW_CALBRIEF:
+                               wprintf("<li class=\"addevent\"><a href=\"display_enter");
+                               if (havebstr("year" )) wprintf("?year=%s", bstr("year"));
+                               if (havebstr("month")) wprintf("?month=%s", bstr("month"));
+                               if (havebstr("day"  )) wprintf("?day=%s", bstr("day"));
+                               wprintf("\">"
+                                       "<img  src=\"static/addevent_24x.gif\" "
+                                       "alt=\"\"><span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("Add new event")
                                        );
-                                       break;
-                               case VIEW_CALENDAR:
-                               case VIEW_CALBRIEF:
-                                       wprintf("<li class=\"addevent\"><a href=\"display_enter");
-                                       if (!IsEmptyStr(bstr("year" ))) wprintf("?year=%s", bstr("year"));
-                                       if (!IsEmptyStr(bstr("month"))) wprintf("?month=%s", bstr("month"));
-                                       if (!IsEmptyStr(bstr("day"  ))) wprintf("?day=%s", bstr("day"));
-                                       wprintf("\">"
-                                               "<img align=\"middle\" src=\"static/addevent_24x.gif\" "
-                                               "border=\"0\"><span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("Add new event")
+                               break;
+                       case VIEW_TASKS:
+                               wprintf(
+                                       "<li class=\"newmess\">"
+                                       "<a href=\"display_enter\">"
+                                       "<img  src=\"static/newmess3_24x.gif\" "
+                                       "alt=\"\"><span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("Add new task")
                                        );
-                                       break;
-                               case VIEW_TASKS:
-                                       wprintf(
-                                               "<li class=\"newmess\">"
-                                               "<a href=\"display_enter\">"
-                                               "<img align=\"middle\" src=\"static/newmess3_24x.gif\" "
-                                               "border=\"0\"><span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("Add new task")
+                               break;
+                       case VIEW_NOTES:
+                               wprintf(
+                                       "<li class=\"enternewnote\">"
+                                       "<a href=\"add_new_note\">"
+                                       "<img  src=\"static/enternewnote_24x.gif\" "
+                                       "alt=\"\"><span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("Add new note")
                                        );
-                                       break;
-                               case VIEW_NOTES:
-                                       wprintf(
-                                               "<li class=\"enternewnote\">"
-                                               "<a href=\"javascript:add_new_note();\">"
-                                               "<img align=\"middle\" src=\"static/enternewnote_24x.gif\" "
-                                               "border=\"0\"><span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("Add new note")
+                               break;
+                       case VIEW_WIKI:
+                               safestrncpy(buf, bstr("page"), sizeof buf);
+                               str_wiki_index(buf);
+                               wprintf(
+                                       "<li class=\"newmess\">"
+                                       "<a href=\"display_enter?wikipage=%s\">"
+                                       "<img  src=\"static/newmess3_24x.gif\" "
+                                       "alt=\"\"><span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", buf, _("Edit this page")
                                        );
-                                       break;
-                               case VIEW_WIKI:
-                                       safestrncpy(buf, bstr("page"), sizeof buf);
-                                       str_wiki_index(buf);
-                                       wprintf(
-                                               "<li class=\"newmess\">"
-                                               "<a href=\"display_enter?wikipage=%s\">"
-                                               "<img align=\"middle\" src=\"static/newmess3_24x.gif\" "
-                                               "border=\"0\"><span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", buf, _("Edit this page")
+                               break;
+                       case VIEW_MAILBOX:
+                               wprintf(
+                                       "<li class=\"newmess\">"
+                                       "<a href=\"display_enter\">"
+                                       "<img  src=\"static/newmess3_24x.gif\" "
+                                       "alt=\"\"><span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("Write mail")
                                        );
-                                       break;
-                               case VIEW_MAILBOX:
-                                       wprintf(
-                                               "<li class=\"newmess\">"
-                                               "<a href=\"display_enter\">"
-                                               "<img align=\"middle\" src=\"static/newmess3_24x.gif\" "
-                                               "border=\"0\"><span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("Write mail")
+                               wprintf(
+                                       "<li class=\"newmess\">"
+                                       "<a href=\"javascript:deleteAllSelectedMessages();\">"
+                                       "<img  src=\"static/delete.gif\" "
+                                       "alt=\"\"><span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("Delete")
                                        );
-                                       break;
-                               default:
-                                       wprintf(
-                                               "<li class=\"newmess\">"
-                                               "<a href=\"display_enter\">"
-                                               "<img align=\"middle\" src=\"static/newmess3_24x.gif\" "
-                                               "border=\"0\"><span class=\"navbar_link\">"
-                                               "%s"
-                                               "</span></a></li>\n", _("Enter a message")
+                               break;
+                       default:
+                               wprintf(
+                                       "<li class=\"newmess\">"
+                                       "<a href=\"display_enter\">"
+                                       "<img  src=\"static/newmess3_24x.gif\" "
+                                       "alt=\"\"><span class=\"navbar_link\">"
+                                       "%s"
+                                       "</span></a></li>\n", _("Enter a message")
                                        );
-                                       break;
+                               break;
                        }
                }
 
@@ -705,23 +756,23 @@ void embed_room_banner(char *got, int navbar_style) {
                        "<li class=\"skipthisroom\">"
                        "<a href=\"skip\" "
                        "title=\"%s\">"
-                       "<img align=\"middle\" src=\"static/skipthisroom_24x.gif\" border=\"0\">"
+                       "<img  src=\"static/skipthisroom_24x.gif\" alt=\"\">"
                        "<span class=\"navbar_link\">%s</span></a>"
                        "</li>\n",
                        _("Leave all messages marked as unread, go to next room with unread messages"),
                        _("Skip this room")
-               );
+                       );
 
                if (navbar_style == navbar_default) wprintf(
                        "<li class=\"markngo\">"
                        "<a href=\"gotonext\" "
                        "title=\"%s\">"
-                       "<img align=\"middle\" src=\"static/markngo_24x.gif\" border=\"0\">"
+                       "<img  src=\"static/markngo_24x.gif\" alt=\"\">"
                        "<span class=\"navbar_link\">%s</span></a>"
                        "</li>\n",
                        _("Mark all messages as read, go to next room with unread messages"),
                        _("Goto next room")
-               );
+                       );
 
                wprintf("</ul></div>\n");
        }
@@ -729,56 +780,64 @@ void embed_room_banner(char *got, int navbar_style) {
 }
 
 
-/**
- * \brief back end routine to take the session to a new room
- * \param gname room to go to
- *
+/*
+ * back end routine to take the session to a new room
  */
-int gotoroom(char *gname)
+long gotoroom(const StrBuf *gname)
 {
-       char buf[SIZ];
+       StrBuf *Buf;
        static long ls = (-1L);
-       int err = 0;
+       long err = 0;
 
-       /** store ungoto information */
-       strcpy(WC->ugname, WC->wc_roomname);
+       /* store ungoto information */
+       strcpy(WC->ugname, ChrPtr(WC->wc_roomname));
        WC->uglsn = ls;
-
+       Buf = NewStrBuf();
        /** move to the new room */
-       serv_printf("GOTO %s", gname);
-       serv_getln(buf, sizeof buf);
-       if (buf[0] != '2') {
-               buf[3] = 0;
-               err = atoi(buf);
+       serv_printf("GOTO %s", ChrPtr(gname));
+       StrBuf_ServGetln(Buf);
+       if  (GetServerStatus(Buf, &err) != 2) {
                serv_puts("GOTO _BASEROOM_");
-               serv_getln(buf, sizeof buf);
-       }
-       if (buf[0] != '2') {
-               buf[3] = 0;
-               err = atoi(buf);
-               return err;
+               StrBuf_ServGetln(Buf);
+               /* 
+                * well, we know that this is the fallback case, 
+                * but we're interested that the first command 
+                * didn't work out in first place.
+                */
+               if (GetServerStatus(Buf, NULL) != 2) {
+                       FreeStrBuf(&Buf);
+                       return err;
+               }
        }
-       extract_token(WC->wc_roomname, &buf[4], 0, '|', sizeof WC->wc_roomname);
-       WC->room_flags = extract_int(&buf[4], 4);
+
+       if (WC->wc_roomname == NULL)
+               WC->wc_roomname = NewStrBuf();
+       else
+               FlushStrBuf(WC->wc_roomname);
+
+       StrBufExtract_token(WC->wc_roomname, Buf, 0, '|');
+       StrBufCutLeft(WC->wc_roomname, 4);
+       WC->room_flags = StrBufExtract_int(Buf, 4, '|');
        /* highest_msg_read = extract_int(&buf[4],6);
           maxmsgnum = extract_int(&buf[4],5);
-        */
-       WC->is_mailbox = extract_int(&buf[4],7);
-       ls = extract_long(&buf[4], 6);
-       WC->wc_floor = extract_int(&buf[4], 10);
-       WC->wc_view = extract_int(&buf[4], 11);
-       WC->wc_default_view = extract_int(&buf[4], 12);
-       WC->wc_is_trash = extract_int(&buf[4], 13);
-       WC->room_flags2 = extract_int(&buf[4], 14);
+       */
+       WC->is_mailbox = StrBufExtract_int(Buf, 7, '|');   
+       ls = StrBufExtract_long(Buf, 6, '|');
+       WC->wc_floor = StrBufExtract_int(Buf, 10, '|');
+       WC->wc_view = StrBufExtract_int(Buf, 11, '|');
+       WC->wc_default_view = StrBufExtract_int(Buf, 12, '|');
+       WC->wc_is_trash = StrBufExtract_int(Buf, 13, '|');
+       WC->room_flags2 = StrBufExtract_int(Buf, 14, '|');
 
        if (WC->is_aide)
                WC->is_room_aide = WC->is_aide;
        else
-               WC->is_room_aide = (char) extract_int(&buf[4], 8);
+               WC->is_room_aide = (char) StrBufExtract_int(Buf, 8, '|');
 
        remove_march(WC->wc_roomname);
-       if (!strcasecmp(gname, "_BASEROOM_"))
+       if (!strcasecmp(ChrPtr(gname), "_BASEROOM_"))
                remove_march(gname);
+       FreeStrBuf(&Buf);
 
        return err;
 }
@@ -826,8 +885,9 @@ char *pop_march(int desired_floor)
 
 
 
-/**
- *\brief Goto next room having unread messages.
+/*
+ * Goto next room having unread messages.
+ *
  * We want to skip over rooms that the user has already been to, and take the
  * user back to the lobby when done.  The room we end up in is placed in
  * newroom - which is set to 0 (the lobby) initially.
@@ -837,11 +897,13 @@ char *pop_march(int desired_floor)
 void gotonext(void)
 {
        char buf[256];
-       struct march *mptr, *mptr2;
+       struct march *mptr = NULL;
+       struct march *mptr2 = NULL;
        char room_name[128];
-       char next_room[128];
+       StrBuf *next_room;
+       int ELoop = 0;
 
-       /**
+       /*
         * First check to see if the march-mode list is already allocated.
         * If it is, pop the first room off the list and go there.
         */
@@ -851,24 +913,30 @@ void gotonext(void)
                serv_getln(buf, sizeof buf);
                if (buf[0] == '1')
                        while (serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
+                               if (IsEmptyStr(buf)) {
+                                       if (ELoop > 10000)
+                                               return;
+                                       if (ELoop % 100 == 0)
+                                               sleeeeeeeeeep(1);
+                                       ELoop ++;
+                                       continue;                                       
+                               }
                                extract_token(room_name, buf, 0, '|', sizeof room_name);
-                               if (strcasecmp(room_name, WC->wc_roomname)) {
+                               if (strcasecmp(room_name, ChrPtr(WC->wc_roomname))) {
                                        mptr = (struct march *) malloc(sizeof(struct march));
                                        mptr->next = NULL;
                                        safestrncpy(mptr->march_name, room_name, sizeof mptr->march_name);
                                        mptr->march_floor = extract_int(buf, 2);
                                        mptr->march_order = extract_int(buf, 3);
-                                       if (WC->march == NULL) {
+                                       if (WC->march == NULL) 
                                                WC->march = mptr;
-                                       } else {
-                                               mptr2 = WC->march;
-                                               while (mptr2->next != NULL)
-                                                       mptr2 = mptr2->next;
+                                       else 
                                                mptr2->next = mptr;
-                                       }
+                                       mptr2 = mptr;
                                }
+                               buf[0] = '\0';
                        }
-               /**
+               /*
                 * add _BASEROOM_ to the end of the march list, so the user will end up
                 * in the system base room (usually the Lobby>) at the end of the loop
                 */
@@ -885,36 +953,36 @@ void gotonext(void)
                                mptr2 = mptr2->next;
                        mptr2->next = mptr;
                }
-               /**
+               /*
                 * ...and remove the room we're currently in, so a <G>oto doesn't make us
                 * walk around in circles
                 */
                remove_march(WC->wc_roomname);
        }
        if (WC->march != NULL) {
-               strcpy(next_room, pop_march(-1));
+               next_room = NewStrBufPlain(pop_march(-1), -1);/*TODO: migrate march to strbuf */
        } else {
-               strcpy(next_room, "_BASEROOM_");
+               next_room = NewStrBufPlain(HKEY("_BASEROOM_"));
        }
 
 
        smart_goto(next_room);
+       FreeStrBuf(&next_room);
 }
 
 
-/**
- * \brief goto next room
- * \param next_room next room to go to
+/*
+ * goto next room
  */
-void smart_goto(char *next_room) {
+void smart_goto(const StrBuf *next_room) {
        gotoroom(next_room);
-       readloop("readnew");
+       readloop(readnew);
 }
 
 
 
-/**
- * \brief mark all messages in current room as having been read
+/*
+ * mark all messages in current room as having been read
  */
 void slrp_highest(void)
 {
@@ -925,30 +993,34 @@ void slrp_highest(void)
 }
 
 
-/**
- * \brief un-goto the previous room
+/*
+ * un-goto the previous room
  */
 void ungoto(void)
 {
-       char buf[SIZ];
+       StrBuf *Buf;
 
        if (!strcmp(WC->ugname, "")) {
                smart_goto(WC->wc_roomname);
                return;
        }
        serv_printf("GOTO %s", WC->ugname);
-       serv_getln(buf, sizeof buf);
-       if (buf[0] != '2') {
+       Buf = NewStrBuf();
+       StrBuf_ServGetln(Buf);
+       if (GetServerStatus(Buf, NULL) != 2) {
                smart_goto(WC->wc_roomname);
+               FreeStrBuf(&Buf);
                return;
        }
        if (WC->uglsn >= 0L) {
                serv_printf("SLRP %ld", WC->uglsn);
-               serv_getln(buf, sizeof buf);
+               StrBuf_ServGetln(Buf);
        }
-       strcpy(buf, WC->ugname);
+       FlushStrBuf(Buf);
+       StrBufAppendBufPlain(Buf, WC->ugname, -1, 0);
        strcpy(WC->ugname, "");
-       smart_goto(buf);
+       smart_goto(Buf);
+       FreeStrBuf(&Buf);
 }
 
 typedef struct __room_states {
@@ -960,16 +1032,16 @@ typedef struct __room_states {
        int order;
        int view;
        int flags2;
-}room_states;
+} room_states;
 
 
 
 
-/**
- * \brief Set/clear/read the "self-service list subscribe" flag for a room
+/*
+ * Set/clear/read the "self-service list subscribe" flag for a room
  * 
- * \param newval set to 0 to clear, 1 to set, any other value to leave unchanged.
- * \return return the new value.
+ * set newval to 0 to clear, 1 to set, any other value to leave unchanged.
+ * returns the new value.
  */
 
 int self_service(int newval) {
@@ -1013,8 +1085,8 @@ int self_service(int newval) {
 
        if (newval != current_value) {
                serv_printf("SETR %s|%s|%s|%d|0|%d|%d|%d|%d",
-                       name, password, dirname, flags,
-                       floor, order, view, flags2);
+                           name, password, dirname, flags,
+                           floor, order, view, flags2);
                serv_getln(buf, sizeof buf);
        }
 
@@ -1022,29 +1094,26 @@ int self_service(int newval) {
 
 }
 
-int
-is_selflist(room_states *RoomFlags)
+int is_selflist(room_states *RoomFlags)
 {
        return ((RoomFlags->flags2 & QR2_SELFLIST) != 0);
 }
 
-int
-is_publiclist(room_states *RoomFlags)
+int is_publiclist(room_states *RoomFlags)
 {
        return ((RoomFlags->flags2 & QR2_SMTP_PUBLIC) != 0);
 }
 
-int
-is_moderatedlist(room_states *RoomFlags)
+int is_moderatedlist(room_states *RoomFlags)
 {
        return ((RoomFlags->flags2 & QR2_MODERATED) != 0);
 }
 
-/**
- * \brief Set/clear/read the "self-service list subscribe" flag for a room
+/*
+ * Set/clear/read the "self-service list subscribe" flag for a room
  * 
- * \param newval set to 0 to clear, 1 to set, any other value to leave unchanged.
- * \return return the new value.
+ * set newval to 0 to clear, 1 to set, any other value to leave unchanged.
+ * returns the new value.
  */
 
 int get_roomflags(room_states *RoomOps) 
@@ -1088,11 +1157,12 @@ int set_roomflags(room_states *RoomOps)
 
 
 
-/**
- * \brief display the form for editing a room
+/*
+ * display the form for editing a room
  */
 void display_editroom(void)
 {
+       StrBuf *Buf;
        char buf[SIZ];
        char cmd[1024];
        char node[256];
@@ -1120,15 +1190,22 @@ void display_editroom(void)
        tab = bstr("tab");
        if (IsEmptyStr(tab)) tab = "admin";
 
-       load_floorlist();
+       Buf = NewStrBuf();
+       load_floorlist(Buf);
+       FreeStrBuf(&Buf);
        output_headers(1, 1, 1, 0, 0, 0);
 
        wprintf("<div class=\"fix_scrollbar_bug\">");
 
-       /** print the tabbed dialog */
-       wprintf("<ul class=\"tabbed_dialog\">\n");
+       wprintf("<br />\n");
 
-       wprintf("<li class=\"tablabel ");
+       /* print the tabbed dialog */
+       wprintf("<div align=\"center\">");
+       wprintf("<table id=\"AdminTabs\" border=\"0\" cellspacing=\"0\" cellpadding=\"0\""
+               "<tr align=\"center\" style=\"cursor:pointer\"><td>&nbsp;</td>"
+               );
+
+       wprintf("<td class=\"");
        if (!strcmp(tab, "admin")) {
                wprintf(" tab_cell_label\">");
                wprintf(_("Administration"));
@@ -1138,11 +1215,12 @@ void display_editroom(void)
                wprintf(_("Administration"));
                wprintf("</a>");
        }
-       wprintf("</li>\n");
+       wprintf("</td>\n");
+       wprintf("<td>&nbsp;</td>\n");
 
        if ( (WC->axlevel >= 6) || (WC->is_room_aide) ) {
 
-               wprintf("<li class=\"tablabel ");
+               wprintf("<td class=\"");
                if (!strcmp(tab, "config")) {
                        wprintf(" tab_cell_label\">");
                        wprintf(_("Configuration"));
@@ -1152,9 +1230,10 @@ void display_editroom(void)
                        wprintf(_("Configuration"));
                        wprintf("</a>");
                }
-               wprintf("</li>\n");
+               wprintf("</td>\n");
+               wprintf("<td>&nbsp;</td>\n");
 
-               wprintf("<li class=\"tablabel ");
+               wprintf("<td class=\"");
                if (!strcmp(tab, "expire")) {
                        wprintf(" tab_cell_label\">");
                        wprintf(_("Message expire policy"));
@@ -1164,9 +1243,10 @@ void display_editroom(void)
                        wprintf(_("Message expire policy"));
                        wprintf("</a>");
                }
-               wprintf("</li>\n");
+               wprintf("</td>\n");
+               wprintf("<td>&nbsp;</td>\n");
        
-               wprintf("<li class=\"tablabel ");
+               wprintf("<td class=\"");
                if (!strcmp(tab, "access")) {
                        wprintf(" tab_cell_label\">");
                        wprintf(_("Access controls"));
@@ -1176,9 +1256,10 @@ void display_editroom(void)
                        wprintf(_("Access controls"));
                        wprintf("</a>");
                }
-               wprintf("</li>\n");
+               wprintf("</td>\n");
+               wprintf("<td>&nbsp;</td>\n");
 
-               wprintf("<li class=\"tablabel ");
+               wprintf("<td class=\"");
                if (!strcmp(tab, "sharing")) {
                        wprintf(" tab_cell_label\">");
                        wprintf(_("Sharing"));
@@ -1188,9 +1269,10 @@ void display_editroom(void)
                        wprintf(_("Sharing"));
                        wprintf("</a>");
                }
-               wprintf("</li>\n");
+               wprintf("</td>\n");
+               wprintf("<td>&nbsp;</td>\n");
 
-               wprintf("<li class=\"tablabel ");
+               wprintf("<td class=\"");
                if (!strcmp(tab, "listserv")) {
                        wprintf(" tab_cell_label\">");
                        wprintf(_("Mailing list service"));
@@ -1200,11 +1282,12 @@ void display_editroom(void)
                        wprintf(_("Mailing list service"));
                        wprintf("</a>");
                }
-               wprintf("</li>\n");
+               wprintf("</td>\n");
+               wprintf("<td>&nbsp;</td>\n");
 
        }
 
-       wprintf("<li class=\"tablabel ");
+       wprintf("<td class=\"");
        if (!strcmp(tab, "feeds")) {
                wprintf(" tab_cell_label\">");
                wprintf(_("Remote retrieval"));
@@ -1214,12 +1297,19 @@ void display_editroom(void)
                wprintf(_("Remote retrieval"));
                wprintf("</a>");
        }
-       wprintf("</li>\n");
+       wprintf("</td>\n");
+       wprintf("<td>&nbsp;</td>\n");
 
-       wprintf("</ul>\n");
-       /** end tabbed dialog */        
+       wprintf("</tr></table>\n");
+       wprintf("</div>\n");
+       /* end tabbed dialog */ 
 
-       /** begin content of whatever tab is open now */
+       wprintf("<script type=\"text/javascript\">"
+               " Nifty(\"table#AdminTabs td\", \"small transparent top\");"
+               "</script>"
+               );
+
+       /* begin content of whatever tab is open now */
 
        if (!strcmp(tab, "admin")) {
                wprintf("<div class=\"tabcontent\">");
@@ -1248,7 +1338,7 @@ void display_editroom(void)
                if (!strncmp(buf, "550", 3)) {
                        wprintf("<br><br><div align=center>%s</div><br><br>\n",
                                _("Higher access is required to access this function.")
-                       );
+                               );
                }
                else if (buf[0] != '2') {
                        wprintf("<br><br><div align=center>%s</div><br><br>\n", &buf[4]);
@@ -1262,14 +1352,14 @@ void display_editroom(void)
                        er_flags2 = extract_int(&buf[4], 7);
        
                        wprintf("<form method=\"POST\" action=\"editroom\">\n");
-                       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+                       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
                
                        wprintf("<ul><li>");
                        wprintf(_("Name of room: "));
                        wprintf("<input type=\"text\" NAME=\"er_name\" VALUE=\"%s\" MAXLENGTH=\"%d\">\n",
                                er_name,
                                (sizeof(er_name)-1)
-                       );
+                               );
                
                        wprintf("<li>");
                        wprintf(_("Resides on floor: "));
@@ -1304,7 +1394,7 @@ void display_editroom(void)
        
                        wprintf("<li><input type=\"radio\" NAME=\"type\" VALUE=\"hidden\" ");
                        if ((er_flags & QR_PRIVATE) &&
-                       (er_flags & QR_GUESSNAME))
+                           (er_flags & QR_GUESSNAME))
                                wprintf("CHECKED ");
                        wprintf(" OnChange=\""
                                "       if (this.form.type[1].checked == true) {        "
@@ -1315,7 +1405,7 @@ void display_editroom(void)
                
                        wprintf("\n<li><input type=\"radio\" NAME=\"type\" VALUE=\"passworded\" ");
                        if ((er_flags & QR_PRIVATE) &&
-                       (er_flags & QR_PASSWORDED))
+                           (er_flags & QR_PASSWORDED))
                                wprintf("CHECKED ");
                        wprintf(" OnChange=\""
                                "       if (this.form.type[2].checked == true) {        "
@@ -1328,8 +1418,8 @@ void display_editroom(void)
                
                        wprintf("<li><input type=\"radio\" NAME=\"type\" VALUE=\"invonly\" ");
                        if ((er_flags & QR_PRIVATE)
-                       && ((er_flags & QR_GUESSNAME) == 0)
-                       && ((er_flags & QR_PASSWORDED) == 0))
+                           && ((er_flags & QR_GUESSNAME) == 0)
+                           && ((er_flags & QR_PASSWORDED) == 0))
                                wprintf("CHECKED ");
                        wprintf(" OnChange=\""
                                "       if (this.form.type[3].checked == true) {        "
@@ -1342,10 +1432,10 @@ void display_editroom(void)
                        if (er_flags & QR_MAILBOX)
                                wprintf("CHECKED ");
                        wprintf (" OnChange=\""
-                                     if (this.form.type[4].checked == true) {        "
-                                             this.form.er_floor.disabled = true;     "
-                                     }                                               "
-                               "\"> ");
+                                "      if (this.form.type[4].checked == true) {        "
+                                "              this.form.er_floor.disabled = true;     "
+                                "      }                                               "
+                                "\"> ");
                        wprintf(_("Personal (mailbox for you only)"));
                        
                        wprintf("\n<li><input type=\"checkbox\" NAME=\"bump\" VALUE=\"yes\" ");
@@ -1386,7 +1476,7 @@ void display_editroom(void)
        
                        wprintf("<li><input type=\"checkbox\" NAME=\"ulallowed\" VALUE=\"yes\" ");
                        if (er_flags & QR_UPLOAD)
-                       wprintf("CHECKED ");
+                               wprintf("CHECKED ");
                        wprintf("> ");
                        wprintf(_("Uploading allowed"));
                
@@ -1431,7 +1521,7 @@ void display_editroom(void)
                
                        wprintf("<li><input type=\"radio\" NAME=\"anon\" VALUE=\"no\" ");
                        if (((er_flags & QR_ANONONLY) == 0)
-                       && ((er_flags & QR_ANONOPT) == 0))
+                           && ((er_flags & QR_ANONOPT) == 0))
                                wprintf("CHECKED ");
                        wprintf("> ");
                        wprintf(_("No anonymous messages"));
@@ -1449,7 +1539,7 @@ void display_editroom(void)
                        wprintf(_("Prompt user when entering messages"));
                        wprintf("</ul>\n");
                
-               /* end of anon options */
+                       /* end of anon options */
                
                        wprintf("<li>");
                        wprintf(_("Room aide: "));
@@ -1470,13 +1560,13 @@ void display_editroom(void)
                                "</CENTER>\n",
                                _("Save changes"),
                                _("Cancel")
-                       );
+                               );
                }
                wprintf("</div>");
        }
 
 
-       /** Sharing the room with other Citadel nodes... */
+       /* Sharing the room with other Citadel nodes... */
        if (!strcmp(tab, "sharing")) {
                wprintf("<div class=\"tabcontent\">");
 
@@ -1487,30 +1577,30 @@ void display_editroom(void)
                serv_puts("CONF getsys|application/x-citadel-ignet-config");
                serv_getln(buf, sizeof buf);
                if (buf[0]=='1') while (serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
-                       extract_token(node, buf, 0, '|', sizeof node);
-                       not_shared_with = realloc(not_shared_with,
-                                       strlen(not_shared_with) + 32);
-                       strcat(not_shared_with, node);
-                       strcat(not_shared_with, "\n");
-               }
+                               extract_token(node, buf, 0, '|', sizeof node);
+                               not_shared_with = realloc(not_shared_with,
+                                                         strlen(not_shared_with) + 32);
+                               strcat(not_shared_with, node);
+                               strcat(not_shared_with, "\n");
+                       }
 
                serv_puts("GNET");
                serv_getln(buf, sizeof buf);
                if (buf[0]=='1') while (serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
-                       extract_token(cmd, buf, 0, '|', sizeof cmd);
-                       extract_token(node, buf, 1, '|', sizeof node);
-                       extract_token(remote_room, buf, 2, '|', sizeof remote_room);
-                       if (!strcasecmp(cmd, "ignet_push_share")) {
-                               shared_with = realloc(shared_with,
-                                               strlen(shared_with) + 32);
-                               strcat(shared_with, node);
-                               if (!IsEmptyStr(remote_room)) {
-                                       strcat(shared_with, "|");
-                                       strcat(shared_with, remote_room);
+                               extract_token(cmd, buf, 0, '|', sizeof cmd);
+                               extract_token(node, buf, 1, '|', sizeof node);
+                               extract_token(remote_room, buf, 2, '|', sizeof remote_room);
+                               if (!strcasecmp(cmd, "ignet_push_share")) {
+                                       shared_with = realloc(shared_with,
+                                                             strlen(shared_with) + 32);
+                                       strcat(shared_with, node);
+                                       if (!IsEmptyStr(remote_room)) {
+                                               strcat(shared_with, "|");
+                                               strcat(shared_with, remote_room);
+                                       }
+                                       strcat(shared_with, "\n");
                                }
-                               strcat(shared_with, "\n");
                        }
-               }
 
                for (i=0; i<num_tokens(shared_with, '\n'); ++i) {
                        extract_token(buf, shared_with, i, '\n', sizeof buf);
@@ -1523,7 +1613,7 @@ void display_editroom(void)
                        }
                }
 
-               /** Display the stuff */
+               /* Display the stuff */
                wprintf("<CENTER><br />"
                        "<table border=1 cellpadding=5><tr>"
                        "<td><B><I>");
@@ -1548,7 +1638,7 @@ void display_editroom(void)
                        extract_token(remote_room, buf, 1, '|', sizeof remote_room);
                        if (!IsEmptyStr(node)) {
                                wprintf("<form method=\"POST\" action=\"netedit\">");
-                               wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+                               wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
                                wprintf("<tr><td>%s</td>\n", node);
 
                                wprintf("<td>");
@@ -1589,7 +1679,7 @@ void display_editroom(void)
                        extract_token(node, not_shared_with, i, '\n', sizeof node);
                        if (!IsEmptyStr(node)) {
                                wprintf("<form method=\"POST\" action=\"netedit\">");
-                               wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+                               wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
                                wprintf("<tr><td>");
                                escputs(node);
                                wprintf("</td><td>"
@@ -1617,21 +1707,21 @@ void display_editroom(void)
                        "</table></CENTER><br />\n"
                        "<I><B>%s</B><ul><li>", _("Notes:"));
                wprintf(_("When sharing a room, "
-                       "it must be shared from both ends.  Adding a node to "
-                       "the 'shared' list sends messages out, but in order to"
-                       " receive messages, the other nodes must be configured"
-                       " to send messages out to your system as well. "
-                       "<li>If the remote room name is blank, it is assumed "
-                       "that the room name is identical on the remote node."
-                       "<li>If the remote room name is different, the remote "
-                       "node must also configure the name of the room here."
-                       "</ul></I><br />\n"
-               ));
+                         "it must be shared from both ends.  Adding a node to "
+                         "the 'shared' list sends messages out, but in order to"
+                         " receive messages, the other nodes must be configured"
+                         " to send messages out to your system as well. "
+                         "<li>If the remote room name is blank, it is assumed "
+                         "that the room name is identical on the remote node."
+                         "<li>If the remote room name is different, the remote "
+                         "node must also configure the name of the room here."
+                         "</ul></I><br />\n"
+                               ));
 
                wprintf("</div>");
        }
 
-       /** Mailing list management */
+       /* Mailing list management */
        if (!strcmp(tab, "listserv")) {
                room_states RoomFlags;
                wprintf("<div class=\"tabcontent\">");
@@ -1641,29 +1731,29 @@ void display_editroom(void)
                        "<tr><td VALIGN=TOP>");
 
                wprintf(_("<i>The contents of this room are being "
-                       "mailed <b>as individual messages</b> "
-                       "to the following list recipients:"
-                       "</i><br /><br />\n"));
+                         "mailed <b>as individual messages</b> "
+                         "to the following list recipients:"
+                         "</i><br /><br />\n"));
 
                serv_puts("GNET");
                serv_getln(buf, sizeof buf);
                if (buf[0]=='1') while (serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
-                       extract_token(cmd, buf, 0, '|', sizeof cmd);
-                       if (!strcasecmp(cmd, "listrecp")) {
-                               extract_token(recp, buf, 1, '|', sizeof recp);
+                               extract_token(cmd, buf, 0, '|', sizeof cmd);
+                               if (!strcasecmp(cmd, "listrecp")) {
+                                       extract_token(recp, buf, 1, '|', sizeof recp);
                        
-                               escputs(recp);
-                               wprintf(" <a href=\"netedit&cmd=remove&tab=listserv&line=listrecp|");
-                               urlescputs(recp);
-                               wprintf("\">");
-                               wprintf(_("(remove)"));
-                               wprintf("</A><br />");
+                                       escputs(recp);
+                                       wprintf(" <a href=\"netedit&cmd=remove&tab=listserv&line=listrecp|");
+                                       urlescputs(recp);
+                                       wprintf("\">");
+                                       wprintf(_("(remove)"));
+                                       wprintf("</A><br />");
+                               }
                        }
-               }
                wprintf("<br /><form method=\"POST\" action=\"netedit\">\n"
                        "<input type=\"hidden\" NAME=\"tab\" VALUE=\"listserv\">\n"
                        "<input type=\"hidden\" NAME=\"prefix\" VALUE=\"listrecp|\">\n");
-               wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+               wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
                wprintf("<input type=\"text\" id=\"add_as_listrecp\" NAME=\"line\">\n");
                wprintf("<input type=\"submit\" NAME=\"add_button\" VALUE=\"%s\">", _("Add"));
                wprintf("</form>\n");
@@ -1671,30 +1761,30 @@ void display_editroom(void)
                wprintf("</td><td VALIGN=TOP>\n");
                
                wprintf(_("<i>The contents of this room are being "
-                       "mailed <b>in digest form</b> "
-                       "to the following list recipients:"
-                       "</i><br /><br />\n"));
+                         "mailed <b>in digest form</b> "
+                         "to the following list recipients:"
+                         "</i><br /><br />\n"));
 
                serv_puts("GNET");
                serv_getln(buf, sizeof buf);
                if (buf[0]=='1') while (serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
-                       extract_token(cmd, buf, 0, '|', sizeof cmd);
-                       if (!strcasecmp(cmd, "digestrecp")) {
-                               extract_token(recp, buf, 1, '|', sizeof recp);
+                               extract_token(cmd, buf, 0, '|', sizeof cmd);
+                               if (!strcasecmp(cmd, "digestrecp")) {
+                                       extract_token(recp, buf, 1, '|', sizeof recp);
                        
-                               escputs(recp);
-                               wprintf(" <a href=\"netedit&cmd=remove&tab=listserv&line="
-                                       "digestrecp|");
-                               urlescputs(recp);
-                               wprintf("\">");
-                               wprintf(_("(remove)"));
-                               wprintf("</A><br />");
+                                       escputs(recp);
+                                       wprintf(" <a href=\"netedit&cmd=remove&tab=listserv&line="
+                                               "digestrecp|");
+                                       urlescputs(recp);
+                                       wprintf("\">");
+                                       wprintf(_("(remove)"));
+                                       wprintf("</A><br />");
+                               }
                        }
-               }
                wprintf("<br /><form method=\"POST\" action=\"netedit\">\n"
                        "<input type=\"hidden\" NAME=\"tab\" VALUE=\"listserv\">\n"
                        "<input type=\"hidden\" NAME=\"prefix\" VALUE=\"digestrecp|\">\n");
-               wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+               wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
                wprintf("<input type=\"text\" id=\"add_as_digestrecp\" NAME=\"line\">\n");
                wprintf("<input type=\"submit\" NAME=\"add_button\" VALUE=\"%s\">", _("Add"));
                wprintf("</form>\n");
@@ -1712,8 +1802,8 @@ void display_editroom(void)
                        _("Digest"),
                        _("Add recipients from Contacts or other address books"),
                        _("Add recipients from Contacts or other address books")
-               );
-               /** Pop open an address book -- end **/
+                       );
+               /* Pop open an address book -- end **/
 
                wprintf("<br />\n<form method=\"GET\" action=\"toggle_self_service\">\n");
 
@@ -1728,7 +1818,7 @@ void display_editroom(void)
                wprintf(_("The URL for subscribe/unsubscribe is: "));
                wprintf("<TT>%s://%s/listsub</TT></td></tr>\n",
                        (is_https ? "https" : "http"),
-                       WC->http_host);
+                       ChrPtr(WC->http_host));
                /* Public posting? */
                wprintf("<tr><td>");
                wprintf(_("Allow non-subscribers to mail to this room."));
@@ -1752,7 +1842,7 @@ void display_editroom(void)
        }
 
 
-       /** Configuration of The Dreaded Auto-Purger */
+       /* Configuration of The Dreaded Auto-Purger */
        if (!strcmp(tab, "expire")) {
                wprintf("<div class=\"tabcontent\">");
 
@@ -1761,7 +1851,7 @@ void display_editroom(void)
                if (!strncmp(buf, "550", 3)) {
                        wprintf("<br><br><div align=center>%s</div><br><br>\n",
                                _("Higher access is required to access this function.")
-                       );
+                               );
                }
                else if (buf[0] != '2') {
                        wprintf("<br><br><div align=center>%s</div><br><br>\n", &buf[4]);
@@ -1778,12 +1868,12 @@ void display_editroom(void)
                        }
                        
                        wprintf("<br /><form method=\"POST\" action=\"set_room_policy\">\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 border=0 cellspacing=5>\n");
                        wprintf("<tr><td>");
                        wprintf(_("Message expire policy for this room"));
                        wprintf("<br />(");
-                       escputs(WC->wc_roomname);
+                       escputs(ChrPtr(WC->wc_roomname));
                        wprintf(")</td><td>");
                        wprintf("<input type=\"radio\" NAME=\"roompolicy\" VALUE=\"0\" %s>",
                                ((roompolicy == 0) ? "CHECKED" : "") );
@@ -1843,20 +1933,20 @@ void display_editroom(void)
                        wprintf("</table>\n"
                                "<input type=\"hidden\" NAME=\"tab\" VALUE=\"expire\">\n"
                                "</form>\n"
-                       );
+                               );
                }
 
                wprintf("</div>");
        }
 
-       /** Access controls */
+       /* Access controls */
        if (!strcmp(tab, "access")) {
                wprintf("<div class=\"tabcontent\">");
                display_whok();
                wprintf("</div>");
        }
 
-       /** Fetch messages from remote locations */
+       /* Fetch messages from remote locations */
        if (!strcmp(tab, "feeds")) {
                wprintf("<div class=\"tabcontent\">");
 
@@ -1873,51 +1963,55 @@ void display_editroom(void)
                wprintf(_("Password"));
                wprintf("</th><th>");
                wprintf(_("Keep messages on server?"));
+               wprintf("</th><th>");
+               wprintf(_("Interval"));
                wprintf("</th><th> </th></tr>");
 
                serv_puts("GNET");
                serv_getln(buf, sizeof buf);
                bg = 1;
                if (buf[0]=='1') while (serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
-                       extract_token(cmd, buf, 0, '|', sizeof cmd);
-                       if (!strcasecmp(cmd, "pop3client")) {
-                               safestrncpy(recp, &buf[11], sizeof recp);
+                               extract_token(cmd, buf, 0, '|', sizeof cmd);
+                               if (!strcasecmp(cmd, "pop3client")) {
+                                       safestrncpy(recp, &buf[11], sizeof recp);
 
-                                bg = 1 - bg;
-                                wprintf("<tr class=\"%s\">",
-                                        (bg ? "even" : "odd")
-                                );
+                                       bg = 1 - bg;
+                                       wprintf("<tr class=\"%s\">",
+                                               (bg ? "even" : "odd")
+                                               );
 
-                               wprintf("<td>");
-                               extract_token(pop3_host, buf, 1, '|', sizeof pop3_host);
-                               escputs(pop3_host);
-                               wprintf("</td>");
+                                       wprintf("<td>");
+                                       extract_token(pop3_host, buf, 1, '|', sizeof pop3_host);
+                                       escputs(pop3_host);
+                                       wprintf("</td>");
 
-                               wprintf("<td>");
-                               extract_token(pop3_user, buf, 2, '|', sizeof pop3_user);
-                               escputs(pop3_user);
-                               wprintf("</td>");
+                                       wprintf("<td>");
+                                       extract_token(pop3_user, buf, 2, '|', sizeof pop3_user);
+                                       escputs(pop3_user);
+                                       wprintf("</td>");
 
-                               wprintf("<td>*****</td>");              /* Don't show the password */
+                                       wprintf("<td>*****</td>");              /* Don't show the password */
 
-                               wprintf("<td>%s</td>", extract_int(buf, 4) ? _("Yes") : _("No"));
+                                       wprintf("<td>%s</td>", extract_int(buf, 4) ? _("Yes") : _("No"));
 
-                               wprintf("<td class=\"button_link\">");
-                               wprintf(" <a href=\"netedit&cmd=remove&tab=feeds&line=pop3client|");
-                               urlescputs(recp);
-                               wprintf("\">");
-                               wprintf(_("(remove)"));
-                               wprintf("</a></td>");
+                                       wprintf("<td>%ld</td>", extract_long(buf, 5));  /* Fetching interval */
+                       
+                                       wprintf("<td class=\"button_link\">");
+                                       wprintf(" <a href=\"netedit&cmd=remove&tab=feeds&line=pop3client|");
+                                       urlescputs(recp);
+                                       wprintf("\">");
+                                       wprintf(_("(remove)"));
+                                       wprintf("</a></td>");
                        
-                               wprintf("</tr>");
+                                       wprintf("</tr>");
+                               }
                        }
-               }
 
                wprintf("<form method=\"POST\" action=\"netedit\">\n"
                        "<tr>"
                        "<input type=\"hidden\" name=\"tab\" value=\"feeds\">"
                        "<input type=\"hidden\" name=\"prefix\" value=\"pop3client|\">\n");
-               wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+               wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
                wprintf("<td>");
                wprintf("<input type=\"text\" id=\"add_as_pop3host\" NAME=\"line_pop3host\">\n");
                wprintf("</td>");
@@ -1931,6 +2025,9 @@ void display_editroom(void)
                wprintf("<input type=\"checkbox\" id=\"add_as_pop3keep\" NAME=\"line_pop3keep\" VALUE=\"1\">");
                wprintf("</td>");
                wprintf("<td>");
+               wprintf("<input type=\"text\" id=\"add_as_pop3int\" NAME=\"line_pop3int\" MAXLENGTH=\"5\">");
+               wprintf("</td>");
+               wprintf("<td>");
                wprintf("<input type=\"submit\" NAME=\"add_button\" VALUE=\"%s\">", _("Add"));
                wprintf("</td></tr>");
                wprintf("</form></table>\n");
@@ -1952,36 +2049,36 @@ void display_editroom(void)
                serv_getln(buf, sizeof buf);
                bg = 1;
                if (buf[0]=='1') while (serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
-                       extract_token(cmd, buf, 0, '|', sizeof cmd);
-                       if (!strcasecmp(cmd, "rssclient")) {
-                               safestrncpy(recp, &buf[10], sizeof recp);
-
-                                bg = 1 - bg;
-                                wprintf("<tr class=\"%s\">",
-                                        (bg ? "even" : "odd")
-                                );
-
-                               wprintf("<td>");
-                               extract_token(pop3_host, buf, 1, '|', sizeof pop3_host);
-                               escputs(pop3_host);
-                               wprintf("</td>");
-
-                               wprintf("<td class=\"button_link\">");
-                               wprintf(" <a href=\"netedit&cmd=remove&tab=feeds&line=rssclient|");
-                               urlescputs(recp);
-                               wprintf("\">");
-                               wprintf(_("(remove)"));
-                               wprintf("</a></td>");
+                               extract_token(cmd, buf, 0, '|', sizeof cmd);
+                               if (!strcasecmp(cmd, "rssclient")) {
+                                       safestrncpy(recp, &buf[10], sizeof recp);
+
+                                       bg = 1 - bg;
+                                       wprintf("<tr class=\"%s\">",
+                                               (bg ? "even" : "odd")
+                                               );
+
+                                       wprintf("<td>");
+                                       extract_token(pop3_host, buf, 1, '|', sizeof pop3_host);
+                                       escputs(pop3_host);
+                                       wprintf("</td>");
+
+                                       wprintf("<td class=\"button_link\">");
+                                       wprintf(" <a href=\"netedit&cmd=remove&tab=feeds&line=rssclient|");
+                                       urlescputs(recp);
+                                       wprintf("\">");
+                                       wprintf(_("(remove)"));
+                                       wprintf("</a></td>");
                        
-                               wprintf("</tr>");
+                                       wprintf("</tr>");
+                               }
                        }
-               }
 
                wprintf("<form method=\"POST\" action=\"netedit\">\n"
                        "<tr>"
                        "<input type=\"hidden\" name=\"tab\" value=\"feeds\">"
                        "<input type=\"hidden\" name=\"prefix\" value=\"rssclient|\">\n");
-               wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+               wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
                wprintf("<td>");
                wprintf("<input type=\"text\" id=\"add_as_pop3host\" size=\"72\" "
                        "maxlength=\"256\" name=\"line_pop3host\">\n");
@@ -1995,7 +2092,7 @@ void display_editroom(void)
        }
 
 
-       /** end content of whatever tab is open now */
+       /* end content of whatever tab is open now */
        wprintf("</div>\n");
 
        address_book_popup();
@@ -2003,25 +2100,32 @@ void display_editroom(void)
 }
 
 
-/** 
- * \brief Toggle self-service list subscription
+/* 
+ * Toggle self-service list subscription
  */
 void toggle_self_service(void) {
-//     int newval = 0;
        room_states RoomFlags;
 
        get_roomflags (&RoomFlags);
 
-       // Yank out the bits we want to change
-       RoomFlags.flags2 = RoomFlags.flags2 &   
-               !(QR2_SELFLIST|QR2_SMTP_PUBLIC|QR2_MODERATED);
-
-       if (!strcasecmp(bstr("QR2_SelfList"), "yes")) 
+       if (yesbstr("QR2_SelfList")) 
                RoomFlags.flags2 = RoomFlags.flags2 | QR2_SELFLIST;
-       if (!strcasecmp(bstr("QR2_SMTP_PUBLIC"), "yes")) 
+       else 
+               RoomFlags.flags2 = RoomFlags.flags2 & ~QR2_SELFLIST;
+
+       if (yesbstr("QR2_SMTP_PUBLIC")) 
                RoomFlags.flags2 = RoomFlags.flags2 | QR2_SMTP_PUBLIC;
-       if (!strcasecmp(bstr("QR2_Moderated"), "yes")) 
+       else
+               RoomFlags.flags2 = RoomFlags.flags2 & ~QR2_SMTP_PUBLIC;
+
+       if (yesbstr("QR2_Moderated")) 
                RoomFlags.flags2 = RoomFlags.flags2 | QR2_MODERATED;
+       else
+               RoomFlags.flags2 = RoomFlags.flags2 & ~QR2_MODERATED;
+       if (yesbstr("QR2_SubsOnly")) 
+               RoomFlags.flags2 = RoomFlags.flags2 | QR2_SMTP_PUBLIC;
+       else
+               RoomFlags.flags2 = RoomFlags.flags2 & ~QR2_SMTP_PUBLIC;
 
        set_roomflags (&RoomFlags);
        
@@ -2030,16 +2134,17 @@ void toggle_self_service(void) {
 
 
 
-/**
- * \brief save new parameters for a room
+/*
+ * save new parameters for a room
  */
 void editroom(void)
 {
-       char buf[SIZ];
-       char er_name[128];
-       char er_password[10];
-       char er_dirname[15];
-       char er_roomaide[26];
+       const StrBuf *Ptr;
+       StrBuf *Buf;
+       StrBuf *er_name;
+       StrBuf *er_password;
+       StrBuf *er_dirname;
+       StrBuf *er_roomaide;
        int er_floor;
        unsigned er_flags;
        int er_listingorder;
@@ -2048,178 +2153,211 @@ void editroom(void)
        int bump;
 
 
-       if (IsEmptyStr(bstr("ok_button"))) {
+       if (!havebstr("ok_button")) {
                strcpy(WC->ImportantMessage,
-                       _("Cancelled.  Changes were not saved."));
+                      _("Cancelled.  Changes were not saved."));
                display_editroom();
                return;
        }
        serv_puts("GETR");
-       serv_getln(buf, sizeof buf);
-
-       if (buf[0] != '2') {
-               strcpy(WC->ImportantMessage, &buf[4]);
+       Buf = NewStrBuf();
+       StrBuf_ServGetln(Buf);
+       if (GetServerStatus(Buf, NULL) != 2) {
+               StrBufCutLeft(Buf, 4);
+               strcpy(WC->ImportantMessage, ChrPtr(Buf));
                display_editroom();
+               FreeStrBuf(&Buf);
                return;
        }
-       extract_token(er_name, &buf[4], 0, '|', sizeof er_name);
-       extract_token(er_password, &buf[4], 1, '|', sizeof er_password);
-       extract_token(er_dirname, &buf[4], 2, '|', sizeof er_dirname);
-       er_flags = extract_int(&buf[4], 3);
-       er_listingorder = extract_int(&buf[4], 5);
-       er_defaultview = extract_int(&buf[4], 6);
-       er_flags2 = extract_int(&buf[4], 7);
 
-       strcpy(er_roomaide, bstr("er_roomaide"));
-       if (IsEmptyStr(er_roomaide)) {
+       er_name = NewStrBuf();
+       er_password = NewStrBuf();
+       er_dirname = NewStrBuf();
+       er_roomaide = NewStrBuf();
+
+       StrBufCutLeft(Buf, 4);
+       StrBufExtract_token(er_name, Buf, 0, '|');
+       StrBufExtract_token(er_password, Buf, 1, '|');
+       StrBufExtract_token(er_dirname, Buf, 2, '|');
+       er_flags = StrBufExtract_int(Buf, 3, '|');
+       er_listingorder = StrBufExtract_int(Buf, 5, '|');
+       er_defaultview = StrBufExtract_int(Buf, 6, '|');
+       er_flags2 = StrBufExtract_int(Buf, 7, '|');
+
+       er_roomaide = NewStrBufDup(sbstr("er_roomaide"));
+       if (StrLength(er_roomaide) == 0) {
                serv_puts("GETA");
-               serv_getln(buf, sizeof buf);
-               if (buf[0] != '2') {
-                       strcpy(er_roomaide, "");
+               StrBuf_ServGetln(Buf);
+               if (GetServerStatus(Buf, NULL) != 2) {
+                       FlushStrBuf(er_roomaide);
                } else {
-                       extract_token(er_roomaide, &buf[4], 0, '|', sizeof er_roomaide);
+                       StrBufCutLeft(Buf, 4);
+                       StrBufExtract_token(er_roomaide, Buf, 0, '|');
                }
        }
-       strcpy(buf, bstr("er_name"));
-       buf[128] = 0;
-       if (!IsEmptyStr(buf)) {
-               strcpy(er_name, buf);
+       Ptr = sbstr("er_name");
+       if (StrLength(Ptr) > 0) {
+               FlushStrBuf(er_name);
+               StrBufAppendBuf(er_name, Ptr, 0);
+       }
+
+       Ptr = sbstr("er_password");
+       if (StrLength(Ptr) > 0) {
+               FlushStrBuf(er_password);
+               StrBufAppendBuf(er_password, Ptr, 0);
        }
+               
 
-       strcpy(buf, bstr("er_password"));
-       buf[10] = 0;
-       if (!IsEmptyStr(buf))
-               strcpy(er_password, buf);
+       Ptr = sbstr("er_dirname");
+       if (StrLength(Ptr) > 0) { /* todo: cut 15 */
+               FlushStrBuf(er_dirname);
+               StrBufAppendBuf(er_dirname, Ptr, 0);
+       }
 
-       strcpy(buf, bstr("er_dirname"));
-       buf[15] = 0;
-       if (!IsEmptyStr(buf))
-               strcpy(er_dirname, buf);
 
-       strcpy(buf, bstr("type"));
+       Ptr = sbstr("type");
        er_flags &= !(QR_PRIVATE | QR_PASSWORDED | QR_GUESSNAME);
 
-       if (!strcmp(buf, "invonly")) {
+       if (!strcmp(ChrPtr(Ptr), "invonly")) {
                er_flags |= (QR_PRIVATE);
        }
-       if (!strcmp(buf, "hidden")) {
+       if (!strcmp(ChrPtr(Ptr), "hidden")) {
                er_flags |= (QR_PRIVATE | QR_GUESSNAME);
        }
-       if (!strcmp(buf, "passworded")) {
+       if (!strcmp(ChrPtr(Ptr), "passworded")) {
                er_flags |= (QR_PRIVATE | QR_PASSWORDED);
        }
-       if (!strcmp(buf, "personal")) {
+       if (!strcmp(ChrPtr(Ptr), "personal")) {
                er_flags |= QR_MAILBOX;
        } else {
                er_flags &= ~QR_MAILBOX;
        }
        
-       if (!strcmp(bstr("prefonly"), "yes")) {
+       if (yesbstr("prefonly")) {
                er_flags |= QR_PREFONLY;
        } else {
                er_flags &= ~QR_PREFONLY;
        }
 
-       if (!strcmp(bstr("readonly"), "yes")) {
+       if (yesbstr("readonly")) {
                er_flags |= QR_READONLY;
        } else {
                er_flags &= ~QR_READONLY;
        }
 
        
-       if (!strcmp(bstr("collabdel"), "yes")) {
+       if (yesbstr("collabdel")) {
                er_flags2 |= QR2_COLLABDEL;
        } else {
                er_flags2 &= ~QR2_COLLABDEL;
        }
 
-       if (!strcmp(bstr("permanent"), "yes")) {
+       if (yesbstr("permanent")) {
                er_flags |= QR_PERMANENT;
        } else {
                er_flags &= ~QR_PERMANENT;
        }
 
-       if (!strcmp(bstr("subjectreq"), "yes")) {
+       if (yesbstr("subjectreq")) {
                er_flags2 |= QR2_SUBJECTREQ;
        } else {
                er_flags2 &= ~QR2_SUBJECTREQ;
        }
 
-       if (!strcmp(bstr("network"), "yes")) {
+       if (yesbstr("network")) {
                er_flags |= QR_NETWORK;
        } else {
                er_flags &= ~QR_NETWORK;
        }
 
-       if (!strcmp(bstr("directory"), "yes")) {
+       if (yesbstr("directory")) {
                er_flags |= QR_DIRECTORY;
        } else {
                er_flags &= ~QR_DIRECTORY;
        }
 
-       if (!strcmp(bstr("ulallowed"), "yes")) {
+       if (yesbstr("ulallowed")) {
                er_flags |= QR_UPLOAD;
        } else {
                er_flags &= ~QR_UPLOAD;
        }
 
-       if (!strcmp(bstr("dlallowed"), "yes")) {
+       if (yesbstr("dlallowed")) {
                er_flags |= QR_DOWNLOAD;
        } else {
                er_flags &= ~QR_DOWNLOAD;
        }
 
-       if (!strcmp(bstr("visdir"), "yes")) {
+       if (yesbstr("visdir")) {
                er_flags |= QR_VISDIR;
        } else {
                er_flags &= ~QR_VISDIR;
        }
 
-       strcpy(buf, bstr("anon"));
+       Ptr = sbstr("anon");
 
        er_flags &= ~(QR_ANONONLY | QR_ANONOPT);
-       if (!strcmp(buf, "anononly"))
+       if (!strcmp(ChrPtr(Ptr), "anononly"))
                er_flags |= QR_ANONONLY;
-       if (!strcmp(buf, "anon2"))
+       if (!strcmp(ChrPtr(Ptr), "anon2"))
                er_flags |= QR_ANONOPT;
 
-       bump = 0;
-       if (!strcmp(bstr("bump"), "yes"))
-               bump = 1;
-
-       er_floor = atoi(bstr("er_floor"));
-
-       sprintf(buf, "SETR %s|%s|%s|%u|%d|%d|%d|%d|%u",
-               er_name, er_password, er_dirname, er_flags, bump, er_floor,
-               er_listingorder, er_defaultview, er_flags2);
-       serv_puts(buf);
-       serv_getln(buf, sizeof buf);
-       if (buf[0] != '2') {
-               strcpy(WC->ImportantMessage, &buf[4]);
+       bump = yesbstr("bump");
+
+       er_floor = ibstr("er_floor");
+
+       StrBufPrintf(Buf, "SETR %s|%s|%s|%u|%d|%d|%d|%d|%u",
+                    ChrPtr(er_name), 
+                    ChrPtr(er_password), 
+                    ChrPtr(er_dirname), 
+                    er_flags, 
+                    bump, 
+                    er_floor,
+                    er_listingorder, 
+                    er_defaultview, 
+                    er_flags2);
+       serv_putbuf(Buf);
+       StrBuf_ServGetln(Buf);
+       if (GetServerStatus(Buf, NULL) != 2) {
+               strcpy(WC->ImportantMessage, &ChrPtr(Buf)[4]);
                display_editroom();
+               FreeStrBuf(&Buf);
+               FreeStrBuf(&er_name);
+               FreeStrBuf(&er_password);
+               FreeStrBuf(&er_dirname);
+               FreeStrBuf(&er_roomaide);
                return;
        }
        gotoroom(er_name);
 
-       if (!IsEmptyStr(er_roomaide)) {
-               sprintf(buf, "SETA %s", er_roomaide);
-               serv_puts(buf);
-               serv_getln(buf, sizeof buf);
-               if (buf[0] != '2') {
-                       strcpy(WC->ImportantMessage, &buf[4]);
+       if (StrLength(er_roomaide) > 0) {
+               serv_printf("SETA %s", ChrPtr(er_roomaide));
+               StrBuf_ServGetln(Buf);
+               if (GetServerStatus(Buf, NULL) != 2) {
+                       strcpy(WC->ImportantMessage, &ChrPtr(Buf)[4]);
                        display_main_menu();
+                       FreeStrBuf(&Buf);
+                       FreeStrBuf(&er_name);
+                       FreeStrBuf(&er_password);
+                       FreeStrBuf(&er_dirname);
+                       FreeStrBuf(&er_roomaide);
                        return;
                }
        }
        gotoroom(er_name);
        strcpy(WC->ImportantMessage, _("Your changes have been saved."));
        display_editroom();
+       FreeStrBuf(&Buf);
+       FreeStrBuf(&er_name);
+       FreeStrBuf(&er_password);
+       FreeStrBuf(&er_dirname);
+       FreeStrBuf(&er_roomaide);
        return;
 }
 
 
-/**
- * \brief Display form for Invite, Kick, and show Who Knows a room
+/*
+ * Display form for Invite, Kick, and show Who Knows a room
  */
 void do_invt_kick(void) {
         char buf[SIZ], room[SIZ], username[SIZ];
@@ -2235,7 +2373,7 @@ void do_invt_kick(void) {
 
         strcpy(username, bstr("username"));
 
-        if (!IsEmptyStr(bstr("kick_button"))) {
+        if (havebstr("kick_button")) {
                 sprintf(buf, "KICK %s", username);
                 serv_puts(buf);
                 serv_getln(buf, sizeof buf);
@@ -2249,7 +2387,7 @@ void do_invt_kick(void) {
                 }
         }
 
-       if (!IsEmptyStr(bstr("invite_button"))) {
+       if (havebstr("invite_button")) {
                 sprintf(buf, "INVT %s", username);
                 serv_puts(buf);
                 serv_getln(buf, sizeof buf);
@@ -2268,8 +2406,8 @@ void do_invt_kick(void) {
 
 
 
-/**
- * \brief Display form for Invite, Kick, and show Who Knows a room
+/*
+ * Display form for Invite, Kick, and show Who Knows a room
  */
 void display_whok(void)
 {
@@ -2287,12 +2425,12 @@ void display_whok(void)
         
        wprintf("<table border=0 CELLSPACING=10><tr VALIGN=TOP><td>");
        wprintf(_("The users listed below have access to this room.  "
-               "To remove a user from the access list, select the user "
-               "name from the list and click 'Kick'."));
+                 "To remove a user from the access list, select the user "
+                 "name from the list and click 'Kick'."));
        wprintf("<br /><br />");
        
         wprintf("<CENTER><form method=\"POST\" action=\"do_invt_kick\">\n");
-       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
        wprintf("<input type=\"hidden\" NAME=\"tab\" VALUE=\"access\">\n");
         wprintf("<select NAME=\"username\" SIZE=\"10\" style=\"width:100%%\">\n");
         serv_puts("WHOK");
@@ -2312,28 +2450,28 @@ void display_whok(void)
 
        wprintf("</td><td>");
        wprintf(_("To grant another user access to this room, enter the "
-               "user name in the box below and click 'Invite'."));
+                 "user name in the box below and click 'Invite'."));
        wprintf("<br /><br />");
 
         wprintf("<CENTER><form method=\"POST\" action=\"do_invt_kick\">\n");
        wprintf("<input type=\"hidden\" NAME=\"tab\" VALUE=\"access\">\n");
-       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
         wprintf(_("Invite:"));
        wprintf(" ");
         wprintf("<input type=\"text\" name=\"username\" id=\"username_id\" style=\"width:100%%\"><br />\n"
                "<input type=\"hidden\" name=\"invite_button\" value=\"Invite\">"
                "<input type=\"submit\" value=\"%s\">"
                "</form></CENTER>\n", _("Invite"));
-               /** Pop open an address book -- begin **/
-               wprintf(
-                       "<a href=\"javascript:PopOpenAddressBook('username_id|%s');\" "
-                       "title=\"%s\">"
-                       "<img align=middle border=0 width=24 height=24 src=\"static/viewcontacts_24x.gif\">"
-                       "&nbsp;%s</a>",
-                       _("User"), 
-                       _("Users"), _("Users")
+       /* Pop open an address book -- begin **/
+       wprintf(
+               "<a href=\"javascript:PopOpenAddressBook('username_id|%s');\" "
+               "title=\"%s\">"
+               "<img align=middle border=0 width=24 height=24 src=\"static/viewcontacts_24x.gif\">"
+               "&nbsp;%s</a>",
+               _("User"), 
+               _("Users"), _("Users")
                );
-               /** Pop open an address book -- end **/
+       /* Pop open an address book -- end **/
 
        wprintf("</td></tr></table>\n");
        address_book_popup();
@@ -2342,30 +2480,33 @@ void display_whok(void)
 
 
 
-/**
- * \brief display the form for entering a new room
+/*
+ * display the form for entering a new room
  */
 void display_entroom(void)
 {
+       StrBuf *Buf;
        int i;
        char buf[SIZ];
 
+       Buf = NewStrBuf();
        serv_puts("CRE8 0");
        serv_getln(buf, sizeof buf);
 
        if (buf[0] != '2') {
                strcpy(WC->ImportantMessage, &buf[4]);
                display_main_menu();
+               FreeStrBuf(&Buf);
                return;
        }
 
        output_headers(1, 1, 1, 0, 0, 0);
 
-       svprintf("BOXTITLE", WCS_STRING, _("Create a new room"));
-       do_template("beginbox");
+       svprintf(HKEY("BOXTITLE"), WCS_STRING, _("Create a new room"));
+       do_template("beginbox", NULL);
 
        wprintf("<form name=\"create_room_form\" method=\"POST\" action=\"entroom\">\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\"> ");
 
@@ -2378,7 +2519,7 @@ void display_entroom(void)
        wprintf("<tr class=\"odd\"><td>");
        wprintf(_("Resides on floor: "));
        wprintf("</td><td>");
-        load_floorlist(); 
+        load_floorlist(Buf); 
         wprintf("<select name=\"er_floor\" size=\"1\">\n");
         for (i = 0; i < 128; ++i)
                 if (!IsEmptyStr(floorlist[i])) {
@@ -2390,12 +2531,12 @@ void display_entroom(void)
         wprintf("</select>\n");
         wprintf("</td></tr>");
 
-               /**
-                * Our clever little snippet of JavaScript automatically selects
-                * a public room if the view is set to Bulletin Board or wiki, and
-                * it selects a mailbox room otherwise.  The user can override this,
-                * of course.  We also disable the floor selector for mailboxes.
-                */
+       /*
+        * Our clever little snippet of JavaScript automatically selects
+        * a public room if the view is set to Bulletin Board or wiki, and
+        * it selects a mailbox room otherwise.  The user can override this,
+        * of course.  We also disable the floor selector for mailboxes.
+        */
        wprintf("<tr class=\"even\"><td>");
        wprintf(_("Default view for room: "));
        wprintf("</td><td>");
@@ -2484,16 +2625,17 @@ void display_entroom(void)
                fmout("LEFT");
        }
 
-       do_template("endbox");
+       do_template("endbox", NULL);
 
        wDumpContent(1);
+       FreeStrBuf(&Buf);
 }
 
 
 
 
-/**
- * \brief support function for entroom() -- sets the default view 
+/*
+ * support function for entroom() -- sets the default view 
  */
 void er_set_default_view(int newview) {
 
@@ -2520,58 +2662,67 @@ void er_set_default_view(int newview) {
        rm_bits2 = extract_int(&buf[4], 7);
 
        serv_printf("SETR %s|%s|%s|%d|0|%d|%d|%d|%d",
-               rm_name, rm_pass, rm_dir, rm_bits1, rm_floor,
-               rm_listorder, newview, rm_bits2
-       );
+                   rm_name, rm_pass, rm_dir, rm_bits1, rm_floor,
+                   rm_listorder, newview, rm_bits2
+               );
        serv_getln(buf, sizeof buf);
 }
 
 
 
-/**
- * \brief enter a new room
+/*
+ * Create a new room
  */
 void entroom(void)
 {
        char buf[SIZ];
-       char er_name[SIZ];
-       char er_type[SIZ];
-       char er_password[SIZ];
+       const StrBuf *er_name;
+       const StrBuf *er_type;
+       const StrBuf *er_password;
        int er_floor;
        int er_num_type;
        int er_view;
 
-       if (IsEmptyStr(bstr("ok_button"))) {
+       if (!havebstr("ok_button")) {
                strcpy(WC->ImportantMessage,
-                       _("Cancelled.  No new room was created."));
+                      _("Cancelled.  No new room was created."));
                display_main_menu();
                return;
        }
-       strcpy(er_name, bstr("er_name"));
-       strcpy(er_type, bstr("type"));
-       strcpy(er_password, bstr("er_password"));
-       er_floor = atoi(bstr("er_floor"));
-       er_view = atoi(bstr("er_view"));
+       er_name = sbstr("er_name");
+       er_type = sbstr("type");
+       er_password = sbstr("er_password");
+       er_floor = ibstr("er_floor");
+       er_view = ibstr("er_view");
 
        er_num_type = 0;
-       if (!strcmp(er_type, "hidden"))
+       if (!strcmp(ChrPtr(er_type), "hidden"))
                er_num_type = 1;
-       if (!strcmp(er_type, "passworded"))
+       else if (!strcmp(ChrPtr(er_type), "passworded"))
                er_num_type = 2;
-       if (!strcmp(er_type, "invonly"))
+       else if (!strcmp(ChrPtr(er_type), "invonly"))
                er_num_type = 3;
-       if (!strcmp(er_type, "personal"))
+       else if (!strcmp(ChrPtr(er_type), "personal"))
                er_num_type = 4;
 
-       sprintf(buf, "CRE8 1|%s|%d|%s|%d|%d|%d", 
-               er_name, er_num_type, er_password, er_floor, 0, er_view);
-       serv_puts(buf);
+       serv_printf("CRE8 1|%s|%d|%s|%d|%d|%d", 
+                   ChrPtr(er_name), 
+                   er_num_type, 
+                   ChrPtr(er_password), 
+                   er_floor, 
+                   0, 
+                   er_view);
+
        serv_getln(buf, sizeof buf);
        if (buf[0] != '2') {
                strcpy(WC->ImportantMessage, &buf[4]);
                display_main_menu();
                return;
        }
+       /** TODO: Room created, now udate the left hand icon bar for this user */
+       burn_folder_cache(0);   /* burn the old folder cache */
+       
+       
        gotoroom(er_name);
        do_change_view(er_view);                /* Now go there */
 }
@@ -2582,21 +2733,28 @@ void entroom(void)
  */
 void display_private(char *rname, int req_pass)
 {
+       WCTemplputParams SubTP;
+       StrBuf *Buf;
        output_headers(1, 1, 1, 0, 0, 0);
 
-       svprintf("BOXTITLE", WCS_STRING, _("Go to a hidden room"));
-       do_template("beginbox");
+       Buf = NewStrBufPlain(_("Go to a hidden room"), -1);
+       memset(&SubTP, 0, sizeof(WCTemplputParams));
+       SubTP.Filter.ContextType = CTX_STRBUF;
+       SubTP.Context = Buf;
+       DoTemplate(HKEY("beginbox"), NULL, &SubTP);
+
+       FreeStrBuf(&Buf);
 
        wprintf("<p>");
        wprintf(_("If you know the name of a hidden (guess-name) or "
-               "passworded room, you can enter that room by typing "
-               "its name below.  Once you gain access to a private "
-               "room, it will appear in your regular room listings "
-               "so you don't have to keep returning here."));
+                 "passworded room, you can enter that room by typing "
+                 "its name below.  Once you gain access to a private "
+                 "room, it will appear in your regular room listings "
+                 "so you don't have to keep returning here."));
        wprintf("</p>");
 
        wprintf("<form method=\"post\" action=\"goto_private\">\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\"> "
                "<tr class=\"even\"><td>");
@@ -2619,10 +2777,10 @@ void display_private(char *rname, int req_pass)
                "<input type=\"submit\" name=\"cancel_button\" value=\"%s\">",
                _("Go there"),
                _("Cancel")
-       );
+               );
        wprintf("</div></form>\n");
 
-       do_template("endbox");
+       do_template("endbox", NULL);
 
        wDumpContent(1);
 }
@@ -2635,20 +2793,18 @@ void goto_private(void)
        char hold_rm[SIZ];
        char buf[SIZ];
 
-       if (IsEmptyStr(bstr("ok_button"))) {
+       if (!havebstr("ok_button")) {
                display_main_menu();
                return;
        }
-       strcpy(hold_rm, WC->wc_roomname);
-       strcpy(buf, "GOTO ");
-       strcat(buf, bstr("gr_name"));
-       strcat(buf, "|");
-       strcat(buf, bstr("gr_pass"));
-       serv_puts(buf);
+       strcpy(hold_rm, ChrPtr(WC->wc_roomname));
+       serv_printf("GOTO %s|%s",
+                   bstr("gr_name"),
+                   bstr("gr_pass"));
        serv_getln(buf, sizeof buf);
 
        if (buf[0] == '2') {
-               smart_goto(bstr("gr_name"));
+               smart_goto(sbstr("gr_name"));
                return;
        }
        if (!strncmp(buf, "540", 3)) {
@@ -2678,11 +2834,11 @@ void display_zap(void)
        wprintf("<div id=\"content\" class=\"service\">\n");
 
        wprintf(_("If you select this option, <em>%s</em> will "
-               "disappear from your room list.  Is this what you wish "
-               "to do?<br />\n"), WC->wc_roomname);
+                 "disappear from your room list.  Is this what you wish "
+                 "to do?<br />\n"), ChrPtr(WC->wc_roomname));
 
        wprintf("<form method=\"POST\" action=\"zap\">\n");
-       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
        wprintf("<input type=\"submit\" NAME=\"ok_button\" VALUE=\"%s\">", _("Zap this room"));
        wprintf("&nbsp;");
        wprintf("<input type=\"submit\" NAME=\"cancel_button\" VALUE=\"%s\">", _("Cancel"));
@@ -2697,26 +2853,28 @@ void display_zap(void)
 void zap(void)
 {
        char buf[SIZ];
-       char final_destination[SIZ];
+       StrBuf *final_destination;
 
        /**
         * If the forget-room routine fails for any reason, we fall back
         * to the current room; otherwise, we go to the Lobby
         */
-       strcpy(final_destination, WC->wc_roomname);
+       final_destination = NewStrBufDup(WC->wc_roomname);
 
-       if (!IsEmptyStr(bstr("ok_button"))) {
-               serv_printf("GOTO %s", WC->wc_roomname);
+       if (havebstr("ok_button")) {
+               serv_printf("GOTO %s", ChrPtr(WC->wc_roomname));
                serv_getln(buf, sizeof buf);
                if (buf[0] == '2') {
                        serv_puts("FORG");
                        serv_getln(buf, sizeof buf);
                        if (buf[0] == '2') {
-                               strcpy(final_destination, "_BASEROOM_");
+                               FlushStrBuf(final_destination);
+                               StrBufAppendBufPlain(final_destination, HKEY("_BASEROOM_"), 0);
                        }
                }
        }
        smart_goto(final_destination);
+       FreeStrBuf(&final_destination);
 }
 
 
@@ -2728,14 +2886,20 @@ void delete_room(void)
 {
        char buf[SIZ];
 
+       
        serv_puts("KILL 1");
        serv_getln(buf, sizeof buf);
+       burn_folder_cache(0);   /* Burn the cahce of known rooms to update the icon bar */
        if (buf[0] != '2') {
                strcpy(WC->ImportantMessage, &buf[4]);
                display_main_menu();
                return;
        } else {
-               smart_goto("_BASEROOM_");
+               StrBuf *Buf;
+               
+               Buf = NewStrBufPlain(HKEY("_BASEROOM_"));
+               smart_goto(Buf);
+               FreeStrBuf(&Buf);
        }
 }
 
@@ -2753,8 +2917,8 @@ void netedit(void) {
        char cmpb0[SIZ];
        char cmpb1[SIZ];
        int i, num_addrs;
-
-       if (!IsEmptyStr(bstr("line_pop3host"))) {
+       /*/ TODO: do line dynamic! */
+       if (havebstr("line_pop3host")) {
                strcpy(line, bstr("prefix"));
                strcat(line, bstr("line_pop3host"));
                strcat(line, "|");
@@ -2762,10 +2926,12 @@ void netedit(void) {
                strcat(line, "|");
                strcat(line, bstr("line_pop3pass"));
                strcat(line, "|");
-               strcat(line, atoi(bstr("line_pop3keep")) ? "1" : "0" );
+               strcat(line, ibstr("line_pop3keep") ? "1" : "0" );
+               strcat(line, "|");
+               sprintf(&line[strlen(line)],"%ld", lbstr("line_pop3int"));
                strcat(line, bstr("suffix"));
        }
-       else if (!IsEmptyStr(bstr("line"))) {
+       else if (havebstr("line")) {
                strcpy(line, bstr("prefix"));
                strcat(line, bstr("line"));
                strcat(line, bstr("suffix"));
@@ -2797,7 +2963,7 @@ void netedit(void) {
                extract_token(cmpb0, line, 0, '|', sizeof cmpb0);
                extract_token(cmpb1, line, 1, '|', sizeof cmpb1);
                if ( (strcasecmp(cmpa0, cmpb0)) 
-                  || (strcasecmp(cmpa1, cmpb1)) ) {
+                    || (strcasecmp(cmpa1, cmpb1)) ) {
                        fprintf(fp, "%s\n", buf);
                }
        }
@@ -2816,7 +2982,7 @@ void netedit(void) {
                serv_puts(buf);
        }
 
-       if (!IsEmptyStr(bstr("add_button"))) {
+       if (havebstr("add_button")) {
                num_addrs = num_tokens(bstr("line"), ',');
                if (num_addrs < 2) {
                        /* just adding one node or address */
@@ -2864,7 +3030,13 @@ void room_to_folder(char *folder, char *room, int floor, int is_mailbox)
         * Otherwise, prefix the floor name as a "public folders" moniker
         */
        else {
-               sprintf(folder, "%s|%s", floorlist[floor], room);
+               if (floor > MAX_FLOORS) {
+                       wc_backtrace ();
+                       sprintf(folder, "%%%%%%|%s", room);
+               }
+               else {
+                       sprintf(folder, "%s|%s", floorlist[floor], room);
+               }
        }
 
        /**
@@ -2900,7 +3072,7 @@ void do_change_view(int newview) {
 void change_view(void) {
        int view;
 
-       view = atol(bstr("view"));
+       view = lbstr("view");
        do_change_view(view);
 }
 
@@ -2939,7 +3111,7 @@ void do_folder_view(struct folder *fold, int max_folders, int num_floors) {
                "       var openGif = 'static/folder_open.gif';         \n"
                "                                                       \n"
                "       rootNode = new TreeNode(1, 'root node - hide'); \n"
-       );
+               );
 
        levels = 0;
        for (i=0; i<max_folders; ++i) {
@@ -2949,7 +3121,7 @@ void do_folder_view(struct folder *fold, int max_folders, int num_floors) {
                        int len;
                        len = strlen(fold[i].name);
                        if ( (!strncasecmp(fold[i].name, fold[i+1].name, len))
-                          && (fold[i+1].name[len] == '|') ) {
+                            && (fold[i+1].name[len] == '|') ) {
                                has_subfolders = 1;
                        }
                }
@@ -3018,7 +3190,7 @@ void do_folder_view(struct folder *fold, int max_folders, int num_floors) {
        wprintf("container = document.getElementById('roomlist_div');   \n"
                "showTree('');  \n"
                "</script>\n"
-       );
+               );
 
        free(parents);
        /** END TREE MENU */
@@ -3034,7 +3206,6 @@ void do_rooms_view(struct folder *fold, int max_folders, int num_floors) {
        char buf[256];
        char floor_name[256];
        char old_floor_name[256];
-       char boxtitle[256];
        int levels, oldlevels;
        int i, t;
        int num_boxes = 0;
@@ -3061,12 +3232,12 @@ void do_rooms_view(struct folder *fold, int max_folders, int num_floors) {
 
                levels = num_tokens(fold[i].name, '|');
                extract_token(floor_name, fold[i].name, 0,
-                       '|', sizeof floor_name);
+                             '|', sizeof floor_name);
 
                if ( (strcasecmp(floor_name, old_floor_name))
-                  && (!IsEmptyStr(old_floor_name)) ) {
+                    && (!IsEmptyStr(old_floor_name)) ) {
                        /* End inner box */
-                       do_template("endbox");
+                       do_template("endbox", NULL);
                        wprintf("<br>");
 
                        ++num_boxes;
@@ -3080,10 +3251,16 @@ void do_rooms_view(struct folder *fold, int max_folders, int num_floors) {
                strcpy(old_floor_name, floor_name);
 
                if (levels == 1) {
-                       /** Begin inner box */
-                       stresc(boxtitle, 256, floor_name, 1, 0);
-                       svprintf("BOXTITLE", WCS_STRING, boxtitle);
-                       do_template("beginbox");
+                       StrBuf *Buf;
+                       WCTemplputParams SubTP;
+
+                       Buf = NewStrBufPlain(floor_name, -1);
+                       memset(&SubTP, 0, sizeof(WCTemplputParams));
+                       SubTP.Filter.ContextType = CTX_STRBUF;
+                       SubTP.Context = Buf;
+                       DoTemplate(HKEY("beginbox"), NULL, &SubTP);
+                       
+                       FreeStrBuf(&Buf);
                }
 
                oldlevels = levels;
@@ -3121,7 +3298,7 @@ void do_rooms_view(struct folder *fold, int max_folders, int num_floors) {
                }
        }
        /** End the final inner box */
-       do_template("endbox");
+       do_template("endbox", NULL);
 
        wprintf("</td></tr></table>\n");
 }
@@ -3130,10 +3307,14 @@ void do_rooms_view(struct folder *fold, int max_folders, int num_floors) {
  * \brief print a floor div???
  * \param which_floordiv name of the floordiv???
  */
-void set_floordiv_expanded(char *which_floordiv) {
-       begin_ajax_response();
-       safestrncpy(WC->floordiv_expanded, which_floordiv, sizeof WC->floordiv_expanded);
-       end_ajax_response();
+void set_floordiv_expanded(void) {
+       wcsession *WCC = WC;
+       StrBuf *FloorDiv;
+       
+       FloorDiv = NewStrBuf();
+       StrBufAppendBuf(FloorDiv, WCC->UrlFragment2, 0);
+       set_preference("floordiv_expanded", FloorDiv, 1);
+       WCC->floordiv_expanded = FloorDiv;
 }
 
 /**
@@ -3150,7 +3331,6 @@ void do_iconbar_view(struct folder *fold, int max_folders, int num_floors) {
        char floordiv_id[32];
        int levels, oldlevels;
        int i, t;
-       int num_drop_targets = 0;
        char *icon = NULL;
 
        strcpy(floor_name, "");
@@ -3162,10 +3342,10 @@ void do_iconbar_view(struct folder *fold, int max_folders, int num_floors) {
 
                levels = num_tokens(fold[i].name, '|');
                extract_token(floor_name, fold[i].name, 0,
-                       '|', sizeof floor_name);
+                             '|', sizeof floor_name);
 
                if ( (strcasecmp(floor_name, old_floor_name))
-                  && (!IsEmptyStr(old_floor_name)) ) {
+                    && (!IsEmptyStr(old_floor_name)) ) {
                        /** End inner box */
                        wprintf("<br>\n");
                        wprintf("</div>\n");    /** floordiv */
@@ -3181,8 +3361,8 @@ void do_iconbar_view(struct folder *fold, int max_folders, int num_floors) {
                                "%s</span><br>\n", floordiv_id, floordivtitle);
                        wprintf("<div id=\"%s\" style=\"display:%s\">",
                                floordiv_id,
-                               (!strcasecmp(floordiv_id, WC->floordiv_expanded) ? "block" : "none")
-                       );
+                               (!strcasecmp(floordiv_id, ChrPtr(WC->floordiv_expanded)) ? "block" : "none")
+                               );
                }
 
                oldlevels = levels;
@@ -3219,7 +3399,7 @@ void do_iconbar_view(struct folder *fold, int max_folders, int num_floors) {
                                wprintf("<a href=\"dotgoto?room=");
                                urlescputs(fold[i].room);
                                wprintf("\">");
-                               wprintf("<img align=\"middle\" border=0 src=\"static/%s\" alt=\"\"> ", icon);
+                               wprintf("<img  border=0 src=\"static/%s\" alt=\"\"> ", icon);
                        }
                        else {
                                wprintf("<i>");
@@ -3249,34 +3429,29 @@ void do_iconbar_view(struct folder *fold, int max_folders, int num_floors) {
        wprintf("</div>\n");    /** floordiv */
 
 
-       /** BEGIN: The old invisible pixel trick, to get our JavaScript to initialize */
-       wprintf("<img src=\"static/blank.gif\" onLoad=\"\n");
+}
 
-       num_drop_targets = 0;
 
-       for (i=0; i<max_folders; ++i) {
-               levels = num_tokens(fold[i].name, '|');
-               if (levels > 1) {
-                       wprintf("drop_targets_elements[%d]=$('roomdiv%d');\n", num_drop_targets, i);
-                       wprintf("drop_targets_roomnames[%d]='", num_drop_targets);
-                       jsescputs(fold[i].room);
-                       wprintf("';\n");
-                       ++num_drop_targets;
-               }
-       }
 
-       wprintf("num_drop_targets = %d;\n", num_drop_targets);
-       if ((WC->floordiv_expanded[0] != '\0')&&
-           (WC->floordiv_expanded[1] != '\0')){
-               wprintf("which_div_expanded = '%s';\n", WC->floordiv_expanded);
-       }
+/**
+ * \brief Burn the cached folder list.  
+ * \param age How old the cahce needs to be before we burn it.
+ */
 
-       wprintf("\">\n");
-       /** END: The old invisible pixel trick, to get our JavaScript to initialize */
+void burn_folder_cache(time_t age)
+{
+       /** If our cached folder list is very old, burn it. */
+       if (WC->cache_fold != NULL) {
+               if ((time(NULL) - WC->cache_timestamp) > age) {
+                       free(WC->cache_fold);
+                       WC->cache_fold = NULL;
+               }
+       }
 }
 
 
 
+
 /**
  * \brief Show the room list.  
  * (only should get called by
@@ -3284,7 +3459,8 @@ void do_iconbar_view(struct folder *fold, int max_folders, int num_floors) {
  * \param viewpref the view preferences???
  */
 
-void list_all_rooms_by_floor(char *viewpref) {
+void list_all_rooms_by_floor(const char *viewpref) {
+       StrBuf *Buf;
        char buf[SIZ];
        int swap = 0;
        struct folder *fold = NULL;
@@ -3294,29 +3470,25 @@ void list_all_rooms_by_floor(char *viewpref) {
        int *floor_mapping;
        int IDMax;
        int i, j;
+       int ShowEmptyFloors;
        int ra_flags = 0;
        int flags = 0;
        int num_floors = 1;     /** add an extra one for private folders */
-       char buf2[SIZ];
        char buf3[SIZ];
        
        /** If our cached folder list is very old, burn it. */
-       if (WC->cache_fold != NULL) {
-               if ((time(NULL) - WC->cache_timestamp) > 300) {
-                       free(WC->cache_fold);
-                       WC->cache_fold = NULL;
-               }
-       }
-
+       burn_folder_cache(300);
+       
        /** Can we do the iconbar roomlist from cache? */
        if ((WC->cache_fold != NULL) && (!strcasecmp(viewpref, "iconbar"))) {
                do_iconbar_view(WC->cache_fold, WC->cache_max_folders, WC->cache_num_floors);
                return;
        }
+       Buf = NewStrBuf();
 
        /** Grab the floor table so we know how to build the list... */
-       load_floorlist();
-
+       load_floorlist(Buf);
+       FreeStrBuf(&Buf);
        /** Start with the mailboxes */
        max_folders = 1;
        alloc_folders = 1;
@@ -3329,18 +3501,18 @@ void list_all_rooms_by_floor(char *viewpref) {
        serv_puts("LFLR");
        serv_getln(buf, sizeof buf);
        if (buf[0]=='1') while(serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
-               if (max_folders >= alloc_folders) {
-                       alloc_folders = max_folders + 100;
-                       fold = realloc(fold,
-                               alloc_folders * sizeof(struct folder));
-               }
-               memset(&fold[max_folders], 0, sizeof(struct folder));
-               extract_token(fold[max_folders].name, buf, 1, '|', sizeof fold[max_folders].name);
-               extract_token(buf3, buf, 0, '|', SIZ);
-               fold[max_folders].floor = atol (buf3);
-               ++max_folders;
-               ++num_floors;
-       }
+                       if (max_folders >= alloc_folders) {
+                               alloc_folders = max_folders + 100;
+                               fold = realloc(fold,
+                                              alloc_folders * sizeof(struct folder));
+                       }
+                       memset(&fold[max_folders], 0, sizeof(struct folder));
+                       extract_token(fold[max_folders].name, buf, 1, '|', sizeof fold[max_folders].name);
+                       extract_token(buf3, buf, 0, '|', SIZ);
+                       fold[max_folders].floor = atol (buf3);
+                       ++max_folders;
+                       ++num_floors;
+               }
        IDMax = 0;
        for (i=0; i<num_floors; i++)
                if (IDMax < fold[i].floor)
@@ -3351,49 +3523,49 @@ void list_all_rooms_by_floor(char *viewpref) {
                floor_mapping[fold[i].floor]=i;
        
        /** refresh the messages index for this room */
-//     serv_puts("GOTO ");
-//     while (serv_getln(buf, sizeof buf), strcmp(buf, "000"));
+/* TODO        serv_puts("GOTO ");
+   while (serv_getln(buf, sizeof buf), strcmp(buf, "000")); */
        /** Now add rooms */
        serv_puts("LKRA");
        serv_getln(buf, sizeof buf);
        if (buf[0]=='1') while(serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
-               if (max_folders >= alloc_folders) {
-                       alloc_folders = max_folders + 100;
-                       fold = realloc(fold,
-                               alloc_folders * sizeof(struct folder));
-               }
-               memset(&fold[max_folders], 0, sizeof(struct folder));
-               extract_token(fold[max_folders].room, buf, 0, '|', sizeof fold[max_folders].room);
-               ra_flags = extract_int(buf, 5);
-               flags = extract_int(buf, 1);
-               fold[max_folders].floor = extract_int(buf, 2);
-               fold[max_folders].hasnewmsgs =
-                       ((ra_flags & UA_HASNEWMSGS) ? 1 : 0 );
-               if (flags & QR_MAILBOX) {
-                       fold[max_folders].is_mailbox = 1;
-               }
-               fold[max_folders].view = extract_int(buf, 6);
-               room_to_folder(fold[max_folders].name,
-                               fold[max_folders].room,
-                               fold[max_folders].floor,
-                               fold[max_folders].is_mailbox);
-               fold[max_folders].selectable = 1;
-               /* Increase the room count for the associtaed floor */
-               if (fold[max_folders].is_mailbox) {
-                       fold[0].num_rooms++;
-               }
-               else {
-                       i = floor_mapping[fold[max_folders].floor];
-                       fold[i].num_rooms++;
+                       if (max_folders >= alloc_folders) {
+                               alloc_folders = max_folders + 100;
+                               fold = realloc(fold,
+                                              alloc_folders * sizeof(struct folder));
+                       }
+                       memset(&fold[max_folders], 0, sizeof(struct folder));
+                       extract_token(fold[max_folders].room, buf, 0, '|', sizeof fold[max_folders].room);
+                       ra_flags = extract_int(buf, 5);
+                       flags = extract_int(buf, 1);
+                       fold[max_folders].floor = extract_int(buf, 2);
+                       fold[max_folders].hasnewmsgs =
+                               ((ra_flags & UA_HASNEWMSGS) ? 1 : 0 );
+                       if (flags & QR_MAILBOX) {
+                               fold[max_folders].is_mailbox = 1;
+                       }
+                       fold[max_folders].view = extract_int(buf, 6);
+                       room_to_folder(fold[max_folders].name,
+                                      fold[max_folders].room,
+                                      fold[max_folders].floor,
+                                      fold[max_folders].is_mailbox);
+                       fold[max_folders].selectable = 1;
+                       /* Increase the room count for the associtaed floor */
+                       if (fold[max_folders].is_mailbox) {
+                               fold[0].num_rooms++;
+                       }
+                       else {
+                               i = floor_mapping[fold[max_folders].floor];
+                               fold[i].num_rooms++;
+                       }
+                       ++max_folders;
                }
-               ++max_folders;
-       }
        
        /*
         * Remove any floors that don't have rooms
         */
-       get_preference("emptyfloors", buf2, sizeof buf2);
-       if (buf2[0]==0 || (strcasecmp(buf2, "no") == 0))
+       get_pref_yesno("emptyfloors", &ShowEmptyFloors, 0);
+       if (ShowEmptyFloors)
        {
                for (i=0; i<num_floors; i++)
                {
@@ -3416,7 +3588,7 @@ void list_all_rooms_by_floor(char *viewpref) {
                        }
                        else {
                                if ( (fold[j+1].is_mailbox)
-                                  && (!fold[j].is_mailbox)) {
+                                    && (!fold[j].is_mailbox)) {
                                        swap = 1;
                                }
                                else {
@@ -3426,9 +3598,9 @@ void list_all_rooms_by_floor(char *viewpref) {
                        if (swap > 0) {
                                memcpy(&ftmp, &fold[j], sizeof(struct folder));
                                memcpy(&fold[j], &fold[j+1],
-                                                       sizeof(struct folder));
+                                      sizeof(struct folder));
                                memcpy(&fold[j+1], &ftmp,
-                                                       sizeof(struct folder));
+                                      sizeof(struct folder));
                        }
                }
        }
@@ -3468,43 +3640,50 @@ void list_all_rooms_by_floor(char *viewpref) {
  */
 void knrooms(void)
 {
-       char listviewpref[SIZ];
+       StrBuf *ListView = NULL;
 
        output_headers(1, 1, 2, 0, 0, 0);
 
        /** Determine whether the user is trying to change views */
-       if (bstr("view") != NULL) {
-               if (!IsEmptyStr(bstr("view"))) {
-                       set_preference("roomlistview", bstr("view"), 1);
-               }
+       if (havebstr("view")) {
+               ListView = NewStrBufPlain(bstr("view"), -1);
+               set_preference("roomlistview", ListView, 1);
        }
-
-       get_preference("roomlistview", listviewpref, sizeof listviewpref);
-
-       if ( (strcasecmp(listviewpref, "folders"))
-          && (strcasecmp(listviewpref, "table")) ) {
-               strcpy(listviewpref, "rooms");
+       /** Sanitize the input so its safe */
+       if(!get_preference("roomlistview", &ListView) ||
+          ((strcasecmp(ChrPtr(ListView), "folders") != 0) &&
+           (strcasecmp(ChrPtr(ListView), "table") != 0))) 
+       {
+               if (ListView == NULL) {
+                       ListView = NewStrBufPlain("rooms", sizeof("rooms") - 1);
+                       set_preference("roomlistview", ListView, 0);
+               }
+               else {
+                       StrBufPrintf(ListView, "rooms");
+                       save_preferences();
+               }
        }
 
        /** title bar */
        wprintf("<div id=\"banner\">\n");
-       wprintf("<div class=\"room_banner\">");
+       wprintf("<div class=\"room_banner\" id=\"room_banner\">");
        wprintf("<h1>");
-       if (!strcasecmp(listviewpref, "rooms")) {
+       if (!strcasecmp(ChrPtr(ListView), "rooms")) {
                wprintf(_("Room list"));
        }
-       if (!strcasecmp(listviewpref, "folders")) {
+       else if (!strcasecmp(ChrPtr(ListView), "folders")) {
                wprintf(_("Folder list"));
        }
-       if (!strcasecmp(listviewpref, "table")) {
+       else if (!strcasecmp(ChrPtr(ListView), "table")) {
                wprintf(_("Room list"));
        }
        wprintf("</h1></div>\n");
-
+       
        /** offer the ability to switch views */
+       wprintf("<div id=\"actiondiv\">");
        wprintf("<ul class=\"room_actions\">\n");
        wprintf("<li class=\"start_page\">");
-       offer_start_page();
+       offer_start_page(NULL, &NoCtx);
        wprintf("</li>");
        wprintf("<li><form name=\"roomlistomatic\">\n"
                "<select name=\"newview\" size=\"1\" "
@@ -3514,23 +3693,23 @@ void knrooms(void)
        wprintf("<option %s value=\"knrooms&view=rooms\">"
                "View as room list"
                "</option>\n",
-               ( !strcasecmp(listviewpref, "rooms") ? "SELECTED" : "" )
-       );
+               ( !strcasecmp(ChrPtr(ListView), "rooms") ? "SELECTED" : "" )
+               );
 
        wprintf("<option %s value=\"knrooms&view=folders\">"
                "View as folder list"
                "</option>\n",
-               ( !strcasecmp(listviewpref, "folders") ? "SELECTED" : "" )
-       );
+               ( !strcasecmp(ChrPtr(ListView), "folders") ? "SELECTED" : "" )
+               );
 
        wprintf("</select>");
        wprintf("</form></li>");
-       wprintf("</ul></div>\n");
+       wprintf("</ul></div></div>\n");
 
        wprintf("<div id=\"content\" class=\"service\">\n");
 
        /** Display the room list in the user's preferred format */
-       list_all_rooms_by_floor(listviewpref);
+       list_all_rooms_by_floor(ChrPtr(ListView));
        wDumpContent(1);
 }
 
@@ -3542,20 +3721,20 @@ void knrooms(void)
 void set_room_policy(void) {
        char buf[SIZ];
 
-       if (IsEmptyStr(bstr("ok_button"))) {
+       if (!havebstr("ok_button")) {
                strcpy(WC->ImportantMessage,
-                       _("Cancelled.  Changes were not saved."));
+                      _("Cancelled.  Changes were not saved."));
                display_editroom();
                return;
        }
 
-       serv_printf("SPEX room|%d|%d", atoi(bstr("roompolicy")), atoi(bstr("roomvalue")));
+       serv_printf("SPEX room|%d|%d", ibstr("roompolicy"), ibstr("roomvalue"));
        serv_getln(buf, sizeof buf);
        strcpy(WC->ImportantMessage, &buf[4]);
 
        if (WC->axlevel >= 6) {
                strcat(WC->ImportantMessage, "<br />\n");
-               serv_printf("SPEX floor|%d|%d", atoi(bstr("floorpolicy")), atoi(bstr("floorvalue")));
+               serv_printf("SPEX floor|%d|%d", ibstr("floorpolicy"), ibstr("floorvalue"));
                serv_getln(buf, sizeof buf);
                strcat(WC->ImportantMessage, &buf[4]);
        }
@@ -3563,4 +3742,400 @@ void set_room_policy(void) {
        display_editroom();
 }
 
+HashList *GetFloorListHash(StrBuf *Target, WCTemplputParams *TP) {
+       /* todo: check context */
+       const char *Err;
+       StrBuf *Buf;
+       StrBuf *Buf2;
+       HashList *floors;
+       HashList *floor;
+       floors = NewHash(1, NULL);
+       Buf = NewStrBuf();
+       serv_puts("LFLR"); /* get floors */
+       StrBufTCP_read_line(Buf, &WC->serv_sock, 0, &Err); /* '100', we hope */
+       if (ChrPtr(Buf)[0] == '1') while(StrBufTCP_read_line(Buf, &WC->serv_sock, 0, &Err), strcmp(ChrPtr(Buf), "000")) {
+                       int a;
+                       const char *floorNum = NULL;
+                       floor = NewHash(1, NULL);
+                       for(a=0; a<FLOOR_PARAM_LEN; a++) {
+                               Buf2 = NewStrBuf();
+                               StrBufExtract_token(Buf2, Buf, a, '|');
+                               if (a==0) {
+                                       floorNum = ChrPtr(Buf2); /* hmm, should we copy Buf2 first? */
+                               }
+                               Put(floor, FPKEY(a), Buf2, NULL);
+                       }
+                       Put(floors, HKEY(floorNum), floor, NULL);
+               }
+       FreeStrBuf(&Buf);
+       return floors;
+}
+
+void tmplput_FLOOR_Value(StrBuf *TemplBuffer, WCTemplputParams *TP) 
+{
+       StrBuf *val;
+       HashList *floor = (HashList *)(TP->Context);
+       void *value;
+       GetHash(floor, TKEY(0), &value);
+       val = (StrBuf *)value;
+       StrECMAEscAppend(TemplBuffer, val, 0);
+}
+HashList *GetRoomListHashLKRA(StrBuf *Target, WCTemplputParams *TP) 
+{
+       serv_puts("LKRA");
+       return GetRoomListHash(Target, TP);
+}
+HashList *GetRoomListHash(StrBuf *Target, WCTemplputParams *TP) 
+{
+       /* TODO: Check context */
+       HashList *rooms;
+       HashList *room;
+       StrBuf *buf;
+       StrBuf *buf2;
+       const char *Err;
+       buf = NewStrBuf();
+       rooms = NewHash(1, NULL);
+       StrBufTCP_read_line(buf, &WC->serv_sock, 0, &Err);
+       if (ChrPtr(buf)[0] == '1') while(StrBufTCP_read_line(buf, &WC->serv_sock, 0, &Err), strcmp(ChrPtr(buf), "000")) {
+                       int i;
+                       const char *rmName = NULL;
+                       room = NewHash(1, NULL);
+                       for(i=0; i<ROOM_PARAM_LEN; i++) {
+                               buf2 = NewStrBuf();
+                               StrBufExtract_token(buf2, buf, i, '|');
+                               if (i==0) {
+                                       rmName = ChrPtr(buf2);
+                               }
+                               Put(room, RPKEY(i), buf2, NULL);
+                       }
+                       Put(rooms, rmName, strlen(rmName), room, NULL);
+               }
+       SortByHashKey(rooms, 1);
+       /*SortByPayload(rooms, SortRoomsByListOrder);  */
+       FreeStrBuf(&buf);
+       return rooms;
+}
+/** Unused function that orders rooms by the listorder flag */
+int SortRoomsByListOrder(const void *room1, const void *room2) 
+{
+       int l1;
+       int l2;
+       HashList *r1 = (HashList *)GetSearchPayload(room1);
+       HashList *r2 = (HashList *)GetSearchPayload(room2);
+       StrBuf *listOrderBuf1;
+       StrBuf *listOrderBuf2;
+  
+       GetHash(r1, RPKEY(3), (void *)&listOrderBuf1);
+       GetHash(r2, RPKEY(3), (void *)&listOrderBuf2);
+       l1 = atoi(ChrPtr(listOrderBuf1));
+       l2 = atoi(ChrPtr(listOrderBuf2));
+       if (l1 < l2) return -1;
+       else if (l1 > l2) return +1;
+       else return 0;
+}
+void tmplput_ROOM_Value(StrBuf *TemplBuffer, WCTemplputParams *TP) 
+{
+       void *value;
+       StrBuf *val;
+       HashList *room = (HashList *)(TP->Context);
+
+       GetHash(room, TKEY(0), &value);
+       val = (StrBuf *)value;
+       StrECMAEscAppend(TemplBuffer, val, 0);
+}
+void jsonRoomFlr(void) {
+       /* Send as our own (application/json) content type */
+  hprintf("HTTP/1.1 200 OK\r\n");
+  hprintf("Content-type: application/json; charset=utf-8\r\n");
+  hprintf("Server: %s / %s\r\n", PACKAGE_STRING, ChrPtr(WC->serv_info->serv_software));
+  hprintf("Connection: close\r\n");
+  hprintf("Pragma: no-cache\r\nCache-Control: no-store\r\nExpires:-1\r\n");
+  begin_burst();
+  DoTemplate(HKEY("json_roomflr"),NULL,&NoCtx);
+  end_burst(); 
+}
+void tmplput_RoomName(StrBuf *Target, WCTemplputParams *TP)
+{
+       StrBufAppendTemplate(Target, TP, WC->wc_roomname, 0);
+}
+
+void _gotonext(void) { slrp_highest(); gotonext(); }
+void dotskip(void) {smart_goto(sbstr("room"));}
+void _display_private(void) { display_private("", 0); }
+void dotgoto(void) {
+       if (WC->wc_view != VIEW_MAILBOX) {      /* dotgoto acts like dotskip when we're in a mailbox view */
+               slrp_highest();
+       }
+       smart_goto(sbstr("room"));
+}
+
+void tmplput_roombanner(StrBuf *Target, WCTemplputParams *TP)
+{
+       wprintf("<div id=\"banner\">\n");
+       embed_room_banner(NULL, navbar_default);
+       wprintf("</div>\n");
+}
+
+
+void tmplput_ungoto(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+
+       if ((WCC!=NULL) && 
+           (!IsEmptyStr(WCC->ugname)))
+               StrBufAppendBufPlain(Target, WCC->ugname, -1, 0);
+}
+
+
+int ConditionalHaveUngoto(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) && 
+               (!IsEmptyStr(WCC->ugname)) && 
+               (strcasecmp(WCC->ugname, ChrPtr(WCC->wc_roomname)) == 0));
+}
+
+int ConditionalRoomHas_QR_PERMANENT(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_PERMANENT) != 0));
+}
+
+int ConditionalRoomHas_QR_INUSE(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_INUSE) != 0));
+}
+
+int ConditionalRoomHas_QR_PRIVATE(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_PRIVATE) != 0));
+}
+
+int ConditionalRoomHas_QR_PASSWORDED(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_PASSWORDED) != 0));
+}
+
+int ConditionalRoomHas_QR_GUESSNAME(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_GUESSNAME) != 0));
+}
+
+int ConditionalRoomHas_QR_DIRECTORY(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_DIRECTORY) != 0));
+}
+
+int ConditionalRoomHas_QR_UPLOAD(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_UPLOAD) != 0));
+}
+
+int ConditionalRoomHas_QR_DOWNLOAD(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_DOWNLOAD) != 0));
+}
+
+int ConditionalRoomHas_QR_VISDIR(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_VISDIR) != 0));
+}
+
+int ConditionalRoomHas_QR_ANONONLY(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_ANONONLY) != 0));
+}
+
+int ConditionalRoomHas_QR_ANONOPT(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_ANONOPT) != 0));
+}
+
+int ConditionalRoomHas_QR_NETWORK(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_NETWORK) != 0));
+}
+
+int ConditionalRoomHas_QR_PREFONLY(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_PREFONLY) != 0));
+}
+
+int ConditionalRoomHas_QR_READONLY(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_READONLY) != 0));
+}
+
+int ConditionalRoomHas_QR_MAILBOX(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+       
+       return ((WCC!=NULL) &&
+               ((WCC->room_flags & QR_MAILBOX) != 0));
+}
+
+
+int ConditionalHaveRoomeditRights(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+
+       return ( (WCC!= NULL) && 
+                ((WCC->axlevel >= 6) || 
+                 (WCC->is_room_aide) || 
+                 (WCC->is_mailbox) ));
+}
+
+int ConditionalIsRoomtype(StrBuf *Target, WCTemplputParams *TP)
+{
+       wcsession *WCC = WC;
+
+       if ((WCC == NULL) ||
+           (TP->Tokens->nParameters < 3) ||
+           (TP->Tokens->Params[2]->Type != TYPE_STR)||
+           (TP->Tokens->Params[2]->len < 7))
+               return 0;
+
+       switch(WCC->wc_view) {
+       case VIEW_BBS:
+               return (!strcasecmp(TP->Tokens->Params[2]->Start, "VIEW_BBS"));
+       case VIEW_MAILBOX:
+               return (!strcasecmp(TP->Tokens->Params[2]->Start, "VIEW_MAILBOX"));
+       case VIEW_ADDRESSBOOK:
+               return (!strcasecmp(TP->Tokens->Params[2]->Start, "VIEW_ADDRESSBOOK"));
+       case VIEW_TASKS:
+               return (!strcasecmp(TP->Tokens->Params[2]->Start, "VIEW_TASKS"));
+       case VIEW_NOTES:
+               return (!strcasecmp(TP->Tokens->Params[2]->Start, "VIEW_NOTES"));
+       case VIEW_WIKI:
+               return (!strcasecmp(TP->Tokens->Params[2]->Start, "VIEW_WIKI"));
+       case VIEW_JOURNAL:
+               return (!strcasecmp(TP->Tokens->Params[2]->Start, "VIEW_JOURNAL"));
+       case VIEW_CALENDAR:
+               return (!strcasecmp(TP->Tokens->Params[2]->Start, "VIEW_CALENDAR"));
+       case VIEW_CALBRIEF:
+               return (!strcasecmp(TP->Tokens->Params[2]->Start, "VIEW_CALBRIEF"));
+       default:
+               return 0;
+       }
+}
+
+void 
+InitModule_ROOMOPS
+(void)
+{
+       RegisterPreference("roomlistview",
+                           _("Room list view"),
+                           PRF_STRING,
+                           NULL);
+        RegisterPreference("emptyfloors", _("Show empty floors"), PRF_YESNO, NULL);
+
+       RegisterNamespace("ROOMNAME", 0, 1, tmplput_RoomName, 0);
+
+       WebcitAddUrlHandler(HKEY("knrooms"), knrooms, 0);
+       WebcitAddUrlHandler(HKEY("gotonext"), _gotonext, NEED_URL);
+       WebcitAddUrlHandler(HKEY("skip"), gotonext, NEED_URL);
+       WebcitAddUrlHandler(HKEY("ungoto"), ungoto, NEED_URL);
+       WebcitAddUrlHandler(HKEY("dotgoto"), dotgoto, NEED_URL);
+       WebcitAddUrlHandler(HKEY("dotskip"), dotskip, NEED_URL);
+       WebcitAddUrlHandler(HKEY("display_private"), _display_private, 0);
+       WebcitAddUrlHandler(HKEY("goto_private"), goto_private, NEED_URL);
+       WebcitAddUrlHandler(HKEY("zapped_list"), zapped_list, 0);
+       WebcitAddUrlHandler(HKEY("display_zap"), display_zap, 0);
+       WebcitAddUrlHandler(HKEY("zap"), zap, 0);
+       WebcitAddUrlHandler(HKEY("display_entroom"), display_entroom, 0);
+       WebcitAddUrlHandler(HKEY("entroom"), entroom, 0);
+       WebcitAddUrlHandler(HKEY("display_whok"), display_whok, 0);
+       WebcitAddUrlHandler(HKEY("do_invt_kick"), do_invt_kick, 0);
+       WebcitAddUrlHandler(HKEY("display_editroom"), display_editroom, 0);
+       WebcitAddUrlHandler(HKEY("netedit"), netedit, 0);
+       WebcitAddUrlHandler(HKEY("editroom"), editroom, 0);
+       WebcitAddUrlHandler(HKEY("delete_room"), delete_room, 0);
+       WebcitAddUrlHandler(HKEY("set_room_policy"), set_room_policy, 0);
+       WebcitAddUrlHandler(HKEY("set_floordiv_expanded"), set_floordiv_expanded, NEED_URL|AJAX);
+       WebcitAddUrlHandler(HKEY("changeview"), change_view, 0);
+       WebcitAddUrlHandler(HKEY("toggle_self_service"), toggle_self_service, 0);
+       WebcitAddUrlHandler(HKEY("json_roomflr"), jsonRoomFlr, 0);
+       RegisterNamespace("ROOMBANNER", 0, 1, tmplput_roombanner, 0);
+
+       RegisterConditional(HKEY("COND:ROOM:TYPE_IS"), 0, ConditionalIsRoomtype, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_PERMANENT"), 0, ConditionalRoomHas_QR_PERMANENT, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_INUSE"), 0, ConditionalRoomHas_QR_INUSE, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_PRIVATE"), 0, ConditionalRoomHas_QR_PRIVATE, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_PASSWORDED"), 0, ConditionalRoomHas_QR_PASSWORDED, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_GUESSNAME"), 0, ConditionalRoomHas_QR_GUESSNAME, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_DIRECTORY"), 0, ConditionalRoomHas_QR_DIRECTORY, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_UPLOAD"), 0, ConditionalRoomHas_QR_UPLOAD, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_DOWNLOAD"), 0, ConditionalRoomHas_QR_DOWNLOAD, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_VISIDIR"), 0, ConditionalRoomHas_QR_VISDIR, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_ANONONLY"), 0, ConditionalRoomHas_QR_ANONONLY, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_ANONOPT"), 0, ConditionalRoomHas_QR_ANONOPT, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_NETWORK"), 0, ConditionalRoomHas_QR_NETWORK, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_PREFONLY"), 0, ConditionalRoomHas_QR_PREFONLY, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_READONLY"), 0, ConditionalRoomHas_QR_READONLY, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:FLAGS:QR_MAILBOX"), 0, ConditionalRoomHas_QR_MAILBOX, CTX_NONE);
+
+       RegisterConditional(HKEY("COND:UNGOTO"), 0, ConditionalHaveUngoto, CTX_NONE);
+       RegisterConditional(HKEY("COND:ROOM:EDITACCESS"), 0, ConditionalHaveRoomeditRights, CTX_NONE);
+
+       RegisterNamespace("ROOM:UNGOTO", 0, 0, tmplput_ungoto, 0);
+       RegisterIterator("FLOORS", 0, NULL, GetFloorListHash, NULL, DeleteHash, CTX_FLOORS, CTX_NONE, IT_NOFLAG);
+       RegisterNamespace("FLOOR:INFO", 1, 2, tmplput_FLOOR_Value, CTX_FLOORS);
+       RegisterIterator("LKRA", 0, NULL, GetRoomListHashLKRA, NULL, NULL, CTX_ROOMS, CTX_NONE, IT_NOFLAG);
+       RegisterNamespace("ROOM:INFO", 1, 2, tmplput_ROOM_Value, CTX_ROOMS);
+}
+
+
+
+void 
+SessionDestroyModule_ROOMOPS
+(wcsession *sess)
+{
+       if (sess->cache_fold != NULL) {
+               free(sess->cache_fold);
+       }
+       
+       free_march_list(sess);
+}
 /*@}*/