Changeset 11925 for trunk/src/bp-messages/bp-messages-star.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/bp-messages-star.php (modified) (1 diff)

trunk/src/bp-messages/bp-messages-star.php

@@ -311 +311 @@
 }
 
-/** SCREENS **************************************************************/
-
-/**
- * Screen handler to display a user's "Starred" private messages page.
- *
- * @since 2.3.0
- */
-function bp_messages_star_screen() {
-    add_action( 'bp_template_content', 'bp_messages_star_content' );
-
-    /**
-     * Fires right before the loading of the "Starred" messages box.
-     *
-     * @since 2.3.0
-     */
-    do_action( 'bp_messages_screen_star' );
-
-    bp_core_load_template( 'members/single/plugins' );
-}
-
-/**
- * Screen content callback to display a user's "Starred" messages page.
- *
- * @since 2.3.0
- */
-function bp_messages_star_content() {
-    // Add our message thread filter.
-    add_filter( 'bp_after_has_message_threads_parse_args', 'bp_messages_filter_starred_message_threads' );
-
-    // Load the message loop template part.
-    bp_get_template_part( 'members/single/messages/messages-loop' );
-
-    // Remove our filter.
-    remove_filter( 'bp_after_has_message_threads_parse_args', 'bp_messages_filter_starred_message_threads' );
-}
-
-/**
- * Filter message threads by those starred by the logged-in user.
- *
- * @since 2.3.0
- *
- * @param  array $r Current message thread arguments.
- * @return array $r Array of starred message threads.
- */
-function bp_messages_filter_starred_message_threads( $r = array() ) {
-    $r['box'] = 'starred';
-    $r['meta_query'] = array( array(
-        'key'   => 'starred_by_user',
-        'value' => $r['user_id']
-    ) );
-
-    return $r;
-}
-
-/** ACTIONS **************************************************************/
-
-/**
- * Action handler to set a message's star status for those not using JS.
- *
- * @since 2.3.0
- */
-function bp_messages_star_action_handler() {
-    if ( ! bp_is_user_messages() ) {
-        return;
-    }
-
-    if ( false === ( bp_is_current_action( 'unstar' ) || bp_is_current_action( 'star' ) ) ) {
-        return;
-    }
-
-    if ( ! wp_verify_nonce( bp_action_variable( 1 ), 'bp-messages-star-' . bp_action_variable( 0 ) ) ) {
-        wp_die( "Oops!  That's a no-no!" );
-    }
-
-    // Check capability.
-    if ( ! is_user_logged_in() || ! bp_core_can_edit_settings() ) {
-        return;
-    }
-
-    // Mark the star.
-    bp_messages_star_set_action( array(
-        'action'     => bp_current_action(),
-        'message_id' => bp_action_variable(),
-        'bulk'       => (bool) bp_action_variable( 2 )
-    ) );
-
-    // Redirect back to previous screen.
-    $redirect = wp_get_referer() ? wp_get_referer() : bp_displayed_user_domain() . bp_get_messages_slug();
-    bp_core_redirect( $redirect );
-    die();
-}
-add_action( 'bp_actions', 'bp_messages_star_action_handler' );
-
-/**
- * Bulk manage handler to set the star status for multiple messages.
- *
- * @since 2.3.0
- */
-function bp_messages_star_bulk_manage_handler() {
-    if ( empty( $_POST['messages_bulk_nonce' ] ) ) {
-        return;
-    }
-
-    // Check the nonce.
-    if ( ! wp_verify_nonce( $_POST['messages_bulk_nonce'], 'messages_bulk_nonce' ) ) {
-        return;
-    }
-
-    // Check capability.
-    if ( ! is_user_logged_in() || ! bp_core_can_edit_settings() ) {
-        return;
-    }
-
-    $action  = ! empty( $_POST['messages_bulk_action'] ) ? $_POST['messages_bulk_action'] : '';
-    $threads = ! empty( $_POST['message_ids'] ) ? $_POST['message_ids'] : '';
-    $threads = wp_parse_id_list( $threads );
-
-    // Bail if action doesn't match our star actions or no IDs.
-    if ( false === in_array( $action, array( 'star', 'unstar' ), true ) || empty( $threads ) ) {
-        return;
-    }
-
-    // It's star time!
-    switch ( $action ) {
-        case 'star' :
-            $count = count( $threads );
-
-            // If we're starring a thread, we only star the first message in the thread.
-            foreach ( $threads as $thread ) {
-                $thread = new BP_Messages_thread( $thread );
-                $mids = wp_list_pluck( $thread->messages, 'id' );
-
-                bp_messages_star_set_action( array(
-                    'action'     => 'star',
-                    'message_id' => $mids[0],
-                ) );
-            }
-
-            bp_core_add_message( sprintf( _n( '%s message was successfully starred', '%s messages were successfully starred', $count, 'buddypress' ), $count ) );
-            break;
-
-        case 'unstar' :
-            $count = count( $threads );
-
-            foreach ( $threads as $thread ) {
-                bp_messages_star_set_action( array(
-                    'action'    => 'unstar',
-                    'thread_id' => $thread,
-                    'bulk'      => true
-                ) );
-            }
-
-            bp_core_add_message( sprintf( _n( '%s message was successfully unstarred', '%s messages were successfully unstarred', $count, 'buddypress' ), $count ) );
-            break;
-    }
-
-    // Redirect back to message box.
-    bp_core_redirect( bp_displayed_user_domain() . bp_get_messages_slug() . '/' . bp_current_action() . '/' );
-    die();
-}
-add_action( 'bp_actions', 'bp_messages_star_bulk_manage_handler', 5 );
-
 /** HOOKS ****************************************************************/