Context Navigation

Back to Ticket #32250

Ticket #32250: 32250.diff

File 32250.diff, 2.7 KB (added by joehoyle, 11 years ago)

src/wp-includes/user.php

diff --git src/wp-includes/user.php src/wp-includes/user.php
index e602a3e..f1a09b0 100644

                class WP_User_Query {
          *                                         of fields. Accepts 'ID', 'display_name', 'login', 'nicename', 'email',
          *                                         'url', 'registered'. Use 'all' for all fields and 'all_with_meta' to
          *                                         include meta fields. Default 'all'.
          *     @type string       $who             Type of users to query. Accepts 'authors'. Default empty (all users).
+         *     @type string       $who             Type of users to query. Accepts 'authors', 'public_author'. Default empty (all users).
          * }
          */
         public function prepare_query( $query = array() ) {
-…
+               class WP_User_Query {
                         $blog_id = absint( $qv['blog_id'] );
+                }
+                if ( isset( $qv['who'] ) && 'authors' == $qv['who'] && $blog_id ) {
+                        $qv['meta_key'] = $wpdb->get_blog_prefix( $blog_id ) . 'user_level';
+                        $qv['meta_value'] = 0;
+                        $qv['meta_compare'] = '!=';
+                if ( isset( $qv['who'] ) && $blog_id ) {
+                        if ( 'authors' == $qv['who'] ) {
+                                $qv['meta_key'] = $wpdb->get_blog_prefix( $blog_id ) . 'user_level';
+                                $qv['meta_value'] = 0;
+                                $qv['meta_compare'] = '!=';
+                        } else if ( 'public_authors' == $qv['who'] ) {
+                                $this->query_from .= " JOIN $wpdb->posts ON $wpdb->posts.post_author = $wpdb->users.ID";
+                                $post_types = get_post_types( array( 'public' => true ) );
+                                $this->query_where .= " AND $wpdb->posts.post_status = 'publish' AND $wpdb->posts.post_type IN ( '" . join( "', '", $post_types ) . "' )";
+                        }
                         $qv['blog_id'] = $blog_id = 0; // Prevent extra meta query
+                }

tests/phpunit/tests/user/query.php

diff --git tests/phpunit/tests/user/query.php tests/phpunit/tests/user/query.php
index 9abf99d..dbd378a 100644

                class Tests_User_Query extends WP_UnitTestCase {
                 $this->assertContains( $users[1], $found );
                 $this->assertNotContains( $users[2], $found );
+        }
+        public function test_who_public_authors() {
+                $users = $this->factory->user->create_many( 3 );
+                $this->factory->post->create( array( 'post_author' => $users[0], 'post_status' => 'publish', 'post_type' => 'post' ) );
+                $this->factory->post->create( array( 'post_author' => $users[1], 'post_status' => 'publish', 'post_type' => 'post' ) );
+                $q = new WP_User_Query( array(
+                        'who' => 'public_authors',
+                ) );
+                $found = wp_list_pluck( $q->get_results(), 'ID' );
+                $this->assertEquals( 2, $q->get_total() );
+                $this->assertContains( $users[0], $found );
+                $this->assertContains( $users[1], $found );
+        }
+}

Download in other formats:

Original Format

Make WordPress Core

Context Navigation

Ticket #32250: 32250.diff

src/wp-includes/user.php

tests/phpunit/tests/user/query.php

Download in other formats: