fix(comment): fix comments

Author: hewenguang

src/pay/weixin.controller.ts

@@ -9,7 +9,6 @@ import {
   Param,
   Body,
   Controller,
-  HttpCode,
   HttpStatus,
 } from '@nestjs/common';
 import { AppException } from 'omniboxd/common/exceptions/app.exception';
@@ -50,7 +49,6 @@ export class WeixinController {
 
   @Public()
   @Post('callback')
-  @HttpCode(204) // Return 204 No Content on success as per WeChat Pay V3 API docs
   callback(@Body() body: WeixinCallbackBody) {
     return this.weixinService.callback(body);
   }

src/pay/weixin.service.ts

@@ -144,6 +144,7 @@ export class WeixinService {
   }
 
   callback(body: WeixinCallbackBody) {
+    throw new Error(JSON.stringify(body));
     try {
       // Decrypt the callback data
       // Note: wechatpay-node-v3 library handles signature verification internally
@@ -195,8 +196,6 @@ export class WeixinService {
     } catch (error) {
       console.log(body, error);
 
-      // If decryption fails (e.g., signature verification failed),
-      // throw an exception to return 4XX/5XX status code with error response
       throw new AppException(
         error.message,
         'SIGNATURE_VERIFICATION_FAILED',
@@ -248,8 +247,16 @@ export class WeixinService {
       await this.ordersService.close(order.orderNo);
     }
 
-    const lastOrder: any = await this.ordersService.findById(userId, orderId);
-    lastOrder.response = response;
-    return lastOrder;
+    if (wechatData.payer && wechatData.payer.openid) {
+      await this.userService.updateUserBindingWhenMetadataEmpty(
+        userId,
+        'wechat',
+        {
+          openid: wechatData.payer.openid,
+        },
+      );
+    }
+
+    return await this.ordersService.findById(userId, orderId);
   }
 }

src/user/user.service.ts

@@ -182,6 +182,18 @@ export class UserService {
     });
   }
 
+  async updateUserBindingWhenMetadataEmpty(
+    userId: string,
+    loginType: string,
+    metadata: Record<string, any>,
+  ) {
+    const userBinding = await this.findUserBinding(userId, loginType);
+    if (userBinding && !userBinding.metadata) {
+      userBinding.metadata = metadata;
+      await this.userBindingRepository.save(userBinding);
+    }
+  }
+
   async updateBinding(oldUnionid: string, newUnionid: string) {
     // Unbind the associated new account
     const existBinding = await this.userBindingRepository.findOne({