Merge pull request #109 from anmcgrath/cell-value-set-take-into-accou…

…nt-type

Don't convert editor input string if the cell type is text.

src/BlazorDatasheet.Core/Edit/Editor.cs

@@ -85,6 +85,7 @@ public Editor(Sheet sheet)
     /// <param name="col"></param>
     /// <param name="isSoftEdit">If true, the editor should hold on to the edit when arrow keys are pressed.</param>
     /// <param name="mode">Details on the method that was used to begin the edit.</param>
+    /// <param name="mode">Details on the method that was used to begin the edit.</param>
     /// <param name="key">The key that was used to begin the edit, if the entry mode was using the keyboard</param>
     public void BeginEdit(int row, int col, bool isSoftEdit = true, EditEntryMode mode = EditEntryMode.None,
         string? key = null)
@@ -155,7 +156,7 @@ public bool AcceptEdit()
         }
 
         var formulaResult = isFormula ? Sheet.FormulaEngine.Evaluate(parsedFormula) : CellValue.Empty;
-        var editValue = isFormula ? formulaResult : new CellValue(this.EditValue);
+        var editValue = isFormula ? formulaResult : GetValueAsCellValue(EditCell.Row, EditCell.Col, EditValue);
 
         var beforeAcceptEdit = new BeforeAcceptEditEventArgs(EditCell, editValue, parsedFormula, formulaString);
         BeforeEditAccepted?.Invoke(this, beforeAcceptEdit);
@@ -196,6 +197,18 @@ public bool AcceptEdit()
         return false;
     }
 
+    private CellValue GetValueAsCellValue(int row, int col, string editValue)
+    {
+        var type = Sheet.Cells.GetCellType(row, col);
+        switch (type)
+        {
+            case "text":
+                return CellValue.Text(editValue);
+        }
+
+        return new CellValue(editValue);
+    }
+
     private void ClearEdit()
     {
         IsEditing = false;

src/BlazorDatasheet/DatasheetRow.razor

@@ -73,6 +73,7 @@
             @switch (visualCell.CellType)
             {
                 case "default":
+                case "text":
                 case "datetime":
                     <div style="width: 100%;">
                         @visualCell.FormattedString
@@ -145,5 +146,4 @@
     }
 
 
-}
-
+}

test/BlazorDatasheet.Test/Edit/EditTests.cs

@@ -1,4 +1,5 @@
 using BlazorDatasheet.Core.Data;
+using FluentAssertions;
 using NUnit.Framework;
 
 namespace BlazorDatasheet.Test.Edit;
@@ -70,4 +71,15 @@ public void Accept_Edit_Is_Undoable()
         sheet.Commands.Undo();
         Assert.AreEqual(null, sheet.Cells.GetValue(0, 0));
     }
+
+    [Test]
+    public void Do_Not_Do_Conversion_If_Cell_Type_Is_Set_To_Text()
+    {
+        var sheet = new Sheet(10, 10);
+        sheet.Cells[0, 0].Type = "text";
+        sheet.Editor.BeginEdit(0, 0);
+        sheet.Editor.EditValue = "04-10-1";
+        sheet.Editor.AcceptEdit();
+        sheet.Cells[0,0].Value.Should().Be("04-10-1");
+    }
 }