Skip to:
Content

BuddyPress.org

Ticket #8398: 8398.2.patch

File 8398.2.patch, 15.5 KB (added by imath, 6 years ago)

Includes unit tests updates

  • src/bp-blogs/bp-blogs-template.php 

    diff --git src/bp-blogs/bp-blogs-template.php src/bp-blogs/bp-blogs-template.php
index cc119d4e9..ac4dd83a7 100644
    function bp_blog_last_active( $args = array() ) {  
    638638                // Backwards compatibility for anyone forcing a 'true' active_format.
    639639                if ( true === $r['active_format'] ) {
    640640                        /* translators: %s: human time diff of the last time the site was active. */
    641                         $r['active_format'] = _x( 'active %s', 'last time the site was active', 'buddypress' );
     641                        $r['active_format'] = _x( 'Active %s', 'last time the site was active', 'buddypress' );
    642642                }
    643643
    644644                // Blog has been posted to at least once.

  • src/bp-core/bp-core-functions.php 

    diff --git src/bp-core/bp-core-functions.php src/bp-core/bp-core-functions.php
index 0e2fe7c96..4b2341013 100644
    add_action( 'wp_head', 'bp_core_record_activity' );  
    15751575 * @since 1.0.0
    15761576 *
    15771577 * @param int|string $last_activity_date The date of last activity.
    1578  * @param string     $string             A sprintf()-able statement of the form 'active %s'.
     1578 * @param string     $string             A sprintf()-able statement of the form 'Active %s'.
    15791579 * @return string $last_active A string of the form '3 years ago'.
    15801580 */
    15811581function bp_core_get_last_activity( $last_activity_date = '', $string = '' ) {
    … … function bp_core_get_last_activity( $last_activity_date = '', $string = '' ) {  
    15971597         *
    15981598         * @param string $last_active        Last activity string based on time since date given.
    15991599         * @param string $last_activity_date The date of last activity.
    1600          * @param string $string             A sprintf()-able statement of the form 'active %s'.
     1600         * @param string $string             A sprintf()-able statement of the form 'Active %s'.
    16011601         */
    16021602        return apply_filters( 'bp_core_get_last_activity', $last_active, $last_activity_date, $string );
    16031603}

  • src/bp-core/classes/class-bp-core-user.php 

    diff --git src/bp-core/classes/class-bp-core-user.php src/bp-core/classes/class-bp-core-user.php
index 51d5dd862..ce4a6d128 100644
    class BP_Core_User {  
    205205                );
    206206
    207207                /* translators: %s: human time diff of the last time the user was active on the site. */
    208                 $this->last_active = bp_core_get_last_activity( bp_get_user_last_activity( $this->id ), _x( 'active %s', 'last time the user was active', 'buddypress' ) );
     208                $this->last_active = bp_core_get_last_activity( bp_get_user_last_activity( $this->id ), _x( 'Active %s', 'last time the user was active', 'buddypress' ) );
    209209        }
    210210
    211211        /**

  • src/bp-groups/bp-groups-blocks.php 

    diff --git src/bp-groups/bp-groups-blocks.php src/bp-groups/bp-groups-blocks.php
index 97f455a3c..063868ee5 100644
    function bp_groups_render_groups_block( $attributes = array() ) {  
    303303                        $output .= sprintf(
    304304                                '<time datetime="%1$s">%2$s</time>',
    305305                                esc_attr( bp_core_get_iso8601_date( $group->last_activity ) ),
    306                                 /* translators: %s: a human time diff. */
     306                                /* translators: %s: last activity timestamp (e.g. "Active 1 hour ago") */
    307307                                sprintf( esc_html__( 'Active %s', 'buddypress' ), bp_get_group_last_active( $group ) )
    308308                        );
    309309                } elseif ( 'popular' === $block_args['extraInfo'] ) {

  • src/bp-groups/bp-groups-widgets.php 

    diff --git src/bp-groups/bp-groups-widgets.php src/bp-groups/bp-groups-widgets.php
index 333398f52..937bb14cb 100644
    function groups_ajax_widget_groups_list() {  
    7676                                                <?php else : ?>
    7777                                                        <span class="activity" data-livestamp="<?php bp_core_iso8601_date( bp_get_group_last_active( 0, array( 'relative' => false ) ) ); ?>">
    7878                                                                <?php
    79                                                                 /* translators: %s: last activity timestamp (e.g. "active 1 hour ago") */
    80                                                                 printf( _x( 'active %s', 'last time the group was active', 'buddypress' ), bp_get_group_last_active() );
     79                                                                /* translators: %s: last activity timestamp (e.g. "Active 1 hour ago") */
     80                                                                printf( _x( 'Active %s', 'last time the group was active', 'buddypress' ), bp_get_group_last_active() );
    8181                                                                ?>
    8282                                                        </span>
    8383                                                <?php endif; ?>

  • src/bp-groups/classes/class-bp-groups-invite-template.php 

    diff --git src/bp-groups/classes/class-bp-groups-invite-template.php src/bp-groups/classes/class-bp-groups-invite-template.php
index dc36224cc..c0d9050d6 100644
    class BP_Groups_Invite_Template {  
    275275                $this->invite->user->user_url  = bp_core_get_user_domain( $user_id, $this->invite->user->user_nicename, $this->invite->user->user_login );
    276276                $this->invite->user->user_link = "<a href='{$this->invite->user->user_url}'>{$this->invite->user->fullname}</a>";
    277277
    278                 /* translators: %s: last activity timestamp (e.g. "active 1 hour ago") */
    279                 $this->invite->user->last_active = bp_core_get_last_activity( $this->invite->user->last_activity, __( 'active %s', 'buddypress' ) );
     278                /* translators: %s: last activity timestamp (e.g. "Active 1 hour ago") */
     279                $this->invite->user->last_active = bp_core_get_last_activity( $this->invite->user->last_activity, __( 'Active %s', 'buddypress' ) );
    280280
    281281                if ( bp_is_active( 'groups' ) ) {
    282282                        $total_groups = BP_Groups_Member::total_group_count( $user_id );

  • src/bp-groups/classes/class-bp-groups-widget.php 

    diff --git src/bp-groups/classes/class-bp-groups-widget.php src/bp-groups/classes/class-bp-groups-widget.php
index 90862063a..08eadf14e 100644
    class BP_Groups_Widget extends WP_Widget {  
    149149                                                                        } elseif ( 'popular' == $instance['group_default'] ) {
    150150                                                                                bp_group_member_count();
    151151                                                                        } else {
    152                                                                                 /* translators: %s: last activity timestamp (e.g. "active 1 hour ago") */
    153                                                                                 printf( __( 'active %s', 'buddypress' ), bp_get_group_last_active() );
     152                                                                                /* translators: %s: last activity timestamp (e.g. "Active 1 hour ago") */
     153                                                                                printf( __( 'Active %s', 'buddypress' ), bp_get_group_last_active() );
    154154                                                                        }
    155155                                                                ?>
    156156                                                                </span>

  • src/bp-members/bp-members-blocks.php 

    diff --git src/bp-members/bp-members-blocks.php src/bp-members/bp-members-blocks.php
index c9944bc32..11886548b 100644
    function bp_members_render_members_block( $attributes = array() ) {  
    332332                                $output .= sprintf(
    333333                                        '<time datetime="%1$s">%2$s</time>',
    334334                                        esc_attr( bp_core_get_iso8601_date( $member->last_activity ) ),
    335                                         /* translators: %s: a human time diff. */
     335                                        /* translators: %s: last activity timestamp (e.g. "Active 1 hour ago") */
    336336                                        sprintf( esc_html__( 'Active %s', 'buddypress' ), bp_core_time_since( $member->last_activity ) )
    337337                                );
    338338                        }

  • src/bp-members/bp-members-template.php 

    diff --git src/bp-members/bp-members-template.php src/bp-members/bp-members-template.php
index e202ea11b..70e73b779 100644
    function bp_member_last_active( $args = array() ) {  
    955955
    956956                // Backwards compatibility for anyone forcing a 'true' active_format.
    957957                if ( true === $r['active_format'] ) {
    958                         $r['active_format'] = __( 'active %s', 'buddypress' );
     958                        /* translators: %s: last activity timestamp (e.g. "Active 1 hour ago") */
     959                        $r['active_format'] = __( 'Active %s', 'buddypress' );
    959960                }
    960961
    961962                // Member has logged in at least one time.
    … … function bp_last_activity( $user_id = 0 ) {  
    16861687         */
    16871688        function bp_get_last_activity( $user_id = 0 ) {
    16881689
    1689                 if ( empty( $user_id ) )
     1690                if ( empty( $user_id ) ) {
    16901691                        $user_id = bp_displayed_user_id();
     1692                }
    16911693
    1692                 $last_activity = bp_core_get_last_activity( bp_get_user_last_activity( $user_id ), __('active %s', 'buddypress') );
     1694                /* translators: %s: last activity timestamp (e.g. "Active 1 hour ago") */
     1695                $last_activity = bp_core_get_last_activity( bp_get_user_last_activity( $user_id ), __('Active %s', 'buddypress') );
    16931696
    16941697                /**
    16951698                 * Filters the 'active [x days ago]' string for a user.

  • src/bp-templates/bp-legacy/buddypress/groups/groups-loop.php 

    diff --git src/bp-templates/bp-legacy/buddypress/groups/groups-loop.php src/bp-templates/bp-legacy/buddypress/groups/groups-loop.php
index b52165e8a..38e82510d 100644
    do_action( 'bp_before_groups_loop' ); ?>  
    6464
    6565                        <div class="item">
    6666                                <div class="item-title"><?php bp_group_link(); ?></div>
    67                                 <div class="item-meta"><span class="activity" data-livestamp="<?php bp_core_iso8601_date( bp_get_group_last_active( 0, array( 'relative' => false ) ) ); ?>"><?php printf( __( 'active %s', 'buddypress' ), bp_get_group_last_active() ); ?></span></div>
     67                                <div class="item-meta">
     68                                        <span class="activity" data-livestamp="<?php bp_core_iso8601_date( bp_get_group_last_active( 0, array( 'relative' => false ) ) ); ?>">
     69                                                <?php
     70                                                /* translators: %s: last activity timestamp (e.g. "Active 1 hour ago") */
     71                                                printf( __( 'Active %s', 'buddypress' ), bp_get_group_last_active() );
     72                                                ?>
     73                                        </span>
     74                                </div>
    6875
    6976                                <div class="item-desc"><?php bp_group_description_excerpt(); ?></div>
    7077

  • src/bp-templates/bp-legacy/buddypress/groups/single/cover-image-header.php 

    diff --git src/bp-templates/bp-legacy/buddypress/groups/single/cover-image-header.php src/bp-templates/bp-legacy/buddypress/groups/single/cover-image-header.php
index 34f5c361b..5409895eb 100644
    do_action( 'bp_before_group_header' ); ?>  
    6060                                do_action( 'bp_group_header_meta' ); ?>
    6161
    6262                                <span class="highlight"><?php bp_group_type(); ?></span>
    63                                 <span class="activity" data-livestamp="<?php bp_core_iso8601_date( bp_get_group_last_active( 0, array( 'relative' => false ) ) ); ?>"><?php printf( __( 'active %s', 'buddypress' ), bp_get_group_last_active() ); ?></span>
     63                                <span class="activity" data-livestamp="<?php bp_core_iso8601_date( bp_get_group_last_active( 0, array( 'relative' => false ) ) ); ?>">
     64                                        <?php
     65                                        /* translators: %s: last activity timestamp (e.g. "Active 1 hour ago") */
     66                                        printf( __( 'Active %s', 'buddypress' ), bp_get_group_last_active() );
     67                                        ?>
     68                                </span>
    6469
    6570                                <?php bp_group_description(); ?>
    6671

  • src/bp-templates/bp-legacy/buddypress/groups/single/group-header.php 

    diff --git src/bp-templates/bp-legacy/buddypress/groups/single/group-header.php src/bp-templates/bp-legacy/buddypress/groups/single/group-header.php
index 409e5d13b..b08ea5716 100644
    do_action( 'bp_before_group_header' );  
    6969
    7070<div id="item-header-content">
    7171        <span class="highlight"><?php bp_group_type(); ?></span>
    72         <span class="activity" data-livestamp="<?php bp_core_iso8601_date( bp_get_group_last_active( 0, array( 'relative' => false ) ) ); ?>"><?php printf( __( 'active %s', 'buddypress' ), bp_get_group_last_active() ); ?></span>
     72        <span class="activity" data-livestamp="<?php bp_core_iso8601_date( bp_get_group_last_active( 0, array( 'relative' => false ) ) ); ?>">
     73                <?php
     74                /* translators: %s: last activity timestamp (e.g. "Active 1 hour ago") */
     75                printf( __( 'Active %s', 'buddypress' ), bp_get_group_last_active() );
     76                ?>
     77        </span>
    7378
    7479        <?php
    7580

  • src/bp-templates/bp-nouveau/buddypress/groups/groups-loop.php 

    diff --git src/bp-templates/bp-nouveau/buddypress/groups/groups-loop.php src/bp-templates/bp-nouveau/buddypress/groups/groups-loop.php
index c0c7eb975..c523ad189 100644
    bp_nouveau_before_loop(); ?>  
    4444                                                        <p class="last-activity item-meta">
    4545                                                                <?php
    4646                                                                        printf(
    47                                                                                 /* translators: %s: last activity timestamp (e.g. "active 1 hour ago") */
     47                                                                                /* translators: %s: last activity timestamp (e.g. "Active 1 hour ago") */
    4848                                                                                esc_html__( 'Active %s', 'buddypress' ),
    4949                                                                                sprintf(
    5050                                                                                        '<span data-livestamp="%1$s">%2$s</span>',

  • src/bp-templates/bp-nouveau/buddypress/groups/single/cover-image-header.php 

    diff --git src/bp-templates/bp-nouveau/buddypress/groups/single/cover-image-header.php src/bp-templates/bp-nouveau/buddypress/groups/single/cover-image-header.php
index 356ae26e9..07439f2f8 100644
     
    3131                                <p class="activity">
    3232                                        <?php
    3333                                                printf(
    34                                                         /* translators: %s: last activity timestamp (e.g. "active 1 hour ago") */
     34                                                        /* translators: %s: last activity timestamp (e.g. "Active 1 hour ago") */
    3535                                                        esc_html__( 'Active %s', 'buddypress' ),
    3636                                                        sprintf(
    3737                                                                '<span data-livestamp="%1$s">%2$s</span>',

  • src/bp-templates/bp-nouveau/buddypress/groups/single/group-header.php 

    diff --git src/bp-templates/bp-nouveau/buddypress/groups/single/group-header.php src/bp-templates/bp-nouveau/buddypress/groups/single/group-header.php
index 11ff03a5a..92c18c48d 100644
     
    2828        <p class="activity">
    2929                <?php
    3030                        printf(
    31                                 /* translators: %s: last activity timestamp (e.g. "active 1 hour ago") */
     31                                /* translators: %s: last activity timestamp (e.g. "Active 1 hour ago") */
    3232                                esc_html__( 'Active %s', 'buddypress' ),
    3333                                sprintf(
    3434                                        '<span data-livestamp="%1$s">%2$s</span>',

  • tests/phpunit/testcases/blogs/template.php 

    diff --git tests/phpunit/testcases/blogs/template.php tests/phpunit/testcases/blogs/template.php
index db0dc0f85..e7fe54147 100644
    class BP_Tests_Blogs_Template extends BP_UnitTestCase {  
    1616                $blogs_template->blog = new stdClass;
    1717                $blogs_template->blog->last_activity = $time;
    1818
    19                 $this->assertEquals( bp_core_get_last_activity( $time, __( 'active %s', 'buddypress' ) ), bp_get_blog_last_active() );
     19                $this->assertEquals( bp_core_get_last_activity( $time, __( 'Active %s', 'buddypress' ) ), bp_get_blog_last_active() );
    2020
    2121                $blogs_template->blog = null;
    2222        }
    … … class BP_Tests_Blogs_Template extends BP_UnitTestCase {  
    3333                $blogs_template->blog = new stdClass;
    3434                $blogs_template->blog->last_activity = $time;
    3535
    36                 $this->assertEquals( bp_core_get_last_activity( $time, __( 'active %s', 'buddypress' ) ), bp_get_blog_last_active( array( 'active_format' => true, ) ) );
     36                $this->assertEquals( bp_core_get_last_activity( $time, __( 'Active %s', 'buddypress' ) ), bp_get_blog_last_active( array( 'active_format' => true, ) ) );
    3737
    3838                $blogs_template->blog = null;
    3939        }

  • tests/phpunit/testcases/members/template.php 

    diff --git tests/phpunit/testcases/members/template.php tests/phpunit/testcases/members/template.php
index 88e1614ea..1a7d7e16d 100644
    class BP_Tests_Members_Template extends BP_UnitTestCase {  
    234234                $members_template->member = new stdClass;
    235235                $members_template->member->last_activity = $time;
    236236
    237                 $this->assertEquals( bp_core_get_last_activity( $time, __( 'active %s', 'buddypress' ) ), bp_get_member_last_active() );
     237                $this->assertEquals( bp_core_get_last_activity( $time, __( 'Active %s', 'buddypress' ) ), bp_get_member_last_active() );
    238238        }
    239239
    240240        /**
    … … class BP_Tests_Members_Template extends BP_UnitTestCase {  
    249249                $members_template->member = new stdClass;
    250250                $members_template->member->last_activity = $time;
    251251
    252                 $this->assertEquals( bp_core_get_last_activity( $time, __( 'active %s', 'buddypress' ) ), bp_get_member_last_active( array( 'active_format' => true, ) ) );
     252                $this->assertEquals( bp_core_get_last_activity( $time, __( 'Active %s', 'buddypress' ) ), bp_get_member_last_active( array( 'active_format' => true, ) ) );
    253253        }
    254254
    255255        /**