From 40a03e4cf5ab30a3c50d68ffa50542963c4faeb1 Mon Sep 17 00:00:00 2001
From: Luciano Iam <lucianito@gmail.com>
Date: Sat, 11 Apr 2020 17:31:34 +0200
Subject: [PATCH] WebSockets: update index page to use JS client lib, cleanup
 code

---
 share/web_surfaces/index.html    |  3 +-
 share/web_surfaces/index/main.js | 63 ++++++++++++--------------------
 2 files changed, 25 insertions(+), 41 deletions(-)

diff --git a/share/web_surfaces/index.html b/share/web_surfaces/index.html
index a01a46d417..2f32d9e64c 100644
--- a/share/web_surfaces/index.html
+++ b/share/web_surfaces/index.html
@@ -32,7 +32,6 @@
                 </div>
             </div>
         </div>
-        <script src="shared/ardour.js"></script>
-        <script src="index/main.js"></script>
+        <script type="module" src="index/main.js"></script>
     </body>
 </html>
diff --git a/share/web_surfaces/index/main.js b/share/web_surfaces/index/main.js
index 6c648653cb..2402074e82 100644
--- a/share/web_surfaces/index/main.js
+++ b/share/web_surfaces/index/main.js
@@ -16,48 +16,45 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
+import { Ardour } from '../shared/ardour.js';
+
 (() => {
 
-    const INDEX_RESOURCE = 'index.json';
-
-    async function fetchIndex (url) {
-        const response = await fetch(url);
-        
-        if (response.status == 200) {
-            return await response.json();
-        } else {
-            throw new Error(`HTTP response status ${response.status}`);
+    async function main () {
+        try {
+            const index = await new Ardour().getAvailableSurfaces();
+            printIndex(index);
+        } catch (err) {
+            printError(`Error loading index: ${err.message}`);
         }
-    }
 
-    function buildItem (groupPath, surface) {
-        const li = document.createElement('li');
-        li.innerHTML = `<li>
-                        <a href="${groupPath}/${surface.path}/">${surface.name}</a>
-                        <p>${surface.description}</p>
-                    </li>`;
-        return li;
+        document.getElementById('loading').style.display = 'none';
     }
 
     function printIndex (index) {
-        for (let group of index) {
-            const path = group['path'];
-            const span = document.querySelector(`#${path} span`);
-            span.innerHTML = group['diskPath'];
-
-            let surfaces = group.surfaces;
+        for (const group of index) {
+            const surfaces = group.surfaces;
+            const groupPath = group['path'];
+            const groupPathSpan = document.querySelector(`#${groupPath} span`);
+            
+            groupPathSpan.innerHTML = group['diskPath'];
 
             if (surfaces.length > 0) {
-                const ul = document.querySelector(`#${path} > ul`);
+                const ul = document.querySelector(`#${groupPath} > ul`);
                 surfaces.sort((a, b) => a.name.localeCompare(b.name));
                 
-                for (surface of surfaces) {
-                    ul.appendChild(buildItem(path, surface));
+                for (const surface of surfaces) {
+                    const li = document.createElement('li');
+                    li.innerHTML = `<li>
+                        <a href="${groupPath}/${surface.path}/">${surface.name}</a>
+                        <p>${surface.description}</p>
+                    </li>`;
+                    ul.appendChild(li);
                 }
             } else {
                 const p = document.createElement('p');
                 p.innerHTML = '<p>No surfaces found</p>';
-                document.getElementById(path).appendChild(p);
+                document.getElementById(groupPath).appendChild(p);
             }
         }
 
@@ -70,18 +67,6 @@
         error.style.display = 'inline';
     }
 
-    async function main () {
-        try {
-            const indexUrl = `${location.protocol}//${location.host}/${INDEX_RESOURCE}`;
-            const index = await fetchIndex (indexUrl);
-            printIndex(index);
-        } catch (err) {
-            printError(`Error loading ${INDEX_RESOURCE}: ${err.message}`);
-        }
-
-        document.getElementById('loading').style.display = 'none';
-    }
-
     main();
 
 })();