Add type checking when importing types from recipes #15979

src/OrchardCore.Modules/OrchardCore.ContentTypes/RecipeSteps/ContentDefinitionStep.cs

@@ -58,6 +58,11 @@ private Task UpdateContentTypeAsync(ContentTypeDefinition type, ContentTypeDefin
 
                 foreach (var part in record.ContentTypePartDefinitionRecords)
                 {
+                    if (string.IsNullOrEmpty(part.PartName))
+                    {
+                        throw new InvalidOperationException($"Failed to add part to ContentType '{type.Name}', The PartName property is not allowed to be empty");
+                    }
+
                     builder.WithPart(part.Name, part.PartName, partBuilder => partBuilder.MergeSettings(part.Settings));
                 }
             });

test/OrchardCore.Tests/OrchardCore.Tests.csproj

@@ -29,6 +29,7 @@
     <None Remove="Recipes\RecipeFiles\recipe3.json" />
     <None Remove="Recipes\RecipeFiles\recipe4.json" />
     <None Remove="Recipes\RecipeFiles\recipe5.json" />
+    <None Remove="Recipes\RecipeFiles\recipe6.json" />
   </ItemGroup>
 
   <ItemGroup>
@@ -44,6 +45,7 @@
     <EmbeddedResource Include="Recipes\RecipeFiles\recipe3.json" />
     <EmbeddedResource Include="Recipes\RecipeFiles\recipe4.json" />
     <EmbeddedResource Include="Recipes\RecipeFiles\recipe5.json" />
+    <EmbeddedResource Include="Recipes\RecipeFiles\recipe6.json" />
   </ItemGroup>
 
   <ItemGroup>

test/OrchardCore.Tests/Recipes/RecipeExecutorTests.cs

@@ -8,6 +8,7 @@
 using OrchardCore.Recipes.Models;
 using OrchardCore.Recipes.Services;
 using OrchardCore.Scripting;
+using OrchardCore.Tests.Apis.Context;
 
 namespace OrchardCore.Recipes
 {
@@ -50,6 +51,27 @@ public async Task ShouldTrimValidScriptExpression(string recipeName, string expe
             });
         }
 
+        [Fact]
+        public async Task IncorrectTypeDefinitionImportsShouldBeBlocked()
+        {
+            var context = new BlogContext();
+            await context.InitializeAsync();
+            await context.UsingTenantScopeAsync(async scope =>
+            {
+                var recipeExecutor = scope.ServiceProvider.GetRequiredService<IRecipeExecutor>();
+                // Act
+                var executionId = Guid.NewGuid().ToString("n");
+                var recipeDescriptor = new RecipeDescriptor { RecipeFileInfo = GetRecipeFileInfo("recipe6") };
+
+                var exception = await Assert.ThrowsAsync<InvalidOperationException>(async () =>
+                {
+                    await recipeExecutor.ExecuteAsync(executionId, recipeDescriptor, new Dictionary<string, object>(), CancellationToken.None);
+                });
+
+                Assert.Equal("Failed to add part to ContentType 'Message', The PartName property is not allowed to be empty", exception.Message);
+            });
+        }
+
         private static Task<ShellScope> GetScopeAsync() => ShellScope.Context.CreateScopeAsync();
 
         private static ShellContext CreateShellContext() => new()

test/OrchardCore.Tests/Recipes/RecipeFiles/recipe6.json

@@ -0,0 +1,27 @@
+{
+  "name": "Recipe6",
+  "displayName": "Recipe 6",
+  "description": "This is a recipe for a bug to verify that the type-checking function works as expected",
+  "author": "Tony Han",
+  "website": "",
+  "version": "1.0.0",
+  "issetuprecipe": false,
+  "categories": [ "test" ],
+  "steps": [
+    {
+      "name": "ContentDefinition",
+      "ContentTypes": [
+        {
+          "Name": "Message",
+          "DisplayName": "Message",
+          "ContentTypePartDefinitionRecords": [ 
+            {
+              //"PartName": "TitlePart", // Test PartName validation.
+              "Name": "TitlePart"
+            }
+          ]
+        }
+      ]
+    }
+  ]
+}