Alipay has an optional parameter: store_id, see documentation at: https://docs.open.alipay.com/api_1/alipay.trade.pay

We should send commerce_store id as default, but in the API level, offer a way to let user to customize this optional parameter.

CommentFileSizeAuthor
#6 support_optional_parameter_store_id_03.patch4.09 KBzfming
#5 support_optional_parameter_store_id_02.patch4.21 KBzfming
#3 support_optional_parameter_store_id.patch3.28 KBzfming
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

skyredwang created an issue. See original summary.

zfming’s picture

zfming CreditAttribution: zfming at Sparkpad commented
Assigned: Unassigned » zfming
zfming’s picture

zfming CreditAttribution: zfming at Sparkpad commented
FileSize
support_optional_parameter_store_id.patch3.28 KB
skyredwang’s picture

skyredwang
Location Shanghai
CreditAttribution: skyredwang at INsReady for Sparkpad commented
Status: Active » Needs work 
+++ b/src/Plugin/Commerce/PaymentGateway/CustomerScanQRCodePay.php
@@ -258,6 +260,12 @@ class CustomerScanQRCodePay extends OffsitePaymentGatewayBase implements Support
+    // Check if $store_id is null then get store_id in order
+    if (!$store_id) {
+      $order = Order::load($order_id);
+      $store_id = $order->getStoreId();
+    }
+

I'd like to move this part of logic check to UI or forms. So, the API stays clean.

zfming’s picture

zfming CreditAttribution: zfming at Sparkpad commented
FileSize
support_optional_parameter_store_id_02.patch4.21 KB

ok, i move this part of code to forms

zfming’s picture

zfming CreditAttribution: zfming at Sparkpad commented
FileSize
support_optional_parameter_store_id_03.patch4.09 KB
zfming’s picture

zfming CreditAttribution: zfming at Sparkpad commented

  • skyredwang committed bdc04de on 8.x-1.x authored by zfming 
    Issue #2916695 by zfming: Support store_id as a parameter for upstream
skyredwang’s picture

skyredwang
Location Shanghai
CreditAttribution: skyredwang at INsReady for Sparkpad commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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