networks: Handle unavailability of device

plinth/modules/networks/networks.py

@@ -79,15 +79,13 @@ def show(request, uuid):
         active_connection = None
 
     # Device status
+    device = None
     if active_connection and active_connection.get_devices():
         device = active_connection.get_devices()[0]
     else:
         interface_name = connection_status['interface_name']
         if interface_name:
             device = network.get_device_by_interface_name(interface_name)
-        else:
-            # TODO: What to do in this situation?
-            raise RuntimeError('Could not get device status')
 
     device_status = network.get_status_from_device(device)
 

plinth/modules/networks/templates/connection_show.html

@@ -74,6 +74,7 @@
         </div>
       </div>
 
+      {% if device %}
         <h3>Device</h3>
 
         <div class="list-group">
@@ -104,6 +105,7 @@
             <span class="pull-right">{{ device.description }}</span>
           </div>
         </div>
+      {% endif %}
 
       <h3>Physical Link</h3>
 
@@ -130,6 +132,7 @@
             SSID
             <span class="pull-right">{{ connection.wireless.ssid }}</span>
           </div>
+          {% if device %}
             <div class="list-group-item">
               Speed
               <span class="pull-right">{{ device.wireless.bitrate }} Mbit/s</span>
@@ -138,6 +141,7 @@
               Mode
               <span class="pull-right">{{ device.wireless.mode }}</span>
             </div>
+          {% endif %}
           {% if access_point.channel %}
             <div class="list-group-item">
               Signal strength

plinth/network.py

@@ -161,6 +161,9 @@ def get_status_from_wifi_access_point(device, ssid):
     """Return the current status of an access point."""
     status = {}
 
+    if not ssid or not device:
+        return status
+
     for access_point in device.get_access_points():
         if access_point.get_ssid().get_data() == ssid:
             status['strength'] = access_point.get_strength()