Changeset 4506 for trunk/bp-xprofile/bp-xprofile-screens.php

Timestamp:

06/13/2011 09:54:52 PM (15 years ago)

Author:

boonebgorges

Message:

Introduces bp_do_404() for easy 404ing. Corrects invalid path handling throughout, using bp_do_404(). Fixes #3280. Props DJPaul

File:

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

trunk/bp-xprofile/bp-xprofile-screens.php

@@ -13 +13 @@
  * @uses bp_core_load_template() Looks for and loads a template file within the current member theme (folder/filename)
  */
-function xprofile_screen_display_profile() {
+function xprofile_screen_display_profile() {    
         $new = isset( $_GET['new'] ) ? $_GET['new'] : '';
 
@@ -39 +39 @@
 
         // Check the field group exists
-        if ( !xprofile_get_field_group( $bp->action_variables[1] ) )
-                bp_core_redirect( bp_get_root_domain() );
+        if ( ( !empty( $bp->action_variables[0] ) && 'group' != $bp->action_variables[0] ) || !xprofile_get_field_group( $bp->action_variables[1] ) ) {
+                bp_do_404();
+                return;
+        }
 
         // Check to see if any new information has been submitted
@@ -133 +135 @@
                 return false;
 
+        if ( !empty( $bp->action_variables ) ) {
+                bp_do_404();
+                return;
+        }
+
         $bp->avatar_admin->step = 'upload-image';