From 16d47d3dadc6a6f59b9193a45f4d64d9225dea65 Mon Sep 17 00:00:00 2001 From: Daniel Markstedt Date: Mon, 25 Sep 2023 21:16:29 -0700 Subject: [PATCH] Web UI: Create a system admin page and move options there --- .../static/themes/modern/icons/command.svg | 1 + python/web/src/static/themes/modern/style.css | 21 ++ python/web/src/templates/admin.html | 186 ++++++++++++++++++ python/web/src/templates/base.html | 47 ----- python/web/src/templates/index.html | 104 +--------- python/web/src/web.py | 29 ++- 6 files changed, 231 insertions(+), 157 deletions(-) create mode 100644 python/web/src/static/themes/modern/icons/command.svg create mode 100644 python/web/src/templates/admin.html diff --git a/python/web/src/static/themes/modern/icons/command.svg b/python/web/src/static/themes/modern/icons/command.svg new file mode 100644 index 0000000000..d2778fa6f2 --- /dev/null +++ b/python/web/src/static/themes/modern/icons/command.svg @@ -0,0 +1 @@ + diff --git a/python/web/src/static/themes/modern/style.css b/python/web/src/static/themes/modern/style.css index a21a91ed96..849b6f3b55 100644 --- a/python/web/src/static/themes/modern/style.css +++ b/python/web/src/static/themes/modern/style.css @@ -954,6 +954,27 @@ section#manual a p { margin: 0; } +/* + ------------------------------------------------------------------------------ + Index > Section: Admin + ------------------------------------------------------------------------------ + */ +section#admin { + margin: 2rem 0 1rem; +} + +section#admin a { + margin: auto; + display: block; + padding: 0.25rem 0 0.25rem 2rem; + background: url("icons/command.svg") no-repeat left center; + font-weight: bold; +} + +section#admin a p { + margin: 0; +} + /* ------------------------------------------------------------------------------ Drives page diff --git a/python/web/src/templates/admin.html b/python/web/src/templates/admin.html new file mode 100644 index 0000000000..87a633cc7d --- /dev/null +++ b/python/web/src/templates/admin.html @@ -0,0 +1,186 @@ +{% extends "base.html" %} +{% block content %} + +
+
+ + {{ _("Logging") }} + +
    +
  • {{ _("The current dropdown selection indicates the active log level.") }}
    • +
+
+ +
+
+ + + + + +
+
+ +
+
+ + + +
+
+
+ +
+ +
+
+ + {{ _("Appearance") }} + +
    +
  • {{ _("Theme and language are auto-detected for your user agent. Here you can change the default.") }}
    • +
  • {{ _("The System Name is the \"pretty\" hostname if set, with a fallback to the regular hostname.") }}
    • +
+
+ +
+
+ {{ _("The current theme is %(theme)s.", theme=current_theme) }} + {% if current_theme == "classic" %} + {{ _('Switch to the %(theme)s theme', theme="modern") }} + {% else %} + {{ _('Switch to the %(theme)s theme', theme="classic") }} + {% endif %} +
+ +
+ + + +
+
+ +
+
+ + + +
+
+ + +
+
+
+ +
+ +
+
+ + {{ _("Companion Services") }} + +
    +
  • {{ _("If you want to add a service, run the easyinstall.sh script and choose the one to install.") }}
    • +
  • {{ _("In order to manage the services in the Web UI, you may install Webmin as well.") }}
    • +
+
+
+ +
+ +
+
+ + {{ _("System Operations") }} + +
    +
  • {{ _("IMPORTANT: Always shut down the system before turning off the power. Failing to do so may lead to data loss.") }}
    • +
+
+ +
+
+ +
+
+ +
+
+
+ +

{{ _("Go to Home") }}

+ +{% endblock content %} diff --git a/python/web/src/templates/base.html b/python/web/src/templates/base.html index 38cbe09ba1..258bd4405b 100644 --- a/python/web/src/templates/base.html +++ b/python/web/src/templates/base.html @@ -104,53 +104,6 @@

{% block content %}{% endblock content %}
-
- {% if current_theme == "classic" %} - {{ _('Switch to the %(theme)s theme', theme="modern") }} - {% else %} - {{ _('Switch to the %(theme)s theme', theme="classic") }} - {% endif %} -
- -
- {% if env["netatalk_configured"] %} - {{ _("Mac AFP file sharing is enabled.") }} - {% if env["webmin_configured"] %} - - {{ ("Server administration") }} - - {% endif %} - {% else %} - {{ _("Mac AFP file sharing is disabled.") }} - {% endif %} -
-
- {% if env["samba_configured"] %} - {{ _("Windows SMB file sharing is enabled.") }} - {% if env["webmin_configured"] %} - - {{ ("Server administration") }} - - {% endif %} - {% else %} - {{ _("Windows SMB file sharing is disabled.") }} - {% endif %} -
-
- {% if env["ftp_configured"] %} - {{ _("FTP file sharing is enabled.") }} - {% else %} - {{ _("FTP file sharing is disabled.") }} - {% endif %} -
-
- {% if env["macproxy_configured"] %} - {{ _("Macproxy is running at %(ip_addr)s (default port 5000)", ip_addr=env['ip_addr']) }} - {% else %} - {{ _("Macproxy is disabled.") }} - {% endif %} -
-
{{ _("PiSCSI software version:") }} {{ env["version"] }}
diff --git a/python/web/src/templates/index.html b/python/web/src/templates/index.html index fb3a5b325b..cfec5b300c 100644 --- a/python/web/src/templates/index.html +++ b/python/web/src/templates/index.html @@ -568,7 +568,6 @@
-
@@ -669,109 +668,10 @@
-
- - {{ _("Logging") }} - -
    -
  • {{ _("The current dropdown selection indicates the active log level.") }}
    • -
-
- -
-
- - - - - -
-
- -
-
- - - -
-
+
+

{{ _("System Administration") }}
-
- -
-
- - {{ _("System Operations") }} - -
    -
  • {{ _("The System Name is the \"pretty\" hostname if set, with a fallback to the regular hostname.") }}
    • -
  • {{ _("IMPORTANT: Always shut down the system before turning off the power. Failing to do so may lead to data loss.") }}
    • -
-
- -
-
- - - -
-
-
-
- - - -
-
- - -
-
-
-
- -
-
- -
-
-
- -
-

{{ _("Read the PiSCSI Manual") }}
diff --git a/python/web/src/web.py b/python/web/src/web.py index 6829b3d427..63db290c7a 100644 --- a/python/web/src/web.py +++ b/python/web/src/web.py @@ -105,11 +105,6 @@ def get_env_info(): "version": server_info["version"], "image_dir": server_info["image_dir"], "image_root_dir": Path(server_info["image_dir"]).name, - "netatalk_configured": sys_cmd.running_proc("afpd"), - "samba_configured": sys_cmd.running_proc("smbd"), - "ftp_configured": sys_cmd.running_proc("vsftpd"), - "macproxy_configured": sys_cmd.running_proc("macproxy"), - "webmin_configured": sys_cmd.running_proc("miniserv.pl"), "cd_suffixes": tuple(server_info["sccd"]), "rm_suffixes": tuple(server_info["scrm"]), "mo_suffixes": tuple(server_info["scmo"]), @@ -263,7 +258,6 @@ def index(): return response( template="index.html", page_title=_("PiSCSI Control Page"), - locales=get_supported_locales(), netinfo=piscsi_cmd.get_network_info(), bridge_configured=sys_cmd.is_bridge_setup(), devices=formatted_devices, @@ -273,8 +267,6 @@ def index(): config_files=config_files, device_types=device_types, scan_depth=server_info["scan_depth"], - log_levels=server_info["log_levels"], - current_log_level=server_info["current_log_level"], scsi_ids=scsi_ids, units=units, reserved_scsi_ids=reserved_scsi_ids, @@ -320,6 +312,27 @@ def drive_list(): ) +@APP.route("/sys/admin", methods=["GET"]) +def admin(): + """ + Sets up the data structures and kicks off the rendering of the system admin page + """ + server_info = piscsi_cmd.get_server_info() + + return response( + template="admin.html", + page_title=_("PiSCSI System Administration"), + log_levels=server_info["log_levels"], + current_log_level=server_info["current_log_level"], + locales=get_supported_locales(), + netatalk_configured=sys_cmd.running_proc("afpd"), + samba_configured=sys_cmd.running_proc("smbd"), + ftp_configured=sys_cmd.running_proc("vsftpd"), + macproxy_configured=sys_cmd.running_proc("macproxy"), + webmin_configured=sys_cmd.running_proc("miniserv.pl"), + ) + + @APP.route("/upload", methods=["GET"]) def upload_page(): """