Changeset 10043 for trunk/src/bp-core/classes/class-bp-user-query.php

Timestamp:

08/11/2015 04:54:03 PM (10 years ago)

Author:

r-a-y

Message:

BP User Query: Remove invalid IDs from $user_ids property after query is run.

If the user_ids parameter is passed to BP_User_Query with invalid user
IDs, after the user query is run, the user_ids property still references
these invalid user IDs. This can lead to devs to assume that the user ID
exists when it does not.

This commit removes these invalid user IDs and includes a unit test.

See #6577.

File:

• trunk/src/bp-core/classes/class-bp-user-query.php (modified) (2 diffs)

trunk/src/bp-core/classes/class-bp-user-query.php

@@ -596 +596 @@
 
         // Match up to the user ids from the main query
-        foreach ( $this->user_ids as $uid ) {
+        foreach ( $this->user_ids as $key => $uid ) {
             if ( isset( $r[ $uid ] ) ) {
                 $this->results[ $uid ] = $r[ $uid ];
@@ -603 +603 @@
                 // (as opposed to 'ID') property
                 $this->results[ $uid ]->id = $uid;
+
+            // remove user ID from original user_ids property
+            } else {
+                unset( $this->user_ids[ $key ] );
             }
         }