From 6d021b23e0c3b2f6a4c12764a058d7d1acdc01d1 Mon Sep 17 00:00:00 2001 From: Mikhail Kuznetsov Date: Tue, 9 Sep 2025 10:44:33 +1000 Subject: [PATCH] =?UTF-8?q?feat(api):=20JobGroupController=20=D0=B4=D0=BE?= =?UTF-8?q?=D0=B1=D0=B0=D0=B2=D0=BB=D0=B5=D0=BD=20=D0=BC=D0=B5=D1=82=D0=BE?= =?UTF-8?q?=D0=B4=20Delete?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- PARR.API/Controllers/V1/JobGroupController.cs | 51 +++++++++++-------- 1 file changed, 29 insertions(+), 22 deletions(-) diff --git a/PARR.API/Controllers/V1/JobGroupController.cs b/PARR.API/Controllers/V1/JobGroupController.cs index 73a63e33..8e96b205 100644 --- a/PARR.API/Controllers/V1/JobGroupController.cs +++ b/PARR.API/Controllers/V1/JobGroupController.cs @@ -29,6 +29,7 @@ namespace PARR.API.Controllers.V1 private readonly IMapper mapper; private readonly IUriService uriService; private readonly IJobGroupService groupService; + private readonly IJobService jobService; private readonly IValidator validator; public JobGroupController( @@ -36,6 +37,7 @@ namespace PARR.API.Controllers.V1 IMapper mapper, IUriService uriService, IJobGroupService groupService, + IJobService jobService, IValidator validator ) { @@ -43,6 +45,7 @@ namespace PARR.API.Controllers.V1 this.mapper = mapper; this.uriService = uriService; this.groupService = groupService; + this.jobService = jobService; this.validator = validator; } @@ -256,32 +259,36 @@ namespace PARR.API.Controllers.V1 /// /// /// - //[HttpDelete(ApiRoutes.JobGroup.Delete)] - //public async Task Delete([FromRoute] Guid id) - //{ - // var jobGroup = await groupService.Get() - // .Include(t => t.Jobs) - // .ThenInclude(t => t.Tnk) - // .FirstOrDefaultAsync(t => t.Id == id); + [HttpDelete(ApiRoutes.JobGroup.Delete)] + public async Task Delete([FromRoute] Guid id) + { + var jobGroup = await groupService.Get() + .FirstOrDefaultAsync(t => t.Id == id); - // if (jobGroup == null) - // return BadRequest(new Response(false, new List { new ErrorModel { - // Message = $"Ошибка при удалении группы заданий на выполнение работ. Не найдена группа заданий на выполнение работ Id: {id}" - // } })); + if (jobGroup == null) + return BadRequest(new Response(false, new List { new ErrorModel { + Message = $"Ошибка при удалении группы заданий на выполнение работ. Не найдена группа заданий на выполнение работ Id: {id}" + } })); - // if (!groupService.Delete(jobGroup) || !await groupService.CommitAsync()) - // return BadRequest(new Response(false, new List { new ErrorModel { - // Message = $"Ошибка при удалении группы заданий на выполнение работ" - // } })); + var jobCount = await jobService.Get().CountAsync(t => t.GroupId == id); + if (jobCount > 0) + return BadRequest(new Response(false, new List { new ErrorModel { + Message = $"Ошибка при удалении группы заданий на выполнение работ. С данным группой связаны задания: {jobCount} шт." + } })); - // logger.LogInformation($"Пользователь {User.Identity?.Name} удалил задание на выполнение работ: {jobGroup.Id},{jobGroup.GroupName}," + - // $" {jobGroup.IsUmbrella}, {jobGroup.ShortDescription}, {jobGroup.FullDescription}," + - // $" {jobGroup.Solution}, {jobGroup.TemplateDuration}, {jobGroup.ReferenceDate}" + - // $"{jobGroup.IsAutoDistributionEnabled}, {jobGroup.IsAgent}, {jobGroup.AgentName}" + - // $"{jobGroup.AgentTimeOutSec}, {jobGroup.AgentScript}"); + if (!groupService.Delete(jobGroup) || !await groupService.CommitAsync()) + return BadRequest(new Response(false, new List { new ErrorModel { + Message = $"Ошибка при удалении группы заданий на выполнение работ" + } })); - // return NoContent(); - //} + logger.LogInformation($"Пользователь {User.Identity?.Name} удалил группу заданий на выполнение работ: {jobGroup.Id},{jobGroup.GroupName}," + + $" {jobGroup.IsUmbrella}, {jobGroup.ShortDescription}, {jobGroup.FullDescription}," + + $" {jobGroup.Solution}, {jobGroup.TemplateDuration}, {jobGroup.ReferenceDate}" + + $"{jobGroup.IsAutoDistributionEnabled}, {jobGroup.IsAgent}, {jobGroup.AgentName}" + + $"{jobGroup.AgentTimeOutSec}, {jobGroup.AgentScript}"); + + return NoContent(); + } ///