X-Git-Url: https://code.citadel.org/?a=blobdiff_plain;f=webcit-ng%2Fstatic%2Fjs%2Fviews.js;h=12642210f40a46b3b65b7ecd6e3bc32c4347e038;hb=d41e1a56e4f10994ea88d2dd3680f273c4e8eb8d;hp=bb8f88e69a959f21722f5289c4725c7077b9b24b;hpb=c2130254167cb73d7841fde1ba8536de3bac469a;p=citadel.git

diff --git a/webcit-ng/static/js/views.js b/webcit-ng/static/js/views.js
index bb8f88e69..12642210f 100644
--- a/webcit-ng/static/js/views.js
+++ b/webcit-ng/static/js/views.js
@@ -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,80 @@ 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(innerdivname).innerHTML = this.status ;		// error message
+				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 (var i in msgs)
+	{
+		document.getElementById(prefix + msgs[i]).innerHTML = "<b>Message " + msgs[i] + " got rendered!!!</b>";
+	}
+
+}