Refactored api server project for latest mooncore changes

2025-07-15 21:04:46 +02:00
parent 514f862a9d
commit f22f0c0e51
8 changed files with 271 additions and 110 deletions
--- a/MoonlightServers.ApiServer/Http/Controllers/Admin/Stars/StarDockerImagesController.cs
+++ b/MoonlightServers.ApiServer/Http/Controllers/Admin/Stars/StarDockerImagesController.cs
@@ -15,22 +15,22 @@ using MoonlightServers.Shared.Http.Responses.Admin.StarDockerImages;
 namespace MoonlightServers.ApiServer.Http.Controllers.Admin.Stars;

 [ApiController]
-[Route("api/admin/servers/stars")]
+[Route("api/admin/servers/stars/{starId:int}/dockerImages")]
 public class StarDockerImagesController : Controller
 {
    private readonly DatabaseRepository<Star> StarRepository;
-    private readonly DatabaseRepository<StarDockerImage> StarDockerImageRepository;
+    private readonly DatabaseRepository<StarDockerImage> DockerImageRepository;

    public StarDockerImagesController(
        DatabaseRepository<Star> starRepository,
-        DatabaseRepository<StarDockerImage> starDockerImageRepository
+        DatabaseRepository<StarDockerImage> dockerImageRepository
    )
    {
        StarRepository = starRepository;
-        StarDockerImageRepository = starDockerImageRepository;
+        DockerImageRepository = dockerImageRepository;
    }

-    [HttpGet("{starId:int}/dockerImages")]
+    [HttpGet]
    [Authorize(Policy = "permissions:admin.servers.stars.get")]
    public async Task<IPagedData<StarDockerImageDetailResponse>> Get(
        [FromRoute] int starId,
@@ -41,16 +41,16 @@ public class StarDockerImagesController : Controller
        var starExists = StarRepository
            .Get()
            .Any(x => x.Id == starId);
-        
-        if(starExists)
+
+        if (!starExists)
            throw new HttpApiException("No star with this id found", 404);

-        var query = StarDockerImageRepository
+        var query = DockerImageRepository
            .Get()
            .Where(x => x.Star.Id == starId);

        var count = await query.CountAsync();
-        
+
        var items = await query
            .Skip(page * pageSize)
            .Take(pageSize)
@@ -70,46 +70,95 @@ public class StarDockerImagesController : Controller
        };
    }

-    [HttpGet("{starId:int}/dockerImages/{id:int}")]
-    [Authorize(Policy = "permissions:admin.servers.stars.get")]
+    [HttpGet("{id:int}")]
+    [Authorize(Policy = "permissions:admin.servers.stars.read")]
    public async Task<StarDockerImageDetailResponse> GetSingle([FromRoute] int starId, [FromRoute] int id)
    {
-        await ApplyStar(starId);
+        var starExists = StarRepository
+            .Get()
+            .Any(x => x.Id == starId);

-        return await CrudHelper.GetSingle(id);
+        if (!starExists)
+            throw new HttpApiException("No star with this id found", 404);
+
+        var dockerImage = await DockerImageRepository
+            .Get()
+            .FirstOrDefaultAsync(x => x.Id == id && x.Star.Id == starId);
+
+        if (dockerImage == null)
+            throw new HttpApiException("No star docker image with this id found", 404);
+
+        return DockerImageMapper.ToAdminResponse(dockerImage);
    }

-    [HttpPost("{starId:int}/dockerImages")]
-    [Authorize(Policy = "permissions:admin.servers.stars.create")]
-    public async Task<StarDockerImageDetailResponse> Create([FromRoute] int starId,
-        [FromBody] CreateStarDockerImageRequest request)
+    [HttpPost("")]
+    [Authorize(Policy = "permissions:admin.servers.stars.write")]
+    public async Task<StarDockerImageDetailResponse> Create(
+        [FromRoute] int starId,
+        [FromBody] CreateStarDockerImageRequest request
+    )
    {
-        await ApplyStar(starId);
+        var star = await StarRepository
+            .Get()
+            .FirstOrDefaultAsync(x => x.Id == starId);

-        var starDockerImage = Mapper.Map<StarDockerImage>(request);
-        starDockerImage.Star = Star;
+        if (star == null)
+            throw new HttpApiException("No star with this id found", 404);

-        var finalVariable = await StarDockerImageRepository.Add(starDockerImage);
+        var dockerImage = DockerImageMapper.ToDockerImage(request);
+        dockerImage.Star = star;

-        return CrudHelper.MapToResult(finalVariable);
+        var finalDockerImage = await DockerImageRepository.Add(dockerImage);
+
+        return DockerImageMapper.ToAdminResponse(finalDockerImage);
    }

-    [HttpPatch("{starId:int}/dockerImages/{id:int}")]
-    [Authorize(Policy = "permissions:admin.servers.stars.update")]
-    public async Task<StarDockerImageDetailResponse> Update([FromRoute] int starId, [FromRoute] int id,
-        [FromBody] UpdateStarDockerImageRequest request)
+    [HttpPatch("{id:int}")]
+    [Authorize(Policy = "permissions:admin.servers.stars.write")]
+    public async Task<StarDockerImageDetailResponse> Update(
+        [FromRoute] int starId,
+        [FromRoute] int id,
+        [FromBody] UpdateStarDockerImageRequest request
+    )
    {
-        await ApplyStar(starId);
+        var starExists = StarRepository
+            .Get()
+            .Any(x => x.Id == starId);

-        return await CrudHelper.Update(id, request);
+        if (!starExists)
+            throw new HttpApiException("No star with this id found", 404);
+
+        var dockerImage = await DockerImageRepository
+            .Get()
+            .FirstOrDefaultAsync(x => x.Id == id && x.Star.Id == starId);
+
+        if (dockerImage == null)
+            throw new HttpApiException("No star docker image with this id found", 404);
+
+        dockerImage = DockerImageMapper.Merge(request, dockerImage);
+        await DockerImageRepository.Update(dockerImage);
+
+        return DockerImageMapper.ToAdminResponse(dockerImage);
    }

-    [HttpDelete("{starId:int}/dockerImages/{id:int}")]
-    [Authorize(Policy = "permissions:admin.servers.stars.delete")]
+    [HttpDelete("{id:int}")]
+    [Authorize(Policy = "permissions:admin.servers.stars.write")]
    public async Task Delete([FromRoute] int starId, [FromRoute] int id)
    {
-        await ApplyStar(starId);
+        var starExists = StarRepository
+            .Get()
+            .Any(x => x.Id == starId);

-        await CrudHelper.Delete(id);
+        if (!starExists)
+            throw new HttpApiException("No star with this id found", 404);
+
+        var dockerImage = await DockerImageRepository
+            .Get()
+            .FirstOrDefaultAsync(x => x.Id == id && x.Star.Id == starId);
+
+        if (dockerImage == null)
+            throw new HttpApiException("No star docker image with this id found", 404);
+        
+        await DockerImageRepository.Remove(dockerImage);
    }
 }
--- a/MoonlightServers.ApiServer/Http/Controllers/Admin/Stars/StarImportExportController.cs
+++ b/MoonlightServers.ApiServer/Http/Controllers/Admin/Stars/StarImportExportController.cs
@@ -4,6 +4,7 @@ using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Http;
 using MoonCore.Exceptions;
 using MoonCore.Helpers;
+using MoonlightServers.ApiServer.Mappers;
 using MoonlightServers.ApiServer.Services;
 using MoonlightServers.Shared.Http.Responses.Admin.Stars;

@@ -48,6 +49,6 @@ public class StarImportExportController : Controller

        var star = await ImportExportService.Import(content);

-        return Mapper.Map<StarDetailResponse>(star);
+        return StarMapper.ToAdminResponse(star);
    }
 }
--- a/MoonlightServers.ApiServer/Http/Controllers/Admin/Stars/StarVariablesController.cs
+++ b/MoonlightServers.ApiServer/Http/Controllers/Admin/Stars/StarVariablesController.cs
@@ -1,12 +1,12 @@
+using System.ComponentModel.DataAnnotations;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.AspNetCore.Authorization;
 using MoonCore.Exceptions;
 using MoonCore.Extended.Abstractions;
-using MoonCore.Extended.Helpers;
-using MoonCore.Helpers;
 using MoonCore.Models;
 using MoonlightServers.ApiServer.Database.Entities;
+using MoonlightServers.ApiServer.Mappers;
 using MoonlightServers.Shared.Http.Requests.Admin.StarVariables;
 using MoonlightServers.Shared.Http.Responses.Admin.StarVariables;

@@ -16,91 +16,147 @@ namespace MoonlightServers.ApiServer.Http.Controllers.Admin.Stars;
 [Route("api/admin/servers/stars")]
 public class StarVariablesController : Controller
 {
-    private readonly CrudHelper<StarVariable, StarVariableDetailResponse> CrudHelper;
    private readonly DatabaseRepository<Star> StarRepository;
-    private readonly DatabaseRepository<StarVariable> StarVariableRepository;
-
-    private Star Star;
+    private readonly DatabaseRepository<StarVariable> VariableRepository;

    public StarVariablesController(
-        CrudHelper<StarVariable, StarVariableDetailResponse> crudHelper,
        DatabaseRepository<Star> starRepository,
-        DatabaseRepository<StarVariable> starVariableRepository)
+        DatabaseRepository<StarVariable> variableRepository)
    {
-        CrudHelper = crudHelper;
        StarRepository = starRepository;
-        StarVariableRepository = starVariableRepository;
-    }
-
-    private async Task ApplyStar(int id)
-    {
-        var star = await StarRepository
-            .Get()
-            .FirstOrDefaultAsync(x => x.Id == id);
-
-        if (star == null)
-            throw new HttpApiException("A star with this id could not be found", 404);
-
-        Star = star;
-
-        CrudHelper.QueryModifier = variables =>
-            variables.Where(x => x.Star.Id == star.Id);
+        VariableRepository = variableRepository;
    }

    [HttpGet("{starId:int}/variables")]
    [Authorize(Policy = "permissions:admin.servers.stars.get")]
-    public async Task<IPagedData<StarVariableDetailResponse>> Get([FromRoute] int starId, [FromQuery] int page, [FromQuery] int pageSize)
+    public async Task<IPagedData<StarVariableDetailResponse>> Get(
+        [FromRoute] int starId,
+        [FromQuery] [Range(0, int.MaxValue)] int page,
+        [FromQuery] [Range(1, 100)] int pageSize
+    )
    {
-        await ApplyStar(starId);
+        var starExists = StarRepository
+            .Get()
+            .Any(x => x.Id == starId);

-        return await CrudHelper.Get(page, pageSize);
+        if (!starExists)
+            throw new HttpApiException("No star with this id found", 404);
+
+        var query = VariableRepository
+            .Get()
+            .Where(x => x.Star.Id == starId);
+
+        var count = await query.CountAsync();
+
+        var items = await query
+            .Skip(page * pageSize)
+            .Take(pageSize)
+            .ToArrayAsync();
+
+        var mappedItems = items
+            .Select(StarVariableMapper.ToAdminResponse)
+            .ToArray();
+
+        return new PagedData<StarVariableDetailResponse>()
+        {
+            Items = mappedItems,
+            CurrentPage = page,
+            PageSize = pageSize,
+            TotalItems = count,
+            TotalPages = count == 0 ? 0 : count / pageSize
+        };
    }

    [HttpGet("{starId:int}/variables/{id:int}")]
    [Authorize(Policy = "permissions:admin.servers.stars.get")]
-    public async Task<StarVariableDetailResponse> GetSingle([FromRoute] int starId, [FromRoute] int id)
+    public async Task<StarVariableDetailResponse> GetSingle(
+        [FromRoute] int starId,
+        [FromRoute] int id
+    )
    {
-        await ApplyStar(starId);
-        
-        return await CrudHelper.GetSingle(id);
+        var starExists = StarRepository
+            .Get()
+            .Any(x => x.Id == starId);
+
+        if (!starExists)
+            throw new HttpApiException("No star with this id found", 404);
+
+        var starVariable = await VariableRepository
+            .Get()
+            .FirstOrDefaultAsync(x => x.Id == id && x.Star.Id == starId);
+
+        if (starVariable == null)
+            throw new HttpApiException("No variable with this id found", 404);
+
+        return StarVariableMapper.ToAdminResponse(starVariable);
    }

    [HttpPost("{starId:int}/variables")]
    [Authorize(Policy = "permissions:admin.servers.stars.create")]
-    public async Task<StarVariableDetailResponse> Create([FromRoute] int starId, [FromBody] CreateStarVariableRequest request)
+    public async Task<StarVariableDetailResponse> Create([FromRoute] int starId,
+        [FromBody] CreateStarVariableRequest request)
    {
-        await ApplyStar(starId);
-        
-        var starVariable = Mapper.Map<StarVariable>(request);
-        starVariable.Star = Star;
+        var star = StarRepository
+            .Get()
+            .FirstOrDefault(x => x.Id == starId);

-        var finalVariable = await StarVariableRepository.Add(starVariable);
+        if (star == null)
+            throw new HttpApiException("No star with this id found", 404);

-        return CrudHelper.MapToResult(finalVariable);
+        var starVariable = StarVariableMapper.ToStarVariable(request);
+        starVariable.Star = star;
+
+        await VariableRepository.Add(starVariable);
+
+        return StarVariableMapper.ToAdminResponse(starVariable);
    }

    [HttpPatch("{starId:int}/variables/{id:int}")]
    [Authorize(Policy = "permissions:admin.servers.stars.update")]
-    public async Task<StarVariableDetailResponse> Update([FromRoute] int starId, [FromRoute] int id,
-        [FromBody] UpdateStarVariableRequest request)
+    public async Task<StarVariableDetailResponse> Update(
+        [FromRoute] int starId,
+        [FromRoute] int id,
+        [FromBody] UpdateStarVariableRequest request
+    )
    {
-        await ApplyStar(starId);
-        
-        var variable = await CrudHelper.GetSingleModel(id);
+        var starExists = StarRepository
+            .Get()
+            .Any(x => x.Id == starId);

-        variable = Mapper.Map(variable, request);
+        if (!starExists)
+            throw new HttpApiException("No star with this id found", 404);
+
+        var starVariable = await VariableRepository
+            .Get()
+            .FirstOrDefaultAsync(x => x.Id == id && x.Star.Id == starId);
+
+        if (starVariable == null)
+            throw new HttpApiException("No variable with this id found", 404);
        
-        await StarVariableRepository.Update(variable);
+        starVariable = StarVariableMapper.Merge(request, starVariable);
+        await VariableRepository.Update(starVariable);
        
-        return CrudHelper.MapToResult(variable);
+        return StarVariableMapper.ToAdminResponse(starVariable);
    }

    [HttpDelete("{starId:int}/variables/{id:int}")]
    [Authorize(Policy = "permissions:admin.servers.stars.delete")]
    public async Task Delete([FromRoute] int starId, [FromRoute] int id)
    {
-        await ApplyStar(starId);
+        var starExists = StarRepository
+            .Get()
+            .Any(x => x.Id == starId);
+
+        if (!starExists)
+            throw new HttpApiException("No star with this id found", 404);
+
+        var starVariable = await VariableRepository
+            .Get()
+            .FirstOrDefaultAsync(x => x.Id == id && x.Star.Id == starId);
+
+        if (starVariable == null)
+            throw new HttpApiException("No variable with this id found", 404);
        
-        await CrudHelper.Delete(id);
+        await VariableRepository.Remove(starVariable);
    }
 }
--- a/MoonlightServers.ApiServer/Http/Controllers/Admin/Stars/StarsController.cs
+++ b/MoonlightServers.ApiServer/Http/Controllers/Admin/Stars/StarsController.cs
@@ -1,15 +1,14 @@
+using System.ComponentModel.DataAnnotations;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.AspNetCore.Authorization;
+using MoonCore.Exceptions;
 using MoonCore.Extended.Abstractions;
-using MoonCore.Extended.Helpers;
-using MoonCore.Helpers;
 using MoonCore.Models;
 using MoonlightServers.ApiServer.Database.Entities;
+using MoonlightServers.ApiServer.Mappers;
 using MoonlightServers.Shared.Http.Requests.Admin.Stars;
-using MoonlightServers.Shared.Http.Responses.Admin.StarDockerImages;
 using MoonlightServers.Shared.Http.Responses.Admin.Stars;
-using MoonlightServers.Shared.Http.Responses.Admin.StarVariables;

 namespace MoonlightServers.ApiServer.Http.Controllers.Admin.Stars;

@@ -17,35 +16,62 @@ namespace MoonlightServers.ApiServer.Http.Controllers.Admin.Stars;
 [Route("api/admin/servers/stars")]
 public class StarsController : Controller
 {
-    private readonly CrudHelper<Star, StarDetailResponse> CrudHelper;
    private readonly DatabaseRepository<Star> StarRepository;

-    public StarsController(CrudHelper<Star, StarDetailResponse> crudHelper, DatabaseRepository<Star> starRepository)
+    public StarsController(DatabaseRepository<Star> starRepository)
    {
-        CrudHelper = crudHelper;
        StarRepository = starRepository;
    }

    [HttpGet]
-    [Authorize(Policy = "permissions:admin.servers.stars.get")]
-    public async Task<IPagedData<StarDetailResponse>> Get([FromQuery] int page, [FromQuery] int pageSize)
+    [Authorize(Policy = "permissions:admin.servers.stars.read")]
+    public async Task<IPagedData<StarDetailResponse>> Get(
+        [FromQuery] [Range(0, int.MaxValue)] int page,
+        [FromQuery] [Range(1, 100)] int pageSize
+    )
    {
-        return await CrudHelper.Get(page, pageSize);
+        var count = await StarRepository.Get().CountAsync();
+
+        var items = await StarRepository
+            .Get()
+            .Skip(page * pageSize)
+            .Take(pageSize)
+            .ToArrayAsync();
+
+        var mappedItems = items
+            .Select(StarMapper.ToAdminResponse)
+            .ToArray();
+
+        return new PagedData<StarDetailResponse>()
+        {
+            CurrentPage = page,
+            Items = mappedItems,
+            PageSize = pageSize,
+            TotalItems = count,
+            TotalPages = count == 0 ? 0 : count / pageSize
+        };
    }

    [HttpGet("{id:int}")]
-    [Authorize(Policy = "permissions:admin.servers.stars.get")]
+    [Authorize(Policy = "permissions:admin.servers.stars.read")]
    public async Task<StarDetailResponse> GetSingle([FromRoute] int id)
    {
-        return await CrudHelper.GetSingle(id);
+        var star = await StarRepository
+            .Get()
+            .FirstOrDefaultAsync(x => x.Id == id);
+
+        if (star == null)
+            throw new HttpApiException("No star with that id found", 404);
+
+        return StarMapper.ToAdminResponse(star);
    }

    [HttpPost]
    [Authorize(Policy = "permissions:admin.servers.stars.create")]
    public async Task<StarDetailResponse> Create([FromBody] CreateStarRequest request)
    {
-        var star = Mapper.Map<Star>(request);
-        
+        var star = StarMapper.ToStar(request);
+
        // Default values
        star.DonateUrl = null;
        star.UpdateUrl = null;
@@ -63,20 +89,40 @@ public class StarsController : Controller

        var finalStar = await StarRepository.Add(star);

-        return CrudHelper.MapToResult(finalStar);
+        return StarMapper.ToAdminResponse(finalStar);
    }

    [HttpPatch("{id:int}")]
    [Authorize(Policy = "permissions:admin.servers.stars.update")]
-    public async Task<StarDetailResponse> Update([FromRoute] int id, [FromBody] UpdateStarRequest request)
+    public async Task<StarDetailResponse> Update(
+        [FromRoute] int id,
+        [FromBody] UpdateStarRequest request
+    )
    {
-        return await CrudHelper.Update(id, request);
+        var star = await StarRepository
+            .Get()
+            .FirstOrDefaultAsync(x => x.Id == id);
+
+        if (star == null)
+            throw new HttpApiException("No star with that id found", 404);
+        
+        star = StarMapper.Merge(request, star);
+        await StarRepository.Update(star);
+        
+        return StarMapper.ToAdminResponse(star);
    }

    [HttpDelete("{id:int}")]
    [Authorize(Policy = "permissions:admin.servers.stars.delete")]
    public async Task Delete([FromRoute] int id)
    {
-        await CrudHelper.Delete(id);
+        var star = await StarRepository
+            .Get()
+            .FirstOrDefaultAsync(x => x.Id == id);
+
+        if (star == null)
+            throw new HttpApiException("No star with that id found", 404);
+        
+        await StarRepository.Remove(star);
    }
 }