From edbb8286146920583087080809f6e904ca856cc9 Mon Sep 17 00:00:00 2001
From: Wojtek Kosior <koszko@koszko.org>
Date: Thu, 25 Aug 2022 10:18:33 +0200
Subject: [proxy] make package version info page also display other versions;
 fix&improve some aspects of package loading and dependency recomputing

---
 .../web_ui/templates/packages/index.html.jinja     | 12 +++++---
 .../templates/packages/show_single.html.jinja      | 36 ++++++++++++++--------
 .../packages/show_single_version.html.jinja        | 12 ++++----
 3 files changed, 37 insertions(+), 23 deletions(-)

(limited to 'src/hydrilla/proxy/web_ui/templates/packages')

diff --git a/src/hydrilla/proxy/web_ui/templates/packages/index.html.jinja b/src/hydrilla/proxy/web_ui/templates/packages/index.html.jinja
index bac774e..cbdd3d9 100644
--- a/src/hydrilla/proxy/web_ui/templates/packages/index.html.jinja
+++ b/src/hydrilla/proxy/web_ui/templates/packages/index.html.jinja
@@ -35,11 +35,15 @@ in a proprietary work, I am not going to enforce this in court.
 	  <div>
 	    {{ info.identifier }}
 	  </div>
-	  {% if info.active_version_ref is not none %}
+	  {%-
+	    if info.active_version is not none and
+	        info.active_version.active == ActiveStatus.REQUIRED
+	  %}
+	    {%- set ver      = info.active_version.info.version %}
+	    {%- set ver_str  = versions.version_string(ver) %}
+	    {%- set ver_desc = ver_str + '@' + info.active_version.info.repo %}
 	    <div class="small-print">
-	      {{ versions.version_string(info.active_version_info.version) }}
-	      @
-	      {{ info.active_version_info.repo }}
+	      {{ _('web_ui.packages.enabled_version_{}').format(ver_desc) }}
 	    </div>
 	  {% endif %}
 	</a>
diff --git a/src/hydrilla/proxy/web_ui/templates/packages/show_single.html.jinja b/src/hydrilla/proxy/web_ui/templates/packages/show_single.html.jinja
index 17f958c..e362fa9 100644
--- a/src/hydrilla/proxy/web_ui/templates/packages/show_single.html.jinja
+++ b/src/hydrilla/proxy/web_ui/templates/packages/show_single.html.jinja
@@ -27,22 +27,32 @@ in a proprietary work, I am not going to enforce this in court.
   {% include 'include/item_list_style.css.jinja' %}
 {% endblock %}
 {% block main %}
-  <h3>
-    {{
-      _('web_ui.packages.single.heading.name_{}')
-          .format(display_info.identifier)
-    }}
-  </h3>
-  TODO: add more info...
-  <h4>{{ _('web_ui.packages.single.version_list_heading') }}</h4>
+  {% block main_info %}
+    <h3>
+      {{
+        _('web_ui.packages.single.heading.name_{}')
+            .format(display_info.identifier)
+      }}
+    </h3>
+    TODO: add more info...
+  {% endblock %}
+  <h4>
+    {% if version_display_infos|length > 0 %}
+      {% block version_list_heading %}
+	{{ _('web_ui.packages.single.version_list_heading') }}
+      {% endblock %}
+    {% endif %}
+  </h4>
   <ul id="item_list">
     {%- for info in version_display_infos %}
-      {% if info.info.repo == '<local>' -%}
-	{%- set entry_classes = 'version-entry-local' -%}
-      {%- else -%}
-	{%- set entry_classes = '' -%}
+      {%- set entry_classes = [] %}
+      {% if info.is_local %}
+	{%- do entry_classes.append('version-entry-local') %}
       {%- endif %}
-      <li class="{{ entry_classes }}">
+      {%- if info.is_orphan %}
+	{%- do entry_classes.append('version-entry-orphan') %}
+      {%- endif %}
+      <li class="{{ entry_classes|join(' ') }}">
         {%-
           set href = url_for(
               '.show_package_version',
diff --git a/src/hydrilla/proxy/web_ui/templates/packages/show_single_version.html.jinja b/src/hydrilla/proxy/web_ui/templates/packages/show_single_version.html.jinja
index 1b46da7..9943521 100644
--- a/src/hydrilla/proxy/web_ui/templates/packages/show_single_version.html.jinja
+++ b/src/hydrilla/proxy/web_ui/templates/packages/show_single_version.html.jinja
@@ -19,12 +19,9 @@ I, Wojtek Kosior, thereby promise not to sue for violation of this
 file's licenses. Although I request that you do not make use this code
 in a proprietary work, I am not going to enforce this in court.
 #}
-{% extends "base.html.jinja" %}
+{% extends "packages/show_single.html.jinja" %}
 {% block title %} {{ _('web_ui.packages.single_version.title') }} {% endblock %}
-{% block style %}
-  {{ super() }}
-{% endblock %}
-{% block main %}
+{% block main_info %}
   {%- if uninstall_disallowed is defined %}
       <aside class="error-note">
 	{{ _('web_ui.err.uninstall_disallowed') }}
@@ -86,4 +83,7 @@ in a proprietary work, I am not going to enforce this in court.
       </form>
     </div>
   {%- endif %}
-{%- endblock %}
+{%- endblock main_info %}
+{% block version_list_heading %}
+  {{ _('web_ui.packages.single_version.version_list_heading') }}
+{% endblock %}
-- 
cgit v1.2.3