fix: 修复商家转账参数缺失的问题 (#977)

--------- Co-authored-by: yansongda <me@yansongda.cn>
yansongda · May 8, 2024 · 747806e · 747806e
1 parent 722ffdc
commit 747806e
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 3 deletions.
diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml
@@ -23,6 +23,6 @@ jobs:
       - name: Run Coverage
         run: vendor/bin/phpunit --coverage-clover coverage.xml
       - name: Upload Coverage
-        uses: codecov/codecov-action@v3
+        uses: codecov/codecov-action@v4
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,9 @@
+## v3.7.3
+
+### fixed
+
+- fix: 修复商家转账参数缺失的问题(#977)
+
 ## v3.7.2
 
 ### added

diff --git a/src/Plugin/Wechat/V3/Marketing/Transfer/CreatePlugin.php b/src/Plugin/Wechat/V3/Marketing/Transfer/CreatePlugin.php
@@ -90,10 +90,11 @@ protected function normal(array $params, array $config, Collection $payload): ar
      */
     protected function encryptSensitiveData(array $params, array $config, Collection $payload): array
     {
+        $data['transfer_detail_list'] = $payload->get('transfer_detail_list', []);
         $data['_serial_no'] = get_wechat_serial_no($params);
         $publicKey = get_wechat_public_key($config, $data['_serial_no']);
 
-        foreach ($payload->get('transfer_detail_list', []) as $key => $list) {
+        foreach ($data['transfer_detail_list'] as $key => $list) {
             if (!empty($list['user_name'])) {
                 $data['transfer_detail_list'][$key]['user_name'] = encrypt_wechat_contents($list['user_name'], $publicKey);
             }

diff --git a/tests/Plugin/Wechat/V3/Marketing/Transfer/CreatePluginTest.php b/tests/Plugin/Wechat/V3/Marketing/Transfer/CreatePluginTest.php
@@ -85,7 +85,8 @@ public function testNormalWithName()
             "test" => "111",
             'transfer_detail_list' => [
                 [
-                    'user_name' => 'yansongda'
+                    'user_name' => 'yansongda',
+                    'foo' => 'bar',
                 ]
             ]
         ]));
@@ -102,5 +103,6 @@ public function testNormalWithName()
         self::assertArrayHasKey('_serial_no', $payload);
         self::assertArrayHasKey('user_name', $payload['transfer_detail_list'][0]);
         self::assertNotEquals('yansongda', $payload['transfer_detail_list'][0]['user_name']);
+        self::assertEquals('bar', $payload['transfer_detail_list'][0]['foo']);
     }
 }