From 31be743207c09cc6e6cfff5f65d12126fb41cca3 Mon Sep 17 00:00:00 2001
From: crupest <crupest@outlook.com>
Date: Wed, 28 Apr 2021 16:35:47 +0800
Subject: refactor: ...

---
 BackEnd/Timeline/Services/User/Avatar/UserAvatarService.cs | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

(limited to 'BackEnd/Timeline/Services/User/Avatar/UserAvatarService.cs')

diff --git a/BackEnd/Timeline/Services/User/Avatar/UserAvatarService.cs b/BackEnd/Timeline/Services/User/Avatar/UserAvatarService.cs
index 4f943e3a..92979a68 100644
--- a/BackEnd/Timeline/Services/User/Avatar/UserAvatarService.cs
+++ b/BackEnd/Timeline/Services/User/Avatar/UserAvatarService.cs
@@ -39,7 +39,7 @@ namespace Timeline.Services.User.Avatar
             _clock = clock;
         }
 
-        public async Task<ICacheableDataDigest> GetAvatarDigest(long userId)
+        public async Task<ICacheableDataDigest> GetAvatarDigestAsync(long userId)
         {
             var usernameChangeTime = await _basicUserService.GetUsernameLastModifiedTimeAsync(userId);
 
@@ -61,7 +61,7 @@ namespace Timeline.Services.User.Avatar
             }
         }
 
-        public async Task<ByteData> GetAvatar(long userId)
+        public async Task<ByteData> GetAvatarAsync(long userId)
         {
             await _basicUserService.ThrowIfUserNotExist(userId);
 
@@ -72,7 +72,7 @@ namespace Timeline.Services.User.Avatar
                 return await _defaultUserAvatarProvider.GetDefaultAvatar();
             }
 
-            var data = await _dataManager.GetEntryAndCheck(entity.DataTag, $"This is required by avatar of {userId}.");
+            var data = await _dataManager.GetEntryAndCheck(entity.DataTag, string.Format(Resource.AvatarDataEntryNotExist, userId));
 
             if (entity.Type is null)
             {
@@ -84,7 +84,7 @@ namespace Timeline.Services.User.Avatar
             return new ByteData(data, entity.Type);
         }
 
-        public async Task<ICacheableDataDigest> SetAvatar(long userId, ByteData avatar)
+        public async Task<ICacheableDataDigest> SetAvatarAsync(long userId, ByteData avatar)
         {
             if (avatar is null)
                 throw new ArgumentNullException(nameof(avatar));
@@ -126,10 +126,12 @@ namespace Timeline.Services.User.Avatar
 
             await transaction.CommitAsync();
 
+            _logger.LogInformation(Resource.LogSetAvatar, userId);
+
             return new CacheableDataDigest(tag, now);
         }
 
-        public async Task DeleteAvatar(long userId)
+        public async Task DeleteAvatarAsync(long userId)
         {
             await _basicUserService.ThrowIfUserNotExist(userId);
 
@@ -149,6 +151,8 @@ namespace Timeline.Services.User.Avatar
             await _database.SaveChangesAsync();
 
             await transaction.CommitAsync();
+
+            _logger.LogInformation(Resource.LogRemoveAvatar, userId);
         }
     }
 }
-- 
cgit v1.2.3