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 (5 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 ) );
+        }
+
     }