DrinkRateAPI/ApiModels/UserProfile/UserProfileAdminStatusPut.cs

@@ -0,0 +1,6 @@
+namespace DrinkRateAPI.ApiModels.UserProfile;
+
+public class UserProfileAdminStatusPut
+{
+    public bool ChangeStatusTo { get; set; }
+}

DrinkRateAPI/ApiModels/UserProfile/UserProfileGet.cs

@@ -2,28 +2,5 @@ namespace DrinkRateAPI.ApiModels.UserProfile;
 
 public class UserProfileGet
 {
-    /// <summary>
-    /// User profile ID
-    /// </summary>
-    public string Id { get; set; }
 
-    /// <summary>
-    /// User profile name
-    /// </summary>
-    public string UserName { get; set; }
-
-    /// <summary>
-    /// Is user admin
-    /// </summary>
-    public bool IsAdmin { get; set; }
-
-    /// <summary>
-    /// Is user deleted
-    /// </summary>
-    public bool IsDeleted { get; set; }
-
-    /// <summary>
-    /// Applicaton user ID of the user profile
-    /// </summary>
-    public string ApplicationUserId { get; set; }
 }

DrinkRateAPI/ApiModels/UserProfile/UserProfilePut.cs

@@ -1,9 +1,6 @@
 namespace DrinkRateAPI.ApiModels.UserProfile;
 
-public class UserProfilePut : UserProfileSelfPut
+public class UserProfilePut
 {
-    /// <summary>
+    public string UserName { get; set; }
-    /// Is user admin
-    /// </summary>
-    public bool? IsAdmin { get; set; }
 }

DrinkRateAPI/ApiModels/UserProfile/UserProfileSelfPut.cs

@@ -1,9 +0,0 @@
-namespace DrinkRateAPI.ApiModels.UserProfile;
-
-public class UserProfileSelfPut
-{
-    /// <summary>
-    /// User profile name
-    /// </summary>
-    public string? UserName { get; set; }
-}

DrinkRateAPI/Controllers/UserProfileController.cs

@@ -1,3 +1,4 @@
+using System.Security.Claims;
 using DrinkRateAPI.ApiModels.UserProfile;
 using DrinkRateAPI.AuthorizationPolicies;
 using DrinkRateAPI.Services;
@@ -10,40 +11,30 @@ namespace DrinkRateAPI.Controllers;
 [Route("userProfile")]
 public class UserProfileController : ControllerBase
 {
+    private readonly ILogger<UserProfileController> _logger;
     private readonly UserProfileService _userProfileService;
 
-    public UserProfileController(UserProfileService userProfileService)
+    public UserProfileController(ILogger<UserProfileController> logger, UserProfileService userProfileService)
     {
+        _logger = logger;
         _userProfileService = userProfileService;
     }
 
     [HttpPut]
-    [Produces("application/json")]
+    public UserProfileGet PutUserProfile([FromBody] UserProfilePut userProfile)
-    public async Task<UserProfileGet> PutUserProfileSelf([FromBody] UserProfileSelfPut userProfile)
     {
-        return await _userProfileService.PutUserProfileSelfAsync(User, userProfile);
+        throw new ApplicationException();
+        var x = User.FindFirst(ClaimTypes.NameIdentifier)?.Value; //HttpContext.User.Identities.First();
+        return new();
     }
 
-    [HttpGet]
+    [HttpPut("{userId}/adminStatus")]
-    [Produces("application/json")]
-    public async Task<UserProfileGet> GetUserProfileSelf()
-    {
-        return await _userProfileService.GetUserProfileSelfAsync(User);
-    }
-
-    [HttpPut("{userId}")]
     [Authorize(Policy = PolicyConstants.AdminOnly)]
     [Produces("application/json")]
-    public async Task<UserProfileGet> PutUserProfile(string userId, [FromBody] UserProfilePut userProfile)
+    public async Task<IActionResult> PutUserAdminStatus(string userId, [FromBody] UserProfileAdminStatusPut body)
     {
-        return await _userProfileService.PutUserProfileAsync(User, userProfile, userId);
+        var changedProfile = await _userProfileService.PutUserProfileAdminStatusAsync(userId, body.ChangeStatusTo);
-    }
 
-    [HttpGet("{userId}")]
+        return Ok(changedProfile);
-    [Authorize(Policy = PolicyConstants.AdminOnly)]
-    [Produces("application/json")]
-    public async Task<UserProfileGet> GetUserProfile(string userId)
-    {
-        return await _userProfileService.GetUserProfileAsync(User, userId);
     }
 }

DrinkRateAPI/DbEntities/DbUserProfile.cs

@@ -20,5 +20,6 @@ public class DbUserProfile : DbEntityWithHistory
     public bool IsDeleted { get; set; }
 
     public Guid ApplicationUserId { get; set; }
+
     public virtual DbApplicationUser ApplicationUser { get; set; }
 }

DrinkRateAPI/Services/UserProfileService.cs

@@ -2,8 +2,12 @@ using System.Security.Claims;
 using DrinkRateAPI.ApiModels.UserProfile;
 using DrinkRateAPI.Contexts;
 using DrinkRateAPI.DbEntities;
+using DrinkRateAPI.DbEntities;
 using DrinkRateAPI.Exceptions;
 using Microsoft.EntityFrameworkCore;
+using Microsoft.AspNetCore.Authorization;
+using Microsoft.AspNetCore.Mvc;
+
 
 namespace DrinkRateAPI.Services;
 
@@ -17,102 +21,29 @@ public class UserProfileService(ApplicationDbContext context, ApplicationUserSer
         return userProfile.IsAdmin;
     }
 
-    public async Task<UserProfileGet> PutUserProfileSelfAsync(ClaimsPrincipal identity, UserProfileSelfPut userProfileSelfPut)
+    public async Task<DbUserProfile> PutUserProfileAdminStatusAsync(string userId, bool changeStatusTo)
     {
-        var authenticatedUser = await _applicationUserService.UserProfileByApplicationUserAsync(identity);
+        var userProfile = GetUserProfileById(userId);
-
+        userProfile.IsAdmin = changeStatusTo;
-        var userId = authenticatedUser.Id.ToString();
-        await PutUserProfile(userProfileSelfPut, userId, false);
-
-        return await GetUserProfile(userId);
-    }
-
-    public async Task<UserProfileGet> GetUserProfileSelfAsync(ClaimsPrincipal identity)
-    {
-        var authenticatedUser = await _applicationUserService.UserProfileByApplicationUserAsync(identity);
-
-        var userId = authenticatedUser.Id.ToString();
-
-        return await GetUserProfile(userId);
-    }
-
-    public async Task<UserProfileGet> PutUserProfileAsync(ClaimsPrincipal identity, UserProfilePut userProfilePut, string userId)
-    {
-        var authenticatedUser = await _applicationUserService.UserProfileByApplicationUserAsync(identity);
-
-        if (authenticatedUser.Id.ToString() == userId)
-        {
-            // Prevent admin de-admining him/herself
-            await PutUserProfile(userProfilePut, userId, false);
-        }
-        else
-        {
-            await PutUserProfile(userProfilePut, userId, IsUserProfileAdmin(authenticatedUser));
-        }
-
-        return await GetUserProfile(userId);
-    }
-
-    public async Task<UserProfileGet> GetUserProfileAsync(ClaimsPrincipal identity, string userId)
-    {
-        var authenticatedUser = await _applicationUserService.UserProfileByApplicationUserAsync(identity);
-
-        return await GetUserProfile(userId);
-    }
-
-    private async Task PutUserProfile<TUserProfilePut>(TUserProfilePut userProfilePut, string userId, bool byAdmin) where TUserProfilePut : UserProfileSelfPut
-    {
-        var userProfile = await GetUserProfileById(userId);
-
-        if (!string.IsNullOrEmpty(userProfilePut.UserName) && userProfile.UserName != userProfilePut.UserName)
-        {
-            var userByName = await TryGetUserProfileByUserName(userProfilePut.UserName);
-            if (userByName == null)
-            {
-                userProfile.UserName = userProfilePut.UserName;
-            }
-            else
-            {
-                throw new BadRequestException($"User with username {userProfilePut.UserName} already exists");
-            }
-        }
-
-        if (byAdmin && userProfilePut is UserProfilePut adminPut && adminPut.IsAdmin != null)
-        {
-            userProfile.IsAdmin = (bool)adminPut.IsAdmin;
-        }
-
         _context.UserProfiles.Update(userProfile);
         await _context.SaveChangesAsync();
-    }
-
-    private async Task<UserProfileGet> GetUserProfile(string userId)
-    {
-        var userProfile = await GetUserProfileById(userId);
-
-        var userProfileGet = new UserProfileGet
-        {
-            Id = userProfile.Id.ToString(),
-            UserName = userProfile.UserName,
-            IsAdmin = userProfile.IsAdmin,
-            IsDeleted = userProfile.IsDeleted,
-            ApplicationUserId = userProfile.ApplicationUserId.ToString(),
-        };
-
-        return userProfileGet;
-    }
-
-    private async Task<DbUserProfile> GetUserProfileById(string userId)
-    {
-        var userProfile = await _context.UserProfiles.FirstOrDefaultAsync(x => x.Id.ToString() == userId);
-
-        return userProfile ?? throw new NotFoundException();
-    }
-
-    private async Task<DbUserProfile?> TryGetUserProfileByUserName(string userName)
-    {
-        var userProfile = await _context.UserProfiles.FirstOrDefaultAsync(x => x.UserName == userName);
 
         return userProfile;
     }
+    
+    public async Task<UserProfileGet> PutUserProfileAsync(UserProfilePut userProfile, ClaimsPrincipal identity)
+    {
+        var profile = _applicationUserService.UserProfileByApplicationUserAsync(identity);
+
+
+
+        return new();
+    }
+
+    public DbUserProfile GetUserProfileById(string userId)
+    {
+        var userProfile = _context.UserProfiles.FirstOrDefault(x => x.Id.ToString() == userId);
+
+        return userProfile ?? throw new NotFoundException();
+    }
 }