Changeset 3369 for trunk/bp-groups.php

Timestamp:

11/07/2010 12:20:02 AM (15 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;