Changeset 3369 for trunk/bp-groups.php

Timestamp:

11/07/2010 12:20:02 AM (16 years ago)

Author:

djpaul

Message:

Fixes even more WP_DEBUG warnings.

File:

• trunk/bp-groups.php (modified) (3 diffs)

trunk/bp-groups.php

@@ -217 +217 @@
         global $bp;
 
-        $group_id = $bp->action_variables[1];
-
-        if ( isset($bp->action_variables) && in_array( 'accept', (array)$bp->action_variables ) && is_numeric($group_id) ) {
+        $group_id = 0;
+        if ( isset( $bp->action_variables[1] ) )
+                $group_id = $bp->action_variables[1];
+
+        if ( isset( $bp->action_variables ) && in_array( 'accept', (array)$bp->action_variables ) && is_numeric( $group_id ) ) {
                 /* Check the nonce */
                 if ( !check_admin_referer( 'groups_accept_invite' ) )
@@ -241 +243 @@
                 bp_core_redirect( $bp->loggedin_user->domain . $bp->current_component . '/' . $bp->current_action );
 
-        } else if ( isset($bp->action_variables) && in_array( 'reject', (array)$bp->action_variables ) && is_numeric($group_id) ) {
+        } elseif ( isset( $bp->action_variables ) && in_array( 'reject', (array)$bp->action_variables ) && is_numeric( $group_id ) ) {
                 /* Check the nonce */
                 if ( !check_admin_referer( 'groups_reject_invite' ) )
                         return false;
 
-                if ( !groups_reject_invite( $bp->loggedin_user->id, $group_id ) ) {
+                if ( !groups_reject_invite( $bp->loggedin_user->id, $group_id ) )
                         bp_core_add_message( __('Group invite could not be rejected', 'buddypress'), 'error' );
-                } else {
+                else
                         bp_core_add_message( __('Group invite rejected', 'buddypress') );
-                }
 
                 bp_core_redirect( $bp->loggedin_user->domain . $bp->current_component . '/' . $bp->current_action );
@@ -1244 +1245 @@
         global $bp, $wp_query;
 
-        if ( !bp_is_active( 'activity' ) || $bp->current_component != $bp->groups->slug || !$bp->groups->current_group || $bp->current_action != 'feed' )
+        if ( !bp_is_active( 'activity' ) || $bp->current_component != $bp->groups->slug || !isset( $bp->groups->current_group ) || $bp->current_action != 'feed' )
                 return false;