Changeset 9720 for sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-custom-stats/stats/project.php

Timestamp:

04/14/2020 04:55:52 AM (6 years ago)

Author:

dd32

Message:

Translate: Move the stats generation for /stats from the Stats screen router to the Custom Stats plugin, store the data in the project translation status table rather than the options.

This also avoids the query-timeout problems by running the query as a INSERT INTO ... SELECT FROM which should be more resiliant to timeouts.

See #3155.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-custom-stats/stats/project.php (modified) (4 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-custom-stats/stats/project.php

@@ -9 +9 @@
  *
  * NOTE: The counts includes all sub-projects in the count, as that's more useful for querying (top-level projects excluded)
- * for example, wp-plugins won't exist, but wp-plugins/akismet will include wp-plugins/akismet/stable wp-plugins/akismet/stable-readme
+ * for example, wp-plugins/akismet will include wp-plugins/akismet/stable and wp-plugins/akismet/stable-readme
+ * wp-plugins and wp-themes are special top-level projects which are included here, where their string counts are updated
+ * on a twice-daily cron task.
  *
  * @author dd32
@@ -27 +29 @@
                 add_action( 'shutdown', array( $this, 'shutdown' ) );
 
+                // Cron task to cache the wp-themes/wp-plugins string counts.
+                if ( ! wp_next_scheduled ( 'wporg_gp_stats_cache_waiting_strings' ) ) {
+                        wp_schedule_event( time(), 'twicedaily', 'wporg_gp_stats_cache_waiting_strings' );
+                }
+                add_action( 'wporg_gp_stats_cache_waiting_strings', [ $this, 'cache_wp_themes_wp_plugins_strings' ] );
+
                 $wpdb->project_translation_status = $gp_table_prefix . 'project_translation_status';
         }
@@ -46 +54 @@
         }
 
-        // Counts up all the
+        // Count up all the strings
         function get_project_translation_counts( $project_id, $locale, $locale_slug, &$counts = array() ) {
                 if ( ! $counts ) {
@@ -75 +83 @@
 
                 return $counts;
+        }
+
+        /**
+         * Cron task to cache the string counts for the wp-themes and wp-plugins parent categories.
+         * 
+         * These don't have any translation sets, but we need to be able to query the waiting strings for them.
+         */
+        public function cache_wp_themes_wp_plugins_strings() {
+                global $wpdb;
+
+                $cached_projects = [
+                        GP::$project->by_path( 'wp-plugins' ),
+                        GP::$project->by_path( 'wp-themes' ),
+                ];
+
+                // Store the counts for these parent projects as the sum of their children.
+                $sql = "INSERT INTO {$wpdb->project_translation_status} ( `project_id`, `locale`, `locale_slug`, `all`, `current`, `waiting`, `fuzzy`, `warnings`, `untranslated`, `date_added`, `date_modified`)
+                SELECT
+                        p.parent_project_id as project_id,
+                        locale, locale_slug,
+                        SUM( stats.all ) as `all`, SUM( stats.current ) as `current`, SUM( stats.waiting ) as `waiting`,
+                        SUM( stats.fuzzy ) as `fuzzy`, SUM( stats.warnings ) as `warnings`, SUM( stats.untranslated ) as `untranslated`,
+                        NOW() as `date_added`, NOW() as `date_modified`
+                FROM {$wpdb->project_translation_status} stats
+                        LEFT JOIN {$wpdb->gp_projects} p ON stats.project_id = p.id
+                WHERE
+                        p.parent_project_id = %d
+                        AND p.active = 1
+                GROUP BY locale, locale_slug
+                ON DUPLICATE KEY UPDATE
+                        `all` = VALUES(`all`), `current` = VALUES(`current`),
+                        `waiting` = VALUES(`waiting`), `fuzzy` = VALUES(`fuzzy`),
+                        `warnings` = VALUES(`warnings`), `untranslated` = VALUES(`untranslated`),
+                        `date_modified` = VALUES(`date_modified`);
+                ";
+
+                foreach ( $cached_projects as $project ) {
+                        $wpdb->query( $wpdb->prepare( $sql, $project->id ) );
+                }
+
         }