Feature/transients

Taka wizja jest moja

Jeżeli transienty mają expiration na 0, to są ładowane tym głównym selectem wraz z wszystkimi opcjami - mają ustawiony autoload na 'yes'. Dodatkowo podczas aktualizacji takich transientów nie ma delete, więc eliminujemy te deadlocki. https://developer.wordpress.org/reference/functions/set_transient/

Dodatkowo ustawiłem aby aktualizacja danych wtyczek była co 5 min a nie co 16 sek.

U klienta tak objawiają się te deadlocki:

[STDERR] WordPress database error Deadlock found when trying to get lock; try restarting transaction for query DELETE FROM wp_gz2dmj_options WHERE option_name = '_transient_require_plugins_data' made by require('wp-blog-header.php'), require_once('wp-load.php'), require_once('wp-config.php'), require_once('wp-settings.php'), include_once('/plugins/flexible-shipping-usps-pro/flexible-shipping-usps-pro.php'), require('/plugins/flexible-shipping-usps-pro/vendor_prefixed/wpdesk/wp-plugin-flow/src/plugin-init-php52.php'), require('/plugins/flexible-shipping-usps-pro/vendor_prefixed/wpdesk/wp-plugin-flow-common/src/plugin-init-php52.php'), FlexibleShippingUspsProVendor\WPDesk\Plugin\Flow\PluginBootstrap->run, FlexibleShippingUspsProVendor\WPDesk_Basic_Requirement_Checker_With_Update_Disable->are_requirements_met, FlexibleShippingUspsProVendor\WPDesk_Basic_Requirement_Checker->are_requirements_met, FlexibleShippingUspsProVendor\WPDesk_Basic_Requirement_Checker->prepare_requirement_notices, FlexibleShippingUspsProVendor\WPDesk_Basic_Requirement_Checker->check_minimum_require_plugins_version_and_append_notices, FlexibleShippingUspsProVendor\WPDesk_Basic_Requirement_Checker->retrieve_required_plugins_data, FlexibleShippingUspsProVendor\WPDesk_Basic_Requirement_Checker::retrieve_plugins_data_in_transient, get_transient, delete_option\n

src/Basic_Requirement_Checker.php

@@ -10,43 +10,70 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 	 * have to be compatible with PHP 5.3.x
 	 */
 	class WPDesk_Basic_Requirement_Checker implements WPDesk_Requirement_Checker {
+
 		const EXTENSION_NAME_OPENSSL = 'openssl';
+
 		const HOOK_ADMIN_NOTICES_ACTION = 'admin_notices';
+
 		const HOOK_PLUGIN_DEACTIVATED_ACTION = 'deactivated_plugin';
+
 		const HOOK_PLUGIN_ACTIVATED_ACTION = 'activated_plugin';
 
 		const PLUGIN_INFO_KEY_NICE_NAME = 'nice_name';
+
 		const PLUGIN_INFO_KEY_NAME = 'name';
+
 		const PLUGIN_INFO_VERSION = 'version';
+
 		const PLUGIN_INFO_FAKE_REQUIRED_MINIMUM_VERSION = '0.0';
+
 		const PLUGIN_INFO_APPEND_PLUGIN_DATA = 'required_version';
-		const PLUGIN_INFO_TRANSIENT_NAME = 'require_plugins_data';
-		const PLUGIN_INFO_TRANSIENT_EXPIRATION_TIME = 16;
+
+		const PLUGIN_INFO_TRANSIENT_NAME = 'wpdesk_plugins_data';
+
+		const EXPIRATION_TRANSIENT_NAME = 'wpdesk_plugins_data_exp';
+
+		const CACHE_TIME = 300;
 
 		/** @var string */
 		protected $plugin_name;
+
 		/** @var string */
 		private $plugin_file;
+
 		/** @var string */
 		private $min_php_version;
+
 		/** @var string */
 		private $min_wp_version;
+
 		/** @var string|null */
 		private $min_wc_version = null;
+
 		/** @var int|null */
 		private $min_openssl_version = null;
+
 		/** @var array */
 		protected $plugin_require;
+
 		/** @var bool */
 		protected $should_check_plugin_versions = false;
+
 		/** @var array */
 		private $module_require;
+
 		/** @var array */
 		private $setting_require;
+
 		/** @var array */
 		protected $notices;
+
 		/** @var @string */
 		private $text_domain;
+		/**
+		 * @var mixed|true
+		 */
+		private $use_transients;
 
 		/**
 		 * @param string $plugin_file
@@ -54,11 +81,13 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 		 * @param string $text_domain
 		 * @param string $php_version
 		 * @param string $wp_version
+		 * @param bool   $use_transients
 		 */
-		public function __construct( $plugin_file, $plugin_name, $text_domain, $php_version, $wp_version ) {
-			$this->plugin_file = $plugin_file;
-			$this->plugin_name = $plugin_name;
-			$this->text_domain = $text_domain;
+		public function __construct( $plugin_file, $plugin_name, $text_domain, $php_version, $wp_version, $use_transients = true ) {
+			$this->plugin_file    = $plugin_file;
+			$this->plugin_name    = $plugin_name;
+			$this->text_domain    = $text_domain;
+			$this->use_transients = $use_transients;
 
 			$this->set_min_php_require( $php_version );
 			$this->set_min_wp_require( $wp_version );
@@ -321,32 +350,47 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 		/**
 		 * Check the plugins directory and retrieve all plugin files with plugin data.
 		 *
+		 * @param bool $use_transients
+		 *
 		 * @return array In format [ 'plugindir/pluginfile.php' => ['Name' => 'Plugin Name', 'Version' => '1.0.1', ...],  ]
 		 */
-		private static function retrieve_plugins_data_in_transient() {
-			static $never_executed = true;
-			if ( $never_executed ) {
-				$never_executed = false;
-				/** Required when WC starts later and these data should be in cache */
-				add_filter( 'extra_plugin_headers', function ( $headers = array() ) {
-					$headers[] = 'WC tested up to';
-					$headers[] = 'WC requires at least';
-					$headers[] = 'Woo';
-
-					return array_unique( $headers );
-				} );
+		private static function retrieve_plugins_data_in_transient( $use_transients = true ) {
+			$current_time    = time();
+			if ( $use_transients) {
+				$plugins         = get_transient( self::PLUGIN_INFO_TRANSIENT_NAME );
+				$expiration_time = get_transient( self::EXPIRATION_TRANSIENT_NAME );
+			} else {
+				$plugins         = get_option( self::PLUGIN_INFO_TRANSIENT_NAME );
+				$expiration_time = get_option( self::EXPIRATION_TRANSIENT_NAME );
 			}
+			$is_expired = ! $expiration_time || $current_time > $expiration_time;
+
+			if ( $plugins === false || $is_expired ) {
+				static $never_executed = true;
+				if ( $never_executed ) {
+					$never_executed = false;
+					/** Required when WC starts later and these data should be in cache */
+					add_filter( 'extra_plugin_headers', function( $headers = array() ) {
+						$headers[] = 'WC tested up to';
+						$headers[] = 'WC requires at least';
+						$headers[] = 'Woo';
+
+						return array_unique( $headers );
+					} );
+				}
 
-			$plugins = get_transient( self::PLUGIN_INFO_TRANSIENT_NAME );
-
-			if ( $plugins === false ) {
 				if ( ! function_exists( 'get_plugins' ) ) {
 					require_once ABSPATH . '/wp-admin/includes/plugin.php';
 				}
 
 				$plugins = function_exists( 'get_plugins' ) ? get_plugins() : array();
-				set_transient( self::PLUGIN_INFO_TRANSIENT_NAME, $plugins,
-					self::PLUGIN_INFO_TRANSIENT_EXPIRATION_TIME );
+				if ( $use_transients ) {
+					set_transient( self::PLUGIN_INFO_TRANSIENT_NAME, $plugins, 0 );
+					set_transient( self::EXPIRATION_TRANSIENT_NAME, $current_time + self::CACHE_TIME, 0 );
+				} else {
+					update_option( self::PLUGIN_INFO_TRANSIENT_NAME, $plugins );
+					update_option( self::EXPIRATION_TRANSIENT_NAME, $current_time + self::CACHE_TIME );
+				}
 			}
 
 			return $plugins;
@@ -358,9 +402,8 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 		 * @return array In format [ 'plugindir/pluginfile.php' => ['Name' => 'Plugin Name', 'Version' => '1.0.1', 'required_version' => '1.0.2']...  ]
 		 */
 		private function retrieve_required_plugins_data() {
-
 			$require_plugins = array();
-			$plugins         = self::retrieve_plugins_data_in_transient();
+			$plugins         = self::retrieve_plugins_data_in_transient( $this->use_transients );
 			if ( is_array( $plugins ) ) {
 				if ( count( $plugins ) > 0 ) {
 					if ( ! empty( $this->plugin_require ) ) {
@@ -420,7 +463,7 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 			if ( function_exists( 'wp_nonce_url' ) && function_exists( 'wp_create_nonce' ) ) {
 				$install_url = wp_nonce_url( $install_url, 'install-plugin_' . $slug );
 			}
-			add_filter( 'plugins_api', function ( $api, $action, $args ) use ( $plugin_info, $slug ) {
+			add_filter( 'plugins_api', function( $api, $action, $args ) use ( $plugin_info, $slug ) {
 				if ( 'plugin_information' !== $action ||
 				     false !== $api ||
 				     ! isset( $args->slug ) ||
@@ -450,7 +493,7 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 			$nice_name = $plugin_info[ self::PLUGIN_INFO_KEY_NICE_NAME ];
 
 			if ( ! self::is_wp_plugin_active( $name ) ) {
-				if ( ! self::is_wp_plugin_installed( $name ) ) {
+				if ( ! self::is_wp_plugin_installed( $name, $this->use_transients ) ) {
 					$install_url = $this->prepare_plugin_repository_install_url( $plugin_info );
 
 					return $this->prepare_notice_message( sprintf( wp_kses( __( 'The &#8220;%s&#8221; plugin requires free %s plugin. <a href="%s">Install %s</a>',
@@ -491,11 +534,12 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 		 * Checks if plugin is installed. Needs to be enabled in deferred way.
 		 *
 		 * @param string $plugin_file
+		 * @param bool   $use_transients
 		 *
 		 * @return bool
 		 */
-		public static function is_wp_plugin_installed( $plugin_file ) {
-			$plugins_data = self::retrieve_plugins_data_in_transient();
+		public static function is_wp_plugin_installed( $plugin_file, $use_transients = false ) {
+			$plugins_data = self::retrieve_plugins_data_in_transient( $use_transients );
 
 			return array_key_exists( $plugin_file, (array) $plugins_data );
 		}
@@ -564,7 +608,10 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 		 * @deprecated use render_notices or disable_plugin
 		 */
 		public function disable_plugin_render_notice() {
-			add_action( self::HOOK_ADMIN_NOTICES_ACTION, array( $this, 'handle_render_notices_action' ) );
+			add_action( self::HOOK_ADMIN_NOTICES_ACTION, array(
+				$this,
+				'handle_render_notices_action'
+			) );
 		}
 
 		/**
@@ -573,7 +620,10 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 		 * @return void
 		 */
 		public function render_notices() {
-			add_action( self::HOOK_ADMIN_NOTICES_ACTION, array( $this, 'handle_render_notices_action' ) );
+			add_action( self::HOOK_ADMIN_NOTICES_ACTION, array(
+				$this,
+				'handle_render_notices_action'
+			) );
 		}
 
 		/**
@@ -582,7 +632,10 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 		 * @return void
 		 */
 		public function disable_plugin() {
-			add_action( self::HOOK_ADMIN_NOTICES_ACTION, array( $this, 'handle_deactivate_action' ) );
+			add_action( self::HOOK_ADMIN_NOTICES_ACTION, array(
+				$this,
+				'handle_deactivate_action'
+			) );
 		}
 
 		/**
@@ -594,7 +647,7 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 			if ( isset( $this->plugin_file ) ) {
 				deactivate_plugins( plugin_basename( $this->plugin_file ) );
 
-				delete_transient( self::PLUGIN_INFO_TRANSIENT_NAME );
+				$this->clear_plugin_info_data();
 			}
 		}
 
@@ -604,8 +657,14 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 		 * @return void
 		 */
 		public function transient_delete_on_plugin_version_changed() {
-			add_action( self::HOOK_PLUGIN_DEACTIVATED_ACTION, array( $this, 'handle_transient_delete_action' ) );
-			add_action( self::HOOK_PLUGIN_ACTIVATED_ACTION, array( $this, 'handle_transient_delete_action' ) );
+			add_action( self::HOOK_PLUGIN_DEACTIVATED_ACTION, array(
+				$this,
+				'clear_plugin_info_data'
+			) );
+			add_action( self::HOOK_PLUGIN_ACTIVATED_ACTION, array(
+				$this,
+				'clear_plugin_info_data'
+			) );
 		}
 
 		/**
@@ -613,8 +672,11 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 		 *
 		 * @return void
 		 */
-		public function handle_transient_delete_action() {
+		public function clear_plugin_info_data() {
 			delete_transient( self::PLUGIN_INFO_TRANSIENT_NAME );
+			delete_transient( self::EXPIRATION_TRANSIENT_NAME );
+			delete_option( self::PLUGIN_INFO_TRANSIENT_NAME );
+			delete_option( self::EXPIRATION_TRANSIENT_NAME );
 		}
 
 		/**
@@ -629,5 +691,6 @@ if ( ! class_exists( 'WPDesk_Basic_Requirement_Checker' ) ) {
 				echo $notice;
 			}
 		}
+
 	}
 }