From 9f8bef7e789e6321d61da2b9e4bb0da420c477ba Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Halil=20=C4=B0brahim=20Kalkan?= <hi_kalkan@yahoo.com>
Date: Sat, 14 Apr 2018 11:52:05 +0300
Subject: [PATCH] Refactored AuditPropertySetter

---
 .../Volo/Abp/Auditing/AuditPropertySetter.cs  | 33 +++++++++++++++++--
 1 file changed, 30 insertions(+), 3 deletions(-)

diff --git a/src/Volo.Abp.Auditing/Volo/Abp/Auditing/AuditPropertySetter.cs b/src/Volo.Abp.Auditing/Volo/Abp/Auditing/AuditPropertySetter.cs
index 3f0c393a19..ef79af2b05 100644
--- a/src/Volo.Abp.Auditing/Volo/Abp/Auditing/AuditPropertySetter.cs
+++ b/src/Volo.Abp.Auditing/Volo/Abp/Auditing/AuditPropertySetter.cs
@@ -22,6 +22,24 @@ namespace Volo.Abp.Auditing
         }
 
         public void SetCreationProperties(object targetObject)
+        {
+            SetCreationTime(targetObject);
+            SetCreatorId(targetObject);
+        }
+
+        public void SetModificationProperties(object targetObject)
+        {
+            SetLastModificationTime(targetObject);
+            SetLastModifierId(targetObject);
+        }
+
+        public void SetDeletionProperties(object targetObject)
+        {
+            SetDeletionTime(targetObject);
+            SetDeleterId(targetObject);
+        }
+
+        private void SetCreationTime(object targetObject)
         {
             if (!(targetObject is IHasCreationTime objectWithCreationTime))
             {
@@ -32,7 +50,10 @@ namespace Volo.Abp.Auditing
             {
                 objectWithCreationTime.CreationTime = Clock.Now;
             }
+        }
 
+        private void SetCreatorId(object targetObject)
+        {
             if (!(targetObject is ICreationAudited creationAuditedObject))
             {
                 return;
@@ -66,14 +87,17 @@ namespace Volo.Abp.Auditing
 
             creationAuditedObject.CreatorId = CurrentUser.Id;
         }
-
-        public void SetModificationProperties(object targetObject)
+        
+        private void SetLastModificationTime(object targetObject)
         {
             if (targetObject is IHasModificationTime objectWithModificationTime)
             {
                 objectWithModificationTime.LastModificationTime = Clock.Now;
             }
+        }
 
+        private void SetLastModifierId(object targetObject)
+        {
             if (!(targetObject is IModificationAudited modificationAuditedObject))
             {
                 return;
@@ -106,7 +130,7 @@ namespace Volo.Abp.Auditing
             modificationAuditedObject.LastModifierId = CurrentUser.Id;
         }
 
-        public void SetDeletionProperties(object targetObject)
+        private void SetDeletionTime(object targetObject)
         {
             if (targetObject is IHasDeletionTime objectWithDeletionTime)
             {
@@ -115,7 +139,10 @@ namespace Volo.Abp.Auditing
                     objectWithDeletionTime.DeletionTime = Clock.Now;
                 }
             }
+        }
 
+        private void SetDeleterId(object targetObject)
+        {
             if (!(targetObject is IDeletionAudited deletionAuditedObject))
             {
                 return;