| @@ -1,9 +1,8 @@ | ||
| 1 | 1 | <?php |
| 2 | 2 | namespace Elementor\Core\Experiments; |
| 3 | 3 | |
| 4 | 4 | use Elementor\Core\Base\Base_Object; |
| 5 | -use Elementor\Core\Experiments\Exceptions\Dependency_Exception; | |
| 6 | 5 | use Elementor\Core\Upgrade\Manager as Upgrade_Manager; |
| 7 | 6 | use Elementor\Core\Utils\Collection; |
| 8 | 7 | use Elementor\Modules\System_Info\Module as System_Info; |
| 9 | 8 | use Elementor\Plugin; |
| @@ -11,9 +10,9 @@ | ||
| 11 | 10 | use Elementor\Tracker; |
| 12 | 11 | use Elementor\Utils; |
| 13 | 12 | |
| 14 | 13 | if ( ! defined( 'ABSPATH' ) ) { |
| 15 | - exit; // Exit if accessed directly. | |
| 14 | + exit; // Exit if accessed directly | |
| 16 | 15 | } |
| 17 | 16 | |
| 18 | 17 | class Manager extends Base_Object { |
| 19 | 18 | |
| @@ -22,8 +21,10 @@ | ||
| 22 | 21 | const RELEASE_STATUS_ALPHA = 'alpha'; |
| 23 | 22 | |
| 24 | 23 | const RELEASE_STATUS_BETA = 'beta'; |
| 25 | 24 | |
| 25 | + const RELEASE_STATUS_RC = 'rc'; | |
| 26 | + | |
| 26 | 27 | const RELEASE_STATUS_STABLE = 'stable'; |
| 27 | 28 | |
| 28 | 29 | const STATE_DEFAULT = 'default'; |
| 29 | 30 | |
| @@ -43,26 +44,24 @@ | ||
| 43 | 44 | |
| 44 | 45 | /** |
| 45 | 46 | * Add Feature |
| 46 | 47 | * |
| 47 | - * Each feature has to provide the following information: | |
| 48 | - * [ | |
| 49 | - * 'name' => string, | |
| 50 | - * 'title' => string, | |
| 51 | - * 'description' => string, | |
| 52 | - * 'tag' => string, | |
| 53 | - * 'release_status' => string, | |
| 54 | - * 'default' => string, | |
| 55 | - * 'new_site' => array, | |
| 56 | - * ] | |
| 57 | - * | |
| 58 | 48 | * @since 3.1.0 |
| 59 | 49 | * @access public |
| 60 | 50 | * |
| 61 | - * @param array $options Feature options. | |
| 51 | + * @param array $options { | |
| 52 | + * @type string $name | |
| 53 | + * @type string $title | |
| 54 | + * @type string $tag | |
| 55 | + * @type array $tags | |
| 56 | + * @type string $description | |
| 57 | + * @type string $release_status | |
| 58 | + * @type string $default | |
| 59 | + * @type callable $on_state_change | |
| 60 | + * } | |
| 61 | + * | |
| 62 | 62 | * @return array|null |
| 63 | - * | |
| 64 | - * @throws Dependency_Exception If can't change feature state. | |
| 63 | + * @throws \Exception | |
| 65 | 64 | */ |
| 66 | 65 | public function add_feature( array $options ) { |
| 67 | 66 | if ( isset( $this->features[ $options['name'] ] ) ) { |
| 68 | 67 | return null; |
| @@ -67,14 +66,48 @@ | ||
| 67 | 66 | if ( isset( $this->features[ $options['name'] ] ) ) { |
| 68 | 67 | return null; |
| 69 | 68 | } |
| 70 | 69 | |
| 71 | - $experimental_data = $this->set_feature_initial_options( $options ); | |
| 70 | + $default_experimental_data = [ | |
| 71 | + 'tag' => '', // Deprecated, use 'tags' instead. | |
| 72 | + 'tags' => [], | |
| 73 | + 'description' => '', | |
| 74 | + 'release_status' => self::RELEASE_STATUS_ALPHA, | |
| 75 | + 'default' => self::STATE_INACTIVE, | |
| 76 | + 'mutable' => true, | |
| 77 | + static::TYPE_HIDDEN => false, | |
| 78 | + 'new_site' => [ | |
| 79 | + 'always_active' => false, | |
| 80 | + 'default_active' => false, | |
| 81 | + 'default_inactive' => false, | |
| 82 | + 'minimum_installation_version' => null, | |
| 83 | + ], | |
| 84 | + 'on_state_change' => null, | |
| 85 | + 'generator_tag' => false, | |
| 86 | + ]; | |
| 72 | 87 | |
| 88 | + $allowed_options = [ 'name', 'title', 'tag', 'tags', 'description', 'release_status', 'default', 'mutable', static::TYPE_HIDDEN, 'new_site', 'on_state_change', 'dependencies', 'generator_tag', 'messages' ]; | |
| 89 | + | |
| 90 | + $experimental_data = $this->merge_properties( $default_experimental_data, $options, $allowed_options ); | |
| 91 | + | |
| 92 | + $experimental_data = $this->unify_feature_tags( $experimental_data ); | |
| 93 | + | |
| 73 | 94 | $new_site = $experimental_data['new_site']; |
| 74 | 95 | |
| 75 | 96 | if ( $new_site['default_active'] || $new_site['always_active'] || $new_site['default_inactive'] ) { |
| 76 | - $experimental_data = $this->set_new_site_default_state( $new_site, $experimental_data ); | |
| 97 | + $is_new_installation = $this->install_compare( $new_site['minimum_installation_version'] ); | |
| 98 | + | |
| 99 | + if ( $is_new_installation ) { | |
| 100 | + if ( $new_site['always_active'] ) { | |
| 101 | + $experimental_data['state'] = self::STATE_ACTIVE; | |
| 102 | + | |
| 103 | + $experimental_data['mutable'] = false; | |
| 104 | + } elseif ( $new_site['default_active'] ) { | |
| 105 | + $experimental_data['default'] = self::STATE_ACTIVE; | |
| 106 | + } elseif ( $new_site['default_inactive'] ) { | |
| 107 | + $experimental_data['default'] = self::STATE_INACTIVE; | |
| 108 | + } | |
| 109 | + } | |
| 77 | 110 | } |
| 78 | 111 | |
| 79 | 112 | if ( $experimental_data['mutable'] ) { |
| 80 | 113 | $experimental_data['state'] = $this->get_saved_feature_state( $options['name'] ); |
| @@ -84,9 +117,17 @@ | ||
| 84 | 117 | $experimental_data['state'] = self::STATE_DEFAULT; |
| 85 | 118 | } |
| 86 | 119 | |
| 87 | 120 | if ( ! empty( $experimental_data['dependencies'] ) ) { |
| 88 | - $experimental_data = $this->initialize_feature_dependencies( $experimental_data ); | |
| 121 | + foreach ( $experimental_data['dependencies'] as $key => $dependency ) { | |
| 122 | + $feature = $this->get_features( $dependency ); | |
| 123 | + | |
| 124 | + if ( ! empty( $feature[ static::TYPE_HIDDEN ] ) ) { | |
| 125 | + throw new Exceptions\Dependency_Exception( 'Depending on a hidden experiment is not allowed.' ); | |
| 126 | + } | |
| 127 | + | |
| 128 | + $experimental_data['dependencies'][ $key ] = $this->create_dependency_class( $dependency, $feature ); | |
| 129 | + } | |
| 89 | 130 | } |
| 90 | 131 | |
| 91 | 132 | $this->features[ $options['name'] ] = $experimental_data; |
| 92 | 133 | |
| @@ -120,11 +161,9 @@ | ||
| 120 | 161 | private function install_compare( $version ) { |
| 121 | 162 | $installs_history = Upgrade_Manager::get_installs_history(); |
| 122 | 163 | |
| 123 | 164 | if ( empty( $installs_history ) ) { |
| 124 | - // Fresh installation: upgrade manager hasn't written history yet on this first request. | |
| 125 | - // Use the current plugin version as the effective first-install version. | |
| 126 | - return version_compare( ELEMENTOR_VERSION, $version, '>=' ); | |
| 165 | + return false; | |
| 127 | 166 | } |
| 128 | 167 | |
| 129 | 168 | $cleaned_version = preg_replace( '/-(beta|cloud|dev)\d*$/', '', key( $installs_history ) ); |
| 130 | 169 | |
| @@ -141,9 +180,9 @@ | ||
| 141 | 180 | * @param array $experimental_data |
| 142 | 181 | * |
| 143 | 182 | * @return array |
| 144 | 183 | */ |
| 145 | - private function unify_feature_tags( array $experimental_data ): array { | |
| 184 | + private function unify_feature_tags( array $experimental_data ) : array { | |
| 146 | 185 | foreach ( [ 'tag', 'tags' ] as $key ) { |
| 147 | 186 | if ( empty( $experimental_data[ $key ] ) ) { |
| 148 | 187 | continue; |
| 149 | 188 | } |
| @@ -160,21 +199,18 @@ | ||
| 160 | 199 | |
| 161 | 200 | /** |
| 162 | 201 | * Format feature tags into the right format. |
| 163 | 202 | * |
| 164 | - * If an array of tags provided, each tag has to provide the following information: | |
| 165 | - * [ | |
| 166 | - * [ | |
| 167 | - * 'type' => string, | |
| 168 | - * 'label' => string, | |
| 169 | - * ] | |
| 170 | - * ] | |
| 203 | + * @param string|array[ | |
| 204 | + * [ | |
| 205 | + * 'type' => string, | |
| 206 | + * 'label' => string | |
| 207 | + * ] | |
| 208 | + * ] $tag | |
| 171 | 209 | * |
| 172 | - * @param string|array $tags A string of comma separated tags, or an array of tags. | |
| 173 | - * | |
| 174 | 210 | * @return array |
| 175 | 211 | */ |
| 176 | - private function format_feature_tags( $tags ): array { | |
| 212 | + private function format_feature_tags( $tags ) : array { | |
| 177 | 213 | if ( ! is_string( $tags ) && ! is_array( $tags ) ) { |
| 178 | 214 | return []; |
| 179 | 215 | } |
| 180 | 216 | |
| @@ -223,9 +259,9 @@ | ||
| 223 | 259 | * |
| 224 | 260 | * @since 3.1.0 |
| 225 | 261 | * @access public |
| 226 | 262 | * |
| 227 | - * @param string $feature_name Optional. Default is null. | |
| 263 | + * @param string $feature_name Optional. Default is null | |
| 228 | 264 | * |
| 229 | 265 | * @return array|null |
| 230 | 266 | */ |
| 231 | 267 | public function get_features( $feature_name = null ) { |
| @@ -249,11 +285,9 @@ | ||
| 249 | 285 | * |
| 250 | 286 | * @since 3.1.0 |
| 251 | 287 | * @access public |
| 252 | 288 | * |
| 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. | |
| 289 | + * @param string $feature_name | |
| 256 | 290 | * |
| 257 | 291 | * @return bool |
| 258 | 292 | */ |
| 259 | 293 | public function is_feature_active( $feature_name, $check_dependencies = false ) { |
| @@ -264,18 +298,9 @@ | ||
| 264 | 298 | } |
| 265 | 299 | |
| 266 | 300 | if ( $check_dependencies && isset( $feature['dependencies'] ) && is_array( $feature['dependencies'] ) ) { |
| 267 | 301 | foreach ( $feature['dependencies'] as $dependency ) { |
| 268 | - if ( $dependency instanceof Non_Existing_Dependency ) { | |
| 269 | - continue; | |
| 270 | - } | |
| 271 | - | |
| 272 | 302 | $dependent_feature = $this->get_features( $dependency->get_name() ); |
| 273 | - | |
| 274 | - if ( $this->is_removed_or_hidden_dependency( $dependent_feature ) ) { | |
| 275 | - continue; | |
| 276 | - } | |
| 277 | - | |
| 278 | 303 | $feature_state = self::STATE_ACTIVE === $this->get_feature_actual_state( $dependent_feature ); |
| 279 | 304 | |
| 280 | 305 | if ( ! $feature_state ) { |
| 281 | 306 | return false; |
| @@ -353,13 +378,14 @@ | ||
| 353 | 378 | $this->add_feature( [ |
| 354 | 379 | 'name' => 'container', |
| 355 | 380 | 'title' => esc_html__( 'Container', 'elementor' ), |
| 356 | 381 | 'description' => sprintf( |
| 357 | - /* translators: 1: Link opening tag, 2: Link closing tag, 3: Link opening tag, 4: Link closing tag */ | |
| 358 | - esc_html__( 'Create advanced layouts and responsive designs with %1$sFlexbox%2$s and %3$sGrid%4$s container elements.', 'elementor' ), | |
| 382 | + esc_html__( 'Create advanced layouts and responsive designs with %1$sFlexbox%2$s and %3$sGrid%4$s container elements. Give it a try using the %5$sContainer playground%6$s.', 'elementor' ), | |
| 359 | 383 | '<a target="_blank" href="https://go.elementor.com/wp-dash-flex-container/">', |
| 360 | 384 | '</a>', |
| 361 | 385 | '<a target="_blank" href="https://go.elementor.com/wp-dash-grid-container/">', |
| 386 | + '</a>', | |
| 387 | + '<a target="_blank" href="https://go.elementor.com/wp-dash-flex-container-playground/">', | |
| 362 | 388 | '</a>' |
| 363 | 389 | ), |
| 364 | 390 | 'release_status' => self::RELEASE_STATUS_STABLE, |
| 365 | 391 | 'default' => self::STATE_INACTIVE, |
| @@ -375,29 +401,66 @@ | ||
| 375 | 401 | ), |
| 376 | 402 | ], |
| 377 | 403 | ] ); |
| 378 | 404 | |
| 405 | + // TODO: Remove this experiment in v3.28 [ED-15983]. | |
| 379 | 406 | $this->add_feature( [ |
| 407 | + 'name' => 'e_swiper_latest', | |
| 408 | + 'title' => esc_html__( 'Upgrade Swiper Library', 'elementor' ), | |
| 409 | + 'description' => esc_html__( 'Prepare your website for future improvements to carousel features by upgrading the Swiper library integrated into your site from v5.36 to v8.45. This experiment includes markup changes so it might require updating custom code and cause compatibility issues with third party plugins.', 'elementor' ), | |
| 410 | + 'default' => self::STATE_ACTIVE, | |
| 411 | + static::TYPE_HIDDEN => true, | |
| 412 | + 'mutable' => false, | |
| 413 | + ] ); | |
| 414 | + | |
| 415 | + $this->add_feature( [ | |
| 416 | + 'name' => 'e_nested_atomic_repeaters', | |
| 417 | + 'title' => esc_html__( 'Nested Elements Performance', 'elementor' ), | |
| 418 | + 'tag' => esc_html__( 'Performance', 'elementor' ), | |
| 419 | + 'description' => esc_html__( 'Improve the performance of the Nested widgets.', 'elementor' ), | |
| 420 | + static::TYPE_HIDDEN => true, | |
| 421 | + 'release_status' => self::RELEASE_STATUS_DEV, | |
| 422 | + 'default' => self::STATE_ACTIVE, | |
| 423 | + ] ); | |
| 424 | + | |
| 425 | + $this->add_feature( [ | |
| 380 | 426 | 'name' => 'e_optimized_markup', |
| 381 | 427 | 'title' => esc_html__( 'Optimized Markup', 'elementor' ), |
| 382 | 428 | 'tag' => esc_html__( 'Performance', 'elementor' ), |
| 383 | 429 | 'description' => esc_html__( 'Reduce the DOM size by eliminating HTML tags in various elements and widgets. This experiment includes markup changes so it might require updating custom CSS/JS code and cause compatibility issues with third party plugins.', 'elementor' ), |
| 384 | - 'release_status' => self::RELEASE_STATUS_STABLE, | |
| 430 | + 'release_status' => self::RELEASE_STATUS_ALPHA, | |
| 385 | 431 | 'default' => self::STATE_INACTIVE, |
| 386 | - 'new_site' => [ | |
| 387 | - 'default_active' => true, | |
| 388 | - 'minimum_installation_version' => '3.30.0', | |
| 389 | - ], | |
| 390 | 432 | ] ); |
| 391 | 433 | |
| 392 | 434 | $this->add_feature( [ |
| 393 | - 'name' => 'e_optimized_css_files', | |
| 394 | - 'title' => esc_html__( 'Optimized CSS Files', 'elementor' ), | |
| 435 | + 'name' => 'e_onboarding', | |
| 436 | + 'title' => esc_html__( 'Plugin Onboarding', 'elementor' ), | |
| 437 | + 'description' => esc_html__( 'New plugin onboarding.', 'elementor' ), | |
| 438 | + static::TYPE_HIDDEN => true, | |
| 439 | + 'release_status' => self::RELEASE_STATUS_ALPHA, | |
| 440 | + 'default' => self::STATE_ACTIVE, | |
| 441 | + ] ); | |
| 442 | + | |
| 443 | + // TODO: Possibly remove experiment in v3.27.0 [ED-15717]. | |
| 444 | + // Check this reference in Pro: 'sticky_anchor_link_offset'. | |
| 445 | + $this->add_feature( [ | |
| 446 | + 'name' => 'e_css_smooth_scroll', | |
| 447 | + 'title' => esc_html__( 'CSS Smooth Scroll', 'elementor' ), | |
| 395 | 448 | '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' ), | |
| 449 | + 'description' => esc_html__( 'Use CSS Smooth Scroll to improve the user experience on your site. This experiment replaces the default JavaScript-based smooth scroll with a CSS-based solution.', 'elementor' ), | |
| 450 | + 'release_status' => self::RELEASE_STATUS_DEV, | |
| 451 | + static::TYPE_HIDDEN => true, | |
| 452 | + 'default' => self::STATE_ACTIVE, | |
| 453 | + 'mutable' => false, | |
| 454 | + ] ); | |
| 455 | + | |
| 456 | + $this->add_feature( [ | |
| 457 | + 'name' => 'e_head_loading_styles', | |
| 458 | + 'title' => esc_html__( 'Head Loading Styles', 'elementor' ), | |
| 459 | + 'description' => esc_html__( 'Head Loading Styles', 'elementor' ), | |
| 460 | + 'hidden' => true, | |
| 461 | + 'default' => self::STATE_INACTIVE, | |
| 397 | 462 | 'release_status' => self::RELEASE_STATUS_ALPHA, |
| 398 | - 'default' => self::STATE_INACTIVE, | |
| 399 | - 'generator_tag' => true, | |
| 400 | 463 | ] ); |
| 401 | 464 | } |
| 402 | 465 | |
| 403 | 466 | /** |
| @@ -424,8 +487,9 @@ | ||
| 424 | 487 | $this->release_statuses = [ |
| 425 | 488 | self::RELEASE_STATUS_DEV => esc_html__( 'Development', 'elementor' ), |
| 426 | 489 | self::RELEASE_STATUS_ALPHA => esc_html__( 'Alpha', 'elementor' ), |
| 427 | 490 | self::RELEASE_STATUS_BETA => esc_html__( 'Beta', 'elementor' ), |
| 491 | + self::RELEASE_STATUS_RC => esc_html__( 'Release Candidate', 'elementor' ), | |
| 428 | 492 | self::RELEASE_STATUS_STABLE => esc_html__( 'Stable', 'elementor' ), |
| 429 | 493 | ]; |
| 430 | 494 | } |
| 431 | 495 | |
| @@ -449,8 +513,9 @@ | ||
| 449 | 513 | * @param Settings $settings |
| 450 | 514 | * |
| 451 | 515 | * @since 3.1.0 |
| 452 | 516 | * @access private |
| 517 | + * | |
| 453 | 518 | */ |
| 454 | 519 | private function register_settings_fields( Settings $settings ) { |
| 455 | 520 | $features = $this->get_features(); |
| 456 | 521 | |
| @@ -538,9 +603,9 @@ | ||
| 538 | 603 | <?php echo esc_html__( 'Experiments and Features', 'elementor' ); ?> |
| 539 | 604 | </h2> |
| 540 | 605 | <p class="e-experiment__description"> |
| 541 | 606 | <?php |
| 542 | - printf( | |
| 607 | + echo sprintf( | |
| 543 | 608 | /* translators: %1$s Link open tag, %2$s: Link close tag. */ |
| 544 | 609 | esc_html__( 'Personalize your Elementor experience by controlling which features and experiments are active on your site. Help make Elementor better by %1$ssharing your experience and feedback with us%2$s.', 'elementor' ), |
| 545 | 610 | '<a href="https://go.elementor.com/wp-dash-experiments-report-an-issue/" target="_blank">', |
| 546 | 611 | '</a>' |
| @@ -548,9 +613,9 @@ | ||
| 548 | 613 | ?> |
| 549 | 614 | </p> |
| 550 | 615 | <p class="e-experiment__description"> |
| 551 | 616 | <?php |
| 552 | - printf( | |
| 617 | + echo sprintf( | |
| 553 | 618 | '%1$s <a href="https://go.elementor.com/wp-dash-experiments/" target="_blank">%2$s</a>', |
| 554 | 619 | esc_html__( 'To use an experiment or feature on your site, simply click on the dropdown next to it and switch to Active. You can always deactivate them at any time.', 'elementor' ), |
| 555 | 620 | esc_html__( 'Learn more', 'elementor' ), |
| 556 | 621 | ); |
| @@ -642,13 +707,13 @@ | ||
| 642 | 707 | ->find( function ( $dependency ) { |
| 643 | 708 | return $dependency instanceof Non_Existing_Dependency; |
| 644 | 709 | } ); |
| 645 | 710 | |
| 646 | - return (bool) $non_existing_dep; | |
| 711 | + return ! ! $non_existing_dep; | |
| 647 | 712 | } |
| 648 | 713 | |
| 649 | 714 | /** |
| 650 | - * Get Feature Settings Label HTML. | |
| 715 | + * Get Feature Settings Label HTML | |
| 651 | 716 | * |
| 652 | 717 | * @since 3.1.0 |
| 653 | 718 | * @access private |
| 654 | 719 | * |
| @@ -673,16 +738,10 @@ | ||
| 673 | 738 | <div class="e-experiment__title"> |
| 674 | 739 | <div class="<?php echo $indicator_classes; ?>" data-tooltip="<?php echo $indicator_tooltip; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped ?>"></div> |
| 675 | 740 | <label class="e-experiment__title__label" for="e-experiment-<?php echo $feature['name']; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped ?>"><?php echo $feature['title']; ?></label> |
| 676 | 741 | <?php foreach ( $feature['tags'] as $tag ) { ?> |
| 677 | - <?php | |
| 678 | - $tag_classes = 'e-experiment__title__tag e-experiment__title__tag__' . $tag['type'] . ' e-editor-one'; | |
| 679 | - ?> | |
| 680 | - <span class="<?php echo esc_attr( $tag_classes ); ?>"><?php echo $tag['label']; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped ?></span> | |
| 742 | + <span class="e-experiment__title__tag e-experiment__title__tag__<?php echo $tag['type']; ?>"><?php echo $tag['label']; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped ?></span> | |
| 681 | 743 | <?php } ?> |
| 682 | - <?php if ( $feature['deprecated'] ) { ?> | |
| 683 | - <span class="e-experiment__title__tag e-experiment__title__tag__deprecated"><?php echo esc_html__( 'Deprecated', 'elementor' ); ?></span> | |
| 684 | - <?php } ?> | |
| 685 | 744 | </div> |
| 686 | 745 | <?php |
| 687 | 746 | |
| 688 | 747 | return ob_get_clean(); |
| @@ -746,11 +805,12 @@ | ||
| 746 | 805 | * |
| 747 | 806 | * @since 3.1.0 |
| 748 | 807 | * @access private |
| 749 | 808 | * |
| 750 | - * @param array $old_feature_data | |
| 809 | + * @param array $old_feature_data | |
| 751 | 810 | * @param string $new_state |
| 752 | - * @param string $old_state | |
| 811 | + * | |
| 812 | + * @throws \Elementor\Core\Experiments\Exceptions\Dependency_Exception | |
| 753 | 813 | */ |
| 754 | 814 | private function on_feature_state_change( array $old_feature_data, $new_state, $old_state ) { |
| 755 | 815 | $new_feature_data = $this->get_features( $old_feature_data['name'] ); |
| 756 | 816 | $this->validate_dependency( $new_feature_data, $new_state ); |
| @@ -767,9 +827,9 @@ | ||
| 767 | 827 | do_action( 'elementor/experiments/feature-state-change/' . $old_feature_data['name'], $old_state, $new_state ); |
| 768 | 828 | } |
| 769 | 829 | |
| 770 | 830 | /** |
| 771 | - * @throws Exceptions\Dependency_Exception If the feature dependency is not available or not active. | |
| 831 | + * @throws \Elementor\Core\Experiments\Exceptions\Dependency_Exception | |
| 772 | 832 | */ |
| 773 | 833 | private function validate_dependency( array $feature, $new_state ) { |
| 774 | 834 | $rollback = function ( $feature_option_key, $state ) { |
| 775 | 835 | remove_all_actions( 'add_option_' . $feature_option_key ); |
| @@ -790,41 +850,20 @@ | ||
| 790 | 850 | } |
| 791 | 851 | |
| 792 | 852 | // Validate if the current feature dependency is available. |
| 793 | 853 | 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 | - | |
| 805 | 854 | $dependency_feature = $this->get_features( $dependency->get_name() ); |
| 806 | 855 | |
| 807 | 856 | if ( ! $dependency_feature ) { |
| 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 | - } | |
| 857 | + $rollback( $feature_option_key, self::STATE_INACTIVE ); | |
| 817 | 858 | |
| 818 | - if ( $this->is_removed_or_hidden_dependency( $dependency_feature ) ) { | |
| 819 | - $this->warn_removed_or_hidden_dependency( | |
| 859 | + throw new Exceptions\Dependency_Exception( | |
| 820 | 860 | sprintf( |
| 821 | - 'The feature `%1$s` depends on hidden experiment `%2$s`.', | |
| 822 | - esc_html( $feature['name'] ), | |
| 823 | - esc_html( $dependency_feature['name'] ) | |
| 861 | + 'The feature `%s` has a dependency `%s` that is not available.', | |
| 862 | + $feature['name'], | |
| 863 | + $dependency->get_name() | |
| 824 | 864 | ) |
| 825 | 865 | ); |
| 826 | - continue; | |
| 827 | 866 | } |
| 828 | 867 | |
| 829 | 868 | $dependency_state = $this->get_feature_actual_state( $dependency_feature ); |
| 830 | 869 | |
| @@ -834,10 +873,10 @@ | ||
| 834 | 873 | |
| 835 | 874 | throw new Exceptions\Dependency_Exception( |
| 836 | 875 | sprintf( |
| 837 | 876 | 'To turn on `%1$s`, Experiment: `%2$s` activity is required!', |
| 838 | - esc_html( $feature['name'] ), | |
| 839 | - esc_html( $dependency_feature['name'] ) | |
| 877 | + $feature['name'], | |
| 878 | + $dependency_feature['name'] | |
| 840 | 879 | ) |
| 841 | 880 | ); |
| 842 | 881 | } |
| 843 | 882 | } |
| @@ -882,9 +921,11 @@ | ||
| 882 | 921 | * This behavior interferes with the experiments dependency mechanism because the data that's being sent can be in any order, |
| 883 | 922 | * while the dependencies mechanism expects it to be in a specific order (dependencies should be activated before their dependents can). |
| 884 | 923 | * In order to solve this issue, we sort the experiments in the POST data based on their dependencies tree. |
| 885 | 924 | * |
| 886 | - * @param array $allowed_options | |
| 925 | + * @param $allowed_options | |
| 926 | + * | |
| 927 | + * @return mixed | |
| 887 | 928 | */ |
| 888 | 929 | private function sort_allowed_options_by_dependencies( $allowed_options ) { |
| 889 | 930 | if ( ! isset( $allowed_options['elementor'] ) ) { |
| 890 | 931 | return $allowed_options; |
| @@ -972,137 +1013,6 @@ | ||
| 972 | 1013 | // Register CLI commands. |
| 973 | 1014 | if ( Utils::is_wp_cli() ) { |
| 974 | 1015 | \WP_CLI::add_command( 'elementor experiments', WP_CLI::class ); |
| 975 | 1016 | } |
| 976 | - } | |
| 977 | - | |
| 978 | - /** | |
| 979 | - * @param array $experimental_data | |
| 980 | - * @return array | |
| 981 | - */ | |
| 982 | - private function initialize_feature_dependencies( array $experimental_data ): array { | |
| 983 | - foreach ( $experimental_data['dependencies'] as $key => $dependency ) { | |
| 984 | - $feature = $this->get_features( $dependency ); | |
| 985 | - | |
| 986 | - if ( ! isset( $feature ) ) { | |
| 987 | - $this->warn_removed_or_hidden_dependency( | |
| 988 | - sprintf( | |
| 989 | - 'Feature %1$s depends on experiment %2$s that is not registered in Core.', | |
| 990 | - esc_html( $experimental_data['name'] ), | |
| 991 | - esc_html( $dependency ) | |
| 992 | - ) | |
| 993 | - ); | |
| 994 | - | |
| 995 | - $experimental_data['dependencies'][ $key ] = $this->create_dependency_class( $dependency, null ); | |
| 996 | - continue; | |
| 997 | - } | |
| 998 | - | |
| 999 | - if ( ! empty( $feature[ static::TYPE_HIDDEN ] ) ) { | |
| 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 | - ); | |
| 1007 | - } | |
| 1008 | - | |
| 1009 | - $experimental_data['dependencies'][ $key ] = $this->create_dependency_class( $dependency, $feature ); | |
| 1010 | - $experimental_data = $this->set_feature_default_state_to_match_dependencies( $feature, $experimental_data ); | |
| 1011 | - } | |
| 1012 | - | |
| 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 ); | |
| 1031 | - } | |
| 1032 | - | |
| 1033 | - /** | |
| 1034 | - * @param array $feature | |
| 1035 | - * @param array $experimental_data | |
| 1036 | - * @return array | |
| 1037 | - * | |
| 1038 | - * we must validate the state: | |
| 1039 | - * * A user can set a dependant feature to inactive and in upgrade we don't change users settings. | |
| 1040 | - * * A developer can set the default state to be invalid (e.g. dependant feature is inactive). | |
| 1041 | - * if one of the dependencies is inactive, the main feature should be inactive as well. | |
| 1042 | - */ | |
| 1043 | - private function set_feature_default_state_to_match_dependencies( array $feature, array $experimental_data ): array { | |
| 1044 | - if ( self::STATE_INACTIVE !== $this->get_feature_actual_state( $feature ) ) { | |
| 1045 | - return $experimental_data; | |
| 1046 | - } | |
| 1047 | - | |
| 1048 | - if ( self::STATE_ACTIVE === $experimental_data['state'] ) { | |
| 1049 | - $experimental_data['state'] = self::STATE_INACTIVE; | |
| 1050 | - } elseif ( self::STATE_DEFAULT === $experimental_data['state'] ) { | |
| 1051 | - $experimental_data['default'] = self::STATE_INACTIVE; | |
| 1052 | - } | |
| 1053 | - | |
| 1054 | - return $experimental_data; | |
| 1055 | - } | |
| 1056 | - | |
| 1057 | - /** | |
| 1058 | - * @param array $new_site | |
| 1059 | - * @param array $experimental_data | |
| 1060 | - * @return array | |
| 1061 | - */ | |
| 1062 | - private function set_new_site_default_state( $new_site, array $experimental_data ): array { | |
| 1063 | - if ( ! $this->install_compare( $new_site['minimum_installation_version'] ) ) { | |
| 1064 | - return $experimental_data; | |
| 1065 | - } | |
| 1066 | - | |
| 1067 | - if ( $new_site['always_active'] ) { | |
| 1068 | - $experimental_data['state'] = self::STATE_ACTIVE; | |
| 1069 | - $experimental_data['mutable'] = false; | |
| 1070 | - } elseif ( $new_site['default_active'] ) { | |
| 1071 | - $experimental_data['default'] = self::STATE_ACTIVE; | |
| 1072 | - } elseif ( $new_site['default_inactive'] ) { | |
| 1073 | - $experimental_data['default'] = self::STATE_INACTIVE; | |
| 1074 | - } | |
| 1075 | - | |
| 1076 | - return $experimental_data; | |
| 1077 | - } | |
| 1078 | - | |
| 1079 | - /** | |
| 1080 | - * @param array $options | |
| 1081 | - * @return array | |
| 1082 | - */ | |
| 1083 | - private function set_feature_initial_options( array $options ): array { | |
| 1084 | - $default_experimental_data = [ | |
| 1085 | - 'tag' => '', // Deprecated, use 'tags' instead. | |
| 1086 | - 'tags' => [], | |
| 1087 | - 'description' => '', | |
| 1088 | - 'release_status' => self::RELEASE_STATUS_ALPHA, | |
| 1089 | - 'default' => self::STATE_INACTIVE, | |
| 1090 | - 'mutable' => true, | |
| 1091 | - static::TYPE_HIDDEN => false, | |
| 1092 | - 'new_site' => [ | |
| 1093 | - 'always_active' => false, | |
| 1094 | - 'default_active' => false, | |
| 1095 | - 'default_inactive' => false, | |
| 1096 | - 'minimum_installation_version' => null, | |
| 1097 | - ], | |
| 1098 | - 'on_state_change' => null, | |
| 1099 | - 'generator_tag' => false, | |
| 1100 | - 'deprecated' => false, | |
| 1101 | - ]; | |
| 1102 | - | |
| 1103 | - $allowed_options = [ 'name', 'title', 'tag', 'tags', 'description', 'release_status', 'default', 'mutable', static::TYPE_HIDDEN, 'new_site', 'on_state_change', 'dependencies', 'generator_tag', 'messages', 'deprecated' ]; | |
| 1104 | - $experimental_data = $this->merge_properties( $default_experimental_data, $options, $allowed_options ); | |
| 1105 | - | |
| 1106 | - return $this->unify_feature_tags( $experimental_data ); | |
| 1107 | 1017 | } |
| 1108 | 1018 | } |
