Clean up board.js syntax

static-data/www/board/board.js

@@ -17,10 +17,6 @@
     along with this program.  If not, see <https://www.gnu.org/licenses/>.
 */
 
-requested = []
-
-var windowHeight = window.innerHeight;
-webpassword = webpass
 newPostForm = document.getElementById('addMsg')
 firstLoad = true
 lastLoadedBoard = 'global'
@@ -30,8 +26,6 @@ loadingTimeout = 8000
 
 let toggleLoadingMessage = function(){
     switch (loadingMessage.style.display){
-        case "block":
-        case "inline":
         case "inline-block":
             loadingMessage.style.display = "none"
             break;
@@ -45,37 +39,36 @@ fetch('/flow/version', {
     method: 'GET',
     headers: {
       "token": webpass
-    }})
+}})
-.then((resp) => resp.text())
+.then((ver) => ver.text())
-.then(function(data) {
+.then(function(ver) {
-    document.getElementById('circlesVersion').innerText = data
+    document.getElementById('circlesVersion').innerText = ver
 })
 
-function appendMessages(msg, blockHash, beforeHash, channel){
+function appendMessages(msg, blockHash, beforeHash, channel) {
-    if (channel !== document.getElementById('feedIDInput').value){return}
+    if (channel !== document.getElementById('feedIDInput').value) return // ignore if channel name isn't matching
+    if (msg.length == 0) return // ignore empty messages
+
     var humanDate = new Date(0)
-    if (msg.length == 0){
-        return
-    }
-    //var msg = JSON.parse(msg)
-    var el = document.createElement('div')
     var msgDate = msg['meta']['time']
     var feed = document.getElementById("feed")
     var beforeEl = null
 
     if (msgDate === undefined){
         msgDate = 'unknown'
-    }
+    } else {
-    else{
         humanDate.setUTCSeconds(msgDate)
         msgDate = humanDate.toLocaleTimeString() + ' ' + humanDate.toLocaleDateString()
     }
+
+    var el = document.createElement('div')
     el.className = 'entry'
     el.innerText = msg['content']
-    if (beforeHash !== null){
+
-        for (x = 0; x < feed.children.length; x++){
+    if (beforeHash !== null) {
-            if (feed.children[x].getAttribute('data-bl') === beforeHash){
+        for (i = 0; i < feed.children.length; i++) {
-                beforeEl = feed.children[x]
+            if (feed.children[i].getAttribute('data-bl') === beforeHash) {
+                beforeEl = feed.children[i]
             }
         }
     }
@@ -133,12 +126,13 @@ function getBlocks(){
     var feed = document.getElementById("feed")
     var ch = document.getElementById('feedIDInput').value
     if (lastLoadedBoard !== ch){
+        requested = []
+    
         toggleLoadingMessage()
         loadedAny = false
-        while (feed.firstChild) {
+
-            feed.removeChild(feed.firstChild);
+        while (feed.firstChild) feed.removeChild(feed.firstChild); // remove all messages from feed
-        }
+
-        requested = [] // reset requested list
         setTimeout(function(){
             if (! loadedAny && ch == document.getElementById('feedIDInput').value){
                 PNotify.notice("There are no posts for " + ch + ". You can be the first!")
@@ -156,13 +150,12 @@ function getBlocks(){
     var blockList = feedText.split(',')
 
     for (i = 0; i < blockList.length; i++){
-        while (blockList[i].length < 64) { blockList[i] = "0" + blockList[i] }
+        blockList[i] = "0".repeat(64 - blockList[i].length) + blockList[i] // pad hash with zeroes
+
         if (! requested.includes(blockList[i])){
-            if (blockList[i].length == 0){
+            if (blockList[i].length == 0) continue
-                continue
+            else requested.push(blockList[i])
-            }
+            loadMessage(blockList[i], blockList, i, ch);
-            requested.push(blockList[i])
-            loadMessage(blockList[i], blockList, i, ch)
         }
     }
 }
@@ -179,8 +172,7 @@ function loadMessage(blockHash, blockList, count, channel){
         let delay = 2000
         if (typeof before == "undefined"){
             before = null
-        }
+        } else {
-        else{
             let existing = document.getElementsByClassName('cMsgBox')
             for (x = 0; x < existing.length; x++){
                 if (existing[x].getAttribute('data-bl') === before){
@@ -193,7 +185,7 @@ function loadMessage(blockHash, blockList, count, channel){
     })
 }
 
-document.getElementById('refreshFeed').onclick = function(){
+document.getElementById('refreshFeed').onclick = function() {
     getBlocks()
 }
 
@@ -211,7 +203,8 @@ newPostForm.onsubmit = function(){
         headers: {
           "content-type": "application/json",
           "token": webpass
-        }})
+        }
+    })
     .then((resp) => resp.text()) // Transform the data into json
     .then(function(data) {
         newPostForm.style.display = 'block'