From 9a8732d076266021ecee9f9666afd9175e534fbf Mon Sep 17 00:00:00 2001
From: maliming <malimings@gmail.com>
Date: Sun, 20 Jul 2025 10:31:04 +0800
Subject: [PATCH] Refactor handling of OriginalValue in AbpEntityEntry

---
 .../ChangeTrackers/AbpEntityEntry.cs                   | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/framework/src/Volo.Abp.EntityFrameworkCore/Volo/Abp/EntityFrameworkCore/ChangeTrackers/AbpEntityEntry.cs b/framework/src/Volo.Abp.EntityFrameworkCore/Volo/Abp/EntityFrameworkCore/ChangeTrackers/AbpEntityEntry.cs
index 6083e7d790..5ffe3278c3 100644
--- a/framework/src/Volo.Abp.EntityFrameworkCore/Volo/Abp/EntityFrameworkCore/ChangeTrackers/AbpEntityEntry.cs
+++ b/framework/src/Volo.Abp.EntityFrameworkCore/Volo/Abp/EntityFrameworkCore/ChangeTrackers/AbpEntityEntry.cs
@@ -48,9 +48,7 @@ public class AbpEntityEntry
 
         if (navigationEntry.NavigationEntry is CollectionEntry)
         {
-            navigationEntry.OriginalValue ??= new List<object>();
-            var ls = navigationEntry.OriginalValue.As<List<object>>();
-            ls.AddIfNotContains(entityEntry.Entity);
+            navigationEntry.OriginalValue!.As<List<object>>().Add(entityEntry.Entity);
         }
         else
         {
@@ -75,11 +73,9 @@ public class AbpNavigationEntry
     {
         NavigationEntry = navigationEntry;
         Name = name;
-        if (navigationEntry.CurrentValue != null)
+        if (navigationEntry.CurrentValue != null )
         {
-            OriginalValue = navigationEntry is CollectionEntry collection
-                ? collection.CurrentValue!.Cast<object>().ToList()
-                : navigationEntry.CurrentValue;
+            OriginalValue = navigationEntry is CollectionEntry ? new List<object>() : navigationEntry.CurrentValue;
         }
     }
 }