]> code.citadel.org Git - citadel.git/blobdiff - webcit/messages.c
* Enhance the older/newer logic. It's still not quite right.
[citadel.git] / webcit / messages.c
index 96b35a69a06856086cd68b646399c8be9ed1b58b..b793f308c291554efe48c8ab65c66d9f485c751e 100644 (file)
 HashList *MsgHeaderHandler = NULL;
 HashList *MsgEvaluators = NULL;
 HashList *MimeRenderHandler = NULL;
+HashList *ReadLoopHandler = NULL;
 int dbg_analyze_msg = 0;
 
-#define SUBJ_COL_WIDTH_PCT             50      /**< Mailbox view column width */
-#define SENDER_COL_WIDTH_PCT           30      /**< Mailbox view column width */
-#define DATE_PLUS_BUTTONS_WIDTH_PCT    20      /**< Mailbox view column width */
+#define SUBJ_COL_WIDTH_PCT             50      /* Mailbox view column width */
+#define SENDER_COL_WIDTH_PCT           30      /* Mailbox view column width */
+#define DATE_PLUS_BUTTONS_WIDTH_PCT    20      /* Mailbox view column width */
 
 void jsonMessageListHdr(void);
 
@@ -60,9 +61,9 @@ int load_message(message_summary *Msg,
                return 0;
        }
 
-       /** begin everythingamundo table */
+       /* begin everythingamundo table */
        HdrToken = NewStrBuf();
-       while ((StrBuf_ServGetln(Buf)>=0) && !Done) {
+       while (!Done && StrBuf_ServGetln(Buf)>=0) {
                if ( (StrLength(Buf)==3) && 
                    !strcmp(ChrPtr(Buf), "000")) 
                {
@@ -229,10 +230,12 @@ int read_message(StrBuf *Target, const char *tmpl, long tmpllen, long msgnum, co
                FreeStrBuf(&Error);
        }
 
-       /* strip the bare contenttype, so we ommit charset etc. */
+       /* Extract just the content-type (omit attributes such as "charset") */
        StrBufExtract_token(Buf, Msg->MsgBody->ContentType, 0, ';');
        StrBufTrim(Buf);
-       /* look up the renderer, that will convert this mimeitem into the htmlized form */
+       StrBufLowerCase(Buf);
+
+       /* Locate a renderer capable of converting this MIME part into HTML */
        if (GetHash(MimeRenderHandler, SKEY(Buf), &vHdr) &&
            (vHdr != NULL)) {
                RenderMimeFuncStruct *Render;
@@ -379,10 +382,10 @@ void handle_one_message(void)
                break;
        case eDELETE:
                CmdBuf = NewStrBuf ();
-               if (WCC->wc_is_trash) { /** Delete from Trash is a real delete */
+               if (WCC->wc_is_trash) { /* Delete from Trash is a real delete */
                        serv_printf("DELE %ld", msgnum);        
                }
-               else {                  /** Otherwise move it to Trash */
+               else {                  /* Otherwise move it to Trash */
                        serv_printf("MOVE %ld|_TRASH_|0", msgnum);
                }
                StrBuf_ServGetln(CmdBuf);
@@ -439,10 +442,10 @@ void embed_message(void) {
                break;
        case eDELETE:
                CmdBuf = NewStrBuf ();
-               if (WCC->wc_is_trash) { /** Delete from Trash is a real delete */
+               if (WCC->wc_is_trash) { /* Delete from Trash is a real delete */
                        serv_printf("DELE %ld", msgnum);        
                }
-               else {                  /** Otherwise move it to Trash */
+               else {                  /* Otherwise move it to Trash */
                        serv_printf("MOVE %ld|_TRASH_|0", msgnum);
                }
                StrBuf_ServGetln(CmdBuf);
@@ -513,7 +516,7 @@ void display_headers(void) {
        serv_getln(buf, sizeof buf);
        if (buf[0] == '1') {
                while (serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
-                       wprintf("%s\n", buf);
+                       wc_printf("%s\n", buf);
                }
        }
 
@@ -568,40 +571,37 @@ message_summary *ReadOneMessageSummary(StrBuf *RawMessage, const char *DefaultSu
  * load message pointers from the server for a "read messages" operation
  *
  * servcmd:            the citadel command to send to the citserver
- * with_headers:       also include some of the headers with the message numbers (more expensive)
  */
-int load_msg_ptrs(const char *servcmd, int with_headers)
+int load_msg_ptrs(const char *servcmd, SharedMessageStatus *Stat)
 {
        StrBuf* FoundCharset = NULL;
         wcsession *WCC = WC;
        message_summary *Msg;
        StrBuf *Buf, *Buf2;
-       int nummsgs = 0;
-       int maxload = 0;
        long len;
        int n;
        int skipit;
        const char *Ptr = NULL;
 
+       Stat->lowest_found = LONG_MAX;
+       Stat->highest_found = LONG_MIN;
+
        if (WCC->summ != NULL) {
                DeleteHash(&WCC->summ);
        }
        WCC->summ = NewHash(1, Flathash);
-       maxload = 10000;
        
        Buf = NewStrBuf();
        serv_puts(servcmd);
        StrBuf_ServGetln(Buf);
        if (GetServerStatus(Buf, NULL) != 1) {
                FreeStrBuf(&Buf);
-               return (nummsgs);
+               return (Stat->nummsgs);
        }
        Buf2 = NewStrBuf();
-       while (len = StrBuf_ServGetln(Buf),
-              ((len != 3)  ||
-               strcmp(ChrPtr(Buf), "000")!= 0))
+       while (len = StrBuf_ServGetln(Buf), ((len != 3) || strcmp(ChrPtr(Buf), "000")!= 0))
        {
-               if (nummsgs < maxload) {
+               if (Stat->nummsgs < Stat->maxload) {
                        skipit = 0;
                        Ptr = NULL;
                        Msg = (message_summary*)malloc(sizeof(message_summary));
@@ -610,16 +610,26 @@ int load_msg_ptrs(const char *servcmd, int with_headers)
                        Msg->msgnum = StrBufExtractNext_long(Buf, &Ptr, '|');
                        Msg->date = StrBufExtractNext_long(Buf, &Ptr, '|');
 
+                       if (Stat->nummsgs == 0) {
+                               if (Msg->msgnum < Stat->lowest_found) {
+                                       Stat->lowest_found = Msg->msgnum;
+                               }
+                               if (Msg->msgnum > Stat->highest_found) {
+                                       Stat->highest_found = Msg->msgnum;
+                               }
+                       }
+
                        if ((Msg->msgnum == 0) && (StrLength(Buf) < 32)) {
                                free(Msg);
                                continue;
                        }
+
                        /* 
                         * as citserver probably gives us messages in forward date sorting
                         * nummsgs should be the same order as the message date.
                         */
                        if (Msg->date == 0) {
-                               Msg->date = nummsgs;
+                               Msg->date = Stat->nummsgs;
                                if (StrLength(Buf) < 32) 
                                        skipit = 1;
                        }
@@ -668,11 +678,11 @@ int load_msg_ptrs(const char *servcmd, int with_headers)
                        n = Msg->msgnum;
                        Put(WCC->summ, (const char *)&n, sizeof(n), Msg, DestroyMessageSummary);
                }
-               nummsgs++;
+               Stat->nummsgs++;
        }
        FreeStrBuf(&Buf2);
        FreeStrBuf(&Buf);
-       return (nummsgs);
+       return (Stat->nummsgs);
 }
 
 
@@ -689,93 +699,6 @@ inline message_summary* GetMessagePtrAt(int n, HashList *Summ)
 }
 
 
-long DrawMessageDropdown(StrBuf *Selector, long maxmsgs, long startmsg, int nMessages)
-{
-       StrBuf *TmpBuf;
-       wcsession *WCC = WC;
-       void *vMsg;
-       int lo, hi;
-       long ret;
-       long hklen;
-       const char *key;
-       int done = 0;
-       int nItems;
-       HashPos *At;
-       long vector[16];
-       WCTemplputParams SubTP;
-
-       memset(&SubTP, 0, sizeof(WCTemplputParams));
-       SubTP.Filter.ContextType = CTX_LONGVECTOR;
-       SubTP.Context = &vector;
-       TmpBuf = NewStrBufPlain(NULL, SIZ);
-       At = GetNewHashPos(WCC->summ, nMessages);
-       nItems = GetCount(WCC->summ);
-       ret = nMessages;
-       vector[0] = 7;
-       vector[2] = 1;
-       vector[1] = startmsg;
-       vector[3] = 0;
-
-       while (!done) {
-               vector[3] = abs(nMessages);
-               lo = GetHashPosCounter(At);
-               if (nMessages > 0) {
-                       if (lo + nMessages >= nItems) {
-                               hi = nItems - 1;
-                               vector[3] = nItems - lo;
-                               if (startmsg == lo) 
-                                       ret = vector[3];
-                       }
-                       else {
-                               hi = lo + nMessages - 1;
-                       }
-               } else {
-                       if (lo + nMessages < -1) {
-                               hi = 0;
-                       }
-                       else {
-                               if ((lo % abs(nMessages)) != 0) {
-                                       int offset = (lo % abs(nMessages) *
-                                                     (nMessages / abs(nMessages)));
-                                       hi = lo + offset;
-                                       vector[3] = abs(offset);
-                                       if (startmsg == lo)
-                                                ret = offset;
-                               }
-                               else
-                                       hi = lo + nMessages;
-                       }
-               }
-               done = !GetNextHashPos(WCC->summ, At, &hklen, &key, &vMsg);
-               
-               /*
-                * Bump these because although we're thinking in zero base, the user
-                * is a drooling idiot and is thinking in one base.
-                */
-               vector[4] = lo + 1;
-               vector[5] = hi + 1;
-               vector[6] = lo;
-               FlushStrBuf(TmpBuf);
-               dbg_print_longvector(vector);
-               DoTemplate(HKEY("select_messageindex"), TmpBuf, &SubTP);
-               StrBufAppendBuf(Selector, TmpBuf, 0);
-       }
-       vector[6] = 0;
-       FlushStrBuf(TmpBuf);
-       if (maxmsgs == 9999999) {
-               vector[1] = 1;
-               ret = maxmsgs;
-       }
-       else
-               vector[1] = 0;          
-       vector[2] = 0;
-       dbg_print_longvector(vector);
-       DoTemplate(HKEY("select_messageindex_all"), TmpBuf, &SubTP);
-       StrBufAppendBuf(Selector, TmpBuf, 0);
-       FreeStrBuf(&TmpBuf);
-       DeleteHashPos(&At);
-       return ret;
-}
 
 void load_seen_flags(void)
 {
@@ -799,7 +722,7 @@ void load_seen_flags(void)
        }
        at = GetNewHashPos(WCC->summ, 0);
        while (GetNextHashPos(WCC->summ, at, &HKLen, &HashKey, &vMsg)) {
-               /** Are you a new message, or an old message? */
+               /* Are you a new message, or an old message? */
                Msg = (message_summary*) vMsg;
                if (is_msg_in_mset(ChrPtr(OldMsg), Msg->msgnum)) {
                        Msg->is_new = 0;
@@ -814,333 +737,159 @@ void load_seen_flags(void)
 
 extern readloop_struct rlid[];
 
+typedef struct _RoomRenderer{
+       int RoomType;
+
+       GetParamsGetServerCall_func GetParamsGetServerCall;
+       PrintViewHeader_func PrintViewHeader;
+       LoadMsgFromServer_func LoadMsgFromServer;
+       RenderView_or_Tail_func RenderView_or_Tail;
+       View_Cleanup_func ViewCleanup;
+} RoomRenderer;
+
+
 /*
  * command loop for reading messages
  *
- * Set oper to "readnew" or "readold" or "readfwd" or "headers"
+ * Set oper to "readnew" or "readold" or "readfwd" or "headers" or "readgt" or "readlt" or "do_search"
  */
 void readloop(long oper)
 {
-       StrBuf *MessageDropdown = NULL;
-       StrBuf *BBViewToolBar = NULL;
+       RoomRenderer *ViewMsg;
+       void *vViewMsg;
        void *vMsg;
        message_summary *Msg;
        char cmd[256] = "";
-       char buf[SIZ];
-       int a = 0;
-       int with_headers = 0;
-       int nummsgs;
-       long startmsg = 0;
-       int maxmsgs = 0;
-       long *displayed_msgs = NULL;
-       int num_displayed = 0;
-       int is_singlecard = 0;
-       struct calview calv;
-       int i;
-       int lowest_displayed = (-1);
-       int highest_displayed = 0;
-       addrbookent *addrbook = NULL;
-       int num_ab = 0;
-       int bbs_reverse = 0;
+       int i, r;
        wcsession *WCC = WC;
        HashPos *at;
        const char *HashKey;
        long HKLen;
-       int care_for_empty_list = 0;
-       int load_seen = 0;
-       int sortit = 0;
-       int defaultsortorder = 0;
        WCTemplputParams SubTP;
-       char *ab_name;
-       const StrBuf *Mime;
+       SharedMessageStatus Stat;
+       void *ViewSpecific;
 
-       if (havebstr("is_summary") && (1 == (ibstr("is_summary"))))
+       if (havebstr("is_summary") && (1 == (ibstr("is_summary")))) {
                WCC->wc_view = VIEW_MAILBOX;
+       }
 
-       if (!WCC->is_ajax) {
-               output_headers(1, 1, 1, 0, 0, 0);
-       } else if (WCC->wc_view == VIEW_MAILBOX) {
-               jsonMessageListHdr();
+       if (havebstr("is_ajax") && (1 == (ibstr("is_ajax")))) {
+               WCC->is_ajax = 1;
        }
 
-       switch (WCC->wc_view) {
-       case VIEW_WIKI:
-               sprintf(buf, "wiki?room=%s&page=home", ChrPtr(WCC->wc_roomname));
-               http_redirect(buf);
+       if ((oper == do_search) && (WCC->wc_view == VIEW_WIKI)) {
+               display_wiki_pagelist();
                return;
-       case VIEW_CALBRIEF:
-       case VIEW_CALENDAR:
-               load_seen = 1;
-               strcpy(cmd, "MSGS ALL");
-               maxmsgs = 32767;
-               parse_calendar_view_request(&calv);
-               break;
-       case VIEW_TASKS:
-               strcpy(cmd, "MSGS ALL");
-               maxmsgs = 32767;
-               break;
-       case VIEW_NOTES:
-               strcpy(cmd, "MSGS ALL");
-               maxmsgs = 32767;
-               wprintf("<div id=\"new_notes_here\"></div>\n");
-               break;
-       case VIEW_ADDRESSBOOK:
-               is_singlecard = ibstr("is_singlecard");
-               if (is_singlecard != 1) {
-                       if (oper == do_search) {
-                               snprintf(cmd, sizeof(cmd), "MSGS SEARCH|%s", bstr("query"));
-                       }
-                       else {
-                               strcpy(cmd, "MSGS ALL");
-                       }
-                       maxmsgs = 9999999;
-                       break;
-               }
-               break;
-       case VIEW_MAILBOX: 
-         if (!WCC->is_ajax) {
-           new_summary_view();
-           return;
-         } else {
-               defaultsortorder = 2;
-               sortit = 1;
-               load_seen = 1;
-               care_for_empty_list = 0;
-               with_headers = 1;
-               /* Generally using maxmsgs|startmsg is not required
-                  in mailbox view, but we have a 'safemode' for clients
-                  (*cough* Exploder) that simply can't handle too many */
-               if (havebstr("maxmsgs")) maxmsgs = ibstr("maxmsgs");
-               else maxmsgs = 9999999;
-               if (havebstr("startmsg")) startmsg = lbstr("startmsg");
-               snprintf(cmd, sizeof(cmd), "MSGS %s|%s||1",
-                        (oper == do_search) ? "SEARCH" : "ALL",
-                        (oper == do_search) ? bstr("query") : ""
-                       );
-         }
-               break;
-       case VIEW_BBS:
-       default:
-               defaultsortorder = 1;
-               startmsg = -1;
-               sortit = 1;
-               care_for_empty_list = 1;
-
-               rlid[oper].cmd(cmd, sizeof(cmd));
-               SetAccessCommand(oper);
+       }
 
-               if (havebstr("maxmsgs"))
-                       maxmsgs = ibstr("maxmsgs");
-               if (maxmsgs == 0) maxmsgs = DEFAULT_MAXMSGS;
+       memset(&Stat, 0, sizeof(SharedMessageStatus));
+       Stat.maxload = 10000;
+       Stat.lowest_found = (-1);
+       Stat.highest_found = (-1);
+       GetHash(ReadLoopHandler, IKEY(WCC->wc_view), &vViewMsg);
+       if (vViewMsg == NULL) {
+               WCC->wc_view = VIEW_BBS;
+               GetHash(ReadLoopHandler, IKEY(WCC->wc_view), &vViewMsg);
+       }
+       if (vViewMsg == NULL) {
+               return;                 // TODO: print message
+       }
 
-               if (havebstr("startmsg")) {
-                       startmsg = lbstr("startmsg");
-               }
-               
+       ViewMsg = (RoomRenderer*) vViewMsg;
+       if (!WCC->is_ajax) {
+               output_headers(1, 1, 1, 0, 0, 0);
+       } else if (WCC->wc_view == VIEW_MAILBOX) {
+               jsonMessageListHdr();
        }
 
-       nummsgs = load_msg_ptrs(cmd, with_headers);
-       if (nummsgs == 0) {
-               if (care_for_empty_list) {
-                       wprintf("<div class=\"nomsgs\"><br><em>");
-                       switch (oper) {
-                       case readnew:
-                               wprintf(_("No new messages."));
-                               break;
-                       case readold:
-                               wprintf(_("No old messages."));
-                               break;
-                       default:
-                               wprintf(_("No messages here."));
-                       }
-                       wprintf("</em><br></div>\n");
-                       goto DONE;
-               }
+       if (ViewMsg->GetParamsGetServerCall != NULL) {
+               r = ViewMsg->GetParamsGetServerCall(
+                      &Stat,
+                      &ViewSpecific,
+                      oper,
+                      cmd, sizeof(cmd)
+               );
+       } else {
+               r = 0;
+       }
+       switch(r)
+       {
+       case 400:
+       case 404:
 
+               return;
+       case 300: /* the callback hook should do the work for us here, since he knows what to do. */
+               return;
+       case 200:
+       default:
+               break;
        }
+       if (!IsEmptyStr(cmd))
+               Stat.nummsgs = load_msg_ptrs(cmd, &Stat);
 
-       if (sortit) {
+       if (Stat.sortit) {
                CompareFunc SortIt;
                memset(&SubTP, 0, sizeof(WCTemplputParams));
-               SubTP.Filter.ContextType = CTX_NONE;
+               SubTP.Filter.ContextType = CTX_MAILSUM;
                SubTP.Context = NULL;
                SortIt =  RetrieveSort(&SubTP, NULL, 0,
-                                      HKEY("date"), defaultsortorder);
+                                      HKEY("date"), Stat.defaultsortorder);
                if (SortIt != NULL)
                        SortByPayload(WCC->summ, SortIt);
-               if (WCC->wc_view == VIEW_BBS) {
-                       if (lbstr("SortOrder") == 2) {
-                               bbs_reverse = 1;
-                               num_displayed = -DEFAULT_MAXMSGS;
-                       }
-                       else {
-                               bbs_reverse = 0;
-                               num_displayed = DEFAULT_MAXMSGS;
-                       }
-               }
        }
-       if (startmsg < 0) startmsg = (bbs_reverse) ? nummsgs - 1 : 0;
+       if (Stat.startmsg < 0) 
+               Stat.startmsg = (Stat.reverse) ? Stat.nummsgs - 1 : 0;
 
-       if (load_seen) load_seen_flags();
+       if (Stat.load_seen) load_seen_flags();
        
         /*
         * Print any inforation above the message list...
         */
-       switch (WCC->wc_view) {
-       case VIEW_BBS:
-               BBViewToolBar = NewStrBufPlain(NULL, SIZ);
-               MessageDropdown = NewStrBufPlain(NULL, SIZ);
-
-               maxmsgs = DrawMessageDropdown(MessageDropdown, maxmsgs, startmsg, num_displayed);
-               if (num_displayed < 0) {
-                       startmsg += maxmsgs;
-                       if (num_displayed != maxmsgs)                           
-                               maxmsgs = abs(maxmsgs) + 1;
-                       else
-                               maxmsgs = abs(maxmsgs);
+       if (ViewMsg->PrintViewHeader != NULL)
+               ViewMsg->PrintViewHeader(&Stat, &ViewSpecific);
 
-               }
-               memset(&SubTP, 0, sizeof(WCTemplputParams));
-               SubTP.Filter.ContextType = CTX_STRBUF;
-               SubTP.Context = MessageDropdown;
-               DoTemplate(HKEY("msg_listselector_top"), BBViewToolBar, &SubTP);
-               StrBufAppendBuf(WCC->WBuf, BBViewToolBar, 0);
-               FlushStrBuf(BBViewToolBar);
-               break;
-       }
-       WCC->startmsg =  startmsg;
-       WCC->maxmsgs = maxmsgs;
+       WCC->startmsg =  Stat.startmsg;
+       WCC->maxmsgs = Stat.maxmsgs;
        WCC->num_displayed = 0;
 
        /* Put some helpful data in vars for mailsummary_json */
-       svputlong("READLOOP:TOTALMSGS", nummsgs);
-       svputlong("READLOOP:STARTMSG", startmsg);
+       svputlong("READLOOP:TOTALMSGS", Stat.nummsgs);
+       svputlong("READLOOP:STARTMSG", Stat.startmsg);
        svputlong("WCVIEW", WCC->wc_view);
 
        /*
         * iterate over each message. if we need to load an attachment, do it here. 
         */
-       if (WCC->wc_view == VIEW_MAILBOX) goto NO_MSG_LOOP;
-       at = GetNewHashPos(WCC->summ, 0);
-       num_displayed = i = 0;
-       while (GetNextHashPos(WCC->summ, at, &HKLen, &HashKey, &vMsg)) {
-               Msg = (message_summary*) vMsg;          
-               if ((Msg->msgnum >= startmsg) && (num_displayed <= maxmsgs)) {                  
-                       switch (WCC->wc_view) {
-                       case VIEW_WIKI:
-                               break;
-                       case VIEW_CALBRIEF: /* load the mime attachments for special tasks... */
-                       case VIEW_CALENDAR:
-                               load_calendar_item(Msg, Msg->is_new, &calv);
-                               break;
-                       case VIEW_TASKS:
-                               display_task(Msg, Msg->is_new);
-                               break;
-                       case VIEW_NOTES:
-                               display_note(Msg, Msg->is_new);
-                               break;
-                       case VIEW_ADDRESSBOOK:
-                               ab_name = NULL;
-                               fetch_ab_name(Msg, &ab_name);
-                               if (ab_name == NULL) 
-                                       break;
-                               ++num_ab;
-                               addrbook = realloc(addrbook,
-                                                  (sizeof(addrbookent) * num_ab) );
-                               safestrncpy(addrbook[num_ab-1].ab_name, ab_name,
-                                           sizeof(addrbook[num_ab-1].ab_name));
-                               addrbook[num_ab-1].ab_msgnum = Msg->msgnum;
-                               free(ab_name);
-                               break;
-                       case VIEW_BBS: /* Tag the mails we want to show in bbview... */
-                       default:
-                               if (displayed_msgs == NULL) {
-                                       displayed_msgs = malloc(sizeof(long) *
-                                                               (maxmsgs<nummsgs ? maxmsgs + 1 : nummsgs + 1));
-                               }
-                               if ((i >= startmsg) && (i < startmsg + maxmsgs)) {
-                                       displayed_msgs[num_displayed] = Msg->msgnum;
-                                       if (lowest_displayed < 0) lowest_displayed = a;
-                                       highest_displayed = a;
-                       
-                                       num_displayed++;
-                               }
-                       }
-               } 
-               i++;
+
+       if ((ViewMsg->LoadMsgFromServer != NULL) && 
+           (!IsEmptyStr(cmd)))
+       {
+               at = GetNewHashPos(WCC->summ, 0);
+               Stat.num_displayed = i = 0;
+               while ( GetNextHashPos(WCC->summ, at, &HKLen, &HashKey, &vMsg)) {
+                       Msg = (message_summary*) vMsg;          
+                       if ((Msg->msgnum >= Stat.startmsg) && (Stat.num_displayed <= Stat.maxmsgs)) {
+                               ViewMsg->LoadMsgFromServer(&Stat, &ViewSpecific, Msg, Msg->is_new, i);
+                       } 
+                       i++;
+               }
+               DeleteHashPos(&at);
        }
-       DeleteHashPos(&at);
 
-NO_MSG_LOOP:
        /*
         * Done iterating the message list. now tasks we want to do after.
         */
-       switch (WCC->wc_view) {
-       case VIEW_MAILBOX:
-         DoTemplate(HKEY("mailsummary_json"),NULL, &SubTP);
-         break;
-       case VIEW_BBS:
-               if (displayed_msgs != NULL) {
-                       /** if we do a split bbview in the future, begin messages div here */
-                       
-                       for (a=0; a<num_displayed; ++a) {
-                               read_message(WCC->WBuf, HKEY("view_message"), displayed_msgs[a], NULL, &Mime);
-                       }
-                       
-                       /* if we do a split bbview in the future, end messages div here */
-                       
-                       free(displayed_msgs);
-                       displayed_msgs = NULL;
-               }
-               memset(&SubTP, 0, sizeof(WCTemplputParams));
-               SubTP.Filter.ContextType = CTX_STRBUF;
-               SubTP.Context = MessageDropdown;
-               DoTemplate(HKEY("msg_listselector_bottom"), BBViewToolBar, &SubTP);
-               StrBufAppendBuf(WCC->WBuf, BBViewToolBar, 0);
+       if (ViewMsg->RenderView_or_Tail != NULL)
+               ViewMsg->RenderView_or_Tail(&Stat, &ViewSpecific, oper);
 
-               FreeStrBuf(&BBViewToolBar);
-               FreeStrBuf(&MessageDropdown);
-       }
+       if (ViewMsg->ViewCleanup != NULL)
+               ViewMsg->ViewCleanup(&ViewSpecific);
 
-       
-DONE:
-       switch (WCC->wc_view) {
-       case VIEW_WIKI:
-               break;
-       case VIEW_CALBRIEF:
-       case VIEW_CALENDAR:
-               render_calendar_view(&calv);
-               break;
-       case VIEW_TASKS:
-               do_tasks_view();        /** Render the task list */
-               break;
-       case VIEW_NOTES:
-               break;
-       case VIEW_ADDRESSBOOK:
-               if (is_singlecard)
-                       read_message(WC->WBuf, HKEY("view_message"), lbstr("startmsg"), NULL, &Mime);
-               else
-                       do_addrbook_view(addrbook, num_ab);     /* Render the address book */
-               break;
-       case VIEW_MAILBOX: 
-       case VIEW_BBS:
-       default:
-               break;
-       }
-       /** Note: wDumpContent() will output one additional </div> tag. */
-       if (WCC->wc_view != VIEW_MAILBOX) {
-               /* We ought to move this out into template */
-               wDumpContent(1);
-       } else {
-               end_burst();
-       }
        WCC->startmsg = 0;
        WCC->maxmsgs = 0;
        if (WCC->summ != NULL) {
                DeleteHash(&WCC->summ);
        }
-       if (addrbook != NULL) free(addrbook);
-       FreeStrBuf(&BBViewToolBar);
 }
 
 
@@ -1159,6 +908,7 @@ void post_mime_to_server(void) {
        size_t encoded_length;
        size_t encoded_strlen;
        char *txtmail = NULL;
+       int include_text_alt = 0;       /* Set to nonzero to include multipart/alternative text/plain */
 
        sprintf(top_boundary, "Citadel--Multipart--%s--%04x--%04x",
                ChrPtr(WCC->serv_info->serv_fqdn),
@@ -1180,6 +930,11 @@ void post_mime_to_server(void) {
                is_multipart = 1;
        }
 
+       /* Only do multipart/alternative for mailboxes.  BBS and Wiki rooms don't need it. */
+       if (WC->wc_view == VIEW_MAILBOX) {
+               include_text_alt = 1;
+       }
+
        if (is_multipart) {
                /* Remember, serv_printf() appends an extra newline */
                serv_printf("Content-type: multipart/mixed; boundary=\"%s\"\n", top_boundary);
@@ -1188,19 +943,21 @@ void post_mime_to_server(void) {
        }
 
        /* Remember, serv_printf() appends an extra newline */
-       serv_printf("Content-type: multipart/alternative; "
-               "boundary=\"%s\"\n", alt_boundary);
-       serv_printf("This is a multipart message in MIME format.\n");
-       serv_printf("--%s", alt_boundary);
+       if (include_text_alt) {
+               serv_printf("Content-type: multipart/alternative; "
+                       "boundary=\"%s\"\n", alt_boundary);
+               serv_printf("This is a multipart message in MIME format.\n");
+               serv_printf("--%s", alt_boundary);
 
-       serv_puts("Content-type: text/plain; charset=utf-8");
-       serv_puts("Content-Transfer-Encoding: quoted-printable");
-       serv_puts("");
-       txtmail = html_to_ascii(bstr("msgtext"), 0, 80, 0);
-        text_to_server_qp(txtmail);     /* Transmit message in quoted-printable encoding */
-        free(txtmail);
+               serv_puts("Content-type: text/plain; charset=utf-8");
+               serv_puts("Content-Transfer-Encoding: quoted-printable");
+               serv_puts("");
+               txtmail = html_to_ascii(bstr("msgtext"), 0, 80, 0);
+               text_to_server_qp(txtmail);     /* Transmit message in quoted-printable encoding */
+               free(txtmail);
 
-       serv_printf("--%s", alt_boundary);
+               serv_printf("--%s", alt_boundary);
+       }
 
        serv_puts("Content-type: text/html; charset=utf-8");
        serv_puts("Content-Transfer-Encoding: quoted-printable");
@@ -1209,7 +966,9 @@ void post_mime_to_server(void) {
        text_to_server_qp(bstr("msgtext"));     /* Transmit message in quoted-printable encoding */
        serv_puts("</body></html>\r\n");
 
-       serv_printf("--%s--", alt_boundary);
+       if (include_text_alt) {
+               serv_printf("--%s--", alt_boundary);
+       }
        
        if (is_multipart) {
                long len;
@@ -1221,6 +980,8 @@ void post_mime_to_server(void) {
                it = GetNewHashPos(WCC->attachments, 0);
                while (GetNextHashPos(WCC->attachments, it, &len, &Key, &vAtt)) {
                        att = (wc_mime_attachment *)vAtt;
+                       if (att->length == 0)
+                               continue;
                        encoded_length = ((att->length * 150) / 100);
                        encoded = malloc(encoded_length);
                        if (encoded == NULL) break;
@@ -1317,10 +1078,9 @@ void post_message(void)
                 * Transfer control of this memory from the upload struct
                 * to the attachment struct.
                 */
-               att->Data = NewStrBufPlain(WCC->upload, WCC->upload_length);
-               free(WCC->upload);
-               WCC->upload_length = 0;
+               att->Data = WCC->upload;
                WCC->upload = NULL;
+               WCC->upload_length = 0;
                display_enter();
                return;
        }
@@ -1387,7 +1147,7 @@ void post_message(void)
                Recp = sbstr("recp");
                Cc = sbstr("cc");
                Bcc = sbstr("bcc");
-               Wikipage = sbstr("wikipage");
+               Wikipage = sbstr("page");
                my_email_addr = sbstr("my_email_addr");
                
                CmdBuf = NewStrBufPlain(NULL, 
@@ -1474,8 +1234,8 @@ void post_message(void)
        /*
         *  If we were editing a page in a wiki room, go to that page now.
         */
-       else if (havebstr("wikipage")) {
-               snprintf(buf, sizeof buf, "wiki?page=%s", bstr("wikipage"));
+       else if (havebstr("page")) {
+               snprintf(buf, sizeof buf, "wiki?page=%s", bstr("page"));
                http_redirect(buf);
        }
        /*
@@ -1533,7 +1293,7 @@ void display_enter(void)
                subject_required = extract_int(&buf[4], 1);
        }
 
-       /**
+       /*
         * Are we perhaps in an address book view?  If so, then an "enter
         * message" command really means "add new entry."
         */
@@ -1576,7 +1336,7 @@ void display_enter(void)
                Recp = sbstr("recp");
                Cc = sbstr("cc");
                Bcc = sbstr("bcc");
-               Wikipage = sbstr("wikipage");
+               Wikipage = sbstr("page");
                
                CmdBuf = NewStrBufPlain(NULL, 
                                        sizeof (CMD) + 
@@ -1606,7 +1366,7 @@ void display_enter(void)
                        }
                }
                else if (buf[0] != '2') {       /* Any other error means that we cannot continue */
-                       wprintf("<em>%s</em><br />\n", &buf[4]);        /* TODO -> important message */
+                       wc_printf("<em>%s</em><br />\n", &buf[4]);      /* TODO -> important message */
                        return;
                }
        }
@@ -1680,43 +1440,43 @@ void confirm_move_msg(void)
 
 
        output_headers(1, 1, 2, 0, 0, 0);
-       wprintf("<div id=\"banner\">\n");
-       wprintf("<h1>");
-       wprintf(_("Confirm move of message"));
-       wprintf("</h1>");
-       wprintf("</div>\n");
+       wc_printf("<div id=\"banner\">\n");
+       wc_printf("<h1>");
+       wc_printf(_("Confirm move of message"));
+       wc_printf("</h1>");
+       wc_printf("</div>\n");
 
-       wprintf("<div id=\"content\" class=\"service\">\n");
+       wc_printf("<div id=\"content\" class=\"service\">\n");
 
-       wprintf("<CENTER>");
+       wc_printf("<CENTER>");
 
-       wprintf(_("Move this message to:"));
-       wprintf("<br />\n");
+       wc_printf(_("Move this message to:"));
+       wc_printf("<br />\n");
 
-       wprintf("<form METHOD=\"POST\" action=\"move_msg\">\n");
-       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
-       wprintf("<INPUT TYPE=\"hidden\" NAME=\"msgid\" VALUE=\"%s\">\n", bstr("msgid"));
+       wc_printf("<form METHOD=\"POST\" action=\"move_msg\">\n");
+       wc_printf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
+       wc_printf("<INPUT TYPE=\"hidden\" NAME=\"msgid\" VALUE=\"%s\">\n", bstr("msgid"));
 
-       wprintf("<SELECT NAME=\"target_room\" SIZE=5>\n");
+       wc_printf("<SELECT NAME=\"target_room\" SIZE=5>\n");
        serv_puts("LKRA");
        serv_getln(buf, sizeof buf);
        if (buf[0] == '1') {
                while (serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
                        extract_token(targ, buf, 0, '|', sizeof targ);
-                       wprintf("<OPTION>");
+                       wc_printf("<OPTION>");
                        escputs(targ);
-                       wprintf("\n");
+                       wc_printf("\n");
                }
        }
-       wprintf("</SELECT>\n");
-       wprintf("<br />\n");
+       wc_printf("</SELECT>\n");
+       wc_printf("<br />\n");
 
-       wprintf("<INPUT TYPE=\"submit\" NAME=\"move_button\" VALUE=\"%s\">", _("Move"));
-       wprintf("&nbsp;");
-       wprintf("<INPUT TYPE=\"submit\" NAME=\"cancel_button\" VALUE=\"%s\">", _("Cancel"));
-       wprintf("</form></CENTER>\n");
+       wc_printf("<INPUT TYPE=\"submit\" NAME=\"move_button\" VALUE=\"%s\">", _("Move"));
+       wc_printf("&nbsp;");
+       wc_printf("<INPUT TYPE=\"submit\" NAME=\"cancel_button\" VALUE=\"%s\">", _("Cancel"));
+       wc_printf("</form></CENTER>\n");
 
-       wprintf("</CENTER>\n");
+       wc_printf("</CENTER>\n");
        wDumpContent(1);
 }
 
@@ -1750,7 +1510,8 @@ void postpart(StrBuf *partnum, StrBuf *filename, int force_download)
                hprintf("HTTP/1.1 404 %s\n", ChrPtr(partnum));
                output_headers(0, 0, 0, 0, 0, 0);
                hprintf("Content-Type: text/plain\r\n");
-               wprintf(_("An error occurred while retrieving this part: %s/%s\n"), 
+               begin_burst();
+               wc_printf(_("An error occurred while retrieving this part: %s/%s\n"), 
                        ChrPtr(partnum), ChrPtr(filename));
                end_burst();
        }
@@ -1809,7 +1570,8 @@ void mimepart(int force_download)
                hprintf("HTTP/1.1 404 %s\n", ChrPtr(Buf));
                output_headers(0, 0, 0, 0, 0, 0);
                hprintf("Content-Type: text/plain\r\n");
-               wprintf(_("An error occurred while retrieving this part: %s\n"), 
+               begin_burst();
+               wc_printf(_("An error occurred while retrieving this part: %s\n"), 
                        ChrPtr(Buf));
                end_burst();
        }
@@ -1910,6 +1672,8 @@ void h_readold(void) { readloop(readold);}
 void h_readfwd(void) { readloop(readfwd);}
 void h_headers(void) { readloop(headers);}
 void h_do_search(void) { readloop(do_search);}
+void h_readgt(void) { readloop(readgt);}
+void h_readlt(void) { readloop(readlt);}
 
 void jsonMessageListHdr(void) 
 {
@@ -1922,13 +1686,6 @@ void jsonMessageListHdr(void)
        begin_burst();
 }
 
-/* Spit out the new summary view. This is basically a static page, so clients can cache the layout, all the dirty work is javascript :) */
-void new_summary_view(void) {
-       begin_burst();
-       DoTemplate(HKEY("msg_listview"),NULL,&NoCtx);
-       DoTemplate(HKEY("trailing"),NULL,&NoCtx);
-       end_burst();
-}
 
 /* Output message list in JSON format */
 void jsonMessageList(void) {
@@ -1940,6 +1697,29 @@ void jsonMessageList(void) {
        WC->is_ajax = 0;
 }
 
+void RegisterReadLoopHandlerset(
+       int RoomType,
+       GetParamsGetServerCall_func GetParamsGetServerCall,
+       PrintViewHeader_func PrintViewHeader,
+       LoadMsgFromServer_func LoadMsgFromServer,
+       RenderView_or_Tail_func RenderView_or_Tail,
+       View_Cleanup_func ViewCleanup
+       )
+{
+       RoomRenderer *Handler;
+
+       Handler = (RoomRenderer*) malloc(sizeof(RoomRenderer));
+
+       Handler->RoomType = RoomType;
+       Handler->GetParamsGetServerCall = GetParamsGetServerCall;
+       Handler->PrintViewHeader = PrintViewHeader;
+       Handler->LoadMsgFromServer = LoadMsgFromServer;
+       Handler->RenderView_or_Tail = RenderView_or_Tail;
+       Handler->ViewCleanup = ViewCleanup;
+
+       Put(ReadLoopHandler, IKEY(RoomType), Handler, NULL);
+}
+
 void 
 InitModule_MSG
 (void)
@@ -1964,29 +1744,31 @@ InitModule_MSG
                           NULL);
        RegisterPreference("mailbox",_("Mailbox view mode"), PRF_STRING, NULL);
 
-       WebcitAddUrlHandler(HKEY("readnew"), h_readnew, NEED_URL);
-       WebcitAddUrlHandler(HKEY("readold"), h_readold, NEED_URL);
-       WebcitAddUrlHandler(HKEY("readfwd"), h_readfwd, NEED_URL);
-       WebcitAddUrlHandler(HKEY("headers"), h_headers, NEED_URL);
-       WebcitAddUrlHandler(HKEY("do_search"), h_do_search, 0);
-       WebcitAddUrlHandler(HKEY("display_enter"), display_enter, 0);
-       WebcitAddUrlHandler(HKEY("post"), post_message, 0);
-       WebcitAddUrlHandler(HKEY("move_msg"), move_msg, 0);
-       WebcitAddUrlHandler(HKEY("delete_msg"), delete_msg, 0);
-       WebcitAddUrlHandler(HKEY("confirm_move_msg"), confirm_move_msg, 0);
-       WebcitAddUrlHandler(HKEY("msg"), embed_message, NEED_URL);
-       WebcitAddUrlHandler(HKEY("message"), handle_one_message, NEED_URL|XHTTP_COMMANDS|COOKIEUNNEEDED|FORCE_SESSIONCLOSE);
-       WebcitAddUrlHandler(HKEY("printmsg"), print_message, NEED_URL);
-       WebcitAddUrlHandler(HKEY("mobilemsg"), mobile_message_view, NEED_URL);
-       WebcitAddUrlHandler(HKEY("msgheaders"), display_headers, NEED_URL);
-
-       WebcitAddUrlHandler(HKEY("mimepart"), view_mimepart, NEED_URL);
-       WebcitAddUrlHandler(HKEY("mimepart_download"), download_mimepart, NEED_URL);
-       WebcitAddUrlHandler(HKEY("postpart"), view_postpart, NEED_URL);
-       WebcitAddUrlHandler(HKEY("postpart_download"), download_postpart, NEED_URL);
+       WebcitAddUrlHandler(HKEY("readnew"), "", 0, h_readnew, NEED_URL);
+       WebcitAddUrlHandler(HKEY("readold"), "", 0, h_readold, NEED_URL);
+       WebcitAddUrlHandler(HKEY("readfwd"), "", 0, h_readfwd, NEED_URL);
+       WebcitAddUrlHandler(HKEY("headers"), "", 0, h_headers, NEED_URL);
+       WebcitAddUrlHandler(HKEY("readgt"), "", 0, h_readgt, NEED_URL);
+       WebcitAddUrlHandler(HKEY("readlt"), "", 0, h_readlt, NEED_URL);
+       WebcitAddUrlHandler(HKEY("do_search"), "", 0, h_do_search, 0);
+       WebcitAddUrlHandler(HKEY("display_enter"), "", 0, display_enter, 0);
+       WebcitAddUrlHandler(HKEY("post"), "", 0, post_message, 0);
+       WebcitAddUrlHandler(HKEY("move_msg"), "", 0, move_msg, 0);
+       WebcitAddUrlHandler(HKEY("delete_msg"), "", 0, delete_msg, 0);
+       WebcitAddUrlHandler(HKEY("confirm_move_msg"), "", 0, confirm_move_msg, 0);
+       WebcitAddUrlHandler(HKEY("msg"), "", 0, embed_message, NEED_URL);
+       WebcitAddUrlHandler(HKEY("message"), "", 0, handle_one_message, NEED_URL|XHTTP_COMMANDS|COOKIEUNNEEDED|FORCE_SESSIONCLOSE);
+       WebcitAddUrlHandler(HKEY("printmsg"), "", 0, print_message, NEED_URL);
+       WebcitAddUrlHandler(HKEY("mobilemsg"), "", 0, mobile_message_view, NEED_URL);
+       WebcitAddUrlHandler(HKEY("msgheaders"), "", 0, display_headers, NEED_URL);
+
+       WebcitAddUrlHandler(HKEY("mimepart"), "", 0, view_mimepart, NEED_URL);
+       WebcitAddUrlHandler(HKEY("mimepart_download"), "", 0, download_mimepart, NEED_URL);
+       WebcitAddUrlHandler(HKEY("postpart"), "", 0, view_postpart, NEED_URL);
+       WebcitAddUrlHandler(HKEY("postpart_download"), "", 0, download_postpart, NEED_URL);
 
        /* json */
-       WebcitAddUrlHandler(HKEY("roommsgs"), jsonMessageList,0);
+       WebcitAddUrlHandler(HKEY("roommsgs"), "", 0, jsonMessageList,0);
        return ;
 }