Upgrade to ClosedXML 0.95 (#73)

Great work, thanks Alexey.
I will make my small edits and it will be possible to make a release.

* Upgrade ClosedXML to 0.95.0-beta1. Fix compilation issues
* Modify code according to changes in ClosedXML 0.95
* Remove obsolete code
* Upgrade a reference file: consolidate conditional formats and other minor changes
* Use pattern matching and other small refactoring
* Upgrade xUnit version
* Bump version to 0.2.0

Author: Pankraty

ClosedXML.Report/ClosedXML.Report.csproj

@@ -11,7 +11,7 @@
     <RepositoryUrl>https://github.com/ClosedXML/ClosedXML.Report</RepositoryUrl>
     <Authors>Alexey Rozhkov</Authors>
     <Copyright>MIT</Copyright>
-    <Version>0.1.0.0</Version>
+    <Version>0.2.0.0</Version>
     <Product>ClosedXML.Report</Product>
     <PackageReleaseNotes>See https://github.com/ClosedXML/ClosedXML.Report/releases/tag/$(productVersion)</PackageReleaseNotes>
     <Description>ClosedXML.Report is a tool for report generation and data analysis in .NET applications through the use of Microsoft Excel. ClosedXML.Report is a .NET-library for report generation Microsoft Excel without requiring Excel to be installed on the machine that's running the code.</Description>
@@ -44,9 +44,10 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="ClosedXML" Version="0.92.1" />
+    <PackageReference Include="ClosedXML.Signed" Version="0.95.0-beta1" Condition="'$(Configuration)'=='Release.Signed'"/>
+    <PackageReference Include="ClosedXML" Version="0.95.0-beta1" Condition="'$(Configuration)'!='Release.Signed'"/>
+    <PackageReference Include="morelinq" Version="2.9.0" />
     <PackageReference Include="System.Linq.Dynamic.Core" Version="1.0.9" />
-    <PackageReference Include="System.ValueTuple" Version="4.3.0" />
   </ItemGroup>
 
 </Project>

ClosedXML.Report/Excel/AddressChangedEventArgs.cs

@@ -3,7 +3,7 @@
 
 namespace ClosedXML.Report.Excel
 {
-    public class SubtotalGroup : IDisposable
+    public class SubtotalGroup
     {
         public int Level { get; private set; }
         public int Column { get; set; }
@@ -22,11 +22,5 @@ public SubtotalGroup(int level, int column, string groupTitle, IXLRange range, I
             GroupTitle = groupTitle;
             Range = range;
         }
-
-        public void Dispose()
-        {
-            Range?.Dispose();
-            SummaryRow?.Dispose();
-        }
     }
 }

ClosedXML.Report/Excel/Subtotal.cs

@@ -1,8 +1,7 @@
+using ClosedXML.Excel;
 using System;
 using System.Collections.Generic;
 using System.Linq;
-using ClosedXML.Excel;
-using ClosedXML.Report.Utils;
 
 namespace ClosedXML.Report.Excel
 {
@@ -15,8 +14,6 @@ internal class TempSheetBuffer: IReportBuffer
         private int _clmn;
         private int _prevrow;
         private int _prevclmn;
-        private int _maxClmn;
-        private int _maxRow;
 
         public TempSheetBuffer(XLWorkbook wb)
         {
@@ -36,22 +33,27 @@ private void Init()
                     _sheet = _wb.AddWorksheet(SheetName);
                     _sheet.SetCalcEngineCacheExpressions(false);
                 }
-                //_sheet.Visibility = XLWorksheetVisibility.VeryHidden;
+                _sheet.Visibility = XLWorksheetVisibility.VeryHidden;
             }
-            _row = 1;
-            _clmn = 1;
-            _maxRow = _prevrow = 1;
-            _maxClmn = _prevclmn = 1;
+            _row = _prevrow = 1;
+            _clmn = _prevclmn = 1;
             Clear();
+            _sheet.Style = _wb.Worksheets.First().Style;
         }
 
         public void WriteValue(object value, IXLStyle cellStyle)
         {
             var xlCell = _sheet.Cell(_row, _clmn);
-            xlCell.SetValue(value);
+            try
+            {
+                xlCell.SetValue(value);
+            }
+            catch (ArgumentException)
+            {
+                xlCell.SetValue(value?.ToString());
+            }
+
             xlCell.Style = cellStyle ?? _wb.Style;
-            _maxClmn = Math.Max(_maxClmn, _clmn);
-            _maxRow = Math.Max(_maxRow, _row);
             ChangeAddress(_row, _clmn + 1);
         }
 
@@ -60,8 +62,6 @@ public void WriteFormulaR1C1(string formula, IXLStyle cellStyle)
             var xlCell = _sheet.Cell(_row, _clmn);
             xlCell.Style = cellStyle;
             xlCell.SetFormulaR1C1(formula);
-            _maxClmn = Math.Max(_maxClmn, _clmn);
-            _maxRow = Math.Max(_maxRow, _row);
             ChangeAddress(_row, _clmn + 1);
         }
 
@@ -92,15 +92,13 @@ private void ChangeAddress(int row, int clmn)
 
         public IXLRange CopyTo(IXLRange range)
         {
-            // LastCellUsed may produce the wrong result, see https://github.com/ClosedXML/ClosedXML/issues/339
-            var lastCell = _sheet.Cell(
-                _sheet.LastRowUsed(true)?.RowNumber() ?? 1,
-                _sheet.LastColumnUsed(true)?.ColumnNumber()?? 1);
-            var tempRng = _sheet.Range(_sheet.Cell(1, 1), lastCell);
+            var firstCell = _sheet.Cell(1, 1);
+            var lastCell = _sheet.LastCellUsed(XLCellsUsedOptions.All) ?? firstCell;
+            var tempRng = _sheet.Range(firstCell, lastCell);
 
             var rowDiff = tempRng.RowCount() - range.RowCount();
             if (rowDiff > 0)
-                range.LastRow().Unsubscribed().RowAbove().Unsubscribed().InsertRowsBelow(rowDiff, true);
+                range.LastRow().RowAbove().InsertRowsBelow(rowDiff, true);
             else if (rowDiff < 0)
                 range.Worksheet.Range(
                     range.LastRow().RowNumber() + rowDiff + 1,
@@ -126,32 +124,31 @@ public IXLRange CopyTo(IXLRange range)
 
             var tgtSheet = range.Worksheet;
             var tgtStartRow = range.RangeAddress.FirstAddress.RowNumber;
-            using (var srcRows = _sheet.Rows(tempRng.RangeAddress.FirstAddress.RowNumber, tempRng.RangeAddress.LastAddress.RowNumber))
-                foreach (var row in srcRows)
-                {
-                    var xlRow = tgtSheet.Row(row.RowNumber() + tgtStartRow-1);
-                    xlRow.OutlineLevel = row.OutlineLevel;
-                    if (row.IsHidden)
-                        xlRow.Collapse();
-                    else
-                        xlRow.Expand();
-                }
+            var srcRows = _sheet.Rows(tempRng.RangeAddress.FirstAddress.RowNumber, tempRng.RangeAddress.LastAddress.RowNumber);
+            foreach (var row in srcRows)
+            {
+                var xlRow = tgtSheet.Row(row.RowNumber() + tgtStartRow-1);
+                xlRow.OutlineLevel = row.OutlineLevel;
+                if (row.IsHidden)
+                    xlRow.Collapse();
+                else
+                    xlRow.Expand();
+            }
             return range;
         }
 
         public void Clear()
         {
-            using (var srcRows = _sheet.RowsUsed(true))
-                foreach (var row in srcRows)
-                {
-                    row.OutlineLevel = 0;
-                }
+            var srcRows = _sheet.RowsUsed(XLCellsUsedOptions.All);
+            foreach (var row in srcRows)
+            {
+                row.OutlineLevel = 0;
+            }
             _sheet.Clear();
         }
 
         public void AddConditionalFormats(IEnumerable<IXLConditionalFormat> formats, IXLRangeBase fromRange, IXLRangeBase toRange)
         {
-            //var tempRng = _sheet.Range(_sheet.Cell(1, 1), _sheet.Cell(_prevrow, _prevclmn));
             foreach (var format in formats)
             {
                 format.CopyRelative(fromRange, toRange, true);