Changeset 13099 for trunk/src/bp-members/classes/class-bp-signup.php

Timestamp:

08/31/2021 09:47:06 PM (3 years ago)

Author:

dcavins

Message:

Introduce $active parameter for BP_Signup::get().

Allow the BP_Signup::get() method to fetch

• activated signups only (active = 1)
• not-yet-activated signups only (active = 0) - the default case
• signups with any active status (active = false)

This change means that users following an activation key
for an already-activated account will get a more helpful
error message telling them the account is already activated.

Props Jean-David.

Fixes #7938.

File:

• trunk/src/bp-members/classes/class-bp-signup.php (modified) (3 diffs)

trunk/src/bp-members/classes/class-bp-signup.php

@@ -256 +256 @@
      *     @type string      $activation_key Activation key to search for. If specified, all other
      *                                       parameters will be ignored.
+     *     @type int|bool    $active         Pass 0 for inactive signups, 1 for active signups,
+     *                                       and `false` to ignore.
      *     @type string      $user_login     Specific user login to return.
      *     @type string      $fields         Which fields to return. Specify 'ids' to fetch a list of signups IDs.
@@ -279 +281 @@
                 'include'        => false,
                 'activation_key' => '',
+                'active'         => 0,
                 'user_email'     => '',
                 'user_login'     => '',
@@ -320 +323 @@
              */
         } else {
+            // Active.
+            if ( false !== $r['active'] ) {
+                $sql['where'][] = $wpdb->prepare( "active = %d", absint( $r['active'] ) );
+            }
 
             // Search terms.