From 5d715e4aac2c1192297d6f1ae1fff8c001948e24 Mon Sep 17 00:00:00 2001
From: Yasen Pramatarov <yasen@lindeas.com>
Date: Thu, 31 Oct 2024 11:08:53 +0200
Subject: [PATCH] Cleans up the html

---
 app/pages/config.php                     |  1 +
 app/templates/config-configfile-edit.php |  8 ++--
 app/templates/config-configfile.php      |  2 +-
 app/templates/config-host-add.php        | 11 ++---
 app/templates/config-host-delete.php     |  7 ++--
 app/templates/config-host-edit.php       | 52 ++++++++++++++++++++++++
 app/templates/config-host.php            | 21 ++++++----
 app/templates/config-platform-add.php    |  7 ++--
 app/templates/config-platform-delete.php | 10 ++---
 app/templates/config-platform-edit.php   | 10 ++---
 10 files changed, 94 insertions(+), 35 deletions(-)
 create mode 100644 app/templates/config-host-edit.php

diff --git a/app/pages/config.php b/app/pages/config.php
index b333ce2..c4415fd 100644
--- a/app/pages/config.php
+++ b/app/pages/config.php
@@ -159,6 +159,7 @@ if ($_SERVER['REQUEST_METHOD'] == 'POST') {
             if (isset($action) && $action === 'add') {
                 include '../app/templates/config-host-add.php';
             } elseif (isset($action) && $action === 'edit') {
+                $hostDetails = $hostObject->getHostDetails($platform_id, $agent);
                 include '../app/templates/config-host-edit.php';
             } elseif (isset($action) && $action === 'delete') {
                 $hostDetails = $hostObject->getHostDetails($platform_id, $agent);
diff --git a/app/templates/config-configfile-edit.php b/app/templates/config-configfile-edit.php
index eb67959..ce5264d 100644
--- a/app/templates/config-configfile-edit.php
+++ b/app/templates/config-configfile-edit.php
@@ -1,10 +1,9 @@
 
                 <!-- widget "config" -->
                 <div class="card text-center w-75 mx-lef">
-                    <p class="h4 card-header">Jilo configuration file</p>
+                    <p class="h4 card-header">Jilo configuration file :: edit</p>
                     <div class="card-body">
                         <div class="card-text">
-                            <p>editing the Jilo Server configuration file</p>
                             <p class="text-danger"><strong>this may break everything, use with extreme caution</strong></p>
                         </div>
                         <form method="POST" action="<?= htmlspecialchars($app_root) ?>?platform=<?= htmlspecialchars($platform_id) ?>&page=config&item=config_file">
@@ -16,8 +15,9 @@ echo "\n";
 ?>
 
                             <p class="text-danger"><strong>this may break everything, use with extreme caution</strong></p>
-                            <a class="btn btn-secondary" href="<?= htmlspecialchars($app_root) ?>?page=config&item=config_file" />Cancel</a>
-                            <input type="submit" class="btn btn-danger" value="Save" />
+                            <a class="btn btn-outline-secondary btn-sm" href="<?= htmlspecialchars($app_root) ?>?page=config&item=config_file" />Cancel</a>
+                            &nbsp;&nbsp;
+                            <input type="submit" class="btn btn-danger btn-sm" value="Save" />
                         </form>
 
 
diff --git a/app/templates/config-configfile.php b/app/templates/config-configfile.php
index d628d94..e61ec96 100644
--- a/app/templates/config-configfile.php
+++ b/app/templates/config-configfile.php
@@ -11,7 +11,7 @@ echo "\n";
 ?>
 
                         <br />
-                        <a class="btn btn-danger" href="<?= htmlspecialchars($app_root) ?>?page=config&item=config_file&action=edit" />Edit</a>
+                        <a class="btn btn-outline-danger btn-sm" href="<?= htmlspecialchars($app_root) ?>?page=config&item=config_file&action=edit" />Edit</a>
                     </div>
                 </div>
                 <!-- /widget "config" -->
diff --git a/app/templates/config-host-add.php b/app/templates/config-host-add.php
index e8354ce..8f220b2 100644
--- a/app/templates/config-host-add.php
+++ b/app/templates/config-host-add.php
@@ -1,6 +1,6 @@
 
-                <!-- widget "config" -->
-                <div class="card text-center w-50 mx-auto">
+                <!-- widget "hosts" -->
+                <div class="card text-center w-50 mx-lef">
                     <p class="h4 card-header">Add new host in Jitsi platform <strong><?= htmlspecialchars($platformDetails[0]['name']) ?></strong></p>
                     <div class="card-body">
                         <form method="POST" action="<?= htmlspecialchars($app_root) ?>?page=config&item=host">
@@ -41,9 +41,10 @@
                             <input type="hidden" name="new" value="true" />
 
                             <br />
-                            <a class="btn btn-secondary" href="<?= htmlspecialchars($app_root) ?>?page=config&item=host" />Cancel</a>
-                            <input type="submit" class="btn btn-primary" value="Save" />
+                            <a class="btn btn-outline-secondary btn-sm" href="<?= htmlspecialchars($app_root) ?>?page=config&item=host" />Cancel</a>
+                            &nbsp;&nbsp;
+                            <input type="submit" class="btn btn-primary btn-sm" value="Save" />
                         </form>
                     </div>
                 </div>
-                <!-- /widget "config" -->
+                <!-- /widget "hosts" -->
diff --git a/app/templates/config-host-delete.php b/app/templates/config-host-delete.php
index 8a84230..835b03a 100644
--- a/app/templates/config-host-delete.php
+++ b/app/templates/config-host-delete.php
@@ -1,6 +1,6 @@
 
                 <!-- widget "hosts" -->
-                <div class="card text-center w-50 mx-auto">
+                <div class="card text-center w-50 mx-lef">
                     <p class="h4 card-header">Jilo configuration for Jitsi platform <strong>"<?= htmlspecialchars($platformDetails[0]['name']) ?>"</strong></p>
                     <div class="card-body">
                         <p class="card-text">delete a host:</p>
@@ -24,8 +24,9 @@ foreach ($hostDetails[0] as $key => $value) {
                             <input type="hidden" name="delete" value="true" />
                             <p class="h5 text-danger">Are you sure you want to delete this host?</p>
                             <br />
-                            <a class="btn btn-secondary" href="<?= htmlspecialchars($app_root) ?>?page=config&item=host#platform<?= htmlspecialchars($platform_id) ?>host<?= htmlspecialchars($hostDetails[0]['id']) ?>" />Cancel</a>
-                            <input type="submit" class="btn btn-danger" value="Delete" />
+                            <a class="btn btn-outline-secondary btn-sm" href="<?= htmlspecialchars($app_root) ?>?page=config&item=host#platform<?= htmlspecialchars($platform_id) ?>host<?= htmlspecialchars($hostDetails[0]['id']) ?>" />Cancel</a>
+                            &nbsp;&nbsp;
+                            <input type="submit" class="btn btn-danger btn-sm" value="Delete" />
                         </form>
                     </div>
                 </div>
diff --git a/app/templates/config-host-edit.php b/app/templates/config-host-edit.php
new file mode 100644
index 0000000..c1fccbc
--- /dev/null
+++ b/app/templates/config-host-edit.php
@@ -0,0 +1,52 @@
+
+                <!-- widget "hosts" -->
+                <div class="card text-center w-50 mx-lef">
+                    <p class="h4 card-header">Jilo configuration for Jitsi platform <strong>"<?= htmlspecialchars($platformDetails[0]['name']) ?>"</strong></p>
+                    <div class="card-body">
+                        <p class="card-text">edit host details:</p>
+                        <form method="POST" action="<?= htmlspecialchars($app_root) ?>?page=config&item=host">
+
+                            <div class="row mb-3">
+                                <div class="col-md-4 text-end">
+                                    <label for="address" class="form-label">address</label>
+                                    <span class="text-danger" style="margin-right: -12px;">*</span>
+                                </div>
+                                <div class="col-md-8">
+                                    <input class="form-control" type="text" name="address" value="<?= htmlspecialchars($hostDetails[0]['address'] ?? '') ?>" required autofocus />
+                                    <p class="text-start"><small>DNS name or IP address of the machine</small></p>
+                                </div>
+                            </div>
+
+                            <div class="row mb-3">
+                                <div class="col-md-4 text-end">
+                                    <label for="port" class="form-label">port</label>
+                                    <span class="text-danger" style="margin-right: -12px;">*</span>
+                                </div>
+                                <div class="col-md-8">
+                                    <input class="form-control" type="text" name="port" value="<?= htmlspecialchars($hostDetails[0]['port'] ?? '') ?>" required />
+                                    <p class="text-start"><small>port on which the Jilo Agent is listening</small></p>
+                                </div>
+                            </div>
+
+                            <div class="row mb-3">
+                                <div class="col-md-4 text-end">
+                                    <label for="name" class="form-label">name</label>
+                                </div>
+                                <div class="col-md-8">
+                                    <input class="form-control" type="text" name="name" value="<?= htmlspecialchars($hostDetails[0]['name'] ?? '') ?>" />
+                                    <p class="text-start"><small>description or name of the host (optional)</small></p>
+                                </div>
+                            </div>
+
+                            <input type="hidden" name="platform" value="<?= htmlspecialchars($platform_id) ?>" />
+                            <input type="hidden" name="item" value="host" />
+                            <input type="hidden" name="host" value="<?= htmlspecialchars($hostDetails[0]['id']) ?>" />
+
+                            <br />
+                            <a class="btn btn-outline-secondary btn-sm" href="<?= htmlspecialchars($app_root) ?>?page=config&item=host#platform<?= htmlspecialchars($platform_id) ?>host<?= htmlspecialchars($hostDetails[0]['id']) ?>" />Cancel</a>
+                            &nbsp;&nbsp;
+                            <input type="submit" class="btn btn-primary btn-sm" value="Save" />
+                        </form>
+                    </div>
+                </div>
+                <!-- /widget "hosts" -->
diff --git a/app/templates/config-host.php b/app/templates/config-host.php
index 459c568..1c39565 100644
--- a/app/templates/config-host.php
+++ b/app/templates/config-host.php
@@ -13,18 +13,21 @@
                             <p class="text-start">
                                 platform <strong><?= htmlspecialchars($platform_array['name']) ?></strong>
                             </p>
-
+                            <ul class="text-start" style="padding-left: 50px;">
 <?php foreach ($hosts as $host_array) { ?>
-                            <a name="platform<?= htmlspecialchars($platform_array['id']) ?>host<?= htmlspecialchars($host_array['id']) ?>"></a>
-                            <p class="text-start" style="padding-left: 50px;">
-                                <?= htmlspecialchars($host_array['address']) ?>:<?= htmlspecialchars($host_array['port']) ?>
-                                &nbsp;
-                                <a class="btn btn-outline-secondary btn-sm" href="<?= htmlspecialchars($app_root) ?>?page=config&item=host&platform=<?= htmlspecialchars($host_array['platform_id']) ?>&host=<?= htmlspecialchars($host_array['id']) ?>&action=edit">edit host</a>
-                                <a class="btn btn-outline-danger btn-sm" href="<?= htmlspecialchars($app_root) ?>?page=config&item=host&platform=<?= htmlspecialchars($host_array['platform_id']) ?>&host=<?= htmlspecialchars($host_array['id']) ?>&action=delete">delete host</a>
-                            </p>
+                                <li style="padding-bottom: 10px;">
+                                    <a name="platform<?= htmlspecialchars($platform_array['id']) ?>host<?= htmlspecialchars($host_array['id']) ?>"></a>
+                                    <span>
+                                        <?= htmlspecialchars($host_array['address']) ?>:<?= htmlspecialchars($host_array['port']) ?>
+                                        &nbsp;
+                                        <a class="btn btn-outline-secondary btn-sm" href="<?= htmlspecialchars($app_root) ?>?page=config&item=host&platform=<?= htmlspecialchars($host_array['platform_id']) ?>&host=<?= htmlspecialchars($host_array['id']) ?>&action=edit">edit host</a>
+                                        <a class="btn btn-outline-danger btn-sm" href="<?= htmlspecialchars($app_root) ?>?page=config&item=host&platform=<?= htmlspecialchars($host_array['platform_id']) ?>&host=<?= htmlspecialchars($host_array['id']) ?>&action=delete">delete host</a>
+                                    </span>
+                                </li>
 <?php } ?>
+                            </ul>
                             <p class="text-start" style="padding-left: 50px;">
-                                total <?= htmlspecialchars(count($hosts)) ?> <?= htmlspecialchars(count($hosts)) === 1 ? 'jilo host' : 'jilo hosts' ?>&nbsp;
+                                total <?= htmlspecialchars(count($hosts)) ?> jilo <?= htmlspecialchars(count($hosts)) === '1' ? 'host' : 'hosts' ?>&nbsp;
                                 &nbsp;
                                 <a class="btn btn-outline-secondary btn-sm" href="<?= htmlspecialchars($app_root) ?>?page=config&item=host&platform=<?= htmlspecialchars($platform_array['id']) ?>&action=add">add new</a>
                             </p>
diff --git a/app/templates/config-platform-add.php b/app/templates/config-platform-add.php
index 2c18911..e9c23b9 100644
--- a/app/templates/config-platform-add.php
+++ b/app/templates/config-platform-add.php
@@ -1,6 +1,6 @@
 
                 <!-- widget "config" -->
-                <div class="card text-center w-50 mx-auto">
+                <div class="card text-center w-50 mx-lef">
                     <p class="h4 card-header">Add new Jitsi platform</p>
                     <div class="card-body">
                         <!--p class="card-text">add new platform:</p-->
@@ -42,8 +42,9 @@
                             <input type="hidden" name="new" value="true" />
 
                             <br />
-                            <a class="btn btn-secondary" href="<?= htmlspecialchars($app_root) ?>?page=config&item=platform" />Cancel</a>
-                            <input type="submit" class="btn btn-primary" value="Save" />
+                            <a class="btn btn-outline-secondary btn-sm" href="<?= htmlspecialchars($app_root) ?>?page=config&item=platform" />Cancel</a>
+                            &nbsp;&nbsp;
+                            <input type="submit" class="btn btn-primary btn-sm" value="Save" />
                         </form>
                     </div>
                 </div>
diff --git a/app/templates/config-platform-delete.php b/app/templates/config-platform-delete.php
index d672041..a0f3db7 100644
--- a/app/templates/config-platform-delete.php
+++ b/app/templates/config-platform-delete.php
@@ -1,9 +1,8 @@
 
                 <!-- widget "config" -->
-                <div class="card text-center w-50 mx-auto">
-                    <p class="h4 card-header">Jilo web configuration for Jitsi platform <strong>"<?= htmlspecialchars($platformDetails[0]['name']) ?>"</strong></p>
+                <div class="card text-center w-50 mx-lef">
+                    <p class="h4 card-header">Jilo configuration for Jitsi platform <strong>"<?= htmlspecialchars($platformDetails[0]['name']) ?>"</strong> :: delete</p>
                     <div class="card-body">
-                        <p class="card-text">delete a platform:</p>
                         <form method="POST" action="<?= htmlspecialchars($app_root) ?>?platform=<?= htmlspecialchars($platform_id) ?>&page=config">
 <?php
 foreach ($platformDetails[0] as $key => $value) {
@@ -24,8 +23,9 @@ foreach ($platformDetails[0] as $key => $value) {
                             <input type="hidden" name="delete" value="true" />
                             <p class="h5 text-danger">Are you sure you want to delete this platform?</p>
                             <br />
-                            <a class="btn btn-secondary" href="<?= htmlspecialchars($app_root) ?>?page=config&item=platform#platform<?= htmlspecialchars($platform_id) ?>" />Cancel</a>
-                            <input type="submit" class="btn btn-danger" value="Delete" />
+                            <a class="btn btn-outline-secondary btn-sm" href="<?= htmlspecialchars($app_root) ?>?page=config&item=platform#platform<?= htmlspecialchars($platform_id) ?>" />Cancel</a>
+                            &nbsp;&nbsp;
+                            <input type="submit" class="btn btn-danger btn-sm" value="Delete" />
                         </form>
                     </div>
                 </div>
diff --git a/app/templates/config-platform-edit.php b/app/templates/config-platform-edit.php
index 10f8054..5e5651f 100644
--- a/app/templates/config-platform-edit.php
+++ b/app/templates/config-platform-edit.php
@@ -1,9 +1,8 @@
 
                 <!-- widget "config" -->
-                <div class="card text-center w-50 mx-auto">
-                    <p class="h4 card-header">Jilo web configuration for Jitsi platform <strong>"<?= htmlspecialchars($platformDetails[0]['name']) ?>"</strong></p>
+                <div class="card text-center w-50 mx-lef">
+                    <p class="h4 card-header">Jilo configuration for Jitsi platform <strong>"<?= htmlspecialchars($platformDetails[0]['name']) ?>"</strong> :: edit</p>
                     <div class="card-body">
-                        <p class="card-text">edit the platform details:</p>
                         <form method="POST" action="<?= htmlspecialchars($app_root) ?>?platform=<?= htmlspecialchars($platform_id) ?>&page=config">
 <?php
 foreach ($platformDetails[0] as $key => $value) {
@@ -28,8 +27,9 @@ foreach ($platformDetails[0] as $key => $value) {
 <?php } ?>
                             <br />
                             <input type="hidden" name="platform" value="<?= htmlspecialchars($platform_id) ?>" />
-                            <a class="btn btn-secondary" href="<?= htmlspecialchars($app_root) ?>?page=config&item=platform#platform<?= htmlspecialchars($platform_id) ?>" />Cancel</a>
-                            <input type="submit" class="btn btn-primary" value="Save" />
+                            <a class="btn btn-outline-secondary btn-sm" href="<?= htmlspecialchars($app_root) ?>?page=config&item=platform#platform<?= htmlspecialchars($platform_id) ?>" />Cancel</a>
+                            &nbsp;&nbsp;
+                            <input type="submit" class="btn btn-primary btn-sm" value="Save" />
                         </form>
                     </div>
                 </div>