Changeset 57750 for trunk/tests/phpunit/tests/multisite/wpNetworkQuery.php

Timestamp:

03/02/2024 01:36:02 PM (2 years ago)

Author:

swissspidy

Message:

Query: Remove leading whitespace from certain database queries.

Unintended leading whitespace at the beginning of a raw MySQL query led to unexpected behavior such as broken pagination. Eliminating said whitespace avoids that.

Adds unit tests to prevent regressions.

Props wpfed, swissspidy, ironprogrammer, tadamarketing, afercia.
Fixes #56841.

File:

• trunk/tests/phpunit/tests/multisite/wpNetworkQuery.php (modified) (1 diff)

trunk/tests/phpunit/tests/multisite/wpNetworkQuery.php

@@ -609 +609 @@
             return array( get_network( self::$network_ids['wordpress.org/'] ) );
         }
+
+        /**
+         * @ticket 56841
+         */
+        public function test_wp_network_query_does_not_have_leading_whitespace() {
+            $q = new WP_Network_Query();
+            $q->query(
+                array(
+                    'fields'               => 'all',
+                    'number'               => 3,
+                    'order'                => 'ASC',
+                    'update_network_cache' => true,
+                )
+            );
+
+            $this->assertSame( ltrim( $q->request ), $q->request, 'The query has leading whitespace' );
+        }
     }