Change code to always explicitly handle arrays.

[mkws-moved-to-github.git] / tools / htdocs / mkws.js

diff --git a/tools/htdocs/mkws.js b/tools/htdocs/mkws.js
index 6a202f8..e41e165 100644 (file)
--- a/tools/htdocs/mkws.js
+++ b/tools/htdocs/mkws.js
@@ -564,7 +564,7 @@ function renderDetails(data, marker)
     if (sources.length == 0) {
        details += '<tr><td colspan="2">No sources for record!</td></tr>';
     } else if (sources.length == 1) {
-       details += renderField("Source", sources[0]);
+       details += renderField("Source", sources);
     } else {
        details += renderField("Sources", sources);
     }
@@ -585,26 +585,27 @@ function renderField(caption, data, data2, data3) {
        return "";
     }
 
-    if (caption == "URL") {
-       var tmp = "";
-       for (var i = 0; i < data.length; i++) {
-           var s = data[i];
-           if (i > 0)
-               tmp += ", ";
-           tmp += '<a href="' + s + '" target="_blank">' + s + '</a>';
-       }
-       data = tmp;
+    var res = "";
+    for (var i = 0; i < data.length; i++) {
+       var s = data[i];
+       if (i > 0)
+           res += ", ";
+
+       if (caption == "URL")
+           s = '<a href="' + s + '" target="_blank">' + s + '</a>';
+
+       res += s
     }
 
     if (data2 != undefined) {
-       data = data + " (" + data2 + ")";
+       res = res + " (" + data2 + ")";
     }
 
     if (data3 != undefined) {
-       data = data + " <i>" + data3 + "</i>";
+       res = res + " <i>" + data3 + "</i>";
     }
 
-    return '<tr><th>' + M(caption) + '</th><td>' + data + '</td></tr>';
+    return '<tr><th>' + M(caption) + '</th><td>' + res + '</td></tr>';
 }