X-Git-Url: https://code.citadel.org/?a=blobdiff_plain;f=webcit%2Fbbsview_renderer.c;h=78fc554c225769e007120333df4803c0819891de;hb=e1d8e996c4de045ce050319bbfcbb6a801180d73;hp=4bcb1935df9cc362c671b833fc60a6635e05bf4b;hpb=eabf3658ecfa4cc952a553696e917808d1109c1c;p=citadel.git diff --git a/webcit/bbsview_renderer.c b/webcit/bbsview_renderer.c index 4bcb1935d..78fc554c2 100644 --- a/webcit/bbsview_renderer.c +++ b/webcit/bbsview_renderer.c @@ -375,21 +375,19 @@ int bbsview_LoadMsgFromServer(SharedMessageStatus *Stat, lprintf(9, "bbsview_LoadMsgFromServer() has been called.\n"); - if (BBS->num_msgs < Stat->maxmsgs) { + if (BBS->alloc_msgs == 0) { + BBS->alloc_msgs = Stat->maxmsgs; + BBS->msgs = malloc(BBS->alloc_msgs * sizeof(long)); + } - if (BBS->alloc_msgs == 0) { - BBS->alloc_msgs = Stat->maxmsgs; - BBS->msgs = malloc(BBS->alloc_msgs * sizeof(long)); - } - - /* Theoretically this never happens because the initial allocation == maxmsgs */ - if (BBS->num_msgs >= BBS->alloc_msgs) { - BBS->alloc_msgs *= 2; - BBS->msgs = realloc(BBS->msgs, (BBS->alloc_msgs * sizeof(long))); - } - - BBS->msgs[BBS->num_msgs++] = Msg->msgnum; + /* Theoretically this never happens because the initial allocation == maxmsgs */ + if (BBS->num_msgs >= BBS->alloc_msgs) { + BBS->alloc_msgs *= 2; + BBS->msgs = realloc(BBS->msgs, (BBS->alloc_msgs * sizeof(long))); } + + BBS->msgs[BBS->num_msgs++] = Msg->msgnum; + return 200; } @@ -426,10 +424,78 @@ int bbsview_RenderView_or_Tail(SharedMessageStatus *Stat, struct bbsview *BBS = (struct bbsview *) *ViewSpecific; int i; 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()); lprintf(9, "starting bbsview_RenderView_or_Tail() - there are %d messages.\n", BBS->num_msgs); + /* 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 = '
%s ...


'; \n" + " p = gt_or_lt + '=' + gt_or_lt_value + '&maxmsgs=' + maxmsgs \n" + " + '&is_summary=0&SortOrder=' + sortorder + '&is_ajax=1' \n" + " + '>_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; + lprintf(9, "\033[31m ** OLDER MESSAGES ** \033[0m\n"); + } + else if (!strcasecmp(bstr("gt_or_lt"), "gt")) { + doing_older_messages = 0; + doing_newer_messages = 1; + lprintf(9, "\033[32m ** NEWER MESSAGES ** \033[0m\n"); + } + else { + doing_older_messages = 0; + doing_newer_messages = 0; + lprintf(9, "\033[33m ** INITIAL PAGE LOAD ** \033[0m\n"); + } + + + /* Supply the link to prepend the previous 20 messages */ + + if (doing_newer_messages == 0) { + wc_printf("
", olderdiv); + /* if (Stat->nummsgs > 0) { */ + if (Stat->nummsgs > 0) { + wc_printf("", + olderdiv, + BBS->msgs[0], + Stat->maxmsgs, + (Stat->reverse ? 2 : 1) + ); + + wc_printf("
" + "↑ ↑ ↑ %s ↑ ↑ ↑" + "
", _("click here for older messages") + ); + wc_printf(""); + } + wc_printf("
"); + } + + + /* Handle the empty message set gracefully... */ if (Stat->nummsgs == 0) { if (!WC->is_ajax) { @@ -443,6 +509,25 @@ int bbsview_RenderView_or_Tail(SharedMessageStatus *Stat, else { 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)); + + /* 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; + } + } + + /* Now render them */ for (i=0; inum_msgs; ++i) { read_message(WC->WBuf, HKEY("view_message"), BBS->msgs[i], NULL, &Mime); @@ -450,59 +535,48 @@ int bbsview_RenderView_or_Tail(SharedMessageStatus *Stat, } - snprintf(newerdiv, sizeof newerdiv, "newerdiv%08lx%08x", time(NULL), rand()); - if (!WC->is_ajax) { /* only supply the script during the initial page load */ - StrBufAppendPrintf(WC->trailing_javascript, - " function moremsgs(target_div, subcmd_name, subcmd_value, maxmsgs, sortorder) { \n" - " $(target_div).innerHTML = '
%s ...


'; \n" - " p = subcmd_name + '=' + subcmd_value + '&maxmsgs=' + maxmsgs \n" - " + '&is_summary=0&SortOrder=' + sortorder + '&is_ajax=1' \n" - " + '&r=' + CtdlRandomString(); \n" - " new Ajax.Updater(target_div, 'readgt', \n" - " { method: 'get', parameters: p, evalScripts: true } ); \n" - " } \n" - "", - _("Loading") - ); - } + /* Supply the link to append the next 20 messages */ - wc_printf("
", newerdiv); - /* if (Stat->nummsgs > 0) { */ - if (Stat->nummsgs >= Stat->maxmsgs) { - wc_printf("", - newerdiv, - BBS->msgs[BBS->num_msgs-1], - Stat->maxmsgs, - (Stat->reverse ? 2 : 1) - ); - - wc_printf("
" - "↓ ↓ ↓ %s ↓ ↓ ↓" - "
", _("newer messages") - ); - wc_printf(""); - } - else { - long gt = 0; /* if new messages appear later, where will they begin? */ - if (Stat->nummsgs > 0) { - gt = BBS->msgs[BBS->num_msgs-1]; + if (doing_older_messages == 0) { + wc_printf(""); } - wc_printf("



"); + return(0); }