* New new BBS view
[citadel.git] / webcit / bbsview_renderer.c
index 0ae07c0ee5fd98b0c07eb62510b6e7fd1f807a84..45b67fb6ec75fdbdd8880ea237aa6fdcb5dfc604 100644 (file)
@@ -1,9 +1,13 @@
 /* 
- * $Id$
+ * $Id$
  *
  * BBS View renderer module for WebCit
  *
- * Copyright (c) 1996-2009 by the citadel.org team
+ * Note: we briefly had a dynamic UI for this.  I thought it was cool, but
+ * it was not received well by the user community.  If you want to play
+ * with it, go get r8256 of bbsview_renderer.c and have fun.
+ *
+ * Copyright (c) 1996-2010 by the citadel.org team
  *
  *  This program is free software; you can redistribute it and/or modify
  *  it under the terms of the GNU General Public License as published by
  *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
  */
 
+#define RANGE 5
+
 #include "webcit.h"
 #include "webserver.h"
 #include "groupdav.h"
 
 /*
  * Data which gets passed around between the various functions in this module
+ *
+ * We do this weird "pivot point" thing instead of starting the page numbers at 0 or 1 so that
+ * the border between old and new messages always falls on a page boundary.  We'll renumber them
+ * to page numbers starting at 1 when presenting them to the user.
  */
 struct bbsview {
        long *msgs;             /* Array of msgnums for messages we are displaying */
        int num_msgs;           /* Number of msgnums stored in 'msgs' */
        int alloc_msgs;         /* Currently allocated size of array */
+       long pivot_msgnum;      /* Page numbers are relative to this message number */
+       int requested_page;     /* Which page number did the user request? */
 };
 
 
+/*
+ * Attempt to determine the closest thing to the "last seen message number" using the
+ * results of the GTSN command
+ */
+long bbsview_get_last_seen(void)
+{
+       char buf[SIZ] = "0";
+
+       serv_puts("GTSN");
+       serv_getln(buf, sizeof buf);
+       if (buf[0] == '2') {
+
+               char *comma_pos = strchr(buf, ',');     /* kill first comma and everything to its right */
+               if (comma_pos) {
+                       *comma_pos = 0;
+               }
+
+               char *colon_pos = strchr(buf, ':');     /* kill first colon and everything to its left */
+               if (colon_pos) {
+                       strcpy(buf, ++colon_pos);
+               }
+       }
+
+       return(atol(buf));
+}
+
+
+
 /*
  * Entry point for message read operations.
  */
@@ -47,42 +87,33 @@ int bbsview_GetParamsGetServerCall(SharedMessageStatus *Stat,
        memset(BBS, 0, sizeof(struct bbsview));
        *ViewSpecific = BBS;
 
-       Stat->defaultsortorder = 1;
-       Stat->startmsg = -1;
-       Stat->sortit = 1;
-       
-       rlid[oper].cmd(cmd, len);               /* this performs the server call to fetch the msg list */
-       
-       if (havebstr("maxmsgs")) {
-               Stat->maxmsgs = ibstr("maxmsgs");
+       Stat->startmsg = -1;                                    /* not used here */
+       Stat->sortit = 1;                                       /* not used here */
+       Stat->num_displayed = DEFAULT_MAXMSGS;                  /* not used here */
+       BBS->requested_page = 0;
+
+       if (havebstr("page")) {
+               BBS->requested_page = ibstr("page");
        }
-       if (Stat->maxmsgs == 0) Stat->maxmsgs = DEFAULT_MAXMSGS;
        
-       if (havebstr("startmsg")) {
-               Stat->startmsg = lbstr("startmsg");
+       if (havebstr("pivot")) {
+               BBS->pivot_msgnum = ibstr("pivot");
        }
-       if (lbstr("SortOrder") == 2) {
-               Stat->reverse = 1;
-               Stat->num_displayed = -DEFAULT_MAXMSGS;
+       else if (oper == 2) {   /* 2 == "read all" (otherwise we pivot at the beginning of new msgs) */
+               BBS->pivot_msgnum = 0;                          /* start from the top */
        }
        else {
-               Stat->reverse = 0;
-               Stat->num_displayed = DEFAULT_MAXMSGS;
+               BBS->pivot_msgnum = bbsview_get_last_seen();
        }
 
-       return 200;
-}
-
-
-/*
- * begin_ajax_response() was moved from bbsview_LoadMsgFromServer() to here ...
- */
-int bbsview_PrintViewHeader(SharedMessageStatus *Stat, void **ViewSpecific)
-{
-       if (WC->is_ajax) {
-               begin_ajax_response();          /* for non-ajax, headers are output in messages.c */
+       if (havebstr("maxmsgs")) {
+               Stat->maxmsgs = ibstr("maxmsgs");
        }
-
+       if (Stat->maxmsgs == 0) Stat->maxmsgs = DEFAULT_MAXMSGS;
+       
+       /* perform a "read all" call to fetch the message list -- we'll cut it down later */
+       rlid[2].cmd(cmd, len);
+       
        return 200;
 }
 
@@ -99,11 +130,11 @@ int bbsview_LoadMsgFromServer(SharedMessageStatus *Stat,
        struct bbsview *BBS = (struct bbsview *) *ViewSpecific;
 
        if (BBS->alloc_msgs == 0) {
-               BBS->alloc_msgs = Stat->maxmsgs;
+               BBS->alloc_msgs = 1000;
                BBS->msgs = malloc(BBS->alloc_msgs * sizeof(long));
        }
 
-       /* Theoretically this never happens because the initial allocation == maxmsgs */
+       /* Check our buffer size */
        if (BBS->num_msgs >= BBS->alloc_msgs) {
                BBS->alloc_msgs *= 2;
                BBS->msgs = realloc(BBS->msgs, (BBS->alloc_msgs * sizeof(long)));
@@ -114,20 +145,8 @@ int bbsview_LoadMsgFromServer(SharedMessageStatus *Stat,
        return 200;
 }
 
-int bbsview_sortfunc_reverse(const void *s1, const void *s2) {
-       long l1;
-       long l2;
-
-       l1 = *(long *)(s1);
-       l2 = *(long *)(s2);
-
-       if (l1 > l2) return(-1);
-       if (l1 < l2) return(+1);
-       return(0);
-}
-
 
-int bbsview_sortfunc_forward(const void *s1, const void *s2) {
+int bbsview_sortfunc(const void *s1, const void *s2) {
        long l1;
        long l2;
 
@@ -146,164 +165,125 @@ int bbsview_RenderView_or_Tail(SharedMessageStatus *Stat,
 {
        struct bbsview *BBS = (struct bbsview *) *ViewSpecific;
        int i;
+       int seq;
        const StrBuf *Mime;
-       char olderdiv[64];
-       char newerdiv[64];
-       int doing_older_messages = 0;
-       int doing_newer_messages = 0;
-
-       snprintf(olderdiv, sizeof olderdiv, "olderdiv%08lx%08x", time(NULL), rand());
-       snprintf(newerdiv, sizeof newerdiv, "newerdiv%08lx%08x", time(NULL), rand());
-
-       /* If this is the initial page load (and not an update), supply the required JavaScript code */
-       if (!WC->is_ajax) {
-          StrBufAppendPrintf(WC->trailing_javascript,
-               "       function moremsgs(target_div, gt_or_lt, gt_or_lt_value, maxmsgs, sortorder) {   \n"
-               "               $(target_div).innerHTML = '<div class=\"moreprompt\">%s ... <img src=\"static/throbber.gif\"></div>';   \n"
-               "               p = gt_or_lt + '=' + gt_or_lt_value + '&maxmsgs=' + maxmsgs             \n"
-               "                       + '&is_summary=0&SortOrder=' + sortorder + '&is_ajax=1'         \n"
-               "                       + '&gt_or_lt=' + gt_or_lt                                       \n"
-               "                       + '&r=' + CtdlRandomString();                                   \n"
-               "               new Ajax.Updater(target_div, 'read' + gt_or_lt,                                 \n"
-               "                       { method: 'get', parameters: p, evalScripts: true } );          \n"
-               "       }                                                                               \n"
-               "",
-               _("Loading")
-          );
-       }
-
-
-       /* Determine whether we are in the middle of a 'click for older messages' or 'click for
-        * newer messages' operation.  If neither, then we are in the initial page load.
-        */
-       if (!strcasecmp(bstr("gt_or_lt"), "lt")) {
-               doing_older_messages = 1;
-               doing_newer_messages = 0;
-       }
-       else if (!strcasecmp(bstr("gt_or_lt"), "gt")) {
-               doing_older_messages = 0;
-               doing_newer_messages = 1;
-       }
-       else {
-               doing_older_messages = 0;
-               doing_newer_messages = 0;
-       }
-
+       int pivot_index = 0;
+       int page_offset = 0;
+       int start_index = 0;
+       int end_index = 0;
 
        /* Cut the message list down to the requested size */
        if (Stat->nummsgs > 0) {
                lprintf(9, "sorting %d messages\n", BBS->num_msgs);
-               qsort(BBS->msgs, (size_t)(BBS->num_msgs), sizeof(long), (Stat->reverse ? bbsview_sortfunc_reverse : bbsview_sortfunc_forward));
+               qsort(BBS->msgs, (size_t)(BBS->num_msgs), sizeof(long), bbsview_sortfunc);
 
                /* Cut it down to 20 messages (or whatever value Stat->maxmsgs is set to) */
 
                if (BBS->num_msgs > Stat->maxmsgs) {
 
-                       if (doing_older_messages) {
-                               /* LT ... cut it down to the LAST 20 messages received */
-                               memcpy(&BBS->msgs[0], &BBS->msgs[BBS->num_msgs - Stat->maxmsgs],
-                                       (Stat->maxmsgs * sizeof(long))
-                               );
-                               BBS->num_msgs = Stat->maxmsgs;
-                       }
-                       else {
-                               /* GT ... cut it down to the FIRST 20 messages received */
-                               BBS->num_msgs = Stat->maxmsgs;
+                       /* Locate the pivot point in our message index */
+                       for (i=0; i<(BBS->num_msgs); ++i) {
+                               if (BBS->msgs[i] <= BBS->pivot_msgnum) {
+                                       pivot_index = i;
+                               }
                        }
-               }
-       }
 
+                       page_offset = (pivot_index / Stat->maxmsgs) + 2;
 
-       /* Supply the link to prepend the previous 20 messages */
-
-       if (doing_newer_messages == 0) {
-               wc_printf("<div id=\"%s\">", olderdiv);
-               /* if (Stat->nummsgs > 0) { */
-               if (Stat->nummsgs > 0) {
-                       wc_printf("<a href=\"javascript:moremsgs('%s', 'lt', %ld, %ld, %d );\">",
-                               olderdiv,
-                               BBS->msgs[0],
-                               Stat->maxmsgs,
-                               (Stat->reverse ? 2 : 1)
-                       );
-               
-                       wc_printf("<div class=\"moreprompt\">"
-                               "&uarr; &uarr; &uarr; %s &uarr; &uarr; &uarr;"
-                               "</div>", _("older messages")
-                       );
-                       wc_printf("</a>");
                }
-               wc_printf("</div>");
        }
 
+       start_index = pivot_index + (BBS->requested_page * Stat->maxmsgs) ;
+       if (start_index < 0) start_index = 0;
+       end_index = start_index + Stat->maxmsgs - 1;
 
+       for (seq = 0; seq < 3; ++seq) {         /* cheap and sleazy way of rendering the funbar twice */
 
-       /* Handle the empty message set gracefully... */
-       if (Stat->nummsgs == 0) {
-               if (!WC->is_ajax) {
-                       wc_printf("<div class=\"nomsgs\"><br><em>");
-                       wc_printf(_("No messages here."));
-                       wc_printf("</em><br></div>\n");
-               }
-       }
+               if (seq == 1) {
+                       /* display the selected range of messages */
 
-       /* Non-empty message set... */
-       else {
-               /* Render the messages */
-       
-               for (i=0; i<BBS->num_msgs; ++i) {
-                       read_message(WC->WBuf, HKEY("view_message"), BBS->msgs[i], NULL, &Mime);
-               }
-
-       }
-
-
-       /* Supply the link to append the next 20 messages */
-
-       if (doing_older_messages == 0) {
-               wc_printf("<div id=\"%s\">", newerdiv);
-               /* if (Stat->nummsgs > 0) { */
-               if (Stat->nummsgs >= Stat->maxmsgs) {
-                       wc_printf("<a href=\"javascript:moremsgs('%s', 'gt', %ld, %ld, %d );\">",
-                               newerdiv,
-                               BBS->msgs[BBS->num_msgs-1],
-                               Stat->maxmsgs,
-                               (Stat->reverse ? 2 : 1)
-                       );
-               
-                       wc_printf("<div class=\"moreprompt\">"
-                               "&darr; &darr; &darr; %s &darr; &darr; &darr;"
-                               "</div>", _("newer messages")
-                       );
-                       wc_printf("</a>");
-               }
-               else {
-                       long gt = 0;    /* if new messages appear later, where will they begin? */
                        if (Stat->nummsgs > 0) {
-                               gt = BBS->msgs[BBS->num_msgs-1];
+                               wc_printf("<font size=\"-1\">\n");
+                               for (i=start_index; (i<=end_index && i<=BBS->num_msgs); ++i) {
+                                       if (BBS->msgs[i] > 0L) {
+                                               read_message(WC->WBuf, HKEY("view_message"), BBS->msgs[i], NULL, &Mime);
+                                       }
+                               }
+                               wc_printf("</font><br>\n");
                        }
-                       else {
-                               gt = atol(bstr("gt"));
-                       }
-                       wc_printf("<a href=\"javascript:moremsgs('%s', 'gt', %ld, %ld, %d );\">",
-                               newerdiv,
-                               gt,
-                               Stat->maxmsgs,
-                               (Stat->reverse ? 2 : 1)
-                       );
-                       wc_printf("<div class=\"moreprompt\">");
-                       wc_printf("%s", _("no more messages"));
-                       wc_printf("</div>");
-                       wc_printf("</a>");
                }
-               wc_printf("</div>");
-       }
+               else {
+                       /* Display the range selecto-bar */
 
-       /* Leave a little padding at the bottom, but only for the initial page load -- don't keep
-        * adding it every time we extend the visible message set.
-        */
-       if (!WC->is_ajax) {
-               wc_printf("<br><br><br><br>");
+                       wc_printf("<div class=\"moreprompt\">");
+                       wc_printf(_("Go to page: "));
+
+                       int first = 1;
+                       int last = (BBS->num_msgs / Stat->maxmsgs) + 2 ;
+
+                       for (i=1; i<=last; ++i) {
+
+                               if (
+                                       (i == first)
+                                       || (i == last)
+                                       || ((i - page_offset) == BBS->requested_page)
+                                       || (
+                                               ((BBS->requested_page - (i - page_offset)) < RANGE)
+                                               && ((BBS->requested_page - (i - page_offset)) > (0 - RANGE))
+                                       )
+                               ) {
+
+                                       if (
+                                               (i == last) 
+                                               && (last - (BBS->requested_page + page_offset) > RANGE)
+                                       ) {
+                                               wc_printf("...&nbsp;");
+                                       }
+                                       if ((i - page_offset) == BBS->requested_page) {
+                                               wc_printf("[");
+                                       }
+                                       else {
+                                               wc_printf("<a href=\"readfwd?pivot=%ld?page=%d\">",
+                                                       BBS->pivot_msgnum,
+                                                       i - page_offset
+                                               );
+                                               wc_printf("<span class=\"moreprompt_link\">");
+                                       }
+                                       if (
+                                               (i == first)
+                                               && ((BBS->requested_page + page_offset) > (RANGE + 1))
+                                       ) {
+                                               wc_printf(_("First"));
+                                       }
+                                       else if (
+                                               (i == last)
+                                               && (last - (BBS->requested_page + page_offset) > RANGE)
+                                       ) {
+                                               wc_printf(_("Last"));
+                                       }
+                                       else {
+                                               wc_printf("%d", i);
+                                       }
+                                       if ((i - page_offset) == BBS->requested_page) {
+                                               wc_printf("]");
+                                       }
+                                       else {
+                                               wc_printf("</span>");
+                                               wc_printf("</a>");
+                                       }
+                                       if (
+                                               (i == first)
+                                               && ((BBS->requested_page + page_offset) > (RANGE + 1))
+                                       ) {
+                                               wc_printf("&nbsp;...");
+                                       }
+                                       if (i != last) {
+                                               wc_printf("&nbsp;");
+                                       }
+                               }
+                       }
+                       wc_printf("</div>\n");
+               }
        }
 
        return(0);
@@ -319,16 +299,11 @@ int bbsview_Cleanup(void **ViewSpecific)
        }
        free(BBS);
 
-       if (WC->is_ajax) {
-               end_ajax_response();
-               WC->is_ajax = 0;
-       }
-       else {
-               wDumpContent(1);
-       }
+       wDumpContent(1);
        return 0;
 }
 
+
 void 
 InitModule_BBSVIEWRENDERERS
 (void)
@@ -336,8 +311,10 @@ InitModule_BBSVIEWRENDERERS
        RegisterReadLoopHandlerset(
                VIEW_BBS,
                bbsview_GetParamsGetServerCall,
-               bbsview_PrintViewHeader,
+               NULL,
+               NULL, 
                bbsview_LoadMsgFromServer,
                bbsview_RenderView_or_Tail,
-               bbsview_Cleanup);
+               bbsview_Cleanup
+       );
 }