From 82fc05ac5125add0ae5f02ccdbfef2ddc073ff37 Mon Sep 17 00:00:00 2001
From: Sebastiaan Janssen <sebastiaan@umbraco.com>
Date: Mon, 23 Aug 2021 14:48:49 +0200
Subject: [PATCH] Cannot copy page with Block List block with property that
 'looks like' JSON #10879 (#10925)

(cherry picked from commit cd6cb3737e54118d83fd3d76fe750e2689620a8a)
---
 .../Compose/BlockEditorComponent.cs           | 31 +++++++++++++++++--
 1 file changed, 29 insertions(+), 2 deletions(-)

diff --git a/src/Umbraco.Web/Compose/BlockEditorComponent.cs b/src/Umbraco.Web/Compose/BlockEditorComponent.cs
index a8b4cfb8ca..ac92aa6918 100644
--- a/src/Umbraco.Web/Compose/BlockEditorComponent.cs
+++ b/src/Umbraco.Web/Compose/BlockEditorComponent.cs
@@ -5,6 +5,7 @@ using Newtonsoft.Json;
 using Newtonsoft.Json.Linq;
 using Umbraco.Core;
 using Umbraco.Core.Composing;
+using Umbraco.Core.Logging;
 using Umbraco.Core.Models.Blocks;
 using Umbraco.Core.PropertyEditors;
 
@@ -17,6 +18,17 @@ namespace Umbraco.Web.Compose
     {
         private ComplexPropertyEditorContentEventHandler _handler;
         private readonly BlockListEditorDataConverter _converter = new BlockListEditorDataConverter();
+        private readonly ILogger _logger;
+
+        [Obsolete("Use the ctor injecting dependencies.")]
+        public BlockEditorComponent() : this(Current.Logger)
+        {
+        }
+
+        public BlockEditorComponent(ILogger logger)
+        {
+            _logger = logger;
+        }
 
         public void Initialize()
         {
@@ -116,8 +128,23 @@ namespace Umbraco.Web.Compose
                             // this gets a little ugly because there could be some other complex editor that contains another block editor
                             // and since we would have no idea how to parse that, all we can do is try JSON Path to find another block editor
                             // of our type
-                            var json = JToken.Parse(asString);
-                            if (ProcessJToken(json, createGuid, out var result))
+                            JToken json = null;
+                            try
+                            {
+                                json = JToken.Parse(asString);
+                            }
+                            catch (Exception e)
+                            {
+                                // See issue https://github.com/umbraco/Umbraco-CMS/issues/10879
+                                // We are detecting JSON data by seeing if a string is surrounded by [] or {}
+                                // If people enter text like [PLACEHOLDER] JToken  parsing fails, it's safe to ignore though
+                                // Logging this just in case in the future we find values that are not safe to ignore
+                                _logger.Warn<BlockEditorComponent>(
+                                    "The property {PropertyAlias} on content type {ContentTypeKey} has a value of: {BlockItemValue} - this was recognized as JSON but could not be parsed",
+                                    data.Key, propertyAliasToBlockItemData.Key, asString);
+                            }
+
+                            if (json != null && ProcessJToken(json, createGuid, out var result))
                             {
                                 // need to re-save this back to the RawPropertyValues
                                 data.RawPropertyValues[propertyAliasToBlockItemData.Key] = result;