Changeset 11925 for trunk/src/bp-messages/classes/class-bp-messages-component.php

Timestamp:

04/02/2018 02:24:57 AM (8 years ago)

Author:

r-a-y

Message:

Messages: Conditionally load action and screen functions.

This commit conditionally loads action and screen function code for the
Messages 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 messages page. Now, we only load this
code when needed.

See #7218.

File:

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

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

@@ -59 +59 @@
                         'cssjs',
                         'cache',
-                        'actions',
-                        'screens',
                         'filters',
                         'template',
@@ -79 +77 @@
 
                 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;
+                }
+
+                if ( bp_is_messages_component() ) {
+                        // Authenticated actions.
+                        if ( is_user_logged_in() &&
+                                in_array( bp_current_action(), array( 'compose', 'notices', 'view' ), true )
+                        ) {
+                                require $this->path . 'bp-messages/actions/' . bp_current_action() . '.php';
+                        }
+
+                        // Authenticated action variables.
+                        if ( is_user_logged_in() && bp_action_variable( 0 ) &&
+                                in_array( bp_action_variable( 0 ), array( 'delete', 'read', 'unread', 'bulk-manage', 'bulk-delete' ), true )
+                        ) {
+                                require $this->path . 'bp-messages/actions/' . bp_action_variable( 0 ) . '.php';
+                        }
+
+                        // Authenticated actions - Star.
+                        if ( is_user_logged_in() && bp_is_active( $this->id, 'star' ) ) {
+                                // Single action.
+                                if ( in_array( bp_current_action(), array( 'star', 'unstar' ), true ) ) {
+                                        require $this->path . 'bp-messages/actions/star.php';
+                                }
+
+                                // Bulk-manage.
+                                if ( bp_is_action_variable( 'bulk-manage' ) ) {
+                                        require $this->path . 'bp-messages/actions/bulk-manage-star.php';
+                                }
+                        }
+
+                        // Screens - User profile integration.
+                        if ( bp_is_user() ) {
+                                require $this->path . 'bp-messages/screens/inbox.php';
+
+                                /*
+                                 * Nav items.
+                                 *
+                                 * 'view' is not a registered nav item, but we add a screen handler manually.
+                                 */
+                                if ( bp_is_user_messages() && in_array( bp_current_action(), array( 'sentbox', 'compose', 'notices', 'view' ), true ) ) {
+                                        require $this->path . 'bp-messages/screens/' . bp_current_action() . '.php';
+                                }
+
+                                // Nav item - Starred.
+                                if ( bp_is_active( $this->id, 'star' ) && bp_is_current_action( bp_get_messages_starred_slug() ) ) {
+                                        require $this->path . 'bp-messages/screens/starred.php';
+                                }
+                        }
+                }
+
+                // Groups notifications HTML table.
+                if ( bp_is_user_settings_notifications() ) {
+                        require $this->path . 'bp-messages/screens/settings-email.php';
+                }
         }