New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

포인트를 저장하는 부분에 적용여부를 확인하지 않는 문제 고침 #2348

Merged
merged 2 commits into from Dec 10, 2018

Conversation

2 participants
@bjrambo
Copy link
Contributor

bjrambo commented Dec 3, 2018

포인트를 저장하는 부분에 적용여부의 성공이나 실패 여부 등등의 내용을 뿌려주지 않아 확인할 수 없는 문제가 있는 것 같습니다.

적어도 return $output 이 있다면 리턴을 통해서 적용여부라도 확인할 수 있도록 제공해주는 것이 좋다고 생각해서 수정했습니다.

구현된 function 아래에 return $output 이 잇지만 어떤 내용도 리턴되고 잇지 않고 잇었습니다.

rhymix/rhymix#1116
의 코드와 동일하고, 이 코드의 대한 라이선스는 듀얼라이선스로 적용하겠습니다..

bjrambo added some commits Dec 3, 2018

불필요하게 포인트여부를 한번 더 디비 쿼리 하지않도록 함.
보통 기존에 기록이 있을 경우 $current_point의 값이 0보다 이상이기 때문에 먼저 가져온 값을 이용하고 $current_point가 0 일 경우에만 insert를 먼저 실행하도록함.

member_srl칼럼이 primary_key 이기 때문에 이미 디비상에서 존재할경우 insert가 되지 않기 때문에 다시 update를 실행하여 확실한 작동을 유도함. (이때 리턴을 시키지 않음)

@bnu bnu self-assigned this Dec 10, 2018

@bnu bnu added the type/bug label Dec 10, 2018

@bnu bnu added this to the 1.11.2 milestone Dec 10, 2018

@bnu bnu merged commit 9735e16 into xpressengine:develop Dec 10, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@bnu bnu added this to 이슈 종료 in 이슈 진행 상황 via automation Dec 10, 2018

dorami added a commit to daolcms/daolcms that referenced this pull request Dec 23, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment