From 1afa30040d5b8bcf5b67a55e1e025ca852ec7172 Mon Sep 17 00:00:00 2001 From: Blitblank Date: Wed, 22 Apr 2026 21:19:35 -0500 Subject: [PATCH] create userdto constructor out of user --- api/src/Controllers/AuthController.cs | 2 +- api/src/Controllers/UsersController.cs | 14 ++------------ api/src/Models/User.cs | 10 +++++++++- 3 files changed, 12 insertions(+), 14 deletions(-) diff --git a/api/src/Controllers/AuthController.cs b/api/src/Controllers/AuthController.cs index 83f44e7..3364fc3 100644 --- a/api/src/Controllers/AuthController.cs +++ b/api/src/Controllers/AuthController.cs @@ -27,7 +27,7 @@ public class AuthController : ControllerBase { [HttpPost("register")] public async Task Register(RegisterDto dto) { - var user = new User { + User user = new User { UserName = dto.UserName, Email = dto.Email, CreatedAt = DateTime.UtcNow // yeah why not utc diff --git a/api/src/Controllers/UsersController.cs b/api/src/Controllers/UsersController.cs index 195741e..f00325f 100644 --- a/api/src/Controllers/UsersController.cs +++ b/api/src/Controllers/UsersController.cs @@ -32,12 +32,7 @@ public class UsersController : ControllerBase { foreach(User user in rawArray) { // TODO: can you operator overload a cast? if so cast(UserDto) would go hard // if not then just a new custom cast function that returns a dto type will do - UserDto newDto = new UserDto{ - CreatedAt = user.CreatedAt, - Email = user.Email, - Id = user.Id, - UserName = user.UserName - }; + UserDto newDto = new UserDto(user); dtoArray.Add(newDto); } @@ -52,12 +47,7 @@ public class UsersController : ControllerBase { if (user == null) return NotFound(); - UserDto newDto = new UserDto{ - CreatedAt = user.CreatedAt, - Email = user.Email, - Id = user.Id, - UserName = user.UserName - }; + UserDto newDto = new UserDto(user); return Ok(newDto); } diff --git a/api/src/Models/User.cs b/api/src/Models/User.cs index 28b4953..8857f71 100644 --- a/api/src/Models/User.cs +++ b/api/src/Models/User.cs @@ -55,9 +55,17 @@ public class LoginDto { public class UserDto { public DateTime CreatedAt { get; set; } = DateTime.UtcNow; // gets compressed to a string' - public List permissions { get; set; } = []; + public List? Permissions { get; set; } = []; public string? Email { get; set; } = ""; public string Id { get; set; } = ""; public string? UserName { get; set; } = ""; + public UserDto(User user) { + CreatedAt = user.CreatedAt; + Email = user.Email; + Id = user.Id; + UserName = user.UserName; + Permissions = user.Permissions; + } + };