feat: 네이버페이 useCfmYmdt 값을 설정에서 변경 가능하도록 수정 #28
Conversation
iamport-naverpay-ext.php
Outdated
| @@ -250,7 +257,10 @@ public function iamport_payment_info( $order_id ) { | |||
| } | |||
|
|
|||
| $response["naverProducts"] = $naverProducts; | |||
| $response["naverUseCfm"] = "20991231"; | |||
| $naverUseCfm = $this->settings["useCfmYmdt"]; | |||
| if ($naverUseCfm && $naverUseCfm != "") { | |||
There was a problem hiding this comment.
else 케이스의 경우 20991231이라도 값을 넣어야하지 않을까용?
There was a problem hiding this comment.
else의 경우엔 일부로 설정을 안했다고 생각해서 넣어주지 않으려고 했습니다
근데 이전에 설정 안한사람들이랑 구분을 해야겠네요 그러면...
There was a problem hiding this comment.
아니면 설정을 안하려면 'none' 값을 넣어주세요 같은건 어떤가요??
There was a problem hiding this comment.
뭐 설정 안함 이런 체크박스 있고 그거 선택하면 입력된거 없어지고 입력 못하게(input disabled) 막으면 제일 좋은데 말 하는것만으로도 너무 귀찮을 것 같네요...
There was a problem hiding this comment.
오 흠 그런설정 기존에 있던게 있나요? 참고하면 간단할거같은데...
There was a problem hiding this comment.
생각해보니 체크박스 설정도 참 불편한게
기존에 null 체크같은걸 해야해서
기존 사용자는 null -> 20991231
체크한 사람은 true -> 입력한 값
체크 푼 사람은 false -> 안넣음
엄청 귀찮긴 하겠네요..
There was a problem hiding this comment.
모든 게이트웨이 맨 위에 있긴한데!
오랜 숙원 사업 해결해주셔서 감사합니다 보리수
There was a problem hiding this comment.
27909c2
이런 느낌도 코드도 간단하고 젤 괜찮을 것 같아 확인 부탁드림다
네이버페이 결제형 결제시
현가맹점에 제공되지 않는 기능입니다.
라는 문구와 함께 실패하는 문의가 들어와 해당 기능과 관련있는 파라매터를 설정으로 뺐습니다.
naverUseCfm 초기값은 기존에 하드코딩으로 있던 20991231로 설정하였습니다. (업데이트 시 기존 가맹점 영향 최소화를 위해)