Merge pull request #21 from dynamicweb/dbe/18708-Fix-Excel-Read-For-E…

…mpty-Columns Read cells with empty values correctly
dynamicweb · Apr 2, 2024 · c3357f7 · c3357f7
2 parents b3e9f80 + cca799a
commit c3357f7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 8 deletions.
diff --git a/src/Dynamicweb.DataIntegration.Providers.ExcelProvider.csproj b/src/Dynamicweb.DataIntegration.Providers.ExcelProvider.csproj
@@ -1,6 +1,6 @@
 <Project Sdk="Microsoft.NET.Sdk">
 	<PropertyGroup>
-		<Version>10.0.11</Version>
+		<Version>10.0.12</Version>
 		<AssemblyVersion>1.0.0</AssemblyVersion>
 		<Title>Excel Provider</Title>
 		<Description>Excel Provider</Description>

diff --git a/src/ExcelReader.cs b/src/ExcelReader.cs
@@ -59,15 +59,14 @@ private void LoadExcelFile()
                 {
                     var hasValue = false;
                     var wsRow = worksheet.Cells[rowNum, 1, rowNum, worksheet.Dimension.End.Column];
-                    var row = dataTable.Rows.Add();
-                    var c = 0;
-                    foreach (var cell in wsRow)
+                    var row = dataTable.Rows.Add();                    
+                    for(var colNum = 1; colNum <= worksheet.Dimension.End.Column; colNum++)
                     {
-                        row[c] = cell.Text;
+                        string cellText = wsRow[rowNum, colNum].Text;
+                        row[colNum - 1] = cellText;
 
-                        if (!string.IsNullOrWhiteSpace(cell.Text))
-                            hasValue = true;
-                        c++;
+                        if (!string.IsNullOrWhiteSpace(cellText))
+                            hasValue = true;                        
                     }
 
                     if (!hasValue)