Merge pull request #652 from contactashish13/issue-247-pro

rodica-andronache · web-flow · commit 3f28e5c9d4eb · 2020-03-02T11:14:26.000+02:00
Error message when publishing post with Live updating chart
diff --git a/classes/Visualizer/Render.php b/classes/Visualizer/Render.php
@@ -151,4 +151,24 @@ public function render() {
 		$this->_toHTML();
 	}
 
+	/**
+	 * Checks if the request is from Gutenberg.
+	 *
+	 * This happens when the chart is being added to a gutenberg block by inserting a visualizer block.
+	 *
+	 * @since ?
+	 *
+	 * @access protected
+	 * @return bool
+	 */
+	protected function is_request_from_gutenberg() {
+		global $post;
+		require_once ABSPATH . 'wp-admin/includes/post.php';
+		if ( $post && function_exists( 'use_block_editor_for_post' ) && use_block_editor_for_post( $post ) ) {
+			do_action( 'themeisle_log_event', Visualizer_Plugin::NAME, sprintf( 'This request appears to be from Gutenberg block. Ignoring for chart %d', $post->ID ), 'debug', __FILE__, __LINE__ );
+			return true;
+		}
+		return false;
+	}
+
 }
diff --git a/classes/Visualizer/Render/Page/Update.php b/classes/Visualizer/Render/Page/Update.php
@@ -38,6 +38,9 @@ class Visualizer_Render_Page_Update extends Visualizer_Render_Page {
 	 * @access protected
 	 */
 	protected function _toHTML() {
+		if ( $this->is_request_from_gutenberg() ) {
+			return;
+		}
 		echo '<!DOCTYPE html>';
 		echo '<html>';
 			echo '<head>';
