Changeset 13495 for trunk/src/bp-xprofile/classes/class-bp-xprofile-component.php

Timestamp:

06/02/2023 02:05:08 AM (3 years ago)

Author:

imath

Message:

Prevent potential issues with late_includes() in Component classes

Use require_once instead of require to load action & screen files.

See #4954
Fixes #8910
Closes ​https://github.com/buddypress/buddypress/pull/112

File:

• trunk/src/bp-xprofile/classes/class-bp-xprofile-component.php (modified) (2 diffs)

trunk/src/bp-xprofile/classes/class-bp-xprofile-component.php

@@ -112 +112 @@
                 // User nav.
                 if ( bp_is_profile_component() ) {
-                        require $this->path . 'bp-xprofile/screens/public.php';
+                        require_once $this->path . 'bp-xprofile/screens/public.php';
 
                         // Sub-nav items.
                         if ( is_user_logged_in() && 'edit' === bp_current_action() ) {
-                                require $this->path . 'bp-xprofile/screens/edit.php';
+                                require_once $this->path . 'bp-xprofile/screens/edit.php';
                         }
                 }
@@ -122 +122 @@
                 // Settings.
                 if ( is_user_logged_in() && bp_is_user_settings_profile() ) {
-                        require $this->path . 'bp-xprofile/screens/settings-profile.php';
+                        require_once $this->path . 'bp-xprofile/screens/settings-profile.php';
                 }
         }