* fixes
[citadel.git] / webcit / roomops.c
index fcbbf368aefc84748e11511009ca1532147011a8..7014040f17c02edf88e349619b798bae1fc4f82a 100644 (file)
@@ -10,6 +10,8 @@ char floorlist[MAX_FLOORS][SIZ]; /**< list of our floor names */
 
 char *viewdefs[9]; /**< the different kinds of available views */
 
+void display_whok(void);
+
 /*
  * Initialize the viewdefs with localized strings
  */
@@ -37,10 +39,16 @@ int is_view_allowed_as_default(int which_view)
                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 */
+
+#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 */
+               default:                return(0);      /* should never get here */
        }
 }
 
@@ -250,14 +258,14 @@ void zapped_list(void)
 {
        output_headers(1, 1, 1, 0, 0, 0);
 
-       svprintf("BOXTITLE", WCS_STRING, _("Zapped (forgotten) rooms"));
-       do_template("beginbox");
+       svput("BOXTITLE", WCS_STRING, _("Zapped (forgotten) rooms"));
+       do_template("beginbox", NULL);
 
        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);
 }
 
@@ -327,37 +335,37 @@ void embed_room_graphic(void) {
                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"
                        "\">"
                );
@@ -374,7 +382,7 @@ void embed_view_o_matic(void) {
        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> "
@@ -404,7 +412,7 @@ void embed_view_o_matic(void) {
                        wprintf("</option>\n");
                }
        }
-       wprintf("</select></form>\n");
+       wprintf("</select></div></form>\n");
 }
 
 
@@ -413,14 +421,15 @@ void embed_view_o_matic(void) {
  */
 void embed_search_o_matic(void) {
        wprintf("<form name=\"searchomatic\" action=\"do_search\">\n");
-       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+       wprintf("<div style=\"display: inline;\"><input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
        wprintf("<label for=\"search_name\">");
        wprintf(_("Search: "));
-       wprintf("</label> <input "
-               "type=\"text\" name=\"query\" size=\"15\" maxlength=\"128\" "
+       wprintf("</label><input ");
+       wprintf("%s", serv_info.serv_fulltext_enabled ? "" : "disabled ");
+       wprintf("type=\"text\" name=\"query\" size=\"15\" maxlength=\"128\" "
                "id=\"search_name\" class=\"inputbox\">\n"
        );
-       wprintf("</form>\n");
+       wprintf("</div></form>\n");
 }
 
 
@@ -435,7 +444,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;
        
@@ -482,10 +490,8 @@ void embed_room_banner(char *got, int navbar_style) {
        }
        else
                strcpy (with_files, "");
-               
-       stresc(sanitized_roomname, 256, WC->wc_roomname, 1, 1);
-       svprintf("ROOMNAME", WCS_STRING, "%s", sanitized_roomname);
-       svprintf("NUMMSGS", WCS_STRING,
+       
+       svprintf(HKEY("NUMMSGS"), WCS_STRING,
                _("%d new of %d messages%s"),
                extract_int(&got[4], 1),
                extract_int(&got[4], 2),
@@ -493,19 +499,20 @@ void embed_room_banner(char *got, int navbar_style) {
        );
        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")
                );
@@ -514,7 +521,7 @@ void embed_room_banner(char *got, int navbar_style) {
                        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")
                        );
@@ -526,8 +533,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"viewcontacts\">"
                                                "<a href=\"readfwd\">"
-                                               "<img align=\"middle\" src=\"static/viewcontacts_24x.gif\" "
-                                               "border=\"0\">"
+                                               "<img src=\"static/viewcontacts_24x.gif\" "
+                                               "alt=\"\">"
                                                "<span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("View contacts")
@@ -537,8 +544,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"staskday\">"
                                                "<a href=\"readfwd?calview=day\">"
-                                               "<img align=\"middle\" src=\"static/taskday2_24x.gif\" "
-                                               "border=\"0\">"
+                                               "<img src=\"static/taskday2_24x.gif\" "
+                                               "alt=\"\">"
                                                "<span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("Day view")
@@ -546,8 +553,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"monthview\">"
                                                "<a href=\"readfwd?calview=month\">"
-                                               "<img align=\"middle\" src=\"static/monthview2_24x.gif\" "
-                                               "border=\"0\">"
+                                               "<img src=\"static/monthview2_24x.gif\" "
+                                               "alt=\"\">"
                                                "<span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("Month view")
@@ -557,8 +564,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"monthview\">"
                                                "<a href=\"readfwd?calview=month\">"
-                                               "<img align=\"middle\" src=\"static/monthview2_24x.gif\" "
-                                               "border=\"0\">"
+                                               "<img src=\"static/monthview2_24x.gif\" "
+                                               "alt=\"\">"
                                                "<span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("Calendar list")
@@ -568,8 +575,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"taskmanag\">"
                                                "<a href=\"readfwd\">"
-                                               "<img align=\"middle\" src=\"static/taskmanag_24x.gif\" "
-                                               "border=\"0\">"
+                                               "<img src=\"static/taskmanag_24x.gif\" "
+                                               "alt=\"\">"
                                                "<span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("View tasks")
@@ -579,8 +586,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"viewnotes\">"
                                                "<a href=\"readfwd\">"
-                                               "<img align=\"middle\" src=\"static/viewnotes_24x.gif\" "
-                                               "border=\"0\">"
+                                               "<img src=\"static/viewnotes_24x.gif\" "
+                                               "alt=\"\">"
                                                "<span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("View notes")
@@ -590,8 +597,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"readallmess\">"
                                                "<a href=\"readfwd\">"
-                                               "<img align=\"middle\" src=\"static/readallmess3_24x.gif\" "
-                                               "border=\"0\">"
+                                               "<img src=\"static/readallmess3_24x.gif\" "
+                                               "alt=\"\">"
                                                "<span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("View message list")
@@ -601,8 +608,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"readallmess\">"
                                                "<a href=\"readfwd\">"
-                                               "<img align=\"middle\" src=\"static/readallmess3_24x.gif\" "
-                                               "border=\"0\">"
+                                               "<img src=\"static/readallmess3_24x.gif\" "
+                                               "alt=\"\">"
                                                "<span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("Wiki home")
@@ -612,8 +619,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"readallmess\">"
                                                "<a href=\"readfwd\">"
-                                               "<img align=\"middle\" src=\"static/readallmess3_24x.gif\" "
-                                               "border=\"0\">"
+                                               "<img src=\"static/readallmess3_24x.gif\" "
+                                               "alt=\"\">"
                                                "<span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("Read all messages")
@@ -628,8 +635,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"addnewcontact\">"
                                                "<a href=\"display_enter\">"
-                                               "<img align=\"middle\" src=\"static/addnewcontact_24x.gif\" "
-                                               "border=\"0\"><span class=\"navbar_link\">"
+                                               "<img src=\"static/addnewcontact_24x.gif\" "
+                                               "alt=\"\"><span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("Add new contact")
                                        );
@@ -637,12 +644,12 @@ void embed_room_banner(char *got, int navbar_style) {
                                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"));
+                                       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 align=\"middle\" src=\"static/addevent_24x.gif\" "
-                                               "border=\"0\"><span class=\"navbar_link\">"
+                                               "<img  src=\"static/addevent_24x.gif\" "
+                                               "alt=\"\"><span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("Add new event")
                                        );
@@ -651,8 +658,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"newmess\">"
                                                "<a href=\"display_enter\">"
-                                               "<img align=\"middle\" src=\"static/newmess3_24x.gif\" "
-                                               "border=\"0\"><span class=\"navbar_link\">"
+                                               "<img  src=\"static/newmess3_24x.gif\" "
+                                               "alt=\"\"><span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("Add new task")
                                        );
@@ -660,9 +667,9 @@ void embed_room_banner(char *got, int navbar_style) {
                                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\">"
+                                               "<a href=\"add_new_note\">"
+                                               "<img  src=\"static/enternewnote_24x.gif\" "
+                                               "alt=\"\"><span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("Add new note")
                                        );
@@ -673,8 +680,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"newmess\">"
                                                "<a href=\"display_enter?wikipage=%s\">"
-                                               "<img align=\"middle\" src=\"static/newmess3_24x.gif\" "
-                                               "border=\"0\"><span class=\"navbar_link\">"
+                                               "<img  src=\"static/newmess3_24x.gif\" "
+                                               "alt=\"\"><span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", buf, _("Edit this page")
                                        );
@@ -683,8 +690,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"newmess\">"
                                                "<a href=\"display_enter\">"
-                                               "<img align=\"middle\" src=\"static/newmess3_24x.gif\" "
-                                               "border=\"0\"><span class=\"navbar_link\">"
+                                               "<img  src=\"static/newmess3_24x.gif\" "
+                                               "alt=\"\"><span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("Write mail")
                                        );
@@ -693,8 +700,8 @@ void embed_room_banner(char *got, int navbar_style) {
                                        wprintf(
                                                "<li class=\"newmess\">"
                                                "<a href=\"display_enter\">"
-                                               "<img align=\"middle\" src=\"static/newmess3_24x.gif\" "
-                                               "border=\"0\"><span class=\"navbar_link\">"
+                                               "<img  src=\"static/newmess3_24x.gif\" "
+                                               "alt=\"\"><span class=\"navbar_link\">"
                                                "%s"
                                                "</span></a></li>\n", _("Enter a message")
                                        );
@@ -706,7 +713,7 @@ 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"),
@@ -717,7 +724,7 @@ void embed_room_banner(char *got, int navbar_style) {
                        "<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"),
@@ -730,10 +737,8 @@ 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)
 {
@@ -741,7 +746,7 @@ int gotoroom(char *gname)
        static long ls = (-1L);
        int err = 0;
 
-       /** store ungoto information */
+       /* store ungoto information */
        strcpy(WC->ugname, WC->wc_roomname);
        WC->uglsn = ls;
 
@@ -827,8 +832,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.
@@ -838,12 +844,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];
        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.
         */
@@ -876,7 +883,7 @@ void gotonext(void)
                                }
                                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
                 */
@@ -893,7 +900,7 @@ 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
                 */
@@ -910,9 +917,8 @@ void gotonext(void)
 }
 
 
-/**
- * \brief goto next room
- * \param next_room next room to go to
+/*
+ * goto next room
  */
 void smart_goto(char *next_room) {
        gotoroom(next_room);
@@ -921,8 +927,8 @@ void smart_goto(char *next_room) {
 
 
 
-/**
- * \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)
 {
@@ -933,8 +939,8 @@ void slrp_highest(void)
 }
 
 
-/**
- * \brief un-goto the previous room
+/*
+ * un-goto the previous room
  */
 void ungoto(void)
 {
@@ -968,16 +974,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) {
@@ -1030,29 +1036,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) 
@@ -1096,8 +1099,8 @@ 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)
 {
@@ -1133,10 +1136,15 @@ void display_editroom(void)
 
        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"));
@@ -1146,11 +1154,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"));
@@ -1160,9 +1169,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"));
@@ -1172,9 +1182,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"));
@@ -1184,9 +1195,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"));
@@ -1196,9 +1208,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"));
@@ -1208,11 +1221,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"));
@@ -1222,12 +1236,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 */ 
+
+       wprintf("<script type=\"text/javascript\">"
+               " Nifty(\"table#AdminTabs td\", \"small transparent top\");"
+               "</script>"
+       );
 
-       /** begin content of whatever tab is open now */
+       /* begin content of whatever tab is open now */
 
        if (!strcmp(tab, "admin")) {
                wprintf("<div class=\"tabcontent\">");
@@ -1270,7 +1291,7 @@ 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: "));
@@ -1484,7 +1505,7 @@ void display_editroom(void)
        }
 
 
-       /** Sharing the room with other Citadel nodes... */
+       /* Sharing the room with other Citadel nodes... */
        if (!strcmp(tab, "sharing")) {
                wprintf("<div class=\"tabcontent\">");
 
@@ -1531,7 +1552,7 @@ void display_editroom(void)
                        }
                }
 
-               /** Display the stuff */
+               /* Display the stuff */
                wprintf("<CENTER><br />"
                        "<table border=1 cellpadding=5><tr>"
                        "<td><B><I>");
@@ -1556,7 +1577,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>");
@@ -1597,7 +1618,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>"
@@ -1639,7 +1660,7 @@ void display_editroom(void)
                wprintf("</div>");
        }
 
-       /** Mailing list management */
+       /* Mailing list management */
        if (!strcmp(tab, "listserv")) {
                room_states RoomFlags;
                wprintf("<div class=\"tabcontent\">");
@@ -1671,7 +1692,7 @@ void display_editroom(void)
                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");
@@ -1702,7 +1723,7 @@ void display_editroom(void)
                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");
@@ -1721,7 +1742,7 @@ void display_editroom(void)
                        _("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");
 
@@ -1760,7 +1781,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\">");
 
@@ -1786,7 +1807,7 @@ 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"));
@@ -1857,14 +1878,14 @@ void display_editroom(void)
                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\">");
 
@@ -1881,6 +1902,8 @@ 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");
@@ -1910,6 +1933,8 @@ void display_editroom(void)
 
                                wprintf("<td>%s</td>", extract_int(buf, 4) ? _("Yes") : _("No"));
 
+                               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);
@@ -1925,7 +1950,7 @@ void display_editroom(void)
                        "<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>");
@@ -1939,6 +1964,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");
@@ -1989,7 +2017,7 @@ void display_editroom(void)
                        "<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");
@@ -2003,7 +2031,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();
@@ -2011,25 +2039,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);
        
@@ -2038,8 +2073,8 @@ void toggle_self_service(void) {
 
 
 
-/**
- * \brief save new parameters for a room
+/*
+ * save new parameters for a room
  */
 void editroom(void)
 {
@@ -2056,7 +2091,7 @@ void editroom(void)
        int bump;
 
 
-       if (IsEmptyStr(bstr("ok_button"))) {
+       if (!havebstr("ok_button")) {
                strcpy(WC->ImportantMessage,
                        _("Cancelled.  Changes were not saved."));
                display_editroom();
@@ -2122,62 +2157,62 @@ void editroom(void)
                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;
@@ -2195,7 +2230,7 @@ void editroom(void)
        if (!strcmp(bstr("bump"), "yes"))
                bump = 1;
 
-       er_floor = atoi(bstr("er_floor"));
+       er_floor = ibstr("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,
@@ -2226,8 +2261,8 @@ void editroom(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 do_invt_kick(void) {
         char buf[SIZ], room[SIZ], username[SIZ];
@@ -2243,7 +2278,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);
@@ -2257,7 +2292,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);
@@ -2276,8 +2311,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)
 {
@@ -2300,7 +2335,7 @@ void display_whok(void)
        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");
@@ -2325,14 +2360,14 @@ void display_whok(void)
 
         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 **/
+               /* Pop open an address book -- begin **/
                wprintf(
                        "<a href=\"javascript:PopOpenAddressBook('username_id|%s');\" "
                        "title=\"%s\">"
@@ -2341,7 +2376,7 @@ void display_whok(void)
                        _("User"), 
                        _("Users"), _("Users")
                );
-               /** Pop open an address book -- end **/
+               /* Pop open an address book -- end **/
 
        wprintf("</td></tr></table>\n");
        address_book_popup();
@@ -2350,8 +2385,8 @@ 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)
 {
@@ -2369,11 +2404,11 @@ void display_entroom(void)
 
        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\"> ");
 
@@ -2398,7 +2433,7 @@ 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,
@@ -2492,7 +2527,7 @@ void display_entroom(void)
                fmout("LEFT");
        }
 
-       do_template("endbox");
+       do_template("endbox", NULL);
 
        wDumpContent(1);
 }
@@ -2500,8 +2535,8 @@ void display_entroom(void)
 
 
 
-/**
- * \brief support function for entroom() -- sets the default view 
+/*
+ * support function for entroom() -- sets the default view 
  */
 void er_set_default_view(int newview) {
 
@@ -2536,8 +2571,8 @@ void er_set_default_view(int newview) {
 
 
 
-/**
- * \brief enter a new room
+/*
+ * Create a new room
  */
 void entroom(void)
 {
@@ -2549,7 +2584,7 @@ void entroom(void)
        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."));
                display_main_menu();
@@ -2558,8 +2593,8 @@ void entroom(void)
        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_floor = ibstr("er_floor");
+       er_view = ibstr("er_view");
 
        er_num_type = 0;
        if (!strcmp(er_type, "hidden"))
@@ -2596,8 +2631,8 @@ void display_private(char *rname, int req_pass)
 {
        output_headers(1, 1, 1, 0, 0, 0);
 
-       svprintf("BOXTITLE", WCS_STRING, _("Go to a hidden room"));
-       do_template("beginbox");
+       svprintf(HKEY("BOXTITLE"), WCS_STRING, _("Go to a hidden room"));
+       do_template("beginbox", NULL);
 
        wprintf("<p>");
        wprintf(_("If you know the name of a hidden (guess-name) or "
@@ -2608,7 +2643,7 @@ void display_private(char *rname, int req_pass)
        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>");
@@ -2634,7 +2669,7 @@ void display_private(char *rname, int req_pass)
        );
        wprintf("</div></form>\n");
 
-       do_template("endbox");
+       do_template("endbox", NULL);
 
        wDumpContent(1);
 }
@@ -2647,7 +2682,7 @@ void goto_private(void)
        char hold_rm[SIZ];
        char buf[SIZ];
 
-       if (IsEmptyStr(bstr("ok_button"))) {
+       if (!havebstr("ok_button")) {
                display_main_menu();
                return;
        }
@@ -2694,7 +2729,7 @@ void display_zap(void)
                "to do?<br />\n"), 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"));
@@ -2717,7 +2752,7 @@ void zap(void)
         */
        strcpy(final_destination, WC->wc_roomname);
 
-       if (!IsEmptyStr(bstr("ok_button"))) {
+       if (havebstr("ok_button")) {
                serv_printf("GOTO %s", WC->wc_roomname);
                serv_getln(buf, sizeof buf);
                if (buf[0] == '2') {
@@ -2767,8 +2802,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, "|");
@@ -2776,10 +2811,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"));
@@ -2830,7 +2867,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 */
@@ -2920,7 +2957,7 @@ void do_change_view(int newview) {
 void change_view(void) {
        int view;
 
-       view = atol(bstr("view"));
+       view = lbstr("view");
        do_change_view(view);
 }
 
@@ -3086,7 +3123,7 @@ void do_rooms_view(struct folder *fold, int max_folders, int num_floors) {
                if ( (strcasecmp(floor_name, old_floor_name))
                   && (!IsEmptyStr(old_floor_name)) ) {
                        /* End inner box */
-                       do_template("endbox");
+                       do_template("endbox", NULL);
                        wprintf("<br>");
 
                        ++num_boxes;
@@ -3102,8 +3139,8 @@ void do_rooms_view(struct folder *fold, int max_folders, int num_floors) {
                if (levels == 1) {
                        /** Begin inner box */
                        stresc(boxtitle, 256, floor_name, 1, 0);
-                       svprintf("BOXTITLE", WCS_STRING, boxtitle);
-                       do_template("beginbox");
+                       svprintf(HKEY("BOXTITLE"), WCS_STRING, boxtitle);
+                       do_template("beginbox", NULL);
                }
 
                oldlevels = levels;
@@ -3141,7 +3178,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");
 }
@@ -3150,10 +3187,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) {
+       struct wcsession *WCC = WC;
+       
+       if (WCC->floordiv_expanded == NULL)
+               WCC->floordiv_expanded = NewStrBuf();
+       else
+               FlushStrBuf(WC->floordiv_expanded);
+       StrBufAppendBuf(WC->floordiv_expanded, WC->UrlFragment1, 0);
 }
 
 /**
@@ -3201,7 +3242,7 @@ 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")
                        );
                }
 
@@ -3239,7 +3280,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>");
@@ -3286,9 +3327,9 @@ void do_iconbar_view(struct folder *fold, int max_folders, int num_floors) {
        }
 
        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);
+       if ((ChrPtr(WC->floordiv_expanded)[0] != '\0')&&
+           (ChrPtr(WC->floordiv_expanded)[1] != '\0')){
+               wprintf("which_div_expanded = '%s';\n", ChrPtr(WC->floordiv_expanded));
        }
 
        wprintf("\">\n");
@@ -3323,7 +3364,7 @@ void burn_folder_cache(time_t age)
  * \param viewpref the view preferences???
  */
 
-void list_all_rooms_by_floor(char *viewpref) {
+void list_all_rooms_by_floor(const char *viewpref) {
        char buf[SIZ];
        int swap = 0;
        struct folder *fold = NULL;
@@ -3333,10 +3374,10 @@ 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. */
@@ -3426,8 +3467,8 @@ void list_all_rooms_by_floor(char *viewpref) {
        /*
         * 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++)
                {
@@ -3502,35 +3543,41 @@ 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);
+       }
+       /** 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();
                }
-       }
-
-       get_preference("roomlistview", listviewpref, sizeof listviewpref);
-
-       if ( (strcasecmp(listviewpref, "folders"))
-          && (strcasecmp(listviewpref, "table")) ) {
-               strcpy(listviewpref, "rooms");
        }
 
        /** title bar */
        wprintf("<div id=\"banner\">\n");
        wprintf("<div class=\"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");
@@ -3548,13 +3595,13 @@ 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>");
@@ -3564,7 +3611,7 @@ void knrooms(void)
        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);
 }
 
@@ -3576,20 +3623,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."));
                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]);
        }
@@ -3597,4 +3644,51 @@ void set_room_policy(void) {
        display_editroom();
 }
 
+
+void tmplput_RoomName(StrBuf *Target, int nArgs, WCTemplateToken *Tokens, void *Context)
+{
+       StrEscAppend(Target, NULL, WC->wc_roomname, 1, 1);
+}
+
+void _gotonext(void) { slrp_highest(); gotonext(); }
+void dotskip(void) {smart_goto(bstr("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(bstr("room"));
+}
+
+
+void 
+InitModule_ROOMOPS
+(void)
+{
+       RegisterNamespace("ROOMNAME", 0, 0, tmplput_RoomName);
+
+       WebcitAddUrlHandler(HKEY("knrooms"), knrooms, 0);
+       WebcitAddUrlHandler(HKEY("gotonext"), _gotonext, 0);
+       WebcitAddUrlHandler(HKEY("skip"), gotonext, 0);
+       WebcitAddUrlHandler(HKEY("ungoto"), ungoto, 0);
+       WebcitAddUrlHandler(HKEY("dotgoto"), dotgoto, 0);
+       WebcitAddUrlHandler(HKEY("dotskip"), dotskip, 0);
+       WebcitAddUrlHandler(HKEY("display_private"), _display_private, 0);
+       WebcitAddUrlHandler(HKEY("goto_private"), goto_private, 0);
+       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);
+}
+
 /*@}*/