Changeset 12446

Timestamp:

08/15/2019 03:48:14 PM (6 years ago)

Author:

dcavins

Message:

Invitations: Improve migration logic.

Per jjj: "Should be empty( $records ) instead of ! to avoid type juggling."

Props jjj.

See #8127.

File:

• trunk/src/bp-groups/bp-groups-functions.php (modified) (1 diff)

trunk/src/bp-groups/bp-groups-functions.php

@@ -3135 +3135 @@
 
     $records = $wpdb->get_results( "SELECT id, group_id, user_id, inviter_id, date_modified, comments, invite_sent FROM {$bp->groups->table_name_members} WHERE is_confirmed = 0 AND is_banned = 0" );
-    if ( ! $records ) {
+    if ( empty( $records ) ) {
         return;
     }