Changeset 13565

Timestamp:

04/18/2024 10:06:30 AM (15 months ago)

Author:

amieiro

Message:

Translate: Sync "Translation Events" from GitHub

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-translation-events

Files:

• includes/stats/stats-calculator.php (modified) (1 diff)
• templates/event.php (modified) (7 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-translation-events/includes/stats/stats-calculator.php

@@ -303 +303 @@
 
     /**
-     * Check if a user is a first time contributor.
+     * Check if a user is a new translation contributor. A new contributor is a user who has made 10 or fewer translations before event start time.
      *
      * @param Event_Start_Date $event_start The event start date.
      * @param int              $user_id      The user ID.
      *
-     * @return bool True if the user is a first time contributor, false otherwise.
-     */
-    public function is_first_time_contributor( $event_start, $user_id ) {
-        global $wpdb, $gp_table_prefix;
-
+     * @return bool True if the user is a new translation contributor, false otherwise.
+     */
+    public function is_new_translation_contributor( $event_start, $user_id ) {
+        global $wpdb, $gp_table_prefix;
+        $new_contributor_max_translation_count = 10;
+        $event_start_date_time                 = $event_start->__toString();
         // phpcs:disable WordPress.DB.PreparedSQL.InterpolatedNotPrepared
         // phpcs:disable WordPress.DB.DirectDatabaseQuery.DirectQuery
         // phpcs:disable WordPress.DB.DirectDatabaseQuery.NoCaching
         // phpcs:disable WordPress.DB.DirectDatabaseQuery.SchemaChange
-        $users_first_translation_date = $wpdb->get_var(
-            $wpdb->prepare(
-                "
-            select min(date_added) from {$gp_table_prefix}translations where user_id = %d
+        $user_translations_count = $wpdb->get_var(
+            $wpdb->prepare(
+                "
+            select count(*) from {$gp_table_prefix}translations where user_id = %d and date_added < %s
         ",
                 array(
                     $user_id,
-                )
-            )
-        );
-
-        if ( get_userdata( $user_id ) && ! $users_first_translation_date ) {
+                    $event_start_date_time,
+                )
+            )
+        );
+
+        if ( get_userdata( $user_id ) && ! $user_translations_count ) {
             return true;
         }
-        $event_start_date_time  = new DateTimeImmutable( $event_start->__toString(), new DateTimeZone( 'UTC' ) );
-        $first_translation_date = new DateTimeImmutable( $users_first_translation_date, new DateTimeZone( 'UTC' ) );
-        // A first time contributor is someone whose first translation was made not earlier than 24 hours before the event.
-        $event_start_date_time = $event_start_date_time->modify( '-1 day' );
-        return $event_start_date_time <= $first_translation_date;
+        return $user_translations_count <= $new_contributor_max_translation_count;
     }
 }

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-translation-events/templates/event.php

@@ -6 +6 @@
 namespace Wporg\TranslationEvents;
 
+use GP_Locales;
 use WP_User;
 use Wporg\TranslationEvents\Attendee\Attendee;
@@ -44 +45 @@
         <?php if ( ! empty( $contributors ) ) : ?>
             <div class="event-contributors">
-                <h2><?php esc_html_e( 'Contributors', 'gp-translation-events' ); ?></h2>
+                <h2>
+                <?php
+                // translators: %d is the number of contributors.
+                echo esc_html( sprintf( __( 'Contributors (%d)', 'gp-translation-events' ), number_format_i18n( count( $contributors ) ) ) );
+                ?>
+                </h2>
                 <ul>
                     <?php foreach ( $contributors as $contributor ) : ?>
@@ -50 +56 @@
                             <a href="<?php echo esc_url( get_author_posts_url( $contributor->ID ) ); ?>" class="avatar"><?php echo get_avatar( $contributor->ID, 48 ); ?></a>
                             <a href="<?php echo esc_url( get_author_posts_url( $contributor->ID ) ); ?>" class="name"><?php echo esc_html( get_the_author_meta( 'display_name', $contributor->ID ) ); ?></a>
-                            <?php if ( $stats_calculator->is_first_time_contributor( $event_start, $contributor->ID ) ) : ?>
+                            <?php if ( $stats_calculator->is_new_translation_contributor( $event_start, $contributor->ID ) ) : ?>
                                 <span class="first-time-contributor-tada" title="<?php esc_html_e( 'New Translation Contributor', 'gp-translation-events' ); ?>"></span>
                             <?php endif; ?>
@@ -78 +84 @@
         <?php if ( ! empty( $attendees ) && ( ! $event->end()->is_in_the_past() || ( ( $attendee instanceof Attendee && $attendee->is_host() ) || current_user_can( 'manage_options' ) || $user->ID === $event->author_id() ) ) ) : ?>
             <div class="event-attendees">
-                <h2><?php esc_html_e( 'Attendees', 'gp-translation-events' ); ?></h2>
+                <h2>
+                <?php
+                // translators: %d is the number of attendees.
+                echo esc_html( sprintf( __( 'Attendees (%d)', 'gp-translation-events' ), number_format_i18n( count( $attendees ) ) ) );
+                ?>
+                </h2>
                 <ul>
                     <?php foreach ( $attendees as $_user ) : ?>
@@ -84 +95 @@
                             <a href="<?php echo esc_url( get_author_posts_url( $_user->ID ) ); ?>" class="avatar"><?php echo get_avatar( $_user->ID, 48 ); ?></a>
                             <a href="<?php echo esc_url( get_author_posts_url( $_user->ID ) ); ?>" class="name"><?php echo esc_html( get_the_author_meta( 'display_name', $_user->ID ) ); ?></a>
-                            <?php if ( $stats_calculator->is_first_time_contributor( $event_start, $_user->ID ) ) : ?>
+                            <?php if ( $stats_calculator->is_new_translation_contributor( $event_start, $_user->ID ) ) : ?>
                                 <span class="first-time-contributor-tada" title="<?php esc_html_e( 'New Translation Contributor', 'gp-translation-events' ); ?>"></span>
                             <?php endif; ?>
@@ -147 +158 @@
                         <?php
                         foreach ( explode( ',', $row->locales ) as $_locale ) {
-                            $_locale = \GP_Locales::by_slug( $_locale );
+                            $_locale = GP_Locales::by_slug( $_locale );
                             ?>
-                            <a href="<?php echo esc_url( gp_url_project_locale( $row->project, $_locale, 'default' ) ); ?>"><?php echo esc_html( $_locale->english_name ); ?></a>
+                            <a href="<?php echo esc_url( gp_url_project_locale( $row->project, $_locale->slug, 'default' ) ); ?>"><?php echo esc_html( $_locale->english_name ); ?></a>
                             <?php
                         }
@@ -200 +211 @@
                                     function ( $contributor ) use ( $stats_calculator, $event_start ) {
                                         $append_tada = '';
-                                        if ( $stats_calculator->is_first_time_contributor( $event_start, $contributor->ID ) ) {
+                                        if ( $stats_calculator->is_new_translation_contributor( $event_start, $contributor->ID ) ) {
                                             $append_tada = '<span class="first-time-contributor-tada" title="' . esc_html__( 'New Translation Contributor', 'gp-translation-events' ) . '"></span>';
                                         }