make it possible to change repo's name and URL using web UI

2 files changed, 82 insertions, 36 deletions

diff --git a/src/hydrilla/proxy/web_ui/templates/repos.html.jinja b/src/hydrilla/proxy/web_ui/templates/repos.html.jinja
index 3e61a83..54465a1 100644
--- a/src/hydrilla/proxy/web_ui/templates/repos.html.jinja
+++ b/src/hydrilla/proxy/web_ui/templates/repos.html.jinja
@@ -29,7 +29,7 @@ in a proprietary work, I am not going to enforce this in court.
 {% block main %}
   <h3>{{ _('web_ui.repos.heading') }}</h3>
   <div>
-    <a href="{{ url_for('.add_repo_get') }}" class="green-button">
+    <a href="{{ url_for('.add_repo') }}" class="green-button">
       {{ _('web_ui.repos.add_repo_button') }}
     </a>
   </div>
diff --git a/src/hydrilla/proxy/web_ui/templates/repos__show_single.html.jinja b/src/hydrilla/proxy/web_ui/templates/repos__show_single.html.jinja
index 96100ce..5783a20 100644
--- a/src/hydrilla/proxy/web_ui/templates/repos__show_single.html.jinja
+++ b/src/hydrilla/proxy/web_ui/templates/repos__show_single.html.jinja
@@ -27,46 +27,93 @@ in a proprietary work, I am not going to enforce this in court.
   {% include 'include/checkbox_tricks_style.css.jinja' %}
 {% endblock %}
 {% block main %}
-  {% set repo_id = display_info.ref.id -%}
-  <h3>
-    {% if display_info.is_local_semirepo -%}
-      {{ _('web_ui.repos.local_packages_semirepo') }}
-    {% else -%}
+  {%- set repo_id = display_info.ref.id %}
+  {%- if display_info.is_local_semirepo %}
+    <h3>{{ _('web_ui.repos.local_packages_semirepo') }}</h3>
+  {% else -%}
+    <h3>
       {{ _('web_ui.repos.single.heading.name_{}').format(display_info.name) }}
-    {% endif -%}
-  </h3>
-  {% if display_info.deleted and not display_info.is_local_semirepo -%}
-    <div>
-      {{ _('web_ui.repos.single.repo_is_deleted') }}
-    </div>
-  {% elif not display_info.deleted -%}
-    <input id="show_url_edit_form" type="checkbox" class="chbx-tricks-show-hide"
-	   checked="">
+    </h3>
+    {%- if repo_name_invalid is defined or repo_name_taken is defined %}
+      {%- set checked_attr = '' %}
+    {%- else %}
+      {%- set checked_attr = 'checked=""' %}
+    {%- endif %}
+    {%- if not display_info.deleted %}
+      <input id="hide_name_edit_form" type="checkbox"
+	     class="chbx-tricks-show-hide" {{ checked_attr }}>
       <div>
-	<div>
-	  {{ display_info.url }}
-	</div>
-	<div>
-	  <label for="show_url_edit_form" class="green-button">
-	    {{ _('web_ui.repos.single.update_url_button') }}
-	  </label>
-	</div>
+	<label for="hide_name_edit_form" class="green-button">
+	  {{ _('web_ui.repos.single.update_name_button') }}
+	</label>
       </div>
-      {% set action_url = url_for('.update_repo_url', repo_id=repo_id) -%}
-      <form method="POST" action="{{ action_url }}">
-	<input type="hidden" name="action" value="update_url">
+      <form method="POST">
+	<input type="hidden" name="action" value="update_repo_data">
+	{%- if repo_name_invalid is defined %}
+	  <aside class="error-note">
+	    {{ _('web_ui.repos.add.repo_name_invalid') }}
+	  </aside>
+	{%- endif %}
+	{%- if repo_name_taken is defined %}
+	  <aside class="error-note">
+	    {{ _('web_ui.repos.add.repo_name_taken') }}
+	  </aside>
+	{%- endif %}
 	<div>
-	  <input name="url" value="{{ display_info.url }}">
+	  <input name="name" value="{{ display_info.name }}" required="">
 	</div>
 	<div>
 	  <button class="green-button">
-	    {{ _('web_ui.repos.single.commit_update_url_button') }}
+	    {{ _('web_ui.repos.single.commit_update_name_button') }}
 	  </button>
-	  <label for="show_url_edit_form" class="green-button">
-	    {{ _('web_ui.repos.single.abort_update_url_button') }}
+	  <label for="hide_url_edit_form" class="green-button">
+	    {{ _('web_ui.repos.single.abort_update_name_button') }}
 	  </label>
 	</div>
       </form>
+    {%- endif %}{# not display_info.deleted #}
+  {%- endif %}{# display_info.is_local_semirepo (else) #}
+  {% if display_info.deleted and not display_info.is_local_semirepo -%}
+    <div>
+      {{ _('web_ui.repos.single.repo_is_deleted') }}
+    </div>
+  {%- elif not display_info.deleted %}
+    {%- if repo_url_invalid is defined %}
+      {%- set checked_attr = '' %}
+    {%- else %}
+      {%- set checked_attr = 'checked=""' %}
+    {%- endif %}
+    <input id="hide_url_edit_form" type="checkbox" class="chbx-tricks-show-hide"
+	    {{ checked_attr }}>
+    <div>
+      <div>
+	{{ display_info.url }}
+      </div>
+      <div>
+	<label for="hide_url_edit_form" class="green-button">
+	  {{ _('web_ui.repos.single.update_url_button') }}
+	</label>
+      </div>
+    </div>
+    <form method="POST">
+      <input type="hidden" name="action" value="update_repo_data">
+      {%- if repo_url_invalid is defined %}
+	<aside class="error-note">
+	  {{ _('web_ui.repos.add.repo_url_invalid') }}
+	</aside>
+      {%- endif %}
+      <div>
+	<input name="url" value="{{ display_info.url }}" required="">
+      </div>
+      <div>
+	<button class="green-button">
+	  {{ _('web_ui.repos.single.commit_update_url_button') }}
+	</button>
+	<label for="hide_url_edit_form" class="green-button">
+	  {{ _('web_ui.repos.single.abort_update_url_button') }}
+	</label>
+      </div>
+    </form>
     <div>
       {% if display_info.last_refreshed is none -%}
 	{{ _('web_ui.repos.single.repo_never_refreshed') }}
@@ -76,15 +123,14 @@ in a proprietary work, I am not going to enforce this in court.
 	      .format(display_info.last_refreshed.strftime('%F %H:%M'))
 	}}
       {% endif -%}
-      {% set action_url = url_for('.refresh_repo', repo_id=repo_id) -%}
-      <form method="POST" action="{{ action_url }}">
+      <form method="POST">
 	<input type="hidden" name="action" value="refresh_repo">
 	<button class="green-button">
 	  {{ _('web_ui.repos.single.refresh_now_button') }}
 	</button>
       </form>
     </div>
-  {% endif -%}
+  {% endif -%}{# not display_info.deleted (elif) #}
   <div>
     {{
       _('web_ui.repos.item_count_{mappings}_{resources}')
@@ -94,9 +140,9 @@ in a proprietary work, I am not going to enforce this in court.
           )
     }}
   </div>
-  {% if not display_info.is_local_semirepo -%}
-    {% set action_url = url_for('.remove_repo', repo_id=repo_id) -%}
-    <form method="POST" action="{{ action_url }}">
+  {% if not display_info.is_local_semirepo and not display_info.deleted -%}
+    <form method="POST">
+      <input type="hidden" name="action" value="remove_repo">
       <button class="green-button">
 	{{ _('web_ui.repos.single.remove_button') }}
       </button>