Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
08/26/2021 12:53:27 AM (5 years ago)
Author:
espellcaste
Message:

Making PHPDoc Improvements to the BP Messages (component) files.

Also, adding several minor PHP changes.

See #8553

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/bp-messages/bp-messages-template.php

    r12990 r13096  
    9494 * Check whether there are more threads to iterate over.
    9595 *
     96 * @global BP_Messages_Box_Template $messages_template
     97 *
    9698 * @return bool
    9799 */
     
    104106 * Set up the current thread inside the loop.
    105107 *
    106  * @return object
     108 * @global BP_Messages_Box_Template $messages_template
     109 *
     110 * @return BP_Messages_Thread
    107111 */
    108112function bp_message_thread() {
     
    120124         * Get the ID of the current thread in the loop.
    121125         *
     126         * @global BP_Messages_Box_Template $messages_template
     127         *
    122128         * @return int
    123129         */
     
    132138                 * @param int $thread_id ID of the current thread in the loop.
    133139                 */
    134                 return apply_filters( 'bp_get_message_thread_id', $messages_template->thread->thread_id );
     140                return apply_filters( 'bp_get_message_thread_id', (int) $messages_template->thread->thread_id );
    135141        }
    136142
     
    144150         * Get the subject of the current thread in the loop.
    145151         *
     152         * @global BP_Messages_Box_Template $messages_template
     153         *
    146154         * @return string
    147155         */
     
    168176         * Generate an excerpt from the current message in the loop.
    169177         *
     178         * @global BP_Messages_Box_Template $messages_template
     179         *
    170180         * @return string
    171181         */
     
    208218         * @since 2.0.0
    209219         *
     220         * @global BP_Messages_Box_Template $messages_template
     221         *
    210222         * @return string The raw content of the last message in the thread.
    211223         */
     
    232244         * Get a link to the page of the current thread's last author.
    233245         *
     246         * @global BP_Messages_Box_Template $messages_template
     247         *
    234248         * @return string
    235249         */
     
    256270         * Generate HTML links to the pages of the current thread's recipients.
    257271         *
     272         * @global BP_Messages_Box_Template $messages_template
     273         *
    258274         * @return string
    259275         */
     
    268284                 * @param string $value HTML links to the pages of the current thread's recipients.
    269285                 */
    270                 return apply_filters( 'bp_message_thread_to', BP_Messages_Thread::get_recipient_links($messages_template->thread->recipients ) );
     286                return apply_filters( 'bp_message_thread_to', BP_Messages_Thread::get_recipient_links( $messages_template->thread->recipients ) );
    271287        }
    272288
     
    288304         *
    289305         * @since 2.9.0 Introduced `$user_id` parameter.
     306         *
     307         * @global BP_Messages_Box_Template $messages_template
    290308         *
    291309         * @param int $thread_id Optional. ID of the thread. Default: current
     
    304322                }
    305323
    306                 if ( null === $user_id ) {
     324                if ( empty( $user_id ) ) {
    307325                        $user_id = bp_loggedin_user_id();
    308326                }
     
    329347 * @since 2.9.0 Introduced `$user_id` parameter.
    330348 *
    331  * @param int $user_id Optional. ID of the user relative to whom the link
    332  *                     should be generated. Default: ID of logged-in user.
     349 * @param int|null $user_id Optional. ID of the user relative to whom the link
     350 *                          should be generated. Default: ID of logged-in user.
    333351 */
    334352function bp_message_thread_delete_link( $user_id = null ) {
     
    340358         * @since 2.9.0 Introduced `$user_id` parameter.
    341359         *
    342          * @param int $user_id Optional. ID of the user relative to whom the link
    343          *                     should be generated. Default: ID of logged-in user.
     360         * @global BP_Messages_Box_Template $messages_template
     361         *
     362         * @param int|null $user_id Optional. ID of the user relative to whom the link
     363         *                          should be generated. Default: ID of logged-in user.
    344364         * @return string
    345365         */
     
    347367                global $messages_template;
    348368
    349                 if ( null === $user_id ) {
     369                if ( empty( $user_id ) ) {
    350370                        $user_id = bp_loggedin_user_id();
    351371                }
     
    372392 * @since 2.9.0 Introduced `$user_id` parameter.
    373393 *
    374  * @param int $user_id Optional. ID of the user relative to whom the link
    375  *                     should be generated. Default: ID of logged-in user.
     394 * @param int|null $user_id Optional. ID of the user relative to whom the link
     395 *                          should be generated. Default: ID of logged-in user.
    376396 */
    377397function bp_the_message_thread_mark_unread_url( $user_id = null ) {
     
    384404         * @since 2.9.0 Introduced `$user_id` parameter.
    385405         *
    386          * @param int $user_id Optional. ID of the user relative to whom the link
    387          *                     should be generated. Default: ID of logged-in user.
     406         * @param int|null $user_id Optional. ID of the user relative to whom the link
     407         *                          should be generated. Default: ID of logged-in user.
    388408         * @return string
    389409         */
     
    396416                $args = array(
    397417                        'action'     => 'unread',
    398                         'message_id' => $id
     418                        'message_id' => $id,
    399419                );
    400420
    401                 if ( null === $user_id ) {
     421                if ( empty( $user_id ) ) {
    402422                        $user_id = bp_loggedin_user_id();
    403423                }
     
    434454 * @since 2.9.0 Introduced `$user_id` parameter.
    435455 *
    436  * @param int $user_id Optional. ID of the user relative to whom the link
    437  *                     should be generated. Default: ID of logged-in user.
     456 * @param int|null $user_id Optional. ID of the user relative to whom the link
     457 *                          should be generated. Default: ID of logged-in user.
    438458 */
    439459function bp_the_message_thread_mark_read_url( $user_id = null ) {
     
    446466         * @since 2.9.0 Introduced `$user_id` parameter.
    447467         *
    448          * @param int $user_id Optional. ID of the user relative to whom the link
    449          *                     should be generated. Default: ID of logged-in user.
     468         * @param int|null $user_id Optional. ID of the user relative to whom the link
     469         *                          should be generated. Default: ID of logged-in user.
    450470         * @return string
    451471         */
     
    458478                $args = array(
    459479                        'action'     => 'read',
    460                         'message_id' => $id
     480                        'message_id' => $id,
    461481                );
    462482
    463                 if ( null === $user_id ) {
     483                if ( empty( $user_id ) ) {
    464484                        $user_id = bp_loggedin_user_id();
    465485                }
     
    484504                 * @param int    $user_id ID of the user relative to whom the link should be generated.
    485505                 */
    486                 return apply_filters( 'bp_get_the_message_thread_mark_read_url', $url );
     506                return apply_filters( 'bp_get_the_message_thread_mark_read_url', $url, $user_id );
    487507        }
    488508
     
    496516         * Generate the CSS class for the current thread.
    497517         *
     518         * @global BP_Messages_Box_Template $messages_template
     519         *
    498520         * @return string
    499521         */
     
    503525                $class = false;
    504526
    505                 if ( $messages_template->current_thread % 2 == 1 ) {
     527                if ( $messages_template->current_thread % 2 === 1 ) {
    506528                        $class .= 'alt';
    507529                }
     
    520542 * Check whether the current thread has unread items.
    521543 *
     544 * @global BP_Messages_Box_Template $messages_template
     545 *
    522546 * @return bool True if there are unread items, otherwise false.
    523547 */
     
    525549        global $messages_template;
    526550
    527         $retval = ! empty( $messages_template->thread->unread_count )
    528                 ? true
    529                 : false;
     551        $retval = ! empty( $messages_template->thread->unread_count );
    530552
    531553        /**
     
    547569        /**
    548570         * Get the current thread's unread count.
     571         *
     572         * @global BP_Messages_Box_Template $messages_template
    549573         *
    550574         * @return int
     
    564588                 * @param int $count Current thread unread count.
    565589                 */
    566                 return apply_filters( 'bp_get_message_thread_unread_count', $count );
     590                return apply_filters( 'bp_get_message_thread_unread_count', (int) $count );
    567591        }
    568592
     
    654678         * Get the unformatted date of the last post in the current thread.
    655679         *
     680         * @global BP_Messages_Box_Template $messages_template
     681         *
    656682         * @return string
    657683         */
     
    707733         * @see bp_core_fetch_avatar() For a description of arguments and
    708734         *      return values.
     735         *
     736         * @global BP_Messages_Box_Template $messages_template
    709737         *
    710738         * @param array|string $args {
     
    759787 * Output the unread messages count for the current inbox.
    760788 *
    761  * @since 2.6.x Added $user_id argument.
     789 * @since 2.6.x Added the `$user_id` paremeter.
    762790 *
    763791 * @param int $user_id The user ID.
    764  *
    765  * @return int $unread_count Total inbox unread count for user.
    766792 */
    767793function bp_total_unread_messages_count( $user_id = 0 ) {
     
    771797         * Get the unread messages count for the current inbox.
    772798         *
    773          * @since 2.6.x Added $user_id argument.
     799         * @since 2.6.x Added the `$user_id` paremeter.
    774800         *
    775801         * @param int $user_id The user ID.
     
    784810                 * @since 1.0.0
    785811                 *
    786                  * @param int $value Unread messages count for the current inbox.
    787                  */
    788                 return apply_filters( 'bp_get_total_unread_messages_count', BP_Messages_Thread::get_inbox_count( $user_id ) );
     812                 * @param int $value   Unread messages count for the current inbox.
     813                 * @param int $user_id ID of the user the messages are from.
     814                 */
     815                return apply_filters( 'bp_get_total_unread_messages_count', BP_Messages_Thread::get_inbox_count( $user_id ), $user_id );
    789816        }
    790817
     
    798825         * Get the pagination HTML for the current thread loop.
    799826         *
     827         * @global BP_Messages_Box_Template $messages_template
     828         *
    800829         * @return string
    801830         */
     
    815844/**
    816845 * Generate the "Viewing message x to y (of z messages)" string for a loop.
     846 *
     847 * @global BP_Messages_Box_Template $messages_template
    817848 */
    818849function bp_messages_pagination_count() {
     
    10581089 * @since 1.6.0
    10591090 *
     1091 * @global BP_Messages_Box_Template $messages_template
     1092 *
    10601093 * @return bool
    10611094 */
     
    10841117 * @since 1.0.0
    10851118 * @deprecated 1.6.0
    1086  * @return bool
    10871119 */
    10881120function bp_message_is_active_notice() {
     
    11121144 */
    11131145function bp_message_notice_id() {
    1114         echo (int) bp_get_message_notice_id();
     1146        echo bp_get_message_notice_id();
    11151147}
    11161148        /**
    11171149         * Get the ID of the current notice in the loop.
     1150         *
     1151         * @global BP_Messages_Box_Template $messages_template
    11181152         *
    11191153         * @return int
     
    11291163                 * @param int $id ID of the current notice in the loop.
    11301164                 */
    1131                 return apply_filters( 'bp_get_message_notice_id', $messages_template->thread->id );
     1165                return apply_filters( 'bp_get_message_notice_id', (int) $messages_template->thread->id );
    11321166        }
    11331167
     
    11411175         * Get the post date of the current notice in the loop.
    11421176         *
     1177         * @global BP_Messages_Box_Template $messages_template
     1178         *
    11431179         * @return string
    11441180         */
     
    11711207         * @since 5.0.0 The $notice parameter has been added.
    11721208         *
    1173          * @param BP_Messages_Notice $notice The notice object.
     1209         * @global BP_Messages_Box_Template $messages_template
     1210         *
     1211         * @param BP_Messages_Notice|null $notice The notice object.
    11741212         * @return string
    11751213         */
     
    12061244         * @since 5.0.0 The $notice parameter has been added.
    12071245         *
    1208          * @param BP_Messages_Notice $notice The notice object.
     1246         * @global BP_Messages_Box_Template $messages_template
     1247         *
     1248         * @param BP_Messages_Notice|null $notice The notice object.
    12091249         * @return string
    12101250         */
     
    12351275         * Get the URL for deleting the current notice.
    12361276         *
     1277         * @global BP_Messages_Box_Template $messages_template
     1278         *
    12371279         * @return string Delete URL.
    12381280         */
     
    12591301        /**
    12601302         * Get the URL for deactivating the current notice.
     1303         *
     1304         * @global BP_Messages_Box_Template $messages_template
    12611305         *
    12621306         * @return string
     
    12901334         * Generate the text ('Deactivate' or 'Activate') for the notice action link.
    12911335         *
     1336         * @global BP_Messages_Box_Template $messages_template
     1337         *
    12921338         * @return string
    12931339         */
     
    13151361 *
    13161362 * @since 9.0.0
    1317  * @return string URL for dismissing the current notice for the current user.
    13181363 */
    13191364function bp_message_notice_dismiss_link() {
     
    13851430
    13861431        if ( is_array( $closed_notices ) ) {
    1387                 if ( !in_array( $notice->id, $closed_notices ) && $notice->id ) {
     1432                if ( ! in_array( $notice->id, $closed_notices, true ) && $notice->id ) {
    13881433                        ?>
    13891434                        <div id="message" class="info notice" rel="n-<?php echo esc_attr( $notice->id ); ?>">
     
    15771622 * Initialize the messages template loop for a specific thread.
    15781623 *
     1624 * @global BP_Messages_Thread_Template $thread_template
     1625 *
    15791626 * @param array|string $args {
    15801627 *     Array of arguments. All are optional.
     
    16191666         * Get the 'ASC' or 'DESC' messages order string for this loop.
    16201667         *
     1668         * @global BP_Messages_Thread_Template $thread_template
     1669         *
    16211670         * @return string
    16221671         */
    16231672        function bp_get_thread_messages_order() {
    16241673                global $thread_template;
     1674
    16251675                return $thread_template->thread->messages_order;
    16261676        }
     
    16281678/**
    16291679 * Check whether there are more messages to iterate over.
     1680 *
     1681 * @global BP_Messages_Thread_Template $thread_template
    16301682 *
    16311683 * @return bool
     
    16401692 * Set up the current thread inside the loop.
    16411693 *
    1642  * @return object
     1694 * @global BP_Messages_Thread_Template $thread_template
     1695 *
     1696 * @return BP_Messages_Message
    16431697 */
    16441698function bp_thread_the_message() {
     
    16571711         * Get the ID of the thread that the current loop belongs to.
    16581712         *
     1713         * @global BP_Messages_Thread_Template $thread_template
     1714         *
    16591715         * @return int
    16601716         */
     
    16801736        /**
    16811737         * Get the subject of the thread currently being iterated over.
     1738         *
     1739         * @global BP_Messages_Thread_Template $thread_template
    16821740         *
    16831741         * @return string
     
    17131771        }
    17141772
     1773        /**
     1774         * Filters the thread recipients.
     1775         *
     1776         * @since 10.0.0
     1777         *
     1778         * @param string $recipients List of thread recipients.
     1779         */
    17151780        return apply_filters( 'bp_get_the_thread_recipients', $recipients );
    17161781}
     
    17201785 *
    17211786 * @since 2.2.0
     1787 *
     1788 * @global BP_Messages_Thread_Template $thread_template
    17221789 *
    17231790 * @return int
     
    17251792function bp_get_thread_recipients_count() {
    17261793        global $thread_template;
     1794
    17271795        /**
    17281796         * Filters the total number of recipients in a thread.
     
    17651833         *
    17661834         * @since 2.2.0
     1835         *
     1836         * @global BP_Messages_Thread_Template $thread_template
    17671837         *
    17681838         * @return string
     
    18061876 */
    18071877function bp_the_thread_message_id() {
    1808         echo (int) bp_get_the_thread_message_id();
     1878        echo bp_get_the_thread_message_id();
    18091879}
    18101880        /**
     
    18121882         *
    18131883         * @since 1.9.0
     1884         *
     1885         * @global BP_Messages_Thread_Template $thread_template
    18141886         *
    18151887         * @return int
     
    18201892                $thread_message_id = isset( $thread_template->message->id )
    18211893                        ? (int) $thread_template->message->id
    1822                         : null;
     1894                        : 0;
    18231895
    18241896                /**
     
    18291901                 * @param int $thread_message_id ID of the current message in the thread.
    18301902                 */
    1831                 return apply_filters( 'bp_get_the_thread_message_id', $thread_message_id );
     1903                return (int) apply_filters( 'bp_get_the_thread_message_id', (int) $thread_message_id );
    18321904        }
    18331905
     
    18451917         * @since 2.1.0
    18461918         *
     1919         * @global BP_Messages_Thread_Template $thread_template
     1920         *
    18471921         * @return string
    18481922         */
     
    18591933
    18601934                // Whether the sender is the same as the logged-in user.
    1861                 if ( bp_loggedin_user_id() == $thread_template->message->sender_id ) {
     1935                if ( bp_loggedin_user_id() === $thread_template->message->sender_id ) {
    18621936                        $classes[] = 'sent-by-me';
    18631937                }
     
    18841958         * Get the CSS class used for message zebra striping.
    18851959         *
     1960         * @global BP_Messages_Thread_Template $thread_template
     1961         *
    18861962         * @return string
    18871963         */
     
    18891965                global $thread_template;
    18901966
    1891                 if ( $thread_template->current_message % 2 == 1 ) {
     1967                $class = 'odd';
     1968                if ( 1 === $thread_template->current_message % 2 ) {
    18921969                        $class = 'even alt';
    1893                 } else {
    1894                         $class = 'odd';
    18951970                }
    18961971
     
    19111986 */
    19121987function bp_the_thread_message_sender_id() {
    1913         echo (int) bp_get_the_thread_message_sender_id();
     1988        echo bp_get_the_thread_message_sender_id();
    19141989}
    19151990        /**
     
    19171992         *
    19181993         * @since 2.1.0
     1994         *
     1995         * @global BP_Messages_Thread_Template $thread_template
    19191996         *
    19201997         * @return int
     
    19482025        /**
    19492026         * Get the avatar for the current message sender.
     2027         *
     2028         * @global BP_Messages_Thread_Template $thread_template
    19502029         *
    19512030         * @param array|string $args {
     
    19982077         * @since 1.1.0
    19992078         *
     2079         * @global BP_Messages_Thread_Template $thread_template
     2080         *
    20002081         * @return string
    20012082         */
     
    20252106         *
    20262107         * @since 1.1.0
     2108         *
     2109         * @global BP_Messages_Thread_Template $thread_template
    20272110         *
    20282111         * @return string
     
    21222205         * @since 2.1.0
    21232206         *
     2207         * @global BP_Messages_Thread_Template $thread_template
    21242208         *
    21252209         * @return int
     
    21502234         *
    21512235         * @since 1.1.0
     2236         *
     2237         * @global BP_Messages_Thread_Template $thread_template
    21522238         *
    21532239         * @return string
     
    21832269 */
    21842270function bp_messages_embed() {
    2185         add_filter( 'embed_post_id',         'bp_get_the_thread_message_id' );
    2186         add_filter( 'bp_embed_get_cache',    'bp_embed_message_cache',      10, 3 );
     2271        add_filter( 'embed_post_id', 'bp_get_the_thread_message_id' );
     2272        add_filter( 'bp_embed_get_cache', 'bp_embed_message_cache', 10, 3 );
    21872273        add_action( 'bp_embed_update_cache', 'bp_embed_message_save_cache', 10, 3 );
    21882274}
Note: See TracChangeset for help on using the changeset viewer.