WeChat Pay has an optional parameter: device_info, see documentation at: https://pay.weixin.qq.com/wiki/doc/api/micropay.php?chapter=9_10&index=1

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
#15 support_optional_parameter_device_info_6.patch2.5 KBzfming
#14 support_optional_parameter_device_info_5.patch2.58 KBzfming
#13 support_optional_parameter_device_info_4.patch2.57 KBzfming
#11 support_optional_parameter_device_info_3.patch2.41 KBzfming
#9 support_optional_parameter_device_info_2.patch2.07 KBzfming
#8 support_optional_parameter_store_id_wx.patch5.68 KBzfming
#7 support_optional_parameter_device_info.patch4 KBzfming
#3 support_optional_parameter_store_id_wx.patch5.68 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
FileSize
support_optional_parameter_store_id_wx.patch5.68 KB

wechat pay has device_info parameter we can input store_id, this patch add device_info parameter

zfming’s picture

zfming CreditAttribution: zfming at Sparkpad commented
FileSize
support_optional_parameter_store_id_wx.patch5.68 KB

wechat pay has device_info parameter we can input store_id, this patch add device_info parameter

skyredwang’s picture

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

  • skyredwang committed 01741a2 on 8.x-1.x authored by zfming 
    Issue #2921469 by zfming: Support device_info as a parameter for...
zfming’s picture

zfming CreditAttribution: zfming at Sparkpad commented
Status: Fixed » Active
zfming’s picture

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

zfming CreditAttribution: zfming at Sparkpad commented
FileSize
support_optional_parameter_store_id_wx.patch5.68 KB

this patch follow 1.x-dev fixed device_info invalid

zfming’s picture

zfming CreditAttribution: zfming at Sparkpad commented
FileSize
support_optional_parameter_device_info_2.patch2.07 KB

this patch follow 1.x-dev fixed device_info invalid

skyredwang’s picture

skyredwang
Location Shanghai
CreditAttribution: skyredwang at INsReady for Sparkpad commented
Status: Active » Needs work 
+++ b/src/Plugin/Commerce/PaymentGateway/Micropay.php
@@ -43,12 +43,12 @@ class Micropay extends QRCodePaymentMode2 {
+    $wechat_order->set('device_info', $device_info);

This is due to the current limitation of the upstream; I'd like to see an issue opened at the upstream, and referenced here as a comment.

zfming’s picture

zfming CreditAttribution: zfming at Sparkpad commented
FileSize
support_optional_parameter_device_info_3.patch2.41 KB

I add a issue at github EasyWeChat, https://github.com/EasyWeChat/payment/issues/6

skyredwang’s picture

skyredwang
Location Shanghai
CreditAttribution: skyredwang at INsReady for Sparkpad commented

No. wrong repo. This is the correct one: https://github.com/overtrue/wechat

zfming’s picture

zfming CreditAttribution: zfming at Sparkpad commented
FileSize
support_optional_parameter_device_info_4.patch2.57 KB

This patch fixed device_info invalid

zfming’s picture

zfming CreditAttribution: zfming at Sparkpad commented
FileSize
support_optional_parameter_device_info_5.patch2.58 KB

add EasyWeChat issue link

zfming’s picture

zfming CreditAttribution: zfming at Sparkpad commented
FileSize
support_optional_parameter_device_info_6.patch2.5 KB

  • skyredwang committed b269b02 on 8.x-1.x 
    Issue #2921469 by zfming, skyredwang: Support device_info as a parameter...
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.