Skip to content

fix: ManageFixMenuService.isFixedMenu 메서드명이 실제 동작과 반대 #406

Description

@pooreumjung

Fix Report

고쳐야 할 사항

  • ManageFixMenuService.isFixedMenu(menuId)(src/main/java/ssu/eatssu/domain/admin/service/ManageFixMenuService.java)가 실제로는 RestaurantType.isVariableType(restaurant)를 반환한다. 즉 메서드 이름과 실제 반환값의 의미가 반대다. 현재 updateMenu/changeDiscontinuedStatus에서 if (isFixedMenu(menuId)) throw NOT_SUPPORT_RESTAURANT처럼 결과적으로는 맞게 쓰이고 있지만, 이름만 보고 이 메서드를 재사용하면 정반대의 판단을 하게 될 위험이 크다. 메서드명을 실제 의미에 맞게 수정해야 한다.

첨부자료

  • 없음

Metadata

Metadata

Assignees

Labels

fix고치기

Type

No type

Fields

No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions