Changeset 5462 for sites/trunk/api.wordpress.org/public_html/events/1.0/index.php

Timestamp:

05/02/2017 09:33:55 PM (7 years ago)

Author:

iandunn

Message:

Events: Attempts all name-based lookups before resorting to IP lookup

Otherwise, someone searching for a city that requires a fallback lookup would have their request short-circuited by the IP lookup, which could produce unexpected results.

File:

• sites/trunk/api.wordpress.org/public_html/events/1.0/index.php (modified) (2 diffs)

sites/trunk/api.wordpress.org/public_html/events/1.0/index.php

@@ -442 +442 @@
     }
 
-    // IP:
-    if ( ! $location && isset( $args['ip'] ) ) {
-        $guess = guess_location_from_ip( $args['ip'] );
-
-        if ( $guess ) {
-            $location = array(
-                'description' => $guess->ip_city,
-                'latitude' => $guess->ip_latitude,
-                'longitude' => $guess->ip_longitude,
-                'country' => $guess->country_short,
-            );
-        }
-    }
-
     /*
      * If all else fails, cast a wide net and try to find something before giving up, even
@@ -487 +473 @@
             // No specific location details.
             $location = array();
+        }
+    }
+
+    // IP:
+    if ( ! $location && isset( $args['ip'] ) ) {
+        $guess = guess_location_from_ip( $args['ip'] );
+
+        if ( $guess ) {
+            $location = array(
+                'description' => $guess->ip_city,
+                'latitude'    => $guess->ip_latitude,
+                'longitude'   => $guess->ip_longitude,
+                'country'     => $guess->country_short,
+            );
         }
     }