问题描述

WxMaIntracityServiceImpl.balanceQuery 方法中的验证逻辑存在错误，当 wxStoreId 为 null 且 payMode 为 APP 或 COMPONENT 时，会错误地阻止合法的 API 调用。

原始错误代码

if (wxStoreId == null && (payMode != null && payMode != PayMode.STORE)) {
    throw new IllegalArgumentException("payMode是PAY_MODE_STORE或null时，必须传递wxStoreId");
}

这个条件判断是反的，导致在以下情况下抛出异常：

• wxStoreId 为 null 且 payMode 为 APP 或 COMPONENT

但根据 API 文档和 BasicWxMaStoreChargeRefundRequest.java 中的注释说明，wxStoreId 只有在 payMode 为 STORE 或 null（默认为 STORE）时才是必需的。

问题示例

修复前，以下合法调用会错误地抛出异常：

// 这个调用应该是合法的，但会抛出 IllegalArgumentException
intracityService.balanceQuery(null, "service123", PayMode.APP);

用户不得不通过传入一个虚假的 wxStoreId 值来绕过这个问题。

解决方案

修正了验证条件，使其正确地仅在 payMode 为 STORE 或 null 时才要求 wxStoreId：

if (wxStoreId == null && (payMode == null || payMode == PayMode.STORE)) {
    throw new IllegalArgumentException("payMode是PAY_MODE_STORE或null时，必须传递wxStoreId");
}

修复后的行为

• ✅ balanceQuery(null, serviceId, null) → 抛出异常（默认 STORE 模式需要 wxStoreId）
• ✅ balanceQuery(null, serviceId, PayMode.STORE) → 抛出异常（STORE 模式需要 wxStoreId）
• ✅ balanceQuery(null, serviceId, PayMode.APP) → 通过验证（不需要 wxStoreId）
• ✅ balanceQuery(null, serviceId, PayMode.COMPONENT) → 通过验证（不需要 wxStoreId）
• ✅ balanceQuery("store123", serviceId, 任意模式) → 通过验证（已提供 wxStoreId）

影响说明

无破坏性变更 - 原本能正常工作的代码继续正常工作，之前被错误阻止的合法用法现在可以正常使用了。

修复 #3704

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.