Merge pull request #831 from Codeinwp/bugfix/499

Implement transient API for storing cache object

Author: selul

classes/Visualizer/Gutenberg/Block.php

@@ -659,6 +659,12 @@ public function update_chart_data( $data ) {
 
 			wp_update_post( $chart );
 
+			// Clear existing chart cache.
+			$cache_key = Visualizer_Plugin::CF_CHART_CACHE . '_' . $data['id'];
+			if ( get_transient( $cache_key ) ) {
+				delete_transient( $cache_key );
+			}
+
 			$revisions = wp_get_post_revisions( $data['id'], array( 'order' => 'ASC' ) );
 
 			if ( count( $revisions ) > 1 ) {

classes/Visualizer/Module/Chart.php

@@ -598,6 +598,14 @@ public function renderChartPages() {
 			$this->_chart = $this->handleExistingRevisions( $chart_id, $this->_chart );
 		}
 
+		// Clear existing chart cache.
+		if ( isset( $_POST['save'] ) && 1 === intval( $_POST['save'] ) ) {
+			$cache_key = Visualizer_Plugin::CF_CHART_CACHE . '_' . $chart_id;
+			if ( get_transient( $cache_key ) ) {
+				delete_transient( $cache_key );
+			}
+		}
+
 		switch ( $tab ) {
 			case 'settings':
 				$this->_handleDataAndSettingsPage();

classes/Visualizer/Module/Frontend.php

@@ -277,11 +277,16 @@ public function renderChart( $atts ) {
 			),
 			$atts
 		);
-		// if empty id or chart does not exists, then return empty string
-		if ( ! $atts['id'] || ! ( $chart = get_post( $atts['id'] ) ) || $chart->post_type !== Visualizer_Plugin::CPT_VISUALIZER ) {
+
+		$chart_data = $this->getChartData( Visualizer_Plugin::CF_CHART_CACHE, $atts['id'] );
+		// if empty chart does not exists, then return empty string.
+		if ( ! $chart_data ) {
 			return '';
 		}
 
+		$chart        = $chart_data['chart'];
+		$type         = $chart_data['type'];
+		$series       = $chart_data['series'];
 		// do not show the chart?
 		if ( ! apply_filters( 'visualizer_pro_show_chart', true, $atts['id'] ) ) {
 			return '';
@@ -309,18 +314,16 @@ public function renderChart( $atts ) {
 			$attributes['data-lazy-limit'] = $atts['lazy'];
 		}
 
-		$type = get_post_meta( $chart->ID, Visualizer_Plugin::CF_CHART_TYPE, true );
-
 		$chart = apply_filters( 'visualizer_schedule_refresh_chart', $chart, $chart->ID, false );
 
-		// fetch and update settings
-		$settings = get_post_meta( $chart->ID, Visualizer_Plugin::CF_SETTINGS, true );
+		// Get and update settings.
+		$settings = $chart_data['settings'];
 		if ( empty( $settings['height'] ) ) {
 			$settings['height'] = '400';
 		}
 
 		// handle series filter hooks
-		$series = apply_filters( Visualizer_Plugin::FILTER_GET_CHART_SERIES, get_post_meta( $chart->ID, Visualizer_Plugin::CF_SERIES, true ), $chart->ID, $type );
+		$series = apply_filters( Visualizer_Plugin::FILTER_GET_CHART_SERIES, $series, $chart->ID, $type );
 
 		// handle settings filter hooks
 		$settings = apply_filters( Visualizer_Plugin::FILTER_GET_CHART_SETTINGS, $settings, $chart->ID, $type );
@@ -345,7 +348,7 @@ public function renderChart( $atts ) {
 		}
 
 		if ( 'yes' === $atts['use_image'] ) {
-			$chart_image = get_post_meta( $chart->ID, Visualizer_Plugin::CF_CHART_IMAGE, true );
+			$chart_image = $chart_data['chart_image'];
 			if ( $chart_image ) {
 				return '<div id="' . $id . '"' . $this->getHtmlAttributes( $attributes ) . '>' . wp_get_attachment_image( $chart_image, 'full' ) . '</div>';
 			}
@@ -544,4 +547,41 @@ private function addSchema( $id ) {
 
 		return '<script type="application/ld+json">' . $schema . '</script>';
 	}
+
+	/**
+	 * Get chart by ID.
+	 *
+	 * @param string $cache_key Cache key.
+	 * @param int    $chart_id Chart ID.
+	 * @return mixed
+	 */
+	private function getChartData( $cache_key = '', $chart_id = 0 ) {
+		if ( ! $chart_id ) {
+			return false;
+		}
+		// Create unique cache key of each chart.
+		$cache_key .= '_' . $chart_id;
+		// Get chart from cache.
+		$chart = get_transient( $cache_key );
+		if ( $chart ) {
+			return $chart;
+		}
+
+		// Get chart by ID.
+		$chart = get_post( $chart_id );
+		if ( $chart && Visualizer_Plugin::CPT_VISUALIZER === $chart->post_type ) {
+			$chart_data = array(
+				'chart'       => $chart,
+				'type'        => get_post_meta( $chart->ID, Visualizer_Plugin::CF_CHART_TYPE, true ),
+				'settings'    => get_post_meta( $chart->ID, Visualizer_Plugin::CF_SETTINGS, true ),
+				'series'      => get_post_meta( $chart->ID, Visualizer_Plugin::CF_SERIES, true ),
+				'chart_image' => get_post_meta( $chart->ID, Visualizer_Plugin::CF_CHART_IMAGE, true ),
+			);
+			// Put the results in a transient. Expire after 12 hours.
+			set_transient( $cache_key, $chart_data, apply_filters( Visualizer_Plugin::FILTER_HANDLE_CACHE_EXPIRATION_TIME, 12 * HOUR_IN_SECONDS ) );
+			return $chart_data;
+		}
+
+		return false;
+	}
 }

classes/Visualizer/Plugin.php

@@ -89,6 +89,7 @@ class Visualizer_Plugin {
 	const FILTER_UNDO_REVISIONS = 'visualizer-undo-revisions';
 	const FILTER_HANDLE_REVISIONS = 'visualizer-handle-revisions';
 	const FILTER_GET_CHART_DATA_AS = 'visualizer-get-chart-data-as';
+	const FILTER_HANDLE_CACHE_EXPIRATION_TIME = 'visualizer-handle-expiration-time';
 
 	const CF_DB_SCHEDULE = 'visualizer-db-schedule';
 	const CF_DB_QUERY = 'visualizer-db-query';
@@ -99,6 +100,8 @@ class Visualizer_Plugin {
 	const PRO_TEASER_URL = 'https://themeisle.com/plugins/visualizer-charts-and-graphs/upgrade/#pricing';
 	const PRO_TEASER_TITLE = 'Check PRO version ';
 
+	const CF_CHART_CACHE = 'visualizer-chart-cache';
+
 	/**
 	 * Name of the option for WordPress DB.
 	 */