Changeset 36813 for trunk/src/wp-includes/vars.php

Timestamp:

03/03/2016 03:24:02 AM (9 years ago)

Author:

johnbillion

Message:

Uploads: Remove an unnecessary static var from wp_is_mobile() to allow its direct and indirect use within unit tests. The static `$is_m
obile var was only used to avoid a handful of calls to strpos()`, which are exceptionally fast and result in no measurable increase in
processing time on each call to wp_is_mobile().

See #35976, #20014

File:

• trunk/src/wp-includes/vars.php (modified) (1 diff)

trunk/src/wp-includes/vars.php

@@ -121 +121 @@
  * Test if the current browser runs on a mobile device (smart phone, tablet, etc.)
  *
- * @staticvar bool $is_mobile
- *
  * @return bool
  */
 function wp_is_mobile() {
-    static $is_mobile = null;
-
-    if ( isset( $is_mobile ) ) {
-        return $is_mobile;
-    }
-
     if ( empty($_SERVER['HTTP_USER_AGENT']) ) {
         $is_mobile = false;