Changeset 12430 for trunk/src/bp-groups/classes/class-bp-groups-group.php

Timestamp:

08/12/2019 08:43:10 PM (6 years ago)

Author:

dcavins

Message:

Update BP_Groups_Group for Invitation API compatibility.

• Update logic in BP_Groups_Group to avoid direct table lookups and instead use request and invitation functions to fetch and delete invitations.

See #6210.

File:

• trunk/src/bp-groups/classes/class-bp-groups-group.php (modified) (3 diffs)

trunk/src/bp-groups/classes/class-bp-groups-group.php

@@ -772 +772 @@
      */
     public static function get_invites( $user_id, $group_id, $sent = null ) {
-        global $wpdb;
-
-        $bp  = buddypress();
-        $sql = $wpdb->prepare( "SELECT user_id FROM {$bp->groups->table_name_members} WHERE group_id = %d and is_confirmed = 0 AND inviter_id = %d", $group_id, $user_id );
-
-        // Query for a specific invite sent status.
-        if ( ! is_null( $sent ) ) {
-            $sql .= $wpdb->prepare( ' AND invite_sent = %d', $sent );
-        }
-
-        return $wpdb->get_col( $sql );
+        if ( 0 === $sent ) {
+            $sent_arg = 'draft';
+        } else if ( 1 === $sent ) {
+            $sent_arg = 'sent';
+        } else {
+            $sent_arg = 'all';
+        }
+
+        return groups_get_invites( array(
+            'item_id'     => $group_id,
+            'inviter_id'  => $user_id,
+            'invite_sent' => $sent_arg,
+            'fields'      => 'user_ids',
+        ) );
     }
 
@@ -960 +963 @@
      */
     public static function get_membership_requests( $group_id, $limit = null, $page = null ) {
-        global $wpdb;
-
-        if ( !empty( $limit ) && !empty( $page ) ) {
-            $pag_sql = $wpdb->prepare( " LIMIT %d, %d", intval( ( $page - 1 ) * $limit), intval( $limit ) );
-        }
-
-        $bp = buddypress();
-
-        $paged_requests = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM {$bp->groups->table_name_members} WHERE group_id = %d AND is_confirmed = 0 AND inviter_id = 0{$pag_sql}", $group_id ) );
-        $total_requests = $wpdb->get_var( $wpdb->prepare( "SELECT COUNT(id) FROM {$bp->groups->table_name_members} WHERE group_id = %d AND is_confirmed = 0 AND inviter_id = 0", $group_id ) );
-
-        return array( 'requests' => $paged_requests, 'total' => $total_requests );
+        $args = array(
+            'item_id' => $group_id
+        );
+        if ( $limit ) {
+            $args['per_page'] = $limit;
+        }
+        if ( $page ) {
+            $args['page'] = $page;
+        }
+
+        $requests = groups_get_requests( $args );
+        $total    = count( groups_get_membership_requested_user_ids( $group_id ) );
+
+        return array( 'requests' => $requests, 'total' => $total );
     }
 
@@ -1638 +1643 @@
      */
     public static function delete_all_invites( $group_id ) {
-        global $wpdb;
-
-        $bp = buddypress();
-
-        return $wpdb->query( $wpdb->prepare( "DELETE FROM {$bp->groups->table_name_members} WHERE group_id = %d AND invite_sent = 1", $group_id ) );
+        if ( empty( $group_id ) ) {
+            return false;
+        }
+
+        $invites_class = new BP_Groups_Invitation_Manager();
+
+        return $invites_class->delete( array(
+            'item_id' => $group_id,
+        ) );
     }