tor: Use app.html instead of simple_app.html

- Reuse status section.

- Reuse configuration form.

- Reuse internal zone section.

- Cleanly split the page into status, internal zone and configuration blocks.

Signed-off-by: Sunil Mohan Adapa <sunil@medhas.org>
Reviewed-by: James Valleroy <jvalleroy@mailbox.org>

plinth/modules/tor/templates/tor.html

@@ -1,4 +1,4 @@
-{% extends "simple_app.html" %}
+{% extends "app.html" %}
 {% comment %}
 #
 # This file is part of FreedomBox.
@@ -32,33 +32,16 @@
 
 {% endblock %}
 
-
+{% block status %}
-{% block configuration %}
-
-  {% include "clients.html" with clients=clients %}
-
-  <h3>{% trans "Status" %}</h3>
-
   {% if config_running %}
+    <h3>{% trans "Status" %}</h3>
 
     <p class="running-status-parent">
       <span class="running-status loading"></span>
       {% trans "Tor configuration is being updated" %}
     </p>
-
   {% else %}
-
+    {{ block.super }}
-    <p class="running-status-parent">
-      {% if status.is_running %}
-        <span class="running-status active"></span>
-        {% trans "Tor is running" %}
-      {% else %}
-        <span class="running-status inactive"></span>
-        {% trans "Tor is not running" %}
-      {% endif %}
-    </p>
-
-    {% include "diagnostics_button.html" with module="tor" enabled=status.enabled %}
 
     {% if status.hs_enabled %}
       <table class="table table-bordered table-condensed table-striped tor-hs">
@@ -80,19 +63,20 @@
         </tbody>
       </table>
     {% endif %}
+  {% endif %}
+{% endblock %}
 
-    {% include "internal-zone.html" %}
+{% block internal_zone %}
+  {% if not config_running %}
+    {{ block.super }}
+  {% endif %}
+{% endblock %}
 
-    <h3>{% trans "Configuration" %}</h3>
+{% block configuration %}
 
-    <form class="form form-configuration" method="post">
+  {% if not config_running %}
-      {% csrf_token %}
 
-      {{ form|bootstrap }}
+    {{ block.super }}
-
-      <input type="submit" class="btn btn-primary btn-md"
-             value="{% trans "Update setup" %}"/>
-    </form>
 
     {% if status.relay_enabled %}
       <h3>{% trans "Relay" %}</h3>

plinth/modules/tor/views.py

@@ -53,13 +53,18 @@ def index(request):
     return TemplateResponse(
         request, 'tor.html', {
             'title': tor.name,
+            'name': tor.name,
             'description': tor.description,
             'clients': tor.clients,
             'manual_page': tor.manual_page,
             'status': status,
             'config_running': bool(config_process),
             'form': form,
-            'firewall': tor.app.get_components_of_type(Firewall)
+            'firewall': tor.app.get_components_of_type(Firewall),
+            'diagnostics_module_name': 'tor',
+            'is_enabled': status['enabled'],
+            'show_status_block': True,
+            'is_running': status['is_running'],
         })
 
 
@@ -70,8 +75,9 @@ def _apply_changes(request, old_status, new_status):
     except ActionError as exception:
         messages.error(
             request,
-            _('Action error: {0} [{1}] [{2}]').format(
+            _('Action error: {0} [{1}] [{2}]').format(exception.args[0],
-                exception.args[0], exception.args[1], exception.args[2]))
+                                                      exception.args[1],
+                                                      exception.args[2]))
 
 
 def __apply_changes(request, old_status, new_status):
@@ -131,8 +137,9 @@ def __apply_changes(request, old_status, new_status):
         else:
             tor.app.disable()
 
-        config_process = actions.superuser_run(
+        config_process = actions.superuser_run('tor',
-            'tor', ['configure'] + arguments, run_in_background=True)
+                                               ['configure'] + arguments,
+                                               run_in_background=True)
         return
 
     if arguments: