Changeset 8781

Timestamp:

05/13/2019 05:49:46 AM (5 years ago)

Author:

tellyworth

Message:

Translation Teams: show current and past contributors separately.

See #2320.

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/wp-i18n-teams

Files:

• views/locale-details.php (modified) (2 diffs)
• wp-i18n-teams.php (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wp-i18n-teams/views/locale-details.php

@@ -130 +130 @@
 
 <?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>
@@ -136 +136 @@
         $translators = array();
         foreach ( $locale_data['translators'] 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 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>',

sites/trunk/wordpress.org/public_html/wp-content/plugins/wp-i18n-teams/wp-i18n-teams.php

@@ -282 +282 @@
         $locale_data['project_validators'] = $contributors['project_validators'];
         $locale_data['translators'] = $contributors['translators'];
+        $locale_data['translators_past'] = $contributors['translators_past'];
 
         return $locale_data;
@@ -302 +303 @@
         $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 );
@@ -478 +480 @@
      * @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 ) {