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 );
-    }
 }