repro: Minor styling and message changes

- Indentation for HTML template consistent with other templates.

- Style 'repro' like the upstream project does in all small case.

- Better describe the functions of a SIP server and organize the actions
  to be done by the user.

- Set the menu weight so that it does not clash with an existing module.

- Name the application 'SIP Server' instead of 'SIP Proxy' as that
  better describes the capabilities of the repro and is simpler for
  users to understand.

data/etc/apache2/conf-available/repro-plinth.conf

@@ -1,3 +1,7 @@
+##
+## On all sites, provide repro admin interface on a path: /repro
+## Only allow users of admin LDAP group.
+##
 <Location /repro>
     ProxyPass http://localhost:5080
 

plinth/modules/repro/__init__.py

@@ -33,12 +33,12 @@ service = None
 def init():
     """Initialize the repro module."""
     menu = cfg.main_menu.get('apps:index')
-    menu.add_urlname(_('SIP Proxy (Repro)'), 'glyphicon-phone-alt',
+    menu.add_urlname(_('SIP Server (repro)'), 'glyphicon-phone-alt',
-                     'repro:index', 800)
+                     'repro:index', 825)
 
     global service
     service = service_module.Service(
-        'repro', _('repro SIP Proxy'), ['sip-plinth', 'sip-tls-plinth'],
+        'repro', _('repro SIP Server'), ['sip-plinth', 'sip-tls-plinth'],
         is_external=True, enabled=is_enabled())
 
 

plinth/modules/repro/forms.py

@@ -26,5 +26,5 @@ from django.utils.translation import ugettext_lazy as _
 class ReproForm(forms.Form):
     """Configuration form."""
     enabled = forms.BooleanField(
-        label=_('Enable Repro service'),
+        label=_('Enable repro service'),
         required=False)

plinth/modules/repro/templates/repro.html

@@ -23,58 +23,61 @@
 
 {% block content %}
 
-<h2>{% trans "SIP Proxy (Repro)" %}</h2>
+  <h2>{% trans "SIP Server (repro)" %}</h2>
 
-<p>
+  <p>
-  {% blocktrans trimmed %}
+    {% blocktrans trimmed %}
-  Repro is a SIP proxy service that a SIP softphone can connect to. Before
+      repro provides various SIP services that a SIP softphone can utilize to
-  using Repro, the domain and users will need to be configured using the
+      provide audio and video calls as well as presence and instant messaging.
-  web-based configuration panel, available at
+      repro provides a server and SIP user accounts that clients can use to let
-  <a href="/repro/domains.html">/repro/domains.html</a>. Users in the
+      their presence known.  It also acts as a proxy to federate SIP
-  <i>admin</i> group will be able to log in to the Repro configuration panel.
+      communications to other servers on the Internet similar to email.
-  {% endblocktrans %}
+    {% endblocktrans %}
-</p>
+  </p>
 
-<p>
+  <p>
-  {% blocktrans trimmed %}
+    {% blocktrans trimmed %}
-  <b>Note:</b> After setting the domain, it is required to restart the Repro
+      To make SIP calls, a client application is needed. Available clients
-  service. Disable the service using the checkbox and Update setup button
+      include <a href="https://jitsi.org/">Jitsi</a> (for computers) and
-  below. Then, re-enable the service.
+      <a href="https://f-droid.org/repository/browse/?fdid=com.csipsimple">
-  {% endblocktrans %}
+        CSipSimple</a> (for Android phones).
-</p>
+    {% endblocktrans %}
+  </p>
 
-<p>
+  <p>
-  {% blocktrans trimmed %}
+    {% blocktrans trimmed %}
-  To make SIP calls, a client application is needed. Available clients include
+      <strong>Note:</strong>  Before using repro, domains and users will need
-  <a href="https://jitsi.org/">Jitsi</a> (for computers) and
+      to be configured using the <a href="/repro/domains.html">web-based
-  <a href="https://f-droid.org/repository/browse/?fdid=com.csipsimple">
+      configuration panel</a>. Users in the <em>admin</em> group will be able
-    CSipSimple</a> (for Android phones).
+      to log in to the repro configuration panel. After setting the domain, it
-  {% endblocktrans %}
+      is required to restart the repro service. Disable the service and
-</p>
+      re-enable it.
+    {% endblocktrans %}
+  </p>
 
-<h3>{% trans "Status" %}</h3>
+  <h3>{% trans "Status" %}</h3>
 
-<p class="running-status-parent">
+  <p class="running-status-parent">
-  {% if status.is_running %}
+    {% if status.is_running %}
-  <span class="running-status active"></span>
+      <span class="running-status active"></span>
-  {% trans "Repro service is running" %}
+      {% trans "repro service is running" %}
-  {% else %}
+    {% else %}
-  <span class="running-status inactive"></span>
+      <span class="running-status inactive"></span>
-  {% trans "Repro service is not running" %}
+      {% trans "repro service is not running" %}
-  {% endif %}
+    {% endif %}
-</p>
+  </p>
 
-{% include "diagnostics_button.html" with module="repro" %}
+  {% include "diagnostics_button.html" with module="repro" %}
 
-<h3>{% trans "Configuration" %}</h3>
+  <h3>{% trans "Configuration" %}</h3>
 
-<form class="form" method="post">
+  <form class="form" method="post">
-  {% csrf_token %}
+    {% csrf_token %}
 
-  {{ form|bootstrap }}
+    {{ form|bootstrap }}
 
-  <input type="submit" class="btn btn-primary"
+    <input type="submit" class="btn btn-primary"
-	 value="{% trans "Update setup" %}"/>
+           value="{% trans "Update setup" %}"/>
-</form>
+  </form>
 
 {% endblock %}

plinth/modules/repro/views.py

@@ -52,7 +52,7 @@ def index(request):
         form = ReproForm(initial=status, prefix='repro')
 
     return TemplateResponse(request, 'repro.html',
-                            {'title': _('SIP Proxy (Repro)'),
+                            {'title': _('SIP Server (repro)'),
                              'status': status,
                              'form': form})