[change] Simplified checks

The checks in `_need_who_is_lookup` are split for better readability

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

Author: DragnEmperor

openwisp_controller/config/who_is/service.py

@@ -158,20 +158,20 @@ def _need_who_is_lookup(self, initial_ip, new_ip):
         This is used to determine if the WhoIs lookup should be triggered
         when the device is saved.
 
-        The lookup is triggered if:
-            - The new IP address is not None.
-            - The WhoIs information is not already present or the initial IP is
-              different from the new IP.
-            - The new IP address is a global (public) IP address.
-            - WhoIs is enabled in the organization settings. (query from db)
+        The lookup is not triggered if:
+            - The new IP address is None or it is a private IP address.
+            - The WhoIs information of new ip is already present.
+            - WhoIs is disabled in the organization settings. (query from db)
         """
 
         # Check cheap conditions first before hitting the database
-        return (
-            self.is_valid_public_ip_address(new_ip)
-            and (initial_ip != new_ip or not self._get_who_is_info_from_db(new_ip))
-            and self.is_who_is_enabled
-        )
+        if not self.is_valid_public_ip_address(new_ip):
+            return False
+
+        if self._get_who_is_info_from_db(new_ip):
+            return False
+
+        return self.is_who_is_enabled
 
     def get_device_who_is_info(self):
         """
@@ -209,17 +209,16 @@ def fetch_who_is_details(self, device_pk, initial_ip_address, new_ip_address):
         Fetches the WhoIs details of the given IP address
         and creates/updates the WhoIs record.
         """
-        # The task is triggered if last_ip is updated irrespective
-        # if there is WhoIsInfo for the new IP address so we return
-        # from the task if that is the case.
+        # The task can be triggered for same ip address multiple times
+        # so we need to return early if WhoIs is already created.
         if self._get_who_is_info_from_db(new_ip_address):
             return
 
         WhoIsInfo = load_model("config", "WhoIsInfo")
 
-        try:
-            ip_client = WhoIsService._get_geoip2_client()
+        ip_client = WhoIsService._get_geoip2_client()
 
+        try:
             data = ip_client.city(ip_address=new_ip_address)
 
         # Catching all possible exceptions raised by the geoip2 client