Flag saving action

Author: Marco Pereirinha

php/class-delivery.php

@@ -1819,6 +1819,11 @@ public function prepare_delivery( $content ) {
 	 */
 	public function catch_urls( $content, $context = 'view' ) {
 
+		// Check if we are saving. If so, bail.
+		// This is to prevent the replacement from happening in the shutdown, signaling content changes in the editor.
+		if ( $this->plugin->get_component( 'replace' )->doing_save() ) {
+			return;
+		}
 		$this->init_delivery();
 		$this->prepare_delivery( $content );
 

php/class-string-replace.php

@@ -8,7 +8,6 @@
 namespace Cloudinary;
 
 use Cloudinary\Component\Setup;
-use \Traversable;
 
 /**
  * String replace class.
@@ -36,6 +35,13 @@ class String_Replace implements Setup {
 	 */
 	protected static $replacements = array();
 
+	/**
+	 * Holds the list of strings and replacements.
+	 *
+	 * @var bool
+	 */
+	protected static $doing_save = false;
+
 	/**
 	 * Site Cache constructor.
 	 *
@@ -130,7 +136,12 @@ public function pre_filter_rest_echo( $result, $server, $request ) {
 	 */
 	public function pre_filter_rest_content( $response, $post, $request ) {
 		$context = $request->get_param( 'context' );
+
 		if ( 'edit' === $context ) {
+			// Updating or creating a post.
+			if ( in_array( $request->get_method(), array( 'PUT', 'POST' ), true ) ) {
+				static::$doing_save = true;
+			}
 			$data = $response->get_data();
 			$data = $this->replace_strings( $data, $context );
 			$response->set_data( $data );
@@ -323,6 +334,16 @@ public static function do_replace( $content ) {
 		return $content;
 	}
 
+	/**
+	 * Check if we are currently saving a REST request (i.e. Gutenberg).
+	 * This is used to prevent double replacements.
+	 *
+	 * @return bool
+	 */
+	public function doing_save() {
+		return static::$doing_save;
+	}
+
 	/**
 	 * Reset internal replacements.
 	 */