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

Timestamp:

04/02/2018 02:24:57 AM (7 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';
+        }
     }