Got it! Temporarily working with the server-side C renderer
[citadel.git] / webcit-ng / static / js / views.js
index bb8f88e69a959f21722f5289c4725c7077b9b24b..171fdf93576dc4b3fe54d5c336db5249ef0b6121 100644 (file)
@@ -30,15 +30,15 @@ var views = {
 
 
 // This function is the dispatcher that determines the correct view for a room,
-// and calls the correct renderer.
+// and calls the correct renderer.  Greater/Less than bounds are accepted.
 //
-function render_room_view(min_msg, max_msg)
+function render_room_view(gt_msg, lt_msg)
 {
        switch(current_view)
        {
                case views.VIEW_MAILBOX:                                                // FIXME view mail rooms as forums for now
                case views.VIEW_BBS:
-                       forum_readmessages("ctdl-main", min_msg, max_msg);
+                       forum_readmessages("ctdl-main", gt_msg, lt_msg);
                        break;
                default:
                        document.getElementById("ctdl-main").innerHTML = "The view for " + current_room + " is " + current_view + " but there is no renderer." ;
@@ -50,21 +50,21 @@ function render_room_view(min_msg, max_msg)
 
 // Forum view (flat) -- let's have another go at this with the rendering done client-side
 //
-function forum_readmessages(target_div, min_msg, max_msg)
+function forum_readmessages(target_div, gt_msg, lt_msg)
 {
-       var innerdivname = randomString(5);
-       document.getElementById(target_div).innerHTML = "<div id=\"" + innerdivname +
-               "\"><br><br><br><center><h5><i class=\"fas fa-spinner fa-spin\"></i>&nbsp;&nbsp;"
-               + _("Loading messages from server, please wait") + "</h5></center></div>" ;
+       original_text = document.getElementById(target_div).innerHTML;          // in case we need to replace it after an error
+       document.getElementById(target_div).innerHTML = 
+               "<i class=\"fas fa-spinner fa-spin\"></i>&nbsp;&nbsp;"
+               + _("Loading messages from server, please wait") ;
 
        var request = new XMLHttpRequest();
-       if (max_msg < 9999999999)
+       if (lt_msg < 9999999999)
        {
-               request.open("GET", "/ctdl/r/" + escapeHTMLURI(current_room) + "/msgs.lt|" + max_msg, true);
+               request.open("GET", "/ctdl/r/" + escapeHTMLURI(current_room) + "/msgs.lt|" + lt_msg, true);
        }
        else
        {
-               request.open("GET", "/ctdl/r/" + escapeHTMLURI(current_room) + "/msgs.gt|" + min_msg, true);
+               request.open("GET", "/ctdl/r/" + escapeHTMLURI(current_room) + "/msgs.gt|" + gt_msg, true);
        }
        request.onreadystatechange = function()
        {
@@ -73,59 +73,104 @@ function forum_readmessages(target_div, min_msg, max_msg)
                        if ((this.status / 100) == 2)
                        {
                                msgs = JSON.parse(this.responseText);
-
-                               document.getElementById(innerdivname).innerHTML = "" ;
-                                       //"Are we logged in? " + logged_in + "<br>"
-                                       //+ "Last seen: " + last_seen + "<br>"
-                                       //+ "Number of messages: " + msgs.length + "<br>"
-                                       //+ "min_msg=" + min_msg + "<br>"
-                                       //+ "max_msg=" + max_msg + "<br>" ;
-
-                               //if (msgs.length == 0)
-                               //{
-                                               //document.getElementById(innerdivname).innerHTML += "FIXME no msgs" ;
-                               //}
+                               document.getElementById(target_div).innerHTML = "" ;
 
                                // If we were given an explicit starting point, by all means start there.
                                // Note that we don't have to remove them from the array because we did a 'msgs gt|xxx' command to Citadel.
-                               if (min_msg > 0)
+                               if (gt_msg > 0)
                                {
-                                       msgs = msgs.slice(0, msgs_per_page);
+                                       msgs = msgs.slice(0, messages_per_page);
                                }
 
                                // show us the last 20 messages and scroll to the bottom (this will become the not-logged-in behavior)
-                               else if ((logged_in) | (!logged_in) | (max_msg < 9999999999))
+                               else if ((logged_in) | (!logged_in) | (lt_msg < 9999999999))
                                {
                                        if (msgs.length > messages_per_page)
                                        {
                                                msgs = msgs.slice(msgs.length - messages_per_page);
                                        }
-                                       document.getElementById(innerdivname).innerHTML +=
-                                               "<div id=\"older_msgs\">" +
-                                               "<a href=\"javascript:forum_readmessages('older_msgs', 0, " + msgs[0] + ");\">" +
-                                               "link to msgs less than " + msgs[0] + "</a></div>" ;
+                                       new_old_div_name = randomString(5);
+                                       if (msgs.length < 1)
+                                       {
+                                               newlt = lt_msg;
+                                       }
+                                       else
+                                       {
+                                               newlt = msgs[0];
+                                       }
+                                       document.getElementById(target_div).innerHTML +=
+                                               "<div id=\"" + new_old_div_name + "\">" +
+                                               "<a href=\"javascript:forum_readmessages('" + new_old_div_name + "', 0, " + newlt + ");\">" +
+                                               "link to msgs less than " + newlt + "</a></div>" ;
                                }
 
-                               // It's render time, bitchez!
+                               // Render the divs (we will fill them in later)
                                for (var i in msgs)
                                {
-                                       document.getElementById(innerdivname).innerHTML +=
-                                               "<div id=\"ctdl_msg_" + msgs[i] + "\">message #" + msgs[i] + "</div>" ;
+                                       document.getElementById(target_div).innerHTML += "<div id=\"ctdl_msg_" + msgs[i] + "\">#" + msgs[i] + "</div>" ;
                                }
-                               if (max_msg == 9999999999)
+                               if (lt_msg == 9999999999)
                                {
-                                       document.getElementById(innerdivname).innerHTML +=
-                                               "<div id=\"newer_msgs\">" +
-                                               "<a href=\"javascript:forum_readmessages('newer_msgs', msgs[msgs.length-1]+1, 9999999999);\">" +
-                                               "link to msgs greater than " + msgs[msgs.length-1] + "</a></div>" ;
+                                       new_new_div_name = randomString(5);
+                                       if (msgs.length <= 0)
+                                       {
+                                               newgt = gt_msg;
+                                       }
+                                       else
+                                       {
+                                               newgt = msgs[msgs.length-1];
+                                       }
+                                       document.getElementById(target_div).innerHTML +=
+                                               "<div id=\"" + new_new_div_name + "\">" +
+                                               "<a href=\"javascript:forum_readmessages('" + new_new_div_name + "', " + newgt + ", 9999999999);\">" +
+                                               "link to msgs greater than " + newgt + "</a></div>" ;
                                }
+
+                               // Render the individual messages in the divs
+                               render_messages(msgs, "ctdl_msg_", views.VIEW_BBS);
+                       }
+                       else
+                       {
+                               document.getElementById(target_div).innerHTML = original_text;          // this will make the link reappear so the user can try again
+                       }
+               }
+       };
+       request.send();
+       request = null;
+}
+
+
+// Render a range of messages, in the view specified, with the div prefix specified
+//
+function render_messages(msgs, prefix, view)
+{
+       for (i=0; i<msgs.length; ++i)
+       {
+               render_one(prefix+msgs[i], msgs[i], view);
+       }
+}
+
+
+// We have to put each XHR for render_messages() into its own stack frame, otherwise it jumbles them together.  I don't know why.
+function render_one(div, msgnum, view)
+{
+       var request = new XMLHttpRequest();
+       request.open("GET", "/ctdl/r/" + escapeHTMLURI(current_room) + "/" + msgs[i] + "/html", true);
+       request.onreadystatechange = function()
+       {
+               if (this.readyState === 4)
+               {
+                       if ((this.status / 100) == 2)
+                       {
+                               document.getElementById(div).innerHTML = this.responseText;     // FIXME don't let the C server render it.  do JSON now.
                        }
                        else
                        {
-                               document.getElementById(innerdivname).innerHTML = this.status ;         // error message
+                               document.getElementById(div).innerHTML = "ERROR";
                        }
                }
        };
        request.send();
        request = null;
 }
+