Add PATCH endpoint for partial profile updates

XIYUEKONGLING · XIYUEKONGLING · commit b6395dbb705b · 2025-12-23T11:01:46.000+08:00
diff --git a/OpenBioCardServer/Controllers/Classic/ClassicUserController.cs b/OpenBioCardServer/Controllers/Classic/ClassicUserController.cs
@@ -51,38 +51,60 @@ public async Task<IActionResult> GetProfile(string username)
     }
 
     /// <summary>
-    /// Update user profile (requires authentication)
+    /// Patch user profile (Partial Update)
+    /// Only updates fields present in the JSON body.
     /// </summary>
     [HttpPost("{username}")]
-    public async Task<IActionResult> UpdateProfile(string username, [FromBody] ClassicProfile request)
+    public async Task<IActionResult> PatchProfile(string username, [FromBody] ClassicProfilePatch request)
     {
         var token = GetTokenFromHeader();
         
         if (string.IsNullOrEmpty(token))
-        {
             return Unauthorized(new ClassicErrorResponse("Missing authentication token"));
-        }
 
         var (isValid, account) = await _authService.ValidateTokenAsync(token);
 
-        if (!isValid || account == null)
-        {
+        if (!isValid || account == null || account.UserName != username)
             return Unauthorized(new ClassicErrorResponse("Invalid token"));
-        }
 
-        if (account.UserName != username)
+        try
         {
-            return Unauthorized(new ClassicErrorResponse("Invalid token"));
+            var success = await _profileService.PatchProfileAsync(username, request);
+            
+            if (!success)
+                return NotFound(new ClassicErrorResponse("Profile not found"));
+
+            return Ok(new ClassicSuccessResponse(true));
         }
+        catch (Exception)
+        {
+            return StatusCode(500, new ClassicErrorResponse("Profile update failed"));
+        }
+    }
+
+    /// <summary>
+    /// Full Update user profile (Legacy/Full Replace)
+    /// Replaces the entire profile with the provided data.
+    /// </summary>
+    [HttpPost("{username}/update")]
+    public async Task<IActionResult> FullUpdateProfile(string username, [FromBody] ClassicProfile request)
+    {
+        var token = GetTokenFromHeader();
+        
+        if (string.IsNullOrEmpty(token))
+            return Unauthorized(new ClassicErrorResponse("Missing authentication token"));
+
+        var (isValid, account) = await _authService.ValidateTokenAsync(token);
+
+        if (!isValid || account == null || account.UserName != username)
+            return Unauthorized(new ClassicErrorResponse("Invalid token"));
 
         try
         {
             var success = await _profileService.UpdateProfileAsync(username, request);
             
             if (!success)
-            {
                 return NotFound(new ClassicErrorResponse("Profile not found"));
-            }
 
             return Ok(new ClassicSuccessResponse(true));
         }
@@ -102,30 +124,19 @@ public async Task<IActionResult> ExportData(string username)
         var token = GetTokenFromHeader();
         
         if (string.IsNullOrEmpty(token))
-        {
             return Unauthorized(new ClassicErrorResponse("Missing authentication token"));
-        }
 
         var (isValid, account) = await _authService.ValidateTokenAsync(token);
 
-        if (!isValid || account == null)
-        {
-            return Unauthorized(new ClassicErrorResponse("Invalid token"));
-        }
-
-        if (account.UserName != username)
-        {
+        if (!isValid || account == null || account.UserName != username)
             return Unauthorized(new ClassicErrorResponse("Invalid token"));
-        }
 
         try
         {
             var exportData = await _profileService.GetExportDataAsync(username, token);
             
             if (exportData == null)
-            {
                 return NotFound(new ClassicErrorResponse("User data not found"));
-            }
 
             return Ok(exportData);
         }
@@ -145,30 +156,19 @@ public async Task<IActionResult> ImportData(string username, [FromBody] ClassicU
         var token = GetTokenFromHeader();
         
         if (string.IsNullOrEmpty(token))
-        {
             return Unauthorized(new ClassicErrorResponse("Missing authentication token"));
-        }
 
         var (isValid, account) = await _authService.ValidateTokenAsync(token);
 
-        if (!isValid || account == null)
-        {
-            return Unauthorized(new ClassicErrorResponse("Invalid token"));
-        }
-
-        if (account.UserName != username)
-        {
+        if (!isValid || account == null || account.UserName != username)
             return Unauthorized(new ClassicErrorResponse("Invalid token"));
-        }
 
         try
         {
             var success = await _profileService.ImportDataAsync(username, request);
             
             if (!success)
-            {
                 return BadRequest(new ClassicErrorResponse("Import failed or username mismatch"));
-            }
 
             return Ok(new ClassicSuccessResponse(true));
         }
diff --git a/OpenBioCardServer/Models/DTOs/Classic/ClassicProfilePatch.cs b/OpenBioCardServer/Models/DTOs/Classic/ClassicProfilePatch.cs
@@ -0,0 +1,68 @@
+using Newtonsoft.Json;
+
+namespace OpenBioCardServer.Models.DTOs.Classic;
+
+/// <summary>
+/// 用于增量更新的 Profile DTO
+/// 所有字段均为可空，Null 表示不修改该字段
+/// </summary>
+public class ClassicProfilePatch
+{
+    [JsonProperty("username")]
+    public string? Username { get; set; }
+    
+    [JsonProperty("name")]
+    public string? Name { get; set; }
+    
+    [JsonProperty("pronouns")]
+    public string? Pronouns { get; set; }
+    
+    [JsonProperty("avatar")]
+    public string? Avatar { get; set; }
+    
+    [JsonProperty("bio")]
+    public string? Bio { get; set; }
+    
+    [JsonProperty("location")]
+    public string? Location { get; set; }
+    
+    [JsonProperty("website")]
+    public string? Website { get; set; }
+    
+    [JsonProperty("background")]
+    public string? Background { get; set; }
+    
+    [JsonProperty("currentCompany")]
+    public string? CurrentCompany { get; set; }
+    
+    [JsonProperty("currentCompanyLink")]
+    public string? CurrentCompanyLink { get; set; }
+    
+    [JsonProperty("currentSchool")]
+    public string? CurrentSchool { get; set; }
+    
+    [JsonProperty("currentSchoolLink")]
+    public string? CurrentSchoolLink { get; set; }
+    
+    // 集合类型：
+    // Null = 不更新
+    // Empty List [] = 清空列表
+    
+    [JsonProperty("contacts")]
+    public List<ClassicContact>? Contacts { get; set; }
+    
+    [JsonProperty("socialLinks")]
+    public List<ClassicSocialLink>? SocialLinks { get; set; }
+    
+    [JsonProperty("projects")]
+    public List<ClassicProject>? Projects { get; set; }
+    
+    [JsonProperty("workExperiences")]
+    public List<ClassicWorkExperience>? WorkExperiences { get; set; }
+    
+    [JsonProperty("schoolExperiences")]
+    public List<ClassicSchoolExperience>? SchoolExperiences { get; set; }
+    
+    [JsonProperty("gallery")]
+    public List<ClassicGalleryItem>? Gallery { get; set; }
+}
diff --git a/OpenBioCardServer/Services/ClassicProfileService.cs b/OpenBioCardServer/Services/ClassicProfileService.cs
@@ -175,4 +175,107 @@ public async Task<bool> UpdateProfileAsync(string username, ClassicProfile reque
             throw; // Re-throw to let controller handle the 500 response
         }
     }
+    
+    /// <summary>
+    /// 增量更新用户 Profile (Smart Update)
+    /// </summary>
+    public async Task<bool> PatchProfileAsync(string username, ClassicProfilePatch patch)
+    {
+        using var transaction = await _context.Database.BeginTransactionAsync();
+
+        try
+        {
+            var profile = await _context.Profiles
+                .AsTracking()
+                .FirstOrDefaultAsync(p => p.Username == username);
+
+            if (profile == null)
+            {
+                _logger.LogWarning("Attempted to patch non-existent profile: {Username}", username);
+                return false;
+            }
+
+            // 1. Update basic profile fields (Only non-nulls)
+            ClassicMapper.UpdateProfileFromPatch(profile, patch);
+
+            // 2. Handle Collections
+            // 逻辑：如果 Patch 中的集合为 null，则跳过（保持原样）。
+            // 如果 Patch 中的集合不为 null (即使是空列表)，则替换原有集合。
+
+            if (patch.Contacts != null)
+            {
+                await _context.ContactItems.Where(c => c.ProfileId == profile.Id).ExecuteDeleteAsync();
+                if (patch.Contacts.Any())
+                {
+                    var items = ClassicMapper.ToContactEntities(patch.Contacts, profile.Id);
+                    await _context.ContactItems.AddRangeAsync(items);
+                }
+            }
+
+            if (patch.SocialLinks != null)
+            {
+                await _context.SocialLinkItems.Where(s => s.ProfileId == profile.Id).ExecuteDeleteAsync();
+                if (patch.SocialLinks.Any())
+                {
+                    var items = ClassicMapper.ToSocialLinkEntities(patch.SocialLinks, profile.Id);
+                    await _context.SocialLinkItems.AddRangeAsync(items);
+                }
+            }
+
+            if (patch.Projects != null)
+            {
+                await _context.ProjectItems.Where(p => p.ProfileId == profile.Id).ExecuteDeleteAsync();
+                if (patch.Projects.Any())
+                {
+                    var items = ClassicMapper.ToProjectEntities(patch.Projects, profile.Id);
+                    await _context.ProjectItems.AddRangeAsync(items);
+                }
+            }
+
+            if (patch.WorkExperiences != null)
+            {
+                await _context.WorkExperienceItems.Where(w => w.ProfileId == profile.Id).ExecuteDeleteAsync();
+                if (patch.WorkExperiences.Any())
+                {
+                    var items = ClassicMapper.ToWorkExperienceEntities(patch.WorkExperiences, profile.Id);
+                    await _context.WorkExperienceItems.AddRangeAsync(items);
+                }
+            }
+
+            if (patch.SchoolExperiences != null)
+            {
+                await _context.SchoolExperienceItems.Where(s => s.ProfileId == profile.Id).ExecuteDeleteAsync();
+                if (patch.SchoolExperiences.Any())
+                {
+                    var items = ClassicMapper.ToSchoolExperienceEntities(patch.SchoolExperiences, profile.Id);
+                    await _context.SchoolExperienceItems.AddRangeAsync(items);
+                }
+            }
+
+            if (patch.Gallery != null)
+            {
+                await _context.GalleryItems.Where(g => g.ProfileId == profile.Id).ExecuteDeleteAsync();
+                if (patch.Gallery.Any())
+                {
+                    var items = ClassicMapper.ToGalleryEntities(patch.Gallery, profile.Id);
+                    await _context.GalleryItems.AddRangeAsync(items);
+                }
+            }
+
+            await _context.SaveChangesAsync();
+            await transaction.CommitAsync();
+
+            // 3. Invalidate Cache
+            await _cache.RemoveAsync(CacheKeys.GetClassicProfileCacheKey(username));
+            
+            _logger.LogInformation("Profile patched successfully for user: {Username}", username);
+            return true;
+        }
+        catch (Exception ex)
+        {
+            await transaction.RollbackAsync();
+            _logger.LogError(ex, "Error patching profile for user: {Username}", username);
+            throw;
+        }
+    }
 }
diff --git a/OpenBioCardServer/Utilities/Mappers/ClassicMapper.cs b/OpenBioCardServer/Utilities/Mappers/ClassicMapper.cs
@@ -359,4 +359,50 @@ public static List<GalleryItemEntity> ToGalleryEntities(List<ClassicGalleryItem>
 
         return null;
     }
+    // === Patch Update Logic ===
+
+    public static void UpdateProfileFromPatch(ProfileEntity profile, ClassicProfilePatch patch)
+    {
+        // 仅当字段不为 Null 时更新
+        // 注意：如果前端传空字符串 ""，这里会更新为空字符串，符合预期（清空字段）
+        
+        if (patch.Username != null) profile.Username = patch.Username;
+        if (patch.Name != null) profile.NickName = patch.Name;
+        if (patch.Pronouns != null) profile.Pronouns = patch.Pronouns;
+
+        if (patch.Avatar != null)
+        {
+            var (avatarType, avatarText, avatarData) = ParseAsset(patch.Avatar);
+            profile.AvatarType = avatarType;
+            profile.AvatarText = avatarText;
+            profile.AvatarData = avatarData;
+        }
+
+        if (patch.Bio != null) profile.Description = patch.Bio;
+        if (patch.Location != null) profile.Location = patch.Location;
+        if (patch.Website != null) profile.Website = patch.Website;
+
+        if (patch.Background != null)
+        {
+            // 如果传空字符串，视为删除背景
+            if (string.IsNullOrEmpty(patch.Background))
+            {
+                profile.BackgroundType = null;
+                profile.BackgroundText = null;
+                profile.BackgroundData = null;
+            }
+            else
+            {
+                var (bgType, bgText, bgData) = ParseAsset(patch.Background);
+                profile.BackgroundType = bgType;
+                profile.BackgroundText = bgText;
+                profile.BackgroundData = bgData;
+            }
+        }
+
+        if (patch.CurrentCompany != null) profile.CurrentCompany = patch.CurrentCompany;
+        if (patch.CurrentCompanyLink != null) profile.CurrentCompanyLink = patch.CurrentCompanyLink;
+        if (patch.CurrentSchool != null) profile.CurrentSchool = patch.CurrentSchool;
+        if (patch.CurrentSchoolLink != null) profile.CurrentSchoolLink = patch.CurrentSchoolLink;
+    }
 }
