Don't set modified audit if the property is generated by the database.

Resolve #9166

framework/src/Volo.Abp.EntityFrameworkCore/Volo/Abp/EntityFrameworkCore/AbpDbContext.cs

@@ -372,17 +372,20 @@ namespace Volo.Abp.EntityFrameworkCore
 
         protected virtual void ApplyAbpConceptsForModifiedEntity(EntityEntry entry, EntityChangeReport changeReport)
         {
-            UpdateConcurrencyStamp(entry);
+            if (entry.Properties.Any(x => x.IsModified && x.Metadata.ValueGenerated == ValueGenerated.Never))
-            SetModificationAuditProperties(entry);
-
-            if (entry.Entity is ISoftDelete && entry.Entity.As<ISoftDelete>().IsDeleted)
-            {
-                SetDeletionAuditProperties(entry);
-                changeReport.ChangedEntities.Add(new EntityChangeEntry(entry.Entity, EntityChangeType.Deleted));
-            }
-            else
             {
-                changeReport.ChangedEntities.Add(new EntityChangeEntry(entry.Entity, EntityChangeType.Updated));
+                UpdateConcurrencyStamp(entry);
+                SetModificationAuditProperties(entry);
+
+                if (entry.Entity is ISoftDelete && entry.Entity.As<ISoftDelete>().IsDeleted)
+                {
+                    SetDeletionAuditProperties(entry);
+                    changeReport.ChangedEntities.Add(new EntityChangeEntry(entry.Entity, EntityChangeType.Deleted));
+                }
+                else
+                {
+                    changeReport.ChangedEntities.Add(new EntityChangeEntry(entry.Entity, EntityChangeType.Updated));
+                }
             }
         }
 

framework/test/Volo.Abp.EntityFrameworkCore.Tests/Volo/Abp/EntityFrameworkCore/Auditing/Auditing_Tests.cs

@@ -1,9 +1,53 @@
-using Volo.Abp.TestApp.Testing;
+using System;
+using System.Threading.Tasks;
+using Shouldly;
+using Volo.Abp.TestApp;
+using Volo.Abp.TestApp.Testing;
+using Xunit;
 
 namespace Volo.Abp.EntityFrameworkCore.Auditing
 {
     public class Auditing_Tests : Auditing_Tests<AbpEntityFrameworkCoreTestModule>
     {
+        [Fact]
+        public async Task Should_Not_Set_Modification_If_Properties_Generated_By_Database()
+        {
+            await WithUnitOfWorkAsync((async () =>
+            {
+                var douglas = await PersonRepository.GetAsync(TestDataBuilder.UserDouglasId);
+                douglas.LastActiveTime = DateTime.Now;
+            }));
 
+            await WithUnitOfWorkAsync((async () =>
+            {
+                var douglas = await PersonRepository.FindAsync(TestDataBuilder.UserDouglasId);
+
+                douglas.ShouldNotBeNull();
+                douglas.LastModificationTime.ShouldBeNull();
+                douglas.LastModificationTime.ShouldBeNull();
+                douglas.LastModifierId.ShouldBeNull();
+            }));
+        }
+
+        [Fact]
+        public async Task Should_Set_Modification_If_Properties_Not_Generated_By_Database()
+        {
+            await WithUnitOfWorkAsync((async () =>
+            {
+                var douglas = await PersonRepository.GetAsync(TestDataBuilder.UserDouglasId);
+                douglas.LastActiveTime = DateTime.Now;
+                douglas.Age = 100;
+            }));
+
+            await WithUnitOfWorkAsync((async () =>
+            {
+                var douglas = await PersonRepository.FindAsync(TestDataBuilder.UserDouglasId);
+
+                douglas.ShouldNotBeNull();
+                douglas.LastModificationTime.ShouldNotBeNull();
+                douglas.LastModificationTime.Value.ShouldBeLessThanOrEqualTo(Clock.Now);
+                douglas.LastModifierId.ShouldBe(CurrentUserId);
+            }));
+        }
     }
 }

framework/test/Volo.Abp.EntityFrameworkCore.Tests/Volo/Abp/TestApp/EntityFrameworkCore/TestAppDbContext.cs

@@ -36,6 +36,11 @@ namespace Volo.Abp.TestApp.EntityFrameworkCore
                 b.HasKey(p => new {p.PersonId, p.Number});
             });
 
+            modelBuilder.Entity<Person>(b =>
+            {
+                b.Property(x => x.LastActiveTime).ValueGeneratedOnAddOrUpdate();
+            });
+
             modelBuilder
                 .Entity<PersonView>(p =>
                 {

framework/test/Volo.Abp.TestApp/Volo/Abp/TestApp/Domain/Person.cs

@@ -23,9 +23,10 @@ namespace Volo.Abp.TestApp.Domain
 
         public virtual Collection<Phone> Phones { get; set; }
 
+        public virtual DateTime? LastActiveTime { get; set; }
+
         private Person()
         {
-            
         }
 
         public Person(Guid id, string name, int age, Guid? tenantId = null, Guid? cityId = null)
@@ -65,4 +66,4 @@ namespace Volo.Abp.TestApp.Domain
             );
         }
     }
-}
+}