LoadFromCollection will now set the Hyperlink property if a member is Uri or ExcelHyperLink.

Author: JanKallman

src/EPPlus/ExcelRangeBase.cs

@@ -191,7 +191,7 @@ private void SetValueAddress(ExcelAddress address, _setValue valueMethod, object
                 if (value is object[,] && (valueMethod == Set_Value || valueMethod == Set_StyleID))
                 {
                     // only simple set value is supported for bulk copy
-                    _worksheet.SetRangeValueInner(address.Start.Row, address.Start.Column, address.End.Row, address.End.Column, (object[,])value);
+                    _worksheet.SetRangeValueInner(address.Start.Row, address.Start.Column, address.End.Row, address.End.Column, (object[,])value, false);
                 }
                 else
                 {

src/EPPlus/ExcelWorksheet.cs

@@ -3944,16 +3944,65 @@ internal void SetValueStyleIdInner(int row, int col, object value, int styleId)
         /// <param name="toRow">end row</param>
         /// <param name="toColumn">end column</param>
         /// <param name="values">set values</param>
-        internal void SetRangeValueInner(int fromRow, int fromColumn, int toRow, int toColumn, object[,] values)
+        /// <param name="setHyperLinkFromValue">If the value is of type Uri or ExcelHyperlink the Hyperlink property is set.</param>
+        internal void SetRangeValueInner(int fromRow, int fromColumn, int toRow, int toColumn, object[,] values, bool setHyperLinkFromValue)
         {
-            
-            _values.SetValueRange_Value(fromRow, fromColumn, values);
+            if (setHyperLinkFromValue)
+            {
+                SetValuesWithHyperLink(fromRow, fromColumn, values);
+            }
+            else
+            {
+                _values.SetValueRange_Value(fromRow, fromColumn, values);
+            }
             //Clearout formulas and flags, for example the rich text flag.
             _formulas.Clear(fromRow, fromColumn, values.GetUpperBound(0) + 1, values.GetUpperBound(1) + 1); 
             _flags.Clear(fromRow, fromColumn, values.GetUpperBound(0) + 1, values.GetUpperBound(1)+1);
             _metadataStore.Clear(fromRow, fromColumn, values.GetUpperBound(0) + 1, values.GetUpperBound(1) + 1);
         }
 
+        private void SetValuesWithHyperLink(int fromRow, int fromColumn, object[,] values)
+        {
+            var rowBound = values.GetUpperBound(0);
+            var colBound = values.GetUpperBound(1);
+
+            for (int r = 0; r <= rowBound; r++)
+            {
+                for (int c = 0; c <= colBound; c++)
+                {
+                    var v = values[r, c];
+                    var row = fromRow + r;
+                    var col = fromColumn + c;
+                    if (v==null)
+                    {
+                        _values.SetValue_Value(row, col, v);
+                        continue;
+                    }
+                    var t = v.GetType();
+                    if (t == typeof(Uri) || t == typeof(ExcelHyperLink))
+                    {
+                        _hyperLinks.SetValue(row, col, (Uri)v);
+                        if (v is ExcelHyperLink hl)
+                        {
+                            SetValueInner(row, col, hl.Display);
+                        }
+                        else
+                        {
+                            var cv = GetValueInner(row, col);
+                            if (cv == null || cv.ToString() == "")
+                            {
+                                SetValueInner(row, col, ((Uri)v).OriginalString);
+                            }
+                        }
+                    }
+                    else
+                    {
+                        _values.SetValue_Value(row, col, v);
+                    }
+                }
+            }
+        }
+
         /// <summary>
         /// Existance check of sheet value
         /// </summary>

src/EPPlus/LoadFunctions/LoadFunctionBase.cs

@@ -87,7 +87,7 @@ internal ExcelRangeBase Load()
             }
             else
             {
-                ws.SetRangeValueInner(Range._fromRow, Range._fromCol, Range._fromRow + nRows - 1, Range._fromCol + nCols - 1, values);
+                ws.SetRangeValueInner(Range._fromRow, Range._fromCol, Range._fromRow + nRows - 1, Range._fromCol + nCols - 1, values, true);
             }
 
 
@@ -161,7 +161,7 @@ private void SetValuesAndFormulas(int nRows, int nCols, object[,] values, Dictio
                     var fromRow = Range._fromRow;
                     var rangeCol = Range._fromCol + col;
                     var toRow = Range._fromRow + nRows - 1;
-                    ws.SetRangeValueInner(fromRow, rangeCol, toRow, rangeCol, columnValues);
+                    ws.SetRangeValueInner(fromRow, rangeCol, toRow, rangeCol, columnValues, true);
                 }
 
             }

src/EPPlusTest/LoadFunctions/LoadFromCollectionTests.cs

@@ -43,7 +43,7 @@ Date               Author                       Change
 namespace EPPlusTest.LoadFunctions
 {
     [TestClass]
-    public class LoadFromCollectionTests
+    public class LoadFromCollectionTests : TestBase
     {
         internal abstract class BaseClass
         {
@@ -69,6 +69,7 @@ internal class BClass
             public string Id { get; set; }
             [System.ComponentModel.Description("MyName")]
             public string Name { get; set; }
+            [EpplusTableColumn(Order = 3)]
             public int Number { get; set; }
         }
 
@@ -79,6 +80,28 @@ internal class CamelCasedClass
             public string CamelCased_And_Underscored { get; set; }
         }
 
+        internal class UrlClass : BClass
+        {
+            [EpplusIgnore]
+            public string EMailAddress { get; set; }
+            [EpplusTableColumn(Order = 5, Header = "My Mail To")]
+            public ExcelHyperLink MailTo
+            {
+                get
+                {
+                    var url = new ExcelHyperLink("mailto:" + EMailAddress);
+                    url.Display = Name;
+                    return url;
+                }
+            }
+            [EpplusTableColumn(Order = 4)]
+            public Uri Url
+            {
+                get;
+                set;
+            }
+        }
+
         [TestMethod]
         public void ShouldNotIncludeHeadersWhenPrintHeadersIsOmitted()
         {
@@ -366,5 +389,35 @@ public void ShouldLoadExpandoObjects()
                 Assert.AreEqual("TestName 2", sheet.Cells["B3"].Value);
             }
         }
+        [TestMethod]
+        public void ShouldSetHyperlinkForURIs()
+        {
+            var items = new List<UrlClass>()
+            {
+                new UrlClass{Id="1", Name="Person 1", EMailAddress="[email protected]"},
+                new UrlClass{Id="2", Name="Person 2", EMailAddress="[email protected]"},
+                new UrlClass{Id="2", Name="Person with Url", EMailAddress="[email protected]", Url=new Uri("https://epplussoftware.com")},
+            };
+
+            using (var package = OpenPackage("LoadFromCollectionUrls.xlsx", true))
+            {
+                var sheet = package.Workbook.Worksheets.Add("test");
+                var r = sheet.Cells["A1"].LoadFromCollection(items, true, TableStyles.Medium1);
+
+                Assert.AreEqual("MyId", sheet.Cells["A1"].Value);
+                Assert.AreEqual("MyName", sheet.Cells["B1"].Value);
+                Assert.AreEqual("Number", sheet.Cells["C1"].Value);
+                Assert.AreEqual("Url", sheet.Cells["D1"].Value);
+                Assert.AreEqual("My Mail To", sheet.Cells["E1"].Value);
+
+                Assert.AreEqual("1", sheet.Cells["A2"].Value);
+                Assert.AreEqual("Person 2", sheet.Cells["B3"].Value);
+                Assert.IsInstanceOfType(sheet.Cells["E3"].Hyperlink, typeof(ExcelHyperLink));
+                Assert.AreEqual("Person 2", sheet.Cells["E3"].Value);
+
+                SaveAndCleanup(package);
+            }
+        }
+
     }
 }