Context Navigation

← Previous Change
Next Change →

meta.php

Timestamp:

10/08/2014 03:11:14 PM (12 years ago)

Author:

kovshenin

Message:

Use the primary meta_query clause when parsing orderby in WP_Query.

When using legacy meta_key, meta_value, etc. arguments in WP_Query,
they're converted into the first clause of a meta_query. By using that
clause instead of the original arguments, we make sure that behavior is
consistent between the two available formats.

props boonebgorges.
fixes #16814.

File:

: 1 edited

trunk/tests/phpunit/tests/meta.php (modified) (2 diffs)

Legend:

: Unmodified
: Added
: Removed

trunk/tests/phpunit/tests/meta.php

-              r29650
+              r29855
+        }
+        /**
+         * @ticket 16814
+         */
         function test_meta_type_cast() {
                 $post_id1 = $this->factory->post->create();
                 add_post_meta( $post_id1, 'num_as_longtext', 123 );
+                add_post_meta( $post_id1, 'num_as_longtext_desc', 10 );
                 $post_id2 = $this->factory->post->create();
                 add_post_meta( $post_id2, 'num_as_longtext', 99 );
+                add_post_meta( $post_id2, 'num_as_longtext_desc', 100 );
                 $posts = new WP_Query( array(
 …
                 $this->assertEquals( array( $post_id2, $post_id1 ), $posts->posts );
                 $this->assertEquals( 2, substr_count( $posts->request, 'CAST(' ) );
+                // Make sure the newer meta_query syntax behaves in a consistent way
+                $posts = new WP_Query( array(
+                        'fields' => 'ids',
+                        'post_type' => 'any',
+                        'meta_query' => array(
+                                array(
+                                        'key' => 'num_as_longtext',
+                                        'value' => '0',
+                                        'compare' => '>',
+                                        'type' => 'UNSIGNED',
+                                ),
+                        ),
+                        'orderby' => 'meta_value',
+                        'order' => 'ASC'
+                ) );
+                $this->assertEquals( array( $post_id2, $post_id1 ), $posts->posts );
+                $this->assertEquals( 2, substr_count( $posts->request, 'CAST(' ) );
+                // The legacy `meta_key` value should take precedence.
+                $posts = new WP_Query( array(
+                        'fields' => 'ids',
+                        'post_type' => 'any',
+                        'meta_key' => 'num_as_longtext',
+                        'meta_compare' => '>',
+                        'meta_type' => 'UNSIGNED',
+                        'meta_query' => array(
+                                array(
+                                        'key' => 'num_as_longtext_desc',
+                                        'value' => '0',
+                                        'compare' => '>',
+                                        'type' => 'UNSIGNED',
+                                ),
+                        ),
+                        'orderby' => 'meta_value',
+                        'order' => 'ASC'
+                ) );
+                $this->assertEquals( array( $post_id2, $post_id1 ), $posts->posts );
+                $this->assertEquals( 2, substr_count( $posts->request, 'CAST(' ) );
+        }

Note: See TracChangeset for help on using the changeset viewer.

Make WordPress Core

Context Navigation

Changeset 29855 for trunk/tests/phpunit/tests/meta.php

Legend:

trunk/tests/phpunit/tests/meta.php

Download in other formats: