← All changes | core/experiments/manager.php +115 -18 4.2.1 → trunk View file →

		@@ -89,8 +89,10 @@
89	89	}
90	90
91	91	$this->features[ $options['name'] ] = $experimental_data;
92	92
	93	+ $this->resolve_registered_dependants( $options['name'], $experimental_data );
	94	+
93	95	if ( $experimental_data['mutable'] && is_admin() ) {
94	96	$feature_option_key = $this->get_feature_option_key( $options['name'] );
95	97
96	98	$on_state_change_callback = function( $old_state, $new_state ) use ( $experimental_data, $feature_option_key ) {
		@@ -249,9 +251,11 @@
249	251	*
250	252	* @since 3.1.0
251	253	* @access public
252	254	*
253		- * @param string $feature_name
	255	+ * @param string $feature_name Experiment feature name.
	256	+ * @param bool $check_dependencies When true, also require dependency experiments to be active.
	257	+ * Missing dependencies always fail closed. Hidden dependencies are treated as satisfied.
254	258	*
255	259	* @return bool
256	260	*/
257	261	public function is_feature_active( $feature_name, $check_dependencies = false ) {
		@@ -260,11 +264,36 @@
260	264	if ( ! $feature || self::STATE_ACTIVE !== $this->get_feature_actual_state( $feature ) ) {
261	265	return false;
262	266	}
263	267
264		- if ( $check_dependencies && isset( $feature['dependencies'] ) && is_array( $feature['dependencies'] ) ) {
	268	+ if ( isset( $feature['dependencies'] ) && is_array( $feature['dependencies'] ) ) {
265	269	foreach ( $feature['dependencies'] as $dependency ) {
266	270	$dependent_feature = $this->get_features( $dependency->get_name() );
	271	+
	272	+ // A dependency that Core still does not ship can never be verified as active.
	273	+ // Non existing dependencies are re-resolved first to cover later registrations.
	274	+ if ( $dependency instanceof Non_Existing_Dependency && ! $dependent_feature ) {
	275	+ return false;
	276	+ }
	277	+
	278	+ if ( ! $dependent_feature ) {
	279	+ // A Core experiment dependency that no longer resolves fails closed.
	280	+ // Class dependencies are not experiment names and stay exempt.
	281	+ if ( $dependency instanceof Wrap_Core_Dependency ) {
	282	+ return false;
	283	+ }
	284	+
	285	+ continue;
	286	+ }
	287	+
	288	+ if ( ! $check_dependencies ) {
	289	+ continue;
	290	+ }
	291	+
	292	+ if ( ! empty( $dependent_feature[ static::TYPE_HIDDEN ] ) ) {
	293	+ continue;
	294	+ }
	295	+
267	296	$feature_state = self::STATE_ACTIVE === $this->get_feature_actual_state( $dependent_feature );
268	297
269	298	if ( ! $feature_state ) {
270	299	return false;
		@@ -378,14 +407,15 @@
378	407	],
379	408	] );
380	409
381	410	$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,
	411	+ 'name' => 'e_optimized_css_files',
	412	+ 'title' => esc_html__( 'Optimized CSS Files', 'elementor' ),
	413	+ 'tag' => esc_html__( 'Performance', 'elementor' ),
	414	+ 'description' => esc_html__( 'Keeps external CSS files available and consistent for sites behind page caching or a CDN.', 'elementor' ),
	415	+ 'release_status' => self::RELEASE_STATUS_ALPHA,
	416	+ 'default' => self::STATE_INACTIVE,
	417	+ 'generator_tag' => true,
388	418	] );
389	419	}
390	420
391	421	/**
		@@ -778,20 +808,41 @@
778	808	}
779	809
780	810	// Validate if the current feature dependency is available.
781	811	foreach ( $feature['dependencies'] as $dependency ) {
	812	+ if ( $dependency instanceof Non_Existing_Dependency ) {
	813	+ $this->warn_removed_or_hidden_dependency(
	814	+ sprintf(
	815	+ 'The feature `%s` has a dependency `%s` that is not available in Core.',
	816	+ esc_html( $feature['name'] ),
	817	+ esc_html( $dependency->get_name() )
	818	+ )
	819	+ );
	820	+ continue;
	821	+ }
	822	+
782	823	$dependency_feature = $this->get_features( $dependency->get_name() );
783	824
784	825	if ( ! $dependency_feature ) {
785		- $rollback( $feature_option_key, self::STATE_INACTIVE );
	826	+ $this->warn_removed_or_hidden_dependency(
	827	+ sprintf(
	828	+ 'The feature `%s` has a dependency `%s` that is not available in Core.',
	829	+ esc_html( $feature['name'] ),
	830	+ esc_html( $dependency->get_name() )
	831	+ )
	832	+ );
	833	+ continue;
	834	+ }
786	835
787		- throw new Exceptions\Dependency_Exception(
	836	+ if ( $this->is_removed_or_hidden_dependency( $dependency_feature ) ) {
	837	+ $this->warn_removed_or_hidden_dependency(
788	838	sprintf(
789		- 'The feature `%s` has a dependency `%s` that is not available.',
	839	+ 'The feature `%1$s` depends on hidden experiment `%2$s`.',
790	840	esc_html( $feature['name'] ),
791		- esc_html( $dependency->get_name() )
	841	+ esc_html( $dependency_feature['name'] )
792	842	)
793	843	);
	844	+ continue;
794	845	}
795	846
796	847	$dependency_state = $this->get_feature_actual_state( $dependency_feature );
797	848
		@@ -944,10 +995,8 @@
944	995
945	996	/**
946	997	* @param array $experimental_data
947	998	* @return array
948		- *
949		- * @throws Exceptions\Dependency_Exception If the feature dependency is not initialized or depends on a hidden experiment.
950	999	*/
951	1000	private function initialize_feature_dependencies( array $experimental_data ): array {
952	1001	foreach ( $experimental_data['dependencies'] as $key => $dependency ) {
953	1002	$feature = $this->get_features( $dependency );
		@@ -952,20 +1001,28 @@
952	1001	foreach ( $experimental_data['dependencies'] as $key => $dependency ) {
953	1002	$feature = $this->get_features( $dependency );
954	1003
955	1004	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(
	1005	+ $this->warn_removed_or_hidden_dependency(
958	1006	sprintf(
959		- 'Feature %s cannot be initialized before dependency feature: %s.',
	1007	+ 'Feature %1$s depends on experiment %2$s that is not registered in Core.',
960	1008	esc_html( $experimental_data['name'] ),
961	1009	esc_html( $dependency )
962	1010	)
963	1011	);
	1012	+
	1013	+ $experimental_data['dependencies'][ $key ] = $this->create_dependency_class( $dependency, null );
	1014	+ continue;
964	1015	}
965	1016
966	1017	if ( ! empty( $feature[ static::TYPE_HIDDEN ] ) ) {
967		- throw new Exceptions\Dependency_Exception( 'Depending on a hidden experiment is not allowed.' );
	1018	+ $this->warn_removed_or_hidden_dependency(
	1019	+ sprintf(
	1020	+ 'Feature %1$s depends on hidden experiment %2$s.',
	1021	+ esc_html( $experimental_data['name'] ),
	1022	+ esc_html( $dependency )
	1023	+ )
	1024	+ );
968	1025	}
969	1026
970	1027	$experimental_data['dependencies'][ $key ] = $this->create_dependency_class( $dependency, $feature );
971	1028	$experimental_data = $this->set_feature_default_state_to_match_dependencies( $feature, $experimental_data );
		@@ -971,8 +1028,48 @@
971	1028	$experimental_data = $this->set_feature_default_state_to_match_dependencies( $feature, $experimental_data );
972	1029	}
973	1030
974	1031	return $experimental_data;
	1032	+ }
	1033	+
	1034	+ private function is_removed_or_hidden_dependency( $dependency_feature ): bool {
	1035	+ if ( ! $dependency_feature ) {
	1036	+ return true;
	1037	+ }
	1038	+
	1039	+ return ! empty( $dependency_feature[ static::TYPE_HIDDEN ] );
	1040	+ }
	1041	+
	1042	+ /**
	1043	+ * Re-resolve non existing dependency placeholders of dependants that were
	1044	+ * registered before this feature existed.
	1045	+ *
	1046	+ * @param string $feature_name Newly registered feature name.
	1047	+ * @param array $feature_data Newly registered feature data.
	1048	+ */
	1049	+ private function resolve_registered_dependants( string $feature_name, array $feature_data ): void {
	1050	+ foreach ( $this->features as $name => $registered_feature ) {
	1051	+ if ( $name === $feature_name || empty( $registered_feature['dependencies'] ) || ! is_array( $registered_feature['dependencies'] ) ) {
	1052	+ continue;
	1053	+ }
	1054	+
	1055	+ foreach ( $registered_feature['dependencies'] as $key => $dependency ) {
	1056	+ if ( ! ( $dependency instanceof Non_Existing_Dependency ) || $dependency->get_name() !== $feature_name ) {
	1057	+ continue;
	1058	+ }
	1059	+
	1060	+ $this->features[ $name ]['dependencies'][ $key ] = $this->create_dependency_class( $feature_name, $feature_data );
	1061	+ }
	1062	+ }
	1063	+ }
	1064	+
	1065	+ private function warn_removed_or_hidden_dependency( string $message ): void {
	1066	+ if ( ! defined( 'WP_DEBUG' ) || ! WP_DEBUG ) {
	1067	+ return;
	1068	+ }
	1069	+
	1070	+ // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped -- Developer notice; message is escaped.
	1071	+ _doing_it_wrong( __METHOD__, esc_html( $message ), ELEMENTOR_VERSION );
975	1072	}
976	1073
977	1074	/**
978	1075	* @param array $feature