Skip to content

Conversation

@PicturePark1101
Copy link
Contributor

@PicturePark1101 PicturePark1101 commented Jul 12, 2025

작업내용

임장 둘러보기 관련 자잘한 버그 fix

상세설명_ & 캡쳐

요구사항 미반영 문제들

  1. 임장 둘러보기 상세보기에서 도로명 주소 + 상세주소로 리턴되고 있음 -> 도로명 주소만 리턴되도록 변경
  2. 현재 임장 주인인 경우에도 임장을 구매할 수 있는 문제가 있음 -> 임장 주인인 경우 isBuyer 필드 true로 리턴
  • 아래 코드처럼 구매자인지, 소유자인지 여부도 판별
	private boolean getIsBuyerOrOwner(Member requestMember, SharedNote sharedNote) {
		return usedPencilFinder.existsByMemberAndSharedNoteId(requestMember, sharedNote.getSharedNoteId()) ||
			sharedNote.getMember().getMemberId().equals(requestMember.getMemberId());
	}

코드상 버그 fix

  1. 임장 둘러보기에서 전체 갯수 불일치 문제 fix
  • count 쿼리에서 조건을 빠뜨려서 나는 버그인 것을 확인 -> limjang.deleted.isFalse() 추가하여 fix
		JPAQuery<Long> countQuery = queryFactory
			.select(sharedNote.count()).from(sharedNote)
			.join(sharedNote.limjang, limjang)
			.join(sharedNote.member, member)
			.join(limjang.limjangPrice, limjangPrice)
			.join(limjang.addressEntity, address)
			.leftJoin(limjang.report, report)
			.where(
				getBcodesStartsWith(code),
				getWhereByPropertyType(propertyType),
				getWhereByPriceType(priceType),
				keywordCondition(keyword),
				sharedNote.deletedAt.isNull(),
				limjang.deleted.isFalse()
			);
  1. 임장 상세보기에서 주소에 " "이 한번 더들어가는 문제
  • 해당 필드가 DB에 null이 아닌 " "이 들어간 문제였음. 필터 조건에 추가하여 fix
	public String getShortAddress() {
		return Stream.of(sigungo, bname1, bname2)
			.filter(s -> s != null && !s.isBlank())
			.collect(Collectors.joining(" "));
	}

@PicturePark1101 PicturePark1101 self-assigned this Jul 12, 2025
@PicturePark1101 PicturePark1101 added 🐞 bugFix 버그 고침 진이 작업했습니다. 👊🏻 PULL REQUEST pr날릴때 labels Jul 12, 2025
@PicturePark1101 PicturePark1101 merged commit 7f6119d into dev Jul 12, 2025
1 check passed
@PicturePark1101 PicturePark1101 deleted the fix/#437 branch July 12, 2025 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

진이 작업했습니다. 🐞 bugFix 버그 고침 👊🏻 PULL REQUEST pr날릴때

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants