Skip to content

Commit

Permalink
Merge pull request #461 from sangkenlee/policy-name-update-fix
Browse files Browse the repository at this point in the history
정책명 업데이트 버그 수정
  • Loading branch information
Siyeop authored Apr 30, 2024
2 parents 00b732d + aa8b7ef commit 6e09809
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions internal/usecase/policy.go
Original file line number Diff line number Diff line change
Expand Up @@ -245,21 +245,21 @@ func (u *PolicyUsecase) Update(ctx context.Context, organizationId string, polic
updateMap["mandatory"] = mandatory
}

if policyName != nil {
policy, err := u.repo.GetByName(ctx, organizationId, *policyName)
if err != nil {
// 정책명을 업데이트하기로 설정하였고
if policyName != nil && policy.PolicyName != *policyName {
exists, err := u.repo.ExistByName(ctx, organizationId, *policyName)

if err != nil && !errors.IsNotFound(err) {
return err
}

// 이름이 같은 정책이 존재하지만 아이디가 서로 다른 경우, 즉 다른 정책이 해당 이름 사용 중임
if policy != nil && policyId != policy.ID {
if exists {
return httpErrors.NewBadRequestError(httpErrors.DuplicateResource, "P_INVALID_POLICY_NAME", "policy name already exists")
}

// 해당 이름 사용중인 정책이 없으면 업데이트, 있으면 동일 정책이므로 업데이트 안 함
if policy == nil {
updateMap["policy_name"] = policyName
}
updateMap["policy_name"] = policyName
}

if description != nil {
Expand Down

0 comments on commit 6e09809

Please sign in to comment.