feat(EditorForm): merge Items and FieldItems parameter (#7646)

* doc: 更新注释信息

* refactor: 增强过滤功能

* test: 增加单元测试

* refactor: 优化性能

Author: ArgoZhang

src/BootstrapBlazor/Components/EditorForm/EditorForm.razor.cs

@@ -85,8 +85,8 @@ public partial class EditorForm<TModel> : IShowLabel, IDisposable
     public int? LabelWidth { get; set; }
 
     /// <summary>
-    /// <para lang="zh">获得/设置 列模板 设置 <see cref="Items"/> 时本参数不生效</para>
-    /// <para lang="en">Gets or sets Field Items Template. Not effective when <see cref="Items"/> is set</para>
+    /// <para lang="zh">获得/设置 绑定属性集合模板 设置 <see cref="Items"/> 时本参数优先级高</para>
+    /// <para lang="en">Gets or sets the field items collection template.</para>
     /// </summary>
     [Parameter]
     public RenderFragment<TModel>? FieldItems { get; set; }
@@ -158,8 +158,8 @@ public partial class EditorForm<TModel> : IShowLabel, IDisposable
     public List<string>? IgnoreItems { get; set; }
 
     /// <summary>
-    /// <para lang="zh">获得/设置 级联上下文绑定字段信息集合 设置此参数后 <see cref="FieldItems"/> 模板不生效</para>
-    /// <para lang="en">Gets or sets Context Field Items Collection. <see cref="FieldItems"/> template will not be effective if set</para>
+    /// <para lang="zh">获得/设置 绑定字段信息集合 设置此参数后 <see cref="FieldItems"/> 模板优先级更高</para>
+    /// <para lang="en">Gets or sets the items collection.</para>
     /// </summary>
     [Parameter]
     public IEnumerable<IEditorItem>? Items { get; set; }
@@ -294,7 +294,9 @@ private List<IEditorItem> GetRenderItems()
         var items = new List<IEditorItem>();
         if (Items != null)
         {
-            items.AddRange(Items.Where(i => !i.GetIgnore() && !string.IsNullOrEmpty(i.GetFieldName()) && FilterIgnoreItem(i)));
+            var editorFieldNames = new HashSet<string?>(_editorItems.Select(item => item.GetFieldName()));
+            items.AddRange(Items.Where(i => !editorFieldNames.Contains(i.GetFieldName()) && FilterEditorItem(i)));
+            items.AddRange(_editorItems.Where(FilterEditorItem));
             return items;
         }
 
@@ -309,6 +311,8 @@ private List<IEditorItem> GetRenderItems()
         return items;
     }
 
+    private bool FilterEditorItem(IEditorItem i) => !i.GetIgnore() && !string.IsNullOrEmpty(i.GetFieldName()) && FilterIgnoreItem(i);
+
     private bool FilterIgnoreItem(IEditorItem item)
     {
         return IgnoreItems?.Find(i => i.Equals(item.GetFieldName(), StringComparison.OrdinalIgnoreCase)) == null;

test/UnitTest/Components/EditorFormTest.cs

@@ -68,8 +68,29 @@ public void Items_Ok()
             pb.Add(a => a.Items, new List<IEditorItem>
             {
                 new InternalTableColumn("Id", typeof(int)) { Ignore = true },
+                new InternalTableColumn("Name", typeof(string)),
                 new TableTemplateColumn<Foo>()
             });
+
+            pb.Add(a => a.FieldItems, f => builder =>
+            {
+                var index = 0;
+                builder.OpenComponent<EditorItem<Foo, string>>(index++);
+                builder.AddAttribute(index++, nameof(EditorItem<Foo, string>.Field), f.Name);
+                builder.AddAttribute(index++, nameof(EditorItem<Foo, string>.FieldExpression), Utility.GenerateValueExpression(foo, nameof(Foo.Name), typeof(string)));
+
+                builder.AddAttribute(index++, nameof(EditorItem<Foo, string>.Required), true);
+                builder.AddAttribute(index++, nameof(EditorItem<Foo, string>.RequiredErrorMessage), "Test");
+                builder.AddAttribute(index++, nameof(EditorItem<Foo, string>.Readonly), true);
+                builder.AddAttribute(index++, nameof(EditorItem<Foo, string>.SkipValidate), false);
+                builder.AddAttribute(index++, nameof(EditorItem<Foo, string>.Text), "Test-Text");
+                builder.AddAttribute(index++, nameof(EditorItem<Foo, string>.ComponentType), typeof(BootstrapInput<string>));
+                builder.AddAttribute(index++, nameof(EditorItem<Foo, string>.ComponentParameters), new KeyValuePair<string, object>[]
+                {
+                        new("type", "text")
+                });
+                builder.CloseComponent();
+            });
         });
     }