From ce87de1dfb87c80ed320ff11fc7f4f63e81d4aca Mon Sep 17 00:00:00 2001
From: Fioddor Superconcentrado <fioddor@gmail.com>
Date: Wed, 16 Dec 2020 10:34:00 +0100
Subject: [PATCH] networks: Hide deactivate/remove buttons for primary
 connections

Helps: #1962.

Signed-off-by: Fioddor Superconcentrado <fioddor@gmail.com>
Reviewed-by: Sunil Mohan Adapa <sunil@medhas.org>
---
 .../networks/templates/connections_list.html  | 29 ++++++++++---------
 plinth/network.py                             |  7 ++++-
 2 files changed, 22 insertions(+), 14 deletions(-)

diff --git a/plinth/modules/networks/templates/connections_list.html b/plinth/modules/networks/templates/connections_list.html
index 987786794..50b43a119 100644
--- a/plinth/modules/networks/templates/connections_list.html
+++ b/plinth/modules/networks/templates/connections_list.html
@@ -53,12 +53,14 @@
 
             <td class="connection-actions">
               {% if connection.is_active %}
-                <form class="form form-action" method="post"
-                      action="{% url 'networks:deactivate' connection.uuid %}">
-                  {% csrf_token %}
-                  <button type="submit" class="btn btn-default btn-sm">
-                    {% trans "Deactivate" %}</button>
-                </form>
+                {% if not connection.primary %}
+                  <form class="form form-action" method="post"
+                        action="{% url 'networks:deactivate' connection.uuid %}">
+                    {% csrf_token %}
+                    <button type="submit" class="btn btn-default btn-sm">
+                      {% trans "Deactivate" %}</button>
+                  </form>
+                {% endif %}
               {% else %}
                 <form class="form form-action" method="post"
                       action="{% url 'networks:activate' connection.uuid %}">
@@ -68,13 +70,14 @@
                 </form>
               {% endif %}
 
-              <a href="{% url 'networks:delete' connection.uuid %}"
-                 class="btn btn-default btn-sm"
-                 role="button"
-                 title="{% blocktrans with name=connection.name %}Delete connection {{ name }}{% endblocktrans %}">
-                <span class="fa fa-trash-o"
-                      aria-hidden="true"></span>
-              </a>
+              {% if not connection.primary %}
+                <a href="{% url 'networks:delete' connection.uuid %}"
+                   class="btn btn-default btn-sm"
+                   role="button"
+                   title="{% blocktrans with name=connection.name %}Delete connection {{ name }}{% endblocktrans %}">
+                  <span class="fa fa-trash-o" aria-hidden="true"></span>
+                </a>
+              {% endif %}
             </td>
           </tr>
         {% endfor %}
diff --git a/plinth/network.py b/plinth/network.py
index 1a79adfff..1c07efec9 100644
--- a/plinth/network.py
+++ b/plinth/network.py
@@ -217,8 +217,10 @@ def _get_wifi_channel_from_frequency(frequency):
 
 def get_connection_list():
     """Get a list of active and available connections."""
-    active_uuids = []
     client = get_nm_client()
+    primary_connection = client.get_primary_connection()
+
+    active_uuids = []
     for connection in client.get_active_connections():
         active_uuids.append(connection.get_uuid())
 
@@ -241,6 +243,9 @@ def get_connection_list():
             'type': connection_type,
             'type_name': connection_type_name,
             'is_active': connection.get_uuid() in active_uuids,
+            'primary':
+                (primary_connection
+                 and primary_connection.get_uuid() == connection.get_uuid()),
             'zone': zone,
         })
     connections.sort(key=lambda connection: connection['is_active'],