[qa] Minor formatting

Signed-off-by: DragnEmperor <[email protected]>

Author: DragnEmperor

openwisp_controller/config/admin.py

@@ -1364,9 +1364,8 @@ class ConfigSettingsInline(admin.StackedInline):
 
         def get_fields(self, request, obj=None):
             fields = super().get_fields(request, obj)
-            # hide the 'who_is_enabled' field if WhoIs is not Configured
             if not app_settings.WHO_IS_CONFIGURED:
-                fields = [f for f in fields if f != "who_is_enabled"]
+                fields.remove("who_is_enabled")
             return fields
 
     OrganizationAdmin.save_on_top = True

openwisp_controller/config/base/device.py

@@ -120,8 +120,8 @@ class Meta:
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
-        # if WhoIs is configured, we need to look out for changes
-        # in last_ip field as well
+        # Initial value for last_ip is required in WhoIs
+        # to remove WhoIs info related to that ip address.
         if app_settings.WHO_IS_CONFIGURED:
             self._changed_checked_fields.append("last_ip")
 
@@ -286,7 +286,6 @@ def save(self, *args, **kwargs):
                 self.key = self.generate_key(shared_secret)
         state_adding = self._state.adding
         super().save(*args, **kwargs)
-        # WhoIs Lookup runs only when WhoIs is configured.
         if app_settings.WHO_IS_CONFIGURED:
             self._check_last_ip()
         if state_adding and self.group and self.group.templates.exists():