Merge pull request #4295 from Codeinwp/bugfix/4062

Fixed cover layout issue with CPT
Codeinwp · Oct 28, 2024 · 7a1094d · 7a1094d
2 parents cd84333 + 23547d7
commit 7a1094d
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 4 deletions.
diff --git a/e2e-tests/specs/customizer/layout/single-post-settings.spec.ts b/e2e-tests/specs/customizer/layout/single-post-settings.spec.ts
@@ -89,7 +89,6 @@ test.describe('Single Post Check', function () {
 			baseURL,
 		});
 		const HIDDEN = [
-			'.entry-header',
 			'.nv-thumb-wrap',
 			'.entry-content',
 			'.nv-tags-list',

diff --git a/inc/customizer/defaults/single_post.php b/inc/customizer/defaults/single_post.php
@@ -128,7 +128,10 @@ public function get_cpt_context( $allowed = [ 'post', 'page' ] ) {
 		 */
 		$allowed_context = apply_filters( 'neve_allowed_custom_post_types', $allowed, 10, 1 );
 		$context         = get_post_type();
-		$context         = apply_filters( 'neve_context_filter', $context, 10, 1 );
+		if ( ! has_filter( 'neve_context_filter' ) && ! in_array( get_post_type(), $allowed_context, true ) ) {
+			$context = 'post';
+		}
+		$context = apply_filters( 'neve_context_filter', $context, 10, 1 );
 
 		return [ $context, $allowed_context ];
 	}

diff --git a/inc/customizer/options/layout_single_page.php b/inc/customizer/options/layout_single_page.php
@@ -61,6 +61,6 @@ public function add_controls() {
 	 * @return bool
 	 */
 	public static function is_cover_layout() {
-		return get_theme_mod( 'neve_page_header_layout' ) === 'cover';
+		return class_exists( '\Neve_Pro\Admin\Custom_Layouts_Cpt' ) && get_theme_mod( 'neve_page_header_layout' ) === 'cover';
 	}
 }
diff --git a/inc/customizer/options/layout_single_post.php b/inc/customizer/options/layout_single_post.php
@@ -658,7 +658,7 @@ public function sanitize_post_elements_ordering( $value ) {
 	 * @return bool
 	 */
 	public static function is_cover_layout() {
-		return get_theme_mod( 'neve_post_header_layout' ) === 'cover';
+		return class_exists( '\Neve_Pro\Admin\Custom_Layouts_Cpt' ) && get_theme_mod( 'neve_post_header_layout' ) === 'cover';
 	}
 
 	/**

diff --git a/inc/views/cover_header.php b/inc/views/cover_header.php
@@ -33,6 +33,10 @@ public function render_cover_header() {
 			return;
 		}
 
+		if ( ! class_exists( '\Neve_Pro\Admin\Custom_Layouts_Cpt' ) && get_post_type() !== $context ) {
+			return;
+		}
+
 		$header_layout = get_theme_mod( 'neve_' . $context . '_header_layout', 'normal' );
 		if ( $header_layout !== 'cover' ) {
 			return;

diff --git a/inc/views/post_layout.php b/inc/views/post_layout.php
@@ -232,6 +232,11 @@ private function get_content_order() {
 			}
 		}
 
+		list( $context ) = $this->get_cpt_context();
+		if ( ! Layout_Single_Post::is_cover_layout() && get_post_type() !== $context ) {
+			$content_order = array_unique( array_merge( array( 'title-meta' ), $content_order ) );
+		}
+
 		return apply_filters( 'neve_layout_single_post_elements_order', $content_order );
 	}
 }