Ticket #2320: 2320.diff

views/locale-details.php

@@ -129 +129 @@
 <?php endif; ?>
 
 <?php if ( ! empty( $locale_data['translators'] ) ) : ?>
-        <h2><?php printf( __( 'All Translation Contributors (%s)', 'wporg' ), number_format_i18n( count( $locale_data['translators'] ) ) ); ?></h2>
+        <h2><?php printf( __( 'Current Translation Contributors (%s)', 'wporg' ), number_format_i18n( count( $locale_data['translators'] ) ) ); ?></h2>
 
         <p>
                 <?php
@@ -146 +146 @@
         </p>
 <?php endif; ?>
 
+<?php if ( ! empty( $locale_data['translators_past'] ) ) : ?>
+        <h2><?php printf( __( 'Past Translation Contributors (%s)', 'wporg' ), number_format_i18n( count( $locale_data['translators_past'] ) ) ); ?></h2>
+
+        <p>
+                <?php
+                $translators = array();
+                foreach ( $locale_data['translators_past'] as $translator ) {
+                        $translators[] = sprintf(
+                                '<a href="https://profiles.wordpress.org/%s">%s</a>',
+                                esc_attr( $translator['nice_name'] ),
+                                esc_html( $translator['display_name'] )
+                        );
+                }
+                echo wp_sprintf( '%l.', $translators );
+                ?>
+        </p>
+<?php endif; ?>
+
 <?php
 $notice = sprintf(
         '%s <a href="https://translate.wordpress.org/locale/%s">%s</a>',

wp-i18n-teams.php

@@ -281 +281 @@
                 $locale_data['validators'] = $contributors['validators'];
                 $locale_data['project_validators'] = $contributors['project_validators'];
                 $locale_data['translators'] = $contributors['translators'];
+                $locale_data['translators_past'] = $contributors['translators_past'];
 
                 return $locale_data;
         }
@@ -301 +302 @@
                 $contributors['locale_managers'] = $this->get_locale_managers( $locale );
                 $contributors['validators'] = $this->get_general_translation_editors( $locale );
                 $contributors['project_validators'] = $this->get_project_translation_editors( $locale );
-                $contributors['translators'] = $this->get_translation_contributors( $locale );
+                $contributors['translators'] = $this->get_translation_contributors( $locale, 365 ); // Contributors from the past year
+                $contributors['translators_past'] = array_diff_key( $this->get_translation_contributors( $locale ), $contributors['translators'] );
 
                 wp_cache_set( 'contributors-data:' . $locale->wp_locale, $contributors, 'wp-i18n-teams', 2 * HOUR_IN_SECONDS );
 
@@ -477 +479 @@
          * @param GP_Locale $locale
          * @return array
          */
-        private function get_translation_contributors( $locale ) {
+        private function get_translation_contributors( $locale, $max_age_days = null ) {
                 global $wpdb;
 
                 $contributors = array();
 
+                $date_constraint = '';
+                if ( !is_null( $max_age_days ) )
+                        $date_constraint = $wpdb->prepare( " AND date_modified >= CURRENT_DATE - INTERVAL %d DAY", $max_age_days );
+
                 $users = $wpdb->get_col( $wpdb->prepare(
                         "SELECT DISTINCT user_id FROM translate_user_translations_count WHERE accepted > 0 AND locale = %s",
                         $locale->slug
-                ) );
+                ) . $date_constraint );
 
                 if ( ! $users ) {
                         $contributors;