Changeset 2880 for sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-plugin-directory/wporg-gp-plugin-directory.php

Timestamp:

04/02/2016 10:24:29 AM (10 years ago)

Author:

ocean90

Message:

Translate, Plugin Directory: Delay cache purging until shutdown to prevent multiple purges for the same project.

File:

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

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-plugin-directory/wporg-gp-plugin-directory.php

@@ -3 +3 @@
  * Plugin name: GlotPress: Plugin Directory Bridge
  * Description: Clears the content translation cache for plugins (readme/code) hosted on wordpress.org based on actions taken withing translate.wordpress.org.
- * Version:     1.0
+ * Version:     1.1
  * Author:      WordPress.org
  * Author URI:  http://wordpress.org/
@@ -13 +13 @@
         public $i18n_cache_group = 'plugins-i18n';
 
+        private $translation_edits_queue = array();
+
         public function __construct() {
                 add_action( 'init', array( $this, 'add_global_cache_group' ) );
                 add_action( 'gp_originals_imported', array( $this, 'originals_imported' ) );
-                add_action( 'gp_translation_created', array( $this, 'translation_created' ) );
-                add_action( 'gp_translation_saved', array( $this, 'translation_saved' ) );
+                add_action( 'gp_translation_created', array( $this, 'queue_translation_for_cache_purge' ) );
+                add_action( 'gp_translation_saved', array( $this, 'queue_translation_for_cache_purge' ) );
+
+                // Cache purging is delayed until shutdown to prevent multiple purges for the same project.
+                add_action( 'shutdown', array( $this, 'delete_plugin_i18n_cache_on_translation_edits' ) );
 
                 if ( defined( 'WP_CLI' ) && WP_CLI ) {
@@ -59 +64 @@
 
         /**
-         * Triggers a cache purge when a new translation was created.
-         *
-         * @param GP_Translation $translation Created translation.
-         */
-        public function translation_created( $translation ) {
+         * Adds a translation to a cache purge queue when a translation was created
+         * or updated.
+         *
+         * @param GP_Translation $translation Created/updated translation.
+         */
+        public function queue_translation_for_cache_purge( $translation ) {
                 if ( ! $this->project_is_plugin( $_SERVER['REQUEST_URI'] ) ) {
                         return;
                 }
 
-                $this->delete_plugin_i18n_cache_on_translation_edit( $translation );
-        }
-
-        /**
-         * Triggers a cache purge when a translation was updated.
-         *
-         * @param GP_Translation $translation Updated translation.
-         */
-        public function translation_saved( $translation ) {
-                if ( ! $this->project_is_plugin( $_SERVER['REQUEST_URI'] ) ) {
-                        return;
-                }
-
-                $this->delete_plugin_i18n_cache_on_translation_edit( $translation );
+                $this->translation_edits_queue[ $translation->original_id ][ $translation->translation_set_id ] = true;
+        }
+
+        /**
+         * Deletes the cache on a translation edits.
+         *
+         * @param GP_Translation $translation The edited translation.
+         */
+        public function delete_plugin_i18n_cache_on_translation_edits() {
+                if ( empty( $this->translation_edits_queue ) ) {
+                        return;
+                }
+
+                $purged = array();
+                foreach ( $this->translation_edits_queue as $original_id => $set_ids ) {
+                        $original = GP::$original->get( $original_id );
+                        if ( ! $original ) {
+                                return;
+                        }
+
+                        foreach ( array_keys( $set_ids ) as $set_id ) {
+                                if ( in_array( "{$original->project_id}-{$set_id}", $purged ) ) {
+                                        continue;
+                                }
+
+                                $translation_set = GP::$translation_set->get( $set_id );
+                                if ( ! $translation_set ) {
+                                        return;
+                                }
+
+                                $this->delete_plugin_i18n_cache_keys_for_locale( $original->project_id, $translation_set->locale );
+                                $purged[] = "{$original->project_id}-{$set_id}";
+                        }
+                }
         }
 
@@ -177 +203 @@
                 $this->delete_plugin_i18n_cache_keys_for( $prefix, $locale );
         }
-
-        /**
-         * Deletes the cache on a translation edit.
-         *
-         * @param GP_Translation $translation The edited translation.
-         */
-        private function delete_plugin_i18n_cache_on_translation_edit( $translation ) {
-                $original = GP::$original->get( $translation->original_id );
-                if ( ! $original ) {
-                        return;
-                }
-
-                $translation_set = GP::$translation_set->get( $translation->translation_set_id );
-                if ( ! $translation_set ) {
-                        return;
-                }
-
-                $this->delete_plugin_i18n_cache_keys_for_locale( $original->project_id, $translation_set->locale );
-        }
 }