Changeset 51154 for trunk/src/wp-includes/query.php

Timestamp:

06/15/2021 03:21:50 PM (3 years ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Remove a one-time $message variable in some _doing_it_wrong() calls.

This brings some consistency with most other calls.

Follow-up to [23378], [25605], [34745], [36219], [44108], [46111], [48156].

See #52627.

File:

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

trunk/src/wp-includes/query.php

@@ -900 +900 @@
  */
 function is_main_query() {
+    global $wp_query;
+
     if ( 'pre_get_posts' === current_filter() ) {
-        $message = sprintf(
-            /* translators: 1: pre_get_posts, 2: WP_Query->is_main_query(), 3: is_main_query(), 4: Documentation URL. */
-            __( 'In %1$s, use the %2$s method, not the %3$s function. See %4$s.' ),
-            '<code>pre_get_posts</code>',
-            '<code>WP_Query->is_main_query()</code>',
-            '<code>is_main_query()</code>',
-            __( 'https://developer.wordpress.org/reference/functions/is_main_query/' )
+        _doing_it_wrong(
+            __FUNCTION__,
+            sprintf(
+                /* translators: 1: pre_get_posts, 2: WP_Query->is_main_query(), 3: is_main_query(), 4: Documentation URL. */
+                __( 'In %1$s, use the %2$s method, not the %3$s function. See %4$s.' ),
+                '<code>pre_get_posts</code>',
+                '<code>WP_Query->is_main_query()</code>',
+                '<code>is_main_query()</code>',
+                __( 'https://developer.wordpress.org/reference/functions/is_main_query/' )
+            ),
+            '3.7.0'
         );
-        _doing_it_wrong( __FUNCTION__, $message, '3.7.0' );
-    }
-
-    global $wp_query;
+    }
+
     return $wp_query->is_main_query();
 }