From 9c13b3f27f8bf764970a63597d729a839e964a96 Mon Sep 17 00:00:00 2001 From: "n.nakhostin" Date: Tue, 2 Sep 2025 11:56:07 +0330 Subject: [PATCH] Refactor ToggleTenderApproval Logic to Simplify Status Check - Updated the ToggleTenderApproval method to remove the redundant check for SubmissionMode when determining if an existing tender approval should be deleted. - This change enhances the clarity of the approval logic and improves maintainability by focusing solely on the Status comparison. --- internal/tender_approval/service.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/tender_approval/service.go b/internal/tender_approval/service.go index 62bc3b2..fc1b4ba 100644 --- a/internal/tender_approval/service.go +++ b/internal/tender_approval/service.go @@ -995,7 +995,7 @@ func (s *tenderApprovalService) ToggleTenderApproval(ctx context.Context, req *T return approval.ToResponseWithTender(nil), nil } - if existing.Status == req.Status && existing.SubmissionMode == req.SubmissionMode { + if existing.Status == req.Status { err = s.repository.Delete(ctx, existing.GetID()) if err != nil { s.logger.Error("Failed to update tender approval", map[string]interface{}{