CommentFileSizeAuthor
#4 3041410-4.patch1.09 KBcarlxjs
#4 3041410-2-4.interdiff.txt1.08 KBcarlxjs
#2 3041410-2.patch1.22 KBcarlxjs

Comments

carlxjs created an issue. See original summary.

carlxjs’s picture

carlxjs commented
StatusFileSize
new 3041410-2.patch1.22 KB
skyredwang’s picture

skyredwang
Location Shanghai
commented 
+++ b/src/Plugin/Commerce/PaymentGateway/JSAPIpay.php
@@ -41,7 +45,7 @@ class JSAPIpay extends QRCodePaymentMode2 {
+  public function requestUnifiedOrder($openid, $sub_openid, $order_id, Price $total_amount, $notify_url = NULL) {

Do we need "$sub_openid = NULL"?

carlxjs’s picture

carlxjs commented
StatusFileSize
new 3041410-2-4.interdiff.txt1.08 KB
new 3041410-4.patch1.09 KB

Yes, when only use ordinary merchant API, $sub_openid should be null.

  • skyredwang committed 853508e on 8.x-1.x authored by carlxjs 
    Issue #3041410 by carlxjs: Support sub_openid as pramater in JSAPIpay
skyredwang’s picture

skyredwang
Location Shanghai
commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.