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

Timestamp:

04/02/2018 04:32:31 AM (7 years ago)

Author:

r-a-y

Message:

XProfile: Conditionally load action and screen functions.

This commit conditionally loads action and screen function code for the
XProfile component, utilizing the 'bp_late_include' hook introduced in
r11884.

Previously, we loaded these functions at all times, which is unnecessary
when a user is not on a BuddyPress extended profile page. Now, we only load
this code when needed.

See #7218.

File:

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

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

@@ -68 +68 @@
             'cssjs',
             'cache',
-            'actions',
             'activity',
-            'screens',
             'caps',
             'filters',
@@ -84 +82 @@
 
         parent::includes( $includes );
+    }
+
+    /**
+     * Late includes method.
+     *
+     * Only load up certain code when on specific pages.
+     *
+     * @since 3.0.0
+     */
+    public function late_includes() {
+        // Bail if PHPUnit is running.
+        if ( defined( 'BP_TESTS_DIR' ) ) {
+            return;
+        }
+
+        // Bail if not on a user page.
+        if ( ! bp_is_user() ) {
+            return;
+        }
+
+        // User nav.
+        if ( bp_is_profile_component() ) {
+            require $this->path . 'bp-xprofile/screens/public.php';
+
+            // Action - Delete avatar.
+            if ( is_user_logged_in()&& bp_is_user_change_avatar() && bp_is_action_variable( 'delete-avatar', 0 ) ) {
+                require $this->path . 'bp-xprofile/actions/delete-avatar.php';
+            }
+
+            // Sub-nav items.
+            if ( is_user_logged_in() &&
+                in_array( bp_current_action(), array( 'edit', 'change-avatar', 'change-cover-image' ), true )
+            ) {
+                require $this->path . 'bp-xprofile/screens/' . bp_current_action() . '.php';
+            }
+        }
+
+        // Settings.
+        if ( is_user_logged_in() && bp_is_user_settings_profile() ) {
+            require $this->path . 'bp-xprofile/screens/settings-profile.php';
+        }
     }