Test modification of editor value before stored to content

Author: lars-erik

.gitignore

@@ -36,6 +36,8 @@ msbuild.wrn
 # Visual Studio 2015
 .vs/
 
+umbraco/
+
 UmbracoV9.Testing.Tests/umbraco/Data/
 UmbracoV9.Testing.Web/umbraco/Data/
 UmbracoV9.Testing.Web/umbraco/Logs/

Our.Umbraco.Test.Examples/DataEditors/DataEditor_Via_Hosted_Controller.Saves_Property_Value_To_Content.verified.txt

@@ -0,0 +1,3 @@
+{
+  "dataValue": "Data from user"
+}

Our.Umbraco.Test.Examples/DataEditors/DataEditor_Via_Hosted_Controller.cs

@@ -1,7 +1,10 @@
-using Newtonsoft.Json;
+using Azure;
+using Newtonsoft.Json;
 using Newtonsoft.Json.Linq;
 using Umbraco.Cms.Core;
 using Umbraco.Cms.Core.Models;
+using Umbraco.Cms.Core.Models.ContentEditing;
+using Umbraco.Cms.Core.Models.Editors;
 using Umbraco.Cms.Core.PropertyEditors;
 using Umbraco.Cms.Core.Serialization;
 using Umbraco.Cms.Core.Services;
@@ -17,6 +20,8 @@ namespace Our.Umbraco.Test.Examples.DataEditors;
 
 public class DataEditor_Via_Hosted_Controller : UmbracoTestServerTestBase
 {
+    private IContentType contentType;
+
     [Test]
     public async Task Creates_Json_Model_For_New_Document()
     {
@@ -33,6 +38,48 @@ public async Task Creates_Json_Model_For_New_Document()
         await VerifyJson(property.ToString(Formatting.Indented));
     }
 
+    [Test]
+    public async Task Saves_Property_Value_To_Content()
+    {
+        var url = PrepareApiControllerUrl<ContentController>(x => x.PostSave(null!));
+
+        var content = new ContentBuilder()
+            .WithId(0)
+            .WithName("My content")
+            .WithContentType(contentType)
+            .AddPropertyData()
+            .WithKeyValue("myProperty",
+                """
+                {
+                    "dataForBackoffice": "Some data we don't want to store",
+                    "dataValue": "Data from user"
+                }
+                """
+            )
+            .Done()
+            .Build();
+
+        var model = new ContentItemSaveBuilder()
+            .WithContent(content)
+            .WithParentId(-1)
+            .WithAction(ContentSaveAction.PublishNew)
+            .Build();
+
+        var payload = JObject.FromObject(model);
+
+        var response = await Client.PostAsync(url, new MultipartFormDataContent { { new StringContent(payload.ToString()), "contentItem" } });
+
+        var json = await response.Content.ReadAsStringAsync();
+        json = json.TrimStart(AngularJsonMediaTypeFormatter.XsrfPrefix);
+        var responseModel = JObject.Parse(json);
+
+        var id = responseModel.Value<int>("id");
+        var createdContent = GetRequiredService<IContentService>().GetById(id)!;
+        var propValue = createdContent.GetValue("myProperty");
+
+        await Verify(propValue);
+    }
+
     [SetUp]
     public void SetupSchema()
     {
@@ -51,12 +98,12 @@ public void SetupSchema()
 
         GetRequiredService<IDataTypeService>().Save(dataType);
 
-        var contentType = new ContentTypeBuilder()
+        contentType = new ContentTypeBuilder()
             .WithoutIdentity()
             .WithName("My doctype")
             .WithAlias("myDoctype")
             .AddPropertyType()
-                .WithAlias("my.property")
+                .WithAlias("myProperty")
                 .WithName("My property")
                 .WithPropertyEditorAlias("my.editor")
                 .WithDataTypeId(dataType.Id)
@@ -138,5 +185,17 @@ public MyDataValueEditor(ILocalizedTextService localizedTextService, IShortStrin
 
         return jObj;
     }
+
+    public override object? FromEditor(ContentPropertyData editorValue, object? currentValue)
+    {
+        var value = editorValue.Value as string;
+        var jObj = JObject.Parse(value ?? "{}");
+        if (jObj.ContainsKey("dataForBackoffice"))
+        {
+            jObj.Remove("dataForBackoffice");
+        }
+
+        return jObj.ToString();
+    }
 }
 

Our.Umbraco.Test.Examples/Our.Umbraco.Test.Examples.csproj

@@ -6,6 +6,22 @@
         <Nullable>enable</Nullable>
     </PropertyGroup>
 
+    <ItemGroup>
+      <Compile Remove="umbraco\**" />
+      <EmbeddedResource Remove="umbraco\**" />
+      <None Remove="umbraco\**" />
+    </ItemGroup>
+
+    <ItemGroup>
+      <None Remove="appsettings.Tests.json" />
+    </ItemGroup>
+
+    <ItemGroup>
+      <Content Include="appsettings.Tests.json">
+        <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+      </Content>
+    </ItemGroup>
+
     <ItemGroup>
         <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0-release-24177-07" />
         <PackageReference Include="NUnit" Version="3.14.0" />

Our.Umbraco.Test.Examples/appsettings.Tests.json

@@ -0,0 +1,28 @@
+{
+  "$schema": "./appsettings-schema.json",
+  "Logging": {
+    "LogLevel": {
+      "Default": "Warning",
+      "Umbraco.Cms.Tests": "Information"
+    },
+    "Console": {
+      "DisableColors": true
+    }
+  },
+  "Tests": {
+    "Database": {
+      "DatabaseType": "SQLite",
+      "PrepareThreadCount": 4,
+      "SchemaDatabaseCount": 4,
+      "EmptyDatabasesCount": 2,
+      "SQLServerMasterConnectionString": ""
+    }
+  },
+  "Umbraco": {
+    "CMS": {
+      "Content": {
+        "AllowedUploadedFileExtensions": ["jpg", "png", "gif", "svg"]
+      }
+    }
+  }
+}

diary.md

@@ -1,3 +1,7 @@
+Exclude the "umbraco" folder to avoid getting compilation problems from auto-enabled models builder.
+
+-----
+
 Must change appsettings.Tests.json to "Copy to ouput dir" something.
 Also this file has a physical reference when package is installed.
 Change %userprofile%\nuget\packages to $(NuGetPackageRoot)