From ee7275ac4f5ec6fc106989c95b5567165ab3c74f Mon Sep 17 00:00:00 2001 From: AJ Date: Tue, 18 Nov 2025 16:02:27 -0500 Subject: [PATCH] Fix state-mutating getter in LengthItem.Length MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Remove dangerous side effect where the Length property getter was modifying the LengthInputValue field. Property getters should be pure and not cause state mutations, as this violates the principle of least surprise and can cause issues with data binding, serialization, and debugging. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- CutList/Models/LengthItem.cs | 1 - 1 file changed, 1 deletion(-) diff --git a/CutList/Models/LengthItem.cs b/CutList/Models/LengthItem.cs index 90d7ea2..2c46be1 100644 --- a/CutList/Models/LengthItem.cs +++ b/CutList/Models/LengthItem.cs @@ -22,7 +22,6 @@ namespace CutList.Models if (double.TryParse(input, out d)) { - LengthInputValue += "\""; return d; }