Skip to content
This repository has been archived by the owner on Dec 3, 2022. It is now read-only.

[이슈] GUI 중지시 팅김 현상 #17

Closed
cha2hyun opened this issue Aug 5, 2021 · 2 comments
Closed

[이슈] GUI 중지시 팅김 현상 #17

cha2hyun opened this issue Aug 5, 2021 · 2 comments
Assignees
Labels
bug Something isn't working 해결: 수정 이슈를 인지하여 코드를 수정하였다
Milestone

Comments

@cha2hyun
Copy link

cha2hyun commented Aug 5, 2021

재현 절차

  1. 로그인
  2. 브라우저설정
  3. 알럿 클릭
  4. 시간 설정
  5. 시작
  6. 중지

이슈 내용

Traceback (most recent call last):
File "qtmain.py", line 138, in stop_reservation_macro
reservation.interrupt()
NameError: name 'reservation' is not defined

기대 결과

중지되지 않고 팅김

참조

@cha2hyun cha2hyun added the bug Something isn't working label Aug 5, 2021
@binchoo
Copy link
Owner

binchoo commented Aug 5, 2021

@cha2hyun

def create_reservation():
global run_interval
def phase_description(resv):
logger.info('설정하신 계정과 장소를 토대로 백신 예약을 시도합니다.')
def phase_summary(resv):
logger.info('매크로 수행이 끝났습니다.')
run_interval = view.getRunInterval(default=7)
reservation = LegacyVaccineReservation(login_cookie, region, run_interval)
reservation.on_start(phase_description)
reservation.on_end(phase_summary)
return reservation

해당 현상은 reservation이 전역 컨텍스트로 등록되지 않아 발생하는 널 포인터 엑셥션입니다. 이 이슈 참조하여 수정하도록 하겠습니다. 감사합니다.

@binchoo binchoo added this to the QA 1.0.9 milestone Aug 5, 2021
@binchoo binchoo added the 진행중 이슈를 인지하여 코드 개선중 label Aug 5, 2021
@binchoo binchoo added 해결: 수정 이슈를 인지하여 코드를 수정하였다 and removed 진행중 이슈를 인지하여 코드 개선중 labels Aug 5, 2021
@binchoo
Copy link
Owner

binchoo commented Aug 5, 2021

@cha2hyun

정지버튼 클릭 시 널 포인터 엑셉션으로 프로그램 자체가 닫히는 이슈 해결하였습니다. master로 머지하였습니다.

이 변경이 실제 예약 절차에 부수 효과를 낳았을 수도 있기 때문에, 충분한 테스트가 더 필요합니다.

GIF 2021-08-05 오후 3-46-57

@binchoo binchoo closed this as completed Aug 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working 해결: 수정 이슈를 인지하여 코드를 수정하였다
Projects
None yet
Development

No branches or pull requests

2 participants