| @@ -4,13 +4,12 @@ | ||
| 4 | 4 | use Elementor\Core\Base\Module as BaseModule; |
| 5 | 5 | use Elementor\Core\Documents_Manager; |
| 6 | 6 | use Elementor\Core\Experiments\Manager as Experiments_Manager; |
| 7 | 7 | use Elementor\Modules\LandingPages\Documents\Landing_Page; |
| 8 | -use Elementor\Modules\LandingPages\AdminMenuItems\Editor_One_Landing_Pages_Menu; | |
| 9 | 8 | use Elementor\Modules\LandingPages\Module as Landing_Pages_Module; |
| 10 | -use Elementor\Modules\EditorOne\Classes\Menu_Data_Provider; | |
| 11 | 9 | use Elementor\Plugin; |
| 12 | 10 | use Elementor\TemplateLibrary\Source_Local; |
| 11 | +use Elementor\Utils; | |
| 13 | 12 | |
| 14 | 13 | if ( ! defined( 'ABSPATH' ) ) { |
| 15 | 14 | exit; // Exit if accessed directly. |
| 16 | 15 | } |
| @@ -19,11 +18,10 @@ | ||
| 19 | 18 | |
| 20 | 19 | const DOCUMENT_TYPE = 'landing-page'; |
| 21 | 20 | const CPT = 'e-landing-page'; |
| 22 | 21 | const ADMIN_PAGE_SLUG = 'edit.php?post_type=' . self::CPT; |
| 23 | - const ACTIVATION_KEY = 'elementor_landing_pages_activation'; | |
| 24 | 22 | |
| 25 | - private $has_pages = null; | |
| 23 | + private $posts; | |
| 26 | 24 | private $trashed_posts; |
| 27 | 25 | private $new_lp_url; |
| 28 | 26 | private $permalink_structure; |
| 29 | 27 | |
| @@ -31,55 +29,31 @@ | ||
| 31 | 29 | return 'landing-pages'; |
| 32 | 30 | } |
| 33 | 31 | |
| 34 | 32 | /** |
| 35 | - * Register Experimental Feature | |
| 33 | + * Get Experimental Data | |
| 36 | 34 | * |
| 37 | 35 | * Implementation of this method makes the module an experiment. |
| 38 | 36 | * |
| 39 | - * @since 3.28.0 | |
| 37 | + * @since 3.1.0 | |
| 38 | + * | |
| 39 | + * @return array | |
| 40 | 40 | */ |
| 41 | - private function register_experiment() { | |
| 42 | - Plugin::$instance->experiments->add_feature( [ | |
| 41 | + public static function get_experimental_data() { | |
| 42 | + return [ | |
| 43 | 43 | 'name' => 'landing-pages', |
| 44 | 44 | 'title' => esc_html__( 'Landing Pages', 'elementor' ), |
| 45 | 45 | 'description' => esc_html__( 'Adds a new Elementor content type that allows creating beautiful landing pages instantly in a streamlined workflow.', 'elementor' ), |
| 46 | - 'release_status' => Experiments_Manager::RELEASE_STATUS_BETA, | |
| 46 | + 'release_status' => Experiments_Manager::RELEASE_STATUS_STABLE, | |
| 47 | 47 | 'default' => Experiments_Manager::STATE_ACTIVE, |
| 48 | 48 | 'new_site' => [ |
| 49 | - 'default_inactive' => true, | |
| 50 | - 'minimum_installation_version' => '3.22.0', | |
| 49 | + 'default_active' => true, | |
| 50 | + 'minimum_installation_version' => '3.1.0-beta', | |
| 51 | 51 | ], |
| 52 | - 'deprecated' => true, | |
| 53 | - ] ); | |
| 52 | + ]; | |
| 54 | 53 | } |
| 55 | 54 | |
| 56 | 55 | /** |
| 57 | - * Should activate landing pages | |
| 58 | - * | |
| 59 | - * Checks whether the Landing Pages should be activated. | |
| 60 | - * | |
| 61 | - * If the activation key set to `1` in wp_options, the Landing Pages feature should be active. Otherwise not. | |
| 62 | - * This is a backwards compatibility for websites that had Landing Pages, therefore couldn't be deactivated. | |
| 63 | - * When deleting posts in Landing Pages CPT, Elementor checks again whether this feature should be activated. | |
| 64 | - * | |
| 65 | - * @since 3.31.0 | |
| 66 | - */ | |
| 67 | - private function should_activate_landing_pages() { | |
| 68 | - if ( '1' === get_option( self::ACTIVATION_KEY ) ) { | |
| 69 | - return true; | |
| 70 | - } | |
| 71 | - | |
| 72 | - if ( $this->has_landing_pages() ) { | |
| 73 | - update_option( self::ACTIVATION_KEY, '1' ); | |
| 74 | - return true; | |
| 75 | - } | |
| 76 | - | |
| 77 | - update_option( self::ACTIVATION_KEY, '0' ); | |
| 78 | - return false; | |
| 79 | - } | |
| 80 | - | |
| 81 | - /** | |
| 82 | 56 | * Get Trashed Landing Pages Posts |
| 83 | 57 | * |
| 84 | 58 | * Returns the posts property of a WP_Query run for Landing Pages with post_status of 'trash'. |
| 85 | 59 | * |
| @@ -106,13 +80,23 @@ | ||
| 106 | 80 | |
| 107 | 81 | return $this->trashed_posts; |
| 108 | 82 | } |
| 109 | 83 | |
| 110 | - private function has_landing_pages() { | |
| 111 | - if ( null !== $this->has_pages ) { | |
| 112 | - return $this->has_pages; | |
| 84 | + /** | |
| 85 | + * Get Landing Pages Posts | |
| 86 | + * | |
| 87 | + * Returns the posts property of a WP_Query run for posts with the Landing Pages CPT. | |
| 88 | + * | |
| 89 | + * @since 3.1.0 | |
| 90 | + * | |
| 91 | + * @return array posts | |
| 92 | + */ | |
| 93 | + private function get_landing_page_posts() { | |
| 94 | + if ( $this->posts ) { | |
| 95 | + return $this->posts; | |
| 113 | 96 | } |
| 114 | 97 | |
| 98 | + // `'posts_per_page' => 1` is because this is only used as an indicator to whether there are any landing pages. | |
| 115 | 99 | $posts_query = new \WP_Query( [ |
| 116 | 100 | 'no_found_rows' => true, |
| 117 | 101 | 'post_type' => self::CPT, |
| 118 | 102 | 'post_status' => 'any', |
| @@ -120,11 +104,11 @@ | ||
| 120 | 104 | 'meta_key' => '_elementor_template_type', |
| 121 | 105 | 'meta_value' => self::DOCUMENT_TYPE, |
| 122 | 106 | ] ); |
| 123 | 107 | |
| 124 | - $this->has_pages = $posts_query->post_count > 0; | |
| 108 | + $this->posts = $posts_query->posts; | |
| 125 | 109 | |
| 126 | - return $this->has_pages; | |
| 110 | + return $this->posts; | |
| 127 | 111 | } |
| 128 | 112 | |
| 129 | 113 | /** |
| 130 | 114 | * Is Elementor Landing Page. |
| @@ -133,9 +117,9 @@ | ||
| 133 | 117 | * |
| 134 | 118 | * @since 3.1.0 |
| 135 | 119 | * @access public |
| 136 | 120 | * |
| 137 | - * @param \WP_Post $post Post Object. | |
| 121 | + * @param \WP_Post $post Post Object | |
| 138 | 122 | * |
| 139 | 123 | * @return bool Whether the post was built with Elementor. |
| 140 | 124 | */ |
| 141 | 125 | public function is_elementor_landing_page( $post ) { |
| @@ -141,25 +125,38 @@ | ||
| 141 | 125 | public function is_elementor_landing_page( $post ) { |
| 142 | 126 | return self::CPT === $post->post_type; |
| 143 | 127 | } |
| 144 | 128 | |
| 145 | - public function get_menu_args() { | |
| 146 | - if ( $this->has_landing_pages() ) { | |
| 147 | - $menu_slug = self::ADMIN_PAGE_SLUG; | |
| 148 | - $function = null; | |
| 129 | + /** | |
| 130 | + * Add Submenu Page | |
| 131 | + * | |
| 132 | + * Adds the 'Landing Pages' submenu item to the 'Templates' menu item. | |
| 133 | + * | |
| 134 | + * @since 3.1.0 | |
| 135 | + */ | |
| 136 | + private function add_submenu_page() { | |
| 137 | + $posts = $this->get_landing_page_posts(); | |
| 138 | + | |
| 139 | + // If there are no Landing Pages, show the "Create Your First Landing Page" page. | |
| 140 | + // If there are, show the pages table. | |
| 141 | + if ( ! empty( $posts ) ) { | |
| 142 | + $landing_page_menu_slug = self::ADMIN_PAGE_SLUG; | |
| 143 | + $landing_page_menu_callback = null; | |
| 149 | 144 | } else { |
| 150 | - $menu_slug = self::CPT; | |
| 151 | - $function = [ $this, 'print_empty_landing_pages_page' ]; | |
| 145 | + $landing_page_menu_slug = self::CPT; | |
| 146 | + $landing_page_menu_callback = [ $this, 'print_empty_landing_pages_page' ]; | |
| 152 | 147 | } |
| 153 | 148 | |
| 154 | - return [ | |
| 155 | - 'menu_slug' => $menu_slug, | |
| 156 | - 'function' => $function, | |
| 157 | - ]; | |
| 158 | - } | |
| 149 | + $landing_pages_title = esc_html__( 'Landing Pages', 'elementor' ); | |
| 159 | 150 | |
| 160 | - private function register_editor_one_menu( Menu_Data_Provider $menu_data_provider ): void { | |
| 161 | - $menu_data_provider->register_menu( new Editor_One_Landing_Pages_Menu( $this ) ); | |
| 151 | + add_submenu_page( | |
| 152 | + Source_Local::ADMIN_MENU_SLUG, | |
| 153 | + $landing_pages_title, | |
| 154 | + $landing_pages_title, | |
| 155 | + 'manage_options', | |
| 156 | + $landing_page_menu_slug, | |
| 157 | + $landing_page_menu_callback | |
| 158 | + ); | |
| 162 | 159 | } |
| 163 | 160 | |
| 164 | 161 | /** |
| 165 | 162 | * Get 'Add New' Landing Page URL |
| @@ -202,9 +199,9 @@ | ||
| 202 | 199 | <div class="e-trashed-items"> |
| 203 | 200 | <?php |
| 204 | 201 | printf( |
| 205 | 202 | /* translators: %1$s Link open tag, %2$s: Link close tag. */ |
| 206 | - esc_html__( 'Or view %1$sTrashed Items%2$s', 'elementor' ), | |
| 203 | + esc_html__( 'Or view %1$sTrashed Items%1$s', 'elementor' ), | |
| 207 | 204 | '<a href="' . esc_url( admin_url( 'edit.php?post_status=trash&post_type=' . self::CPT ) ) . '">', |
| 208 | 205 | '</a>' |
| 209 | 206 | ); |
| 210 | 207 | ?> |
| @@ -245,9 +242,9 @@ | ||
| 245 | 242 | 'urls' => [ |
| 246 | 243 | 'addNewLandingPageUrl' => $this->get_add_new_landing_page_url(), |
| 247 | 244 | ], |
| 248 | 245 | 'landingPages' => [ |
| 249 | - 'landingPagesHasPages' => $this->has_landing_pages(), | |
| 246 | + 'landingPagesHasPages' => [] !== $this->get_landing_page_posts(), | |
| 250 | 247 | 'isLandingPageAdminEdit' => $this->is_landing_page_admin_edit(), |
| 251 | 248 | ], |
| 252 | 249 | ]; |
| 253 | 250 | |
| @@ -311,9 +308,9 @@ | ||
| 311 | 308 | return $post_link; |
| 312 | 309 | } |
| 313 | 310 | |
| 314 | 311 | // Any slug prefixes need to be removed from the post link. |
| 315 | - return trailingslashit( get_home_url() ) . trailingslashit( $post->post_name ); | |
| 312 | + return get_home_url() . '/' . $post->post_name . '/'; | |
| 316 | 313 | } |
| 317 | 314 | |
| 318 | 315 | /** |
| 319 | 316 | * Adjust Landing Page Query |
| @@ -421,18 +418,8 @@ | ||
| 421 | 418 | return false; |
| 422 | 419 | } |
| 423 | 420 | |
| 424 | 421 | public function __construct() { |
| 425 | - if ( ! $this->should_activate_landing_pages() ) { | |
| 426 | - return; | |
| 427 | - } | |
| 428 | - | |
| 429 | - $this->register_experiment(); | |
| 430 | - | |
| 431 | - if ( ! Plugin::$instance->experiments->is_feature_active( 'landing-pages' ) ) { | |
| 432 | - return; | |
| 433 | - } | |
| 434 | - | |
| 435 | 422 | $this->permalink_structure = get_option( 'permalink_structure' ); |
| 436 | 423 | |
| 437 | 424 | $this->register_landing_page_cpt(); |
| 438 | 425 | |
| @@ -461,20 +448,12 @@ | ||
| 461 | 448 | add_action( 'elementor/documents/register', function( Documents_Manager $documents_manager ) { |
| 462 | 449 | $documents_manager->register_document_type( self::DOCUMENT_TYPE, Landing_Page::get_class_full_name() ); |
| 463 | 450 | } ); |
| 464 | 451 | |
| 465 | - add_action( 'elementor/editor-one/menu/register', function ( Menu_Data_Provider $menu_data_provider ) { | |
| 466 | - $this->register_editor_one_menu( $menu_data_provider ); | |
| 467 | - } ); | |
| 452 | + add_action( 'admin_menu', function() { | |
| 453 | + $this->add_submenu_page(); | |
| 454 | + }, 30 ); | |
| 468 | 455 | |
| 469 | - add_filter( 'elementor/editor-one/menu/elementor_post_types', function ( array $elementor_post_types ): array { | |
| 470 | - $elementor_post_types[ static::CPT ] = [ | |
| 471 | - 'menu_slug' => 'elementor-editor-templates', | |
| 472 | - 'child_slug' => 'edit.php?post_type=' . static::CPT, | |
| 473 | - ]; | |
| 474 | - return $elementor_post_types; | |
| 475 | - } ); | |
| 476 | - | |
| 477 | 456 | // Add the custom 'Add New' link for Landing Pages into Elementor's admin config. |
| 478 | 457 | add_action( 'elementor/admin/localize_settings', function( array $settings ) { |
| 479 | 458 | return $this->admin_localize_settings( $settings ); |
| 480 | 459 | } ); |
| @@ -482,13 +461,8 @@ | ||
| 482 | 461 | add_filter( 'elementor/template_library/sources/local/register_taxonomy_cpts', function( array $cpts ) { |
| 483 | 462 | $cpts[] = self::CPT; |
| 484 | 463 | |
| 485 | 464 | return $cpts; |
| 486 | - } ); | |
| 487 | - | |
| 488 | - // When deleting posts in Landing Page CPT, force Elementor to check again whether this feature should be activated. | |
| 489 | - add_action( 'deleted_post_' . self::CPT, function () { | |
| 490 | - delete_option( self::ACTIVATION_KEY ); | |
| 491 | 465 | } ); |
| 492 | 466 | |
| 493 | 467 | // In the Landing Pages Admin Table page - Overwrite Template type column header title. |
| 494 | 468 | add_action( 'manage_' . Landing_Pages_Module::CPT . '_posts_columns', function( $posts_columns ) { |