Fix List<Dictionary<string,object>> SaveAs the filter button at the bottom when only one column & multiple rows #789

Author: shps951023

src/MiniExcel/OpenXml/ExcelOpenXmlSheetWriter.DefaultOpenXml.cs

@@ -347,7 +347,7 @@ private static string GetDimensionRef(int maxRowIndex, int maxColumnIndex)
             if (maxRowIndex == 0 && maxColumnIndex == 0)
                 dimensionRef = "A1";
             else if (maxColumnIndex <= 1)
-                dimensionRef = $"A{maxRowIndex}";
+                dimensionRef = $"A1:A{maxRowIndex}";
             else if (maxRowIndex == 0)
                 dimensionRef = $"A1:{ColumnHelper.GetAlphabetColumnName(maxColumnIndex - 1)}1";
             else

src/MiniExcel/OpenXml/ExcelOpenXmlSheetWriter.cs

@@ -251,9 +251,7 @@ private int WriteValues(MiniExcelStreamWriter writer, object values)
             writer.Write(WorksheetXml.EndSheetData);
 
             if (_configuration.AutoFilter)
-            {
                 writer.Write(WorksheetXml.Autofilter(GetDimensionRef(maxRowIndex, maxColumnIndex)));
-            }
 
             writer.Write(WorksheetXml.Drawing(_currentSheetIndex));
             writer.Write(WorksheetXml.EndWorksheet);

tests/MiniExcelTests/MiniExcelIssueTests.cs

@@ -4374,4 +4374,23 @@ public void TestIssue773()
         Assert.Equal("c3", rows[6].AA);
         Assert.Equal("Ram", rows[6].B);
     }
+
+    /// <summary>
+    /// https://github.com/mini-software/MiniExcel/issues/789
+    /// </summary>
+    [Fact]
+    public void TestIssue789()
+    {
+        var path = PathHelper.GetTempPath();
+        var value = new[] {
+            new Dictionary<string, object> { {"no","1"} },
+            new Dictionary<string, object> { {"no","2"} },
+            new Dictionary<string, object> { {"no","3"} },
+        };
+        MiniExcel.SaveAs(path, value);
+
+        var xml = Helpers.GetZipFileContent(path.ToString(), "xl/worksheets/sheet1.xml");
+
+        Assert.Contains("<x:autoFilter ref=\"A1:A4\" />", xml);
+    }
 }