Skip to:
Content

BuddyPress.org

Changeset 8791


Ignore:
Timestamp:
08/11/2014 04:56:19 PM (11 years ago)
Author:
johnjamesjacoby
Message:

Code clean-up in bp-messages-template.php.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-messages/bp-messages-template.php

    r8790 r8791  
    193193     */
    194194    function has_threads() {
    195         if ( $this->thread_count )
     195        if ( $this->thread_count ) {
    196196            return true;
     197        }
    197198
    198199        return false;
    … …  
    541542     */
    542543    function bp_get_message_thread_delete_link() {
    543         global $messages_template, $bp;
    544         return apply_filters( 'bp_get_message_thread_delete_link', wp_nonce_url( trailingslashit( bp_loggedin_user_domain() . $bp->messages->slug . '/' . bp_current_action() . '/delete/' . $messages_template->thread->thread_id ), 'messages_delete_thread' ) );
     544        global $messages_template;
     545        return apply_filters( 'bp_get_message_thread_delete_link', wp_nonce_url( trailingslashit( bp_loggedin_user_domain() . bp_get_messages_slug() . '/' . bp_current_action() . '/delete/' . $messages_template->thread->thread_id ), 'messages_delete_thread' ) );
    545546    }
    546547
    … …  
    561562        $class = false;
    562563
    563         if ( $messages_template->current_thread % 2 == 1 )
     564        if ( $messages_template->current_thread % 2 == 1 ) {
    564565            $class .= 'alt';
     566        }
    565567
    566568        return apply_filters( 'bp_get_message_css_class', trim( $class ) );
    … …  
    575577    global $messages_template;
    576578
    577     if ( $messages_template->thread->unread_count )
    578         return true;
    579 
    580     return false;
     579    $retval = ! empty( $messages_template->thread->unread_count )
     580        ? true
     581        : false;
     582
     583    return apply_filters( 'bp_message_thread_has_unread', $retval );
    581584}
    582585
    … …  
    595598        global $messages_template;
    596599
    597         if ( (int) $messages_template->thread->unread_count )
    598             return apply_filters( 'bp_get_message_thread_unread_count', $messages_template->thread->unread_count );
    599 
    600         return false;
     600        $count = ! empty( $messages_template->thread->unread_count )
     601            ? (int) $messages_template->thread->unread_count
     602            : false;
     603
     604        return apply_filters( 'bp_get_message_thread_unread_count', $count );
    601605    }
    602606
    … …  
    713717
    714718    $start_num = intval( ( $messages_template->pag_page - 1 ) * $messages_template->pag_num ) + 1;
    715     $from_num = bp_core_number_format( $start_num );
    716     $to_num = bp_core_number_format( ( $start_num + ( $messages_template->pag_num - 1 ) > $messages_template->total_thread_count ) ? $messages_template->total_thread_count : $start_num + ( $messages_template->pag_num - 1 ) );
    717     $total = bp_core_number_format( $messages_template->total_thread_count );
     719    $from_num  = bp_core_number_format( $start_num );
     720    $to_num    = bp_core_number_format( ( $start_num + ( $messages_template->pag_num - 1 ) > $messages_template->total_thread_count ) ? $messages_template->total_thread_count : $start_num + ( $messages_template->pag_num - 1 ) );
     721    $total     = bp_core_number_format( $messages_template->total_thread_count );
    718722
    719723    echo sprintf( _n( 'Viewing 1 message', 'Viewing %1$s - %2$s of %3$s messages', $total, 'buddypress' ), $from_num, $to_num, number_format_i18n( $total ) );
    … …  
    787791     */
    788792    function bp_get_messages_subject_value() {
    789         $subject = '';
    790         if ( !empty( $_POST['subject'] ) )
    791             $subject = $_POST['subject'];
     793        $subject = ! empty( $_POST['subject'] )
     794            ? $_POST['subject']
     795            : '';
    792796
    793797        return apply_filters( 'bp_get_messages_subject_value', $subject );
    … …  
    809813     */
    810814    function bp_get_messages_content_value() {
    811         $content = '';
    812         if ( !empty( $_POST['content'] ) )
    813             $content = $_POST['content'];
     815
     816        $content = !empty( $_POST['content'] )
     817            ? $_POST['content']
     818            : '';
    814819
    815820        return apply_filters( 'bp_get_messages_content_value', $content );
    … …  
    851856    global $messages_template;
    852857
    853     if ( $messages_template->thread->is_active )
    854         return true;
    855 
    856     return false;
     858    $retval = ! empty( $messages_template->thread->is_active )
     859        ? true
     860        : false;
     861
     862    return apply_filters( 'bp_messages_is_active_notice', $retval );
    857863}
    858864
    … …  
    881887    function bp_get_message_is_active_notice() {
    882888
    883         $string = '';
    884         if ( bp_messages_is_active_notice() )
    885             $string = __( 'Currently Active', 'buddypress' );
     889        $string = bp_messages_is_active_notice()
     890            ? __( 'Currently Active', 'buddypress' )
     891            : '';
    886892
    887893        return apply_filters( 'bp_get_message_is_active_notice', $string );
    … …  
    917923    function bp_get_message_notice_post_date() {
    918924        global $messages_template;
    919         return apply_filters( 'bp_get_message_notice_post_date', bp_format_time( strtotime($messages_template->thread->date_sent) ) );
     925        return apply_filters( 'bp_get_message_notice_post_date', bp_format_time( strtotime( $messages_template->thread->date_sent ) ) );
    920926    }
    921927
    … …  
    964970     */
    965971    function bp_get_message_notice_delete_link() {
    966         global $messages_template, $bp;
    967 
    968         return apply_filters( 'bp_get_message_notice_delete_link', wp_nonce_url( bp_loggedin_user_domain() . $bp->messages->slug . '/notices/delete/' . $messages_template->thread->id, 'messages_delete_thread' ) );
     972        global $messages_template;
     973
     974        return apply_filters( 'bp_get_message_notice_delete_link', wp_nonce_url( bp_loggedin_user_domain() . bp_get_messages_slug() . '/notices/delete/' . $messages_template->thread->id, 'messages_delete_thread' ) );
    969975    }
    970976
    … …  
    981987     */
    982988    function bp_get_message_activate_deactivate_link() {
    983         global $messages_template, $bp;
    984 
    985         if ( 1 == (int) $messages_template->thread->is_active ) {
    986             $link = wp_nonce_url( trailingslashit( bp_loggedin_user_domain() . $bp->messages->slug . '/notices/deactivate/' . $messages_template->thread->id ), 'messages_deactivate_notice' );
     989        global $messages_template;
     990
     991        if ( 1 === (int) $messages_template->thread->is_active ) {
     992            $link = wp_nonce_url( trailingslashit( bp_loggedin_user_domain() . bp_get_messages_slug() . '/notices/deactivate/' . $messages_template->thread->id ), 'messages_deactivate_notice' );
    987993        } else {
    988             $link = wp_nonce_url( trailingslashit( bp_loggedin_user_domain() . $bp->messages->slug . '/notices/activate/' . $messages_template->thread->id ), 'messages_activate_notice' );
     994            $link = wp_nonce_url( trailingslashit( bp_loggedin_user_domain() . bp_get_messages_slug() . '/notices/activate/' . $messages_template->thread->id ), 'messages_activate_notice' );
    989995        }
    990996        return apply_filters( 'bp_get_message_activate_deactivate_link', $link );
    … …  
    10051011        global $messages_template;
    10061012
    1007         if ( 1 == (int) $messages_template->thread->is_active  ) {
     1013        if ( 1 === (int) $messages_template->thread->is_active  ) {
    10081014            $text = __('Deactivate', 'buddypress');
    10091015        } else {
    … …  
    10311037     */
    10321038    function bp_get_messages_slug() {
    1033         global $bp;
    1034         return apply_filters( 'bp_get_messages_slug', $bp->messages->slug );
     1039        return apply_filters( 'bp_get_messages_slug', buddypress()->messages->slug );
    10351040    }
    10361041
    … …  
    10411046    $notice = BP_Messages_Notice::get_active();
    10421047
    1043     if ( empty( $notice ) )
     1048    if ( empty( $notice ) ) {
    10441049        return false;
     1050    }
    10451051
    10461052    $closed_notices = bp_get_user_meta( bp_loggedin_user_id(), 'closed_notices', true );
    10471053
    1048     if ( !$closed_notices )
     1054    if ( empty( $closed_notices ) ) {
    10491055        $closed_notices = array();
    1050 
    1051     if ( is_array($closed_notices) ) {
     1056    }
     1057
     1058    if ( is_array( $closed_notices ) ) {
    10521059        if ( !in_array( $notice->id, $closed_notices ) && $notice->id ) {
    10531060            ?>
    1054             <div id="message" class="info notice" rel="n-<?php echo $notice->id ?>">
     1061            <div id="message" class="info notice" rel="n-<?php echo esc_attr( $notice->id ); ?>">
    10551062                <p>
    10561063                    <strong><?php echo stripslashes( wp_filter_kses( $notice->subject ) ) ?></strong><br />
    … …  
    10771084    function bp_get_send_private_message_link() {
    10781085
    1079         if ( bp_is_my_profile() || !is_user_logged_in() )
     1086        if ( bp_is_my_profile() || ! is_user_logged_in() ) {
    10801087            return false;
     1088        }
    10811089
    10821090        return apply_filters( 'bp_get_send_private_message_link', wp_nonce_url( bp_loggedin_user_domain() . bp_get_messages_slug() . '/compose/?r=' . bp_core_get_username( bp_displayed_user_id() ) ) );
    … …  
    11371145     */
    11381146    function bp_get_message_loading_image_src() {
    1139         global $bp;
    1140         return apply_filters( 'bp_get_message_loading_image_src', $bp->messages->image_base . '/ajax-loader.gif' );
     1147        return apply_filters( 'bp_get_message_loading_image_src', buddypress()->messages->image_base . '/ajax-loader.gif' );
    11411148    }
    11421149
    … …  
    11481155
    11491156    foreach ( $recipients as $recipient ) {
    1150         $user_id = bp_is_username_compatibility_mode() ? bp_core_get_userid( $recipient ) : bp_core_get_userid_from_nicename( $recipient );
    1151 
    1152         if ( $user_id ) : ?>
     1157
     1158        $user_id = bp_is_username_compatibility_mode()
     1159            ? bp_core_get_userid( $recipient )
     1160            : bp_core_get_userid_from_nicename( $recipient );
     1161
     1162        if ( ! empty( $user_id ) ) : ?>
    11531163
    11541164            <li id="un-<?php echo esc_attr( $recipient ); ?>" class="friend-tab">
    … …  
    12831293     */
    12841294    function has_messages() {
    1285         if ( $this->message_count )
     1295        if ( $this->message_count ) {
    12861296            return true;
     1297        }
    12871298
    12881299        return false;
    … …  
    13491360
    13501361        // loop has just started
    1351         if ( 0 == $this->current_message )
    1352             do_action('thread_loop_start');
     1362        if ( 0 == $this->current_message ) {
     1363            do_action( 'thread_loop_start' );
     1364        }
    13531365    }
    13541366}
    … …  
    13691381    global $thread_template;
    13701382
    1371     $defaults = array(
     1383    $r = bp_parse_args( $args, array(
    13721384        'thread_id' => false,
    13731385        'order'     => 'ASC'
    1374     );
    1375 
    1376     $r = wp_parse_args( $args, $defaults );
    1377     extract( $r, EXTR_SKIP );
    1378 
    1379     if ( empty( $thread_id ) && bp_is_messages_component() && bp_is_current_action( 'view' ) )
    1380         $thread_id = (int) bp_action_variable( 0 );
    1381 
    1382     $thread_template = new BP_Messages_Thread_Template( $thread_id, $order );
     1386    ), 'thread_has_messages' );
     1387
     1388    if ( empty( $r['thread_id'] ) && bp_is_messages_component() && bp_is_current_action( 'view' ) ) {
     1389        $r['thread_id'] = (int) bp_action_variable( 0 );
     1390    }
     1391
     1392    $thread_template = new BP_Messages_Thread_Template( $r['thread_id'], $r['order'] );
     1393
    13831394    return $thread_template->has_messages();
    13841395}
    … …  
    14721483        $recipient_links = array();
    14731484
    1474         if ( count( $thread_template->thread->recipients ) >= 5 )
    1475             return apply_filters( 'bp_get_the_thread_recipients', sprintf( __( '%d Recipients', 'buddypress' ), count($thread_template->thread->recipients) ) );
     1485        if ( count( $thread_template->thread->recipients ) >= 5 ) {
     1486            return apply_filters( 'bp_get_the_thread_recipients', sprintf( __( '%d Recipients', 'buddypress' ), count( $thread_template->thread->recipients ) ) );
     1487        }
    14761488
    14771489        foreach( (array) $thread_template->thread->recipients as $recipient ) {
    … …  
    15081520        global $thread_template;
    15091521
    1510         $thread_message_id = isset( $thread_template->message->id ) ? (int) $thread_template->message->id : null;
     1522        $thread_message_id = isset( $thread_template->message->id )
     1523            ? (int) $thread_template->message->id
     1524            : null;
     1525
    15111526        return apply_filters( 'bp_get_the_thread_message_id', $thread_message_id );
    15121527    }
    … …  
    16201635        global $thread_template;
    16211636
    1622         $defaults = array(
     1637        $r = bp_parse_args( $args, array(
    16231638            'type'   => 'thumb',
    16241639            'width'  => false,
    16251640            'height' => false,
    1626         );
    1627 
    1628         $r = wp_parse_args( $args, $defaults );
    1629         extract( $r, EXTR_SKIP );
    1630 
    1631         return apply_filters( 'bp_get_the_thread_message_sender_avatar_thumb', bp_core_fetch_avatar( array( 'item_id' => $thread_template->message->sender_id, 'type' => $type, 'width' => $width, 'height' => $height, 'alt' => bp_core_get_user_displayname( $thread_template->message->sender_id ) ) ) );
     1641        ) );
     1642
     1643        return apply_filters( 'bp_get_the_thread_message_sender_avatar_thumb', bp_core_fetch_avatar( array(
     1644            'item_id' => $thread_template->message->sender_id,
     1645            'type'    => $r['type'],
     1646            'width'   => $r['width'],
     1647            'height'  => $r['height'],
     1648            'alt'     => bp_core_get_user_displayname( $thread_template->message->sender_id )
     1649        ) ) );
    16321650    }
    16331651
    … …  
    16651683        $display_name = bp_core_get_user_displayname( $thread_template->message->sender_id );
    16661684
    1667         if ( empty( $display_name ) )
     1685        if ( empty( $display_name ) ) {
    16681686            $display_name = __( 'Deleted User', 'buddypress' );
     1687        }
    16691688
    16701689        return apply_filters( 'bp_get_the_thread_message_sender_name', $display_name );
    … …  
    16831702     */
    16841703    function bp_get_the_thread_delete_link() {
    1685         global $bp;
    1686 
    1687         return apply_filters( 'bp_get_message_thread_delete_link', wp_nonce_url( bp_loggedin_user_domain() . $bp->messages->slug . '/inbox/delete/' . bp_get_the_thread_id(), 'messages_delete_thread' ) );
     1704        return apply_filters( 'bp_get_message_thread_delete_link', wp_nonce_url( bp_loggedin_user_domain() . bp_get_messages_slug() . '/inbox/delete/' . bp_get_the_thread_id(), 'messages_delete_thread' ) );
    16881705    }
    16891706
Note: See TracChangeset for help on using the changeset viewer.