← All changes | core/experiments/manager.php +73 -17 4.2.1 → 4.3.2 View file →

		@@ -249,9 +249,11 @@
249	249	*
250	250	* @since 3.1.0
251	251	* @access public
252	252	*
253		- * @param string $feature_name
	253	+ * @param string $feature_name Experiment feature name.
	254	+ * @param bool $check_dependencies When true, also require dependency experiments to be active.
	255	+ * Missing or hidden dependencies are treated as active for compatibility.
254	256	*
255	257	* @return bool
256	258	*/
257	259	public function is_feature_active( $feature_name, $check_dependencies = false ) {
		@@ -262,9 +264,18 @@
262	264	}
263	265
264	266	if ( $check_dependencies && isset( $feature['dependencies'] ) && is_array( $feature['dependencies'] ) ) {
265	267	foreach ( $feature['dependencies'] as $dependency ) {
	268	+ if ( $dependency instanceof Non_Existing_Dependency ) {
	269	+ continue;
	270	+ }
	271	+
266	272	$dependent_feature = $this->get_features( $dependency->get_name() );
	273	+
	274	+ if ( $this->is_removed_or_hidden_dependency( $dependent_feature ) ) {
	275	+ continue;
	276	+ }
	277	+
267	278	$feature_state = self::STATE_ACTIVE === $this->get_feature_actual_state( $dependent_feature );
268	279
269	280	if ( ! $feature_state ) {
270	281	return false;
		@@ -378,14 +389,15 @@
378	389	],
379	390	] );
380	391
381	392	$this->add_feature( [
382		- 'name' => 'e_panel_promotions',
383		- 'title' => esc_html__( 'Panel Promotions', 'elementor' ),
384		- 'description' => esc_html__( 'Enable experimental rendering for targeted promotions within the elements panels.', 'elementor' ),
385		- 'release_status' => self::RELEASE_STATUS_DEV,
386		- 'default' => self::STATE_ACTIVE,
387		- 'type' => self::TYPE_HIDDEN,
	393	+ 'name' => 'e_optimized_css_files',
	394	+ 'title' => esc_html__( 'Optimized CSS Files', 'elementor' ),
	395	+ 'tag' => esc_html__( 'Performance', 'elementor' ),
	396	+ 'description' => esc_html__( 'Keeps external CSS files available and consistent for sites behind page caching or a CDN.', 'elementor' ),
	397	+ 'release_status' => self::RELEASE_STATUS_ALPHA,
	398	+ 'default' => self::STATE_INACTIVE,
	399	+ 'generator_tag' => true,
388	400	] );
389	401	}
390	402
391	403	/**
		@@ -778,20 +790,41 @@
778	790	}
779	791
780	792	// Validate if the current feature dependency is available.
781	793	foreach ( $feature['dependencies'] as $dependency ) {
	794	+ if ( $dependency instanceof Non_Existing_Dependency ) {
	795	+ $this->warn_removed_or_hidden_dependency(
	796	+ sprintf(
	797	+ 'The feature `%s` has a dependency `%s` that is not available in Core.',
	798	+ esc_html( $feature['name'] ),
	799	+ esc_html( $dependency->get_name() )
	800	+ )
	801	+ );
	802	+ continue;
	803	+ }
	804	+
782	805	$dependency_feature = $this->get_features( $dependency->get_name() );
783	806
784	807	if ( ! $dependency_feature ) {
785		- $rollback( $feature_option_key, self::STATE_INACTIVE );
	808	+ $this->warn_removed_or_hidden_dependency(
	809	+ sprintf(
	810	+ 'The feature `%s` has a dependency `%s` that is not available in Core.',
	811	+ esc_html( $feature['name'] ),
	812	+ esc_html( $dependency->get_name() )
	813	+ )
	814	+ );
	815	+ continue;
	816	+ }
786	817
787		- throw new Exceptions\Dependency_Exception(
	818	+ if ( $this->is_removed_or_hidden_dependency( $dependency_feature ) ) {
	819	+ $this->warn_removed_or_hidden_dependency(
788	820	sprintf(
789		- 'The feature `%s` has a dependency `%s` that is not available.',
	821	+ 'The feature `%1$s` depends on hidden experiment `%2$s`.',
790	822	esc_html( $feature['name'] ),
791		- esc_html( $dependency->get_name() )
	823	+ esc_html( $dependency_feature['name'] )
792	824	)
793	825	);
	826	+ continue;
794	827	}
795	828
796	829	$dependency_state = $this->get_feature_actual_state( $dependency_feature );
797	830
		@@ -944,10 +977,8 @@
944	977
945	978	/**
946	979	* @param array $experimental_data
947	980	* @return array
948		- *
949		- * @throws Exceptions\Dependency_Exception If the feature dependency is not initialized or depends on a hidden experiment.
950	981	*/
951	982	private function initialize_feature_dependencies( array $experimental_data ): array {
952	983	foreach ( $experimental_data['dependencies'] as $key => $dependency ) {
953	984	$feature = $this->get_features( $dependency );
		@@ -952,20 +983,28 @@
952	983	foreach ( $experimental_data['dependencies'] as $key => $dependency ) {
953	984	$feature = $this->get_features( $dependency );
954	985
955	986	if ( ! isset( $feature ) ) {
956		- // since we must validate the state of each dependency, we have to make sure that dependencies are initialized in the correct order, otherwise, error.
957		- throw new Exceptions\Dependency_Exception(
	987	+ $this->warn_removed_or_hidden_dependency(
958	988	sprintf(
959		- 'Feature %s cannot be initialized before dependency feature: %s.',
	989	+ 'Feature %1$s depends on experiment %2$s that is not registered in Core.',
960	990	esc_html( $experimental_data['name'] ),
961	991	esc_html( $dependency )
962	992	)
963	993	);
	994	+
	995	+ $experimental_data['dependencies'][ $key ] = $this->create_dependency_class( $dependency, null );
	996	+ continue;
964	997	}
965	998
966	999	if ( ! empty( $feature[ static::TYPE_HIDDEN ] ) ) {
967		- throw new Exceptions\Dependency_Exception( 'Depending on a hidden experiment is not allowed.' );
	1000	+ $this->warn_removed_or_hidden_dependency(
	1001	+ sprintf(
	1002	+ 'Feature %1$s depends on hidden experiment %2$s.',
	1003	+ esc_html( $experimental_data['name'] ),
	1004	+ esc_html( $dependency )
	1005	+ )
	1006	+ );
968	1007	}
969	1008
970	1009	$experimental_data['dependencies'][ $key ] = $this->create_dependency_class( $dependency, $feature );
971	1010	$experimental_data = $this->set_feature_default_state_to_match_dependencies( $feature, $experimental_data );
		@@ -971,8 +1010,25 @@
971	1010	$experimental_data = $this->set_feature_default_state_to_match_dependencies( $feature, $experimental_data );
972	1011	}
973	1012
974	1013	return $experimental_data;
	1014	+ }
	1015	+
	1016	+ private function is_removed_or_hidden_dependency( $dependency_feature ): bool {
	1017	+ if ( ! $dependency_feature ) {
	1018	+ return true;
	1019	+ }
	1020	+
	1021	+ return ! empty( $dependency_feature[ static::TYPE_HIDDEN ] );
	1022	+ }
	1023	+
	1024	+ private function warn_removed_or_hidden_dependency( string $message ): void {
	1025	+ if ( ! defined( 'WP_DEBUG' ) || ! WP_DEBUG ) {
	1026	+ return;
	1027	+ }
	1028	+
	1029	+ // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped -- Developer notice; message is escaped.
	1030	+ _doing_it_wrong( __METHOD__, esc_html( $message ), ELEMENTOR_VERSION );
975	1031	}
976	1032
977	1033	/**
978	1034	* @param array $feature