From b1cfb0d619ba3896f961516c6745423d1ccb889c Mon Sep 17 00:00:00 2001
From: Mike Alhayek <mike@crestapps.com>
Date: Thu, 14 Dec 2023 11:52:44 -0800
Subject: [PATCH] Set `_items` in the ShapeViewModel to prevent exception
 (#14899)

---
 .../Views/ShapeViewModel.cs                   | 31 +++++++++----------
 1 file changed, 15 insertions(+), 16 deletions(-)

diff --git a/src/OrchardCore/OrchardCore.DisplayManagement/Views/ShapeViewModel.cs b/src/OrchardCore/OrchardCore.DisplayManagement/Views/ShapeViewModel.cs
index 6df0449c36c..109699a581d 100644
--- a/src/OrchardCore/OrchardCore.DisplayManagement/Views/ShapeViewModel.cs
+++ b/src/OrchardCore/OrchardCore.DisplayManagement/Views/ShapeViewModel.cs
@@ -16,24 +16,18 @@ public ShapeViewModel()
 
         public ShapeViewModel(string shapeType)
         {
-            if (string.IsNullOrEmpty(shapeType))
-            {
-                throw new ArgumentException($"The {nameof(shapeType)} cannot be empty");
-            }
+            ArgumentException.ThrowIfNullOrEmpty(shapeType, nameof(shapeType));
 
             Metadata.Type = shapeType;
         }
 
         private ShapeMetadata _metadata;
+
         public ShapeMetadata Metadata => _metadata ??= new ShapeMetadata();
 
         public string Position
         {
-            get
-            {
-                return Metadata.Position;
-            }
-
+            get => Metadata.Position;
             set
             {
                 Metadata.Position = value;
@@ -41,25 +35,30 @@ public string Position
         }
 
         public string Id { get; set; }
+
         public string TagName { get; set; }
 
         private List<string> _classes;
-        public IList<string> Classes => _classes ??= new List<string>();
+
+        public IList<string> Classes => _classes ??= [];
 
         private Dictionary<string, string> _attributes;
-        public IDictionary<string, string> Attributes => _attributes ??= new Dictionary<string, string>();
+
+        public IDictionary<string, string> Attributes => _attributes ??= [];
 
         private Dictionary<string, object> _properties;
-        public IDictionary<string, object> Properties => _properties ??= new Dictionary<string, object>();
+
+        public IDictionary<string, object> Properties => _properties ??= [];
 
         private bool _sorted = false;
 
         private List<IPositioned> _items;
+
         public IReadOnlyList<IPositioned> Items
         {
             get
             {
-                _items ??= new List<IPositioned>();
+                _items ??= [];
 
                 if (!_sorted)
                 {
@@ -78,8 +77,9 @@ public ValueTask<IShape> AddAsync(object item, string position)
                 return new ValueTask<IShape>(this);
             }
 
-            position ??= "";
+            position ??= string.Empty;
             _sorted = false;
+            _items ??= [];
 
             if (item is IHtmlContent)
             {
@@ -91,8 +91,7 @@ public ValueTask<IShape> AddAsync(object item, string position)
             }
             else
             {
-                var shape = item as IPositioned;
-                if (shape != null)
+                if (item is IPositioned shape)
                 {
                     if (position != null)
                     {