| @@ -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 ) { |
| @@ -342,13 +378,14 @@ | ||
| 342 | 378 | $this->add_feature( [ |
| 343 | 379 | 'name' => 'container', |
| 344 | 380 | 'title' => esc_html__( 'Container', 'elementor' ), |
| 345 | 381 | 'description' => sprintf( |
| 346 | - /* translators: 1: Link opening tag, 2: Link closing tag, 3: Link opening tag, 4: Link closing tag */ | |
| 347 | - 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' ), | |
| 348 | 383 | '<a target="_blank" href="https://go.elementor.com/wp-dash-flex-container/">', |
| 349 | 384 | '</a>', |
| 350 | 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/">', | |
| 351 | 388 | '</a>' |
| 352 | 389 | ), |
| 353 | 390 | 'release_status' => self::RELEASE_STATUS_STABLE, |
| 354 | 391 | 'default' => self::STATE_INACTIVE, |
| @@ -364,29 +401,67 @@ | ||
| 364 | 401 | ), |
| 365 | 402 | ], |
| 366 | 403 | ] ); |
| 367 | 404 | |
| 405 | + // TODO: Remove this experiment in v3.28 [ED-15983]. | |
| 368 | 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( [ | |
| 369 | 426 | 'name' => 'e_optimized_markup', |
| 370 | 427 | 'title' => esc_html__( 'Optimized Markup', 'elementor' ), |
| 371 | 428 | 'tag' => esc_html__( 'Performance', 'elementor' ), |
| 372 | 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' ), |
| 373 | - 'release_status' => self::RELEASE_STATUS_STABLE, | |
| 430 | + 'release_status' => self::RELEASE_STATUS_ALPHA, | |
| 374 | 431 | 'default' => self::STATE_INACTIVE, |
| 375 | - 'new_site' => [ | |
| 376 | - 'default_active' => true, | |
| 377 | - 'minimum_installation_version' => '3.30.0', | |
| 378 | - ], | |
| 379 | 432 | ] ); |
| 380 | 433 | |
| 381 | 434 | $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' ), | |
| 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' ), | |
| 448 | + 'tag' => esc_html__( 'Performance', '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' ), | |
| 385 | 450 | 'release_status' => self::RELEASE_STATUS_DEV, |
| 451 | + static::TYPE_HIDDEN => true, | |
| 386 | 452 | 'default' => self::STATE_ACTIVE, |
| 387 | - 'type' => self::TYPE_HIDDEN, | |
| 453 | + 'mutable' => false, | |
| 388 | 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, | |
| 462 | + 'release_status' => self::RELEASE_STATUS_ALPHA, | |
| 463 | + ] ); | |
| 389 | 464 | } |
| 390 | 465 | |
| 391 | 466 | /** |
| 392 | 467 | * Init States |
| @@ -412,8 +487,9 @@ | ||
| 412 | 487 | $this->release_statuses = [ |
| 413 | 488 | self::RELEASE_STATUS_DEV => esc_html__( 'Development', 'elementor' ), |
| 414 | 489 | self::RELEASE_STATUS_ALPHA => esc_html__( 'Alpha', 'elementor' ), |
| 415 | 490 | self::RELEASE_STATUS_BETA => esc_html__( 'Beta', 'elementor' ), |
| 491 | + self::RELEASE_STATUS_RC => esc_html__( 'Release Candidate', 'elementor' ), | |
| 416 | 492 | self::RELEASE_STATUS_STABLE => esc_html__( 'Stable', 'elementor' ), |
| 417 | 493 | ]; |
| 418 | 494 | } |
| 419 | 495 | |
| @@ -437,8 +513,9 @@ | ||
| 437 | 513 | * @param Settings $settings |
| 438 | 514 | * |
| 439 | 515 | * @since 3.1.0 |
| 440 | 516 | * @access private |
| 517 | + * | |
| 441 | 518 | */ |
| 442 | 519 | private function register_settings_fields( Settings $settings ) { |
| 443 | 520 | $features = $this->get_features(); |
| 444 | 521 | |
| @@ -526,9 +603,9 @@ | ||
| 526 | 603 | <?php echo esc_html__( 'Experiments and Features', 'elementor' ); ?> |
| 527 | 604 | </h2> |
| 528 | 605 | <p class="e-experiment__description"> |
| 529 | 606 | <?php |
| 530 | - printf( | |
| 607 | + echo sprintf( | |
| 531 | 608 | /* translators: %1$s Link open tag, %2$s: Link close tag. */ |
| 532 | 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' ), |
| 533 | 610 | '<a href="https://go.elementor.com/wp-dash-experiments-report-an-issue/" target="_blank">', |
| 534 | 611 | '</a>' |
| @@ -536,9 +613,9 @@ | ||
| 536 | 613 | ?> |
| 537 | 614 | </p> |
| 538 | 615 | <p class="e-experiment__description"> |
| 539 | 616 | <?php |
| 540 | - printf( | |
| 617 | + echo sprintf( | |
| 541 | 618 | '%1$s <a href="https://go.elementor.com/wp-dash-experiments/" target="_blank">%2$s</a>', |
| 542 | 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' ), |
| 543 | 620 | esc_html__( 'Learn more', 'elementor' ), |
| 544 | 621 | ); |
| @@ -630,13 +707,13 @@ | ||
| 630 | 707 | ->find( function ( $dependency ) { |
| 631 | 708 | return $dependency instanceof Non_Existing_Dependency; |
| 632 | 709 | } ); |
| 633 | 710 | |
| 634 | - return (bool) $non_existing_dep; | |
| 711 | + return ! ! $non_existing_dep; | |
| 635 | 712 | } |
| 636 | 713 | |
| 637 | 714 | /** |
| 638 | - * Get Feature Settings Label HTML. | |
| 715 | + * Get Feature Settings Label HTML | |
| 639 | 716 | * |
| 640 | 717 | * @since 3.1.0 |
| 641 | 718 | * @access private |
| 642 | 719 | * |
| @@ -661,16 +738,10 @@ | ||
| 661 | 738 | <div class="e-experiment__title"> |
| 662 | 739 | <div class="<?php echo $indicator_classes; ?>" data-tooltip="<?php echo $indicator_tooltip; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped ?>"></div> |
| 663 | 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> |
| 664 | 741 | <?php foreach ( $feature['tags'] as $tag ) { ?> |
| 665 | - <?php | |
| 666 | - $tag_classes = 'e-experiment__title__tag e-experiment__title__tag__' . $tag['type'] . ' e-editor-one'; | |
| 667 | - ?> | |
| 668 | - <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> | |
| 669 | 743 | <?php } ?> |
| 670 | - <?php if ( $feature['deprecated'] ) { ?> | |
| 671 | - <span class="e-experiment__title__tag e-experiment__title__tag__deprecated"><?php echo esc_html__( 'Deprecated', 'elementor' ); ?></span> | |
| 672 | - <?php } ?> | |
| 673 | 744 | </div> |
| 674 | 745 | <?php |
| 675 | 746 | |
| 676 | 747 | return ob_get_clean(); |
| @@ -734,11 +805,12 @@ | ||
| 734 | 805 | * |
| 735 | 806 | * @since 3.1.0 |
| 736 | 807 | * @access private |
| 737 | 808 | * |
| 738 | - * @param array $old_feature_data | |
| 809 | + * @param array $old_feature_data | |
| 739 | 810 | * @param string $new_state |
| 740 | - * @param string $old_state | |
| 811 | + * | |
| 812 | + * @throws \Elementor\Core\Experiments\Exceptions\Dependency_Exception | |
| 741 | 813 | */ |
| 742 | 814 | private function on_feature_state_change( array $old_feature_data, $new_state, $old_state ) { |
| 743 | 815 | $new_feature_data = $this->get_features( $old_feature_data['name'] ); |
| 744 | 816 | $this->validate_dependency( $new_feature_data, $new_state ); |
| @@ -755,9 +827,9 @@ | ||
| 755 | 827 | do_action( 'elementor/experiments/feature-state-change/' . $old_feature_data['name'], $old_state, $new_state ); |
| 756 | 828 | } |
| 757 | 829 | |
| 758 | 830 | /** |
| 759 | - * @throws Exceptions\Dependency_Exception If the feature dependency is not available or not active. | |
| 831 | + * @throws \Elementor\Core\Experiments\Exceptions\Dependency_Exception | |
| 760 | 832 | */ |
| 761 | 833 | private function validate_dependency( array $feature, $new_state ) { |
| 762 | 834 | $rollback = function ( $feature_option_key, $state ) { |
| 763 | 835 | remove_all_actions( 'add_option_' . $feature_option_key ); |
| @@ -786,10 +858,10 @@ | ||
| 786 | 858 | |
| 787 | 859 | throw new Exceptions\Dependency_Exception( |
| 788 | 860 | sprintf( |
| 789 | 861 | 'The feature `%s` has a dependency `%s` that is not available.', |
| 790 | - esc_html( $feature['name'] ), | |
| 791 | - esc_html( $dependency->get_name() ) | |
| 862 | + $feature['name'], | |
| 863 | + $dependency->get_name() | |
| 792 | 864 | ) |
| 793 | 865 | ); |
| 794 | 866 | } |
| 795 | 867 | |
| @@ -801,10 +873,10 @@ | ||
| 801 | 873 | |
| 802 | 874 | throw new Exceptions\Dependency_Exception( |
| 803 | 875 | sprintf( |
| 804 | 876 | 'To turn on `%1$s`, Experiment: `%2$s` activity is required!', |
| 805 | - esc_html( $feature['name'] ), | |
| 806 | - esc_html( $dependency_feature['name'] ) | |
| 877 | + $feature['name'], | |
| 878 | + $dependency_feature['name'] | |
| 807 | 879 | ) |
| 808 | 880 | ); |
| 809 | 881 | } |
| 810 | 882 | } |
| @@ -849,9 +921,11 @@ | ||
| 849 | 921 | * This behavior interferes with the experiments dependency mechanism because the data that's being sent can be in any order, |
| 850 | 922 | * while the dependencies mechanism expects it to be in a specific order (dependencies should be activated before their dependents can). |
| 851 | 923 | * In order to solve this issue, we sort the experiments in the POST data based on their dependencies tree. |
| 852 | 924 | * |
| 853 | - * @param array $allowed_options | |
| 925 | + * @param $allowed_options | |
| 926 | + * | |
| 927 | + * @return mixed | |
| 854 | 928 | */ |
| 855 | 929 | private function sort_allowed_options_by_dependencies( $allowed_options ) { |
| 856 | 930 | if ( ! isset( $allowed_options['elementor'] ) ) { |
| 857 | 931 | return $allowed_options; |
| @@ -939,114 +1013,6 @@ | ||
| 939 | 1013 | // Register CLI commands. |
| 940 | 1014 | if ( Utils::is_wp_cli() ) { |
| 941 | 1015 | \WP_CLI::add_command( 'elementor experiments', WP_CLI::class ); |
| 942 | 1016 | } |
| 943 | - } | |
| 944 | - | |
| 945 | - /** | |
| 946 | - * @param array $experimental_data | |
| 947 | - * @return array | |
| 948 | - * | |
| 949 | - * @throws Exceptions\Dependency_Exception If the feature dependency is not initialized or depends on a hidden experiment. | |
| 950 | - */ | |
| 951 | - private function initialize_feature_dependencies( array $experimental_data ): array { | |
| 952 | - foreach ( $experimental_data['dependencies'] as $key => $dependency ) { | |
| 953 | - $feature = $this->get_features( $dependency ); | |
| 954 | - | |
| 955 | - 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( | |
| 958 | - sprintf( | |
| 959 | - 'Feature %s cannot be initialized before dependency feature: %s.', | |
| 960 | - esc_html( $experimental_data['name'] ), | |
| 961 | - esc_html( $dependency ) | |
| 962 | - ) | |
| 963 | - ); | |
| 964 | - } | |
| 965 | - | |
| 966 | - if ( ! empty( $feature[ static::TYPE_HIDDEN ] ) ) { | |
| 967 | - throw new Exceptions\Dependency_Exception( 'Depending on a hidden experiment is not allowed.' ); | |
| 968 | - } | |
| 969 | - | |
| 970 | - $experimental_data['dependencies'][ $key ] = $this->create_dependency_class( $dependency, $feature ); | |
| 971 | - $experimental_data = $this->set_feature_default_state_to_match_dependencies( $feature, $experimental_data ); | |
| 972 | - } | |
| 973 | - | |
| 974 | - return $experimental_data; | |
| 975 | - } | |
| 976 | - | |
| 977 | - /** | |
| 978 | - * @param array $feature | |
| 979 | - * @param array $experimental_data | |
| 980 | - * @return array | |
| 981 | - * | |
| 982 | - * we must validate the state: | |
| 983 | - * * A user can set a dependant feature to inactive and in upgrade we don't change users settings. | |
| 984 | - * * A developer can set the default state to be invalid (e.g. dependant feature is inactive). | |
| 985 | - * if one of the dependencies is inactive, the main feature should be inactive as well. | |
| 986 | - */ | |
| 987 | - private function set_feature_default_state_to_match_dependencies( array $feature, array $experimental_data ): array { | |
| 988 | - if ( self::STATE_INACTIVE !== $this->get_feature_actual_state( $feature ) ) { | |
| 989 | - return $experimental_data; | |
| 990 | - } | |
| 991 | - | |
| 992 | - if ( self::STATE_ACTIVE === $experimental_data['state'] ) { | |
| 993 | - $experimental_data['state'] = self::STATE_INACTIVE; | |
| 994 | - } elseif ( self::STATE_DEFAULT === $experimental_data['state'] ) { | |
| 995 | - $experimental_data['default'] = self::STATE_INACTIVE; | |
| 996 | - } | |
| 997 | - | |
| 998 | - return $experimental_data; | |
| 999 | - } | |
| 1000 | - | |
| 1001 | - /** | |
| 1002 | - * @param array $new_site | |
| 1003 | - * @param array $experimental_data | |
| 1004 | - * @return array | |
| 1005 | - */ | |
| 1006 | - private function set_new_site_default_state( $new_site, array $experimental_data ): array { | |
| 1007 | - if ( ! $this->install_compare( $new_site['minimum_installation_version'] ) ) { | |
| 1008 | - return $experimental_data; | |
| 1009 | - } | |
| 1010 | - | |
| 1011 | - if ( $new_site['always_active'] ) { | |
| 1012 | - $experimental_data['state'] = self::STATE_ACTIVE; | |
| 1013 | - $experimental_data['mutable'] = false; | |
| 1014 | - } elseif ( $new_site['default_active'] ) { | |
| 1015 | - $experimental_data['default'] = self::STATE_ACTIVE; | |
| 1016 | - } elseif ( $new_site['default_inactive'] ) { | |
| 1017 | - $experimental_data['default'] = self::STATE_INACTIVE; | |
| 1018 | - } | |
| 1019 | - | |
| 1020 | - return $experimental_data; | |
| 1021 | - } | |
| 1022 | - | |
| 1023 | - /** | |
| 1024 | - * @param array $options | |
| 1025 | - * @return array | |
| 1026 | - */ | |
| 1027 | - private function set_feature_initial_options( array $options ): array { | |
| 1028 | - $default_experimental_data = [ | |
| 1029 | - 'tag' => '', // Deprecated, use 'tags' instead. | |
| 1030 | - 'tags' => [], | |
| 1031 | - 'description' => '', | |
| 1032 | - 'release_status' => self::RELEASE_STATUS_ALPHA, | |
| 1033 | - 'default' => self::STATE_INACTIVE, | |
| 1034 | - 'mutable' => true, | |
| 1035 | - static::TYPE_HIDDEN => false, | |
| 1036 | - 'new_site' => [ | |
| 1037 | - 'always_active' => false, | |
| 1038 | - 'default_active' => false, | |
| 1039 | - 'default_inactive' => false, | |
| 1040 | - 'minimum_installation_version' => null, | |
| 1041 | - ], | |
| 1042 | - 'on_state_change' => null, | |
| 1043 | - 'generator_tag' => false, | |
| 1044 | - 'deprecated' => false, | |
| 1045 | - ]; | |
| 1046 | - | |
| 1047 | - $allowed_options = [ 'name', 'title', 'tag', 'tags', 'description', 'release_status', 'default', 'mutable', static::TYPE_HIDDEN, 'new_site', 'on_state_change', 'dependencies', 'generator_tag', 'messages', 'deprecated' ]; | |
| 1048 | - $experimental_data = $this->merge_properties( $default_experimental_data, $options, $allowed_options ); | |
| 1049 | - | |
| 1050 | - return $this->unify_feature_tags( $experimental_data ); | |
| 1051 | 1017 | } |
| 1052 | 1018 | } |
