Merge pull request #7736 from Automattic/fix/no-sensei-blocks-for-posts

Initialize Sensei blocks for new posts
Automattic · Jan 15, 2025 · fbcb785 · fbcb785
2 parents 0aea63c + e3d50c1
commit fbcb785
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 3 deletions.
diff --git a/changelog/fix-no-sensei-blocks-for-posts b/changelog/fix-no-sensei-blocks-for-posts
@@ -0,0 +1,4 @@
+Significance: patch
+Type: fixed
+
+Initialize Sensei blocks for posts
diff --git a/includes/blocks/class-sensei-blocks-initializer.php b/includes/blocks/class-sensei-blocks-initializer.php
@@ -132,8 +132,12 @@ private function get_admin_page_post_type(): ?string {
 			return $post_type ? $post_type : null;
 		}
 
-		if ( 'post-new.php' === $pagenow && isset( $_GET['post_type'] ) && post_type_exists( $_GET['post_type'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification, WordPress.Security.ValidatedSanitizedInput -- Only reading the post type.
-			return $_GET['post_type']; // phpcs:ignore WordPress.Security.NonceVerification, WordPress.Security.ValidatedSanitizedInput -- Already validated.
+		if ( 'post-new.php' === $pagenow ) {
+			if ( isset( $_GET['post_type'] ) && post_type_exists( $_GET['post_type'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification, WordPress.Security.ValidatedSanitizedInput -- Only reading the post type.
+				return $_GET['post_type']; // phpcs:ignore WordPress.Security.NonceVerification, WordPress.Security.ValidatedSanitizedInput -- Already validated.
+			} else {
+				return 'post';
+			}
 		}
 
 		return null;

diff --git a/tests/unit-tests/blocks/test-class-sensei-blocks-initializer.php b/tests/unit-tests/blocks/test-class-sensei-blocks-initializer.php
@@ -105,7 +105,24 @@ public function testMaybeInitializeBlocks_WhenInAdminOnGutenbergEditSiteScreen_I
 		$initializer_mock->maybe_initialize_blocks();
 	}
 
-	public function testMaybeInitializeBlocks_WhenInAdminOnNewPostScreenAndHasCorrectPostType_InitializesBlocks() {
+	public function testMaybeInitializeBlocks_WhenInAdminOnNewPostScreen_InitializesBlocks() {
+		/* Arrange */
+		$initializer_mock = $this->getMockForAbstractClass( Sensei_Blocks_Initializer::class );
+
+		set_current_screen( 'post' );
+
+		global $pagenow;
+		$pagenow = 'post-new.php';
+
+		/* Assert */
+		$initializer_mock->expects( $this->once() )
+			->method( 'initialize_blocks' );
+
+		/* Act */
+		$initializer_mock->maybe_initialize_blocks();
+	}
+
+	public function testMaybeInitializeBlocks_WhenInAdminOnNewCourseScreenAndHasCorrectPostType_InitializesBlocks() {
 		/* Arrange */
 		$initializer_mock = $this->getMockForAbstractClass( Sensei_Blocks_Initializer::class, [ [ 'course' ] ] );