Skip to content

fix(core): 保留重复标量语义的空数组修复(#150 替代) - #155

Draft
jeequan wants to merge 1 commit into
devfrom
fix/review-150-20261027
Draft

jeequan wants to merge 1 commit into
devfrom
fix/review-150-20261027

Conversation

@jeequan

@jeequan jeequan commented Oct 27, 2026 •

Copy link
Copy Markdown
Owner

定位与来源

参考社区 #150(@rwzdk)发现的空数组问题及样例,提供独立最小兼容替代草稿。不关闭或修改原PR;本方案不采用全局重复参数逗号拼接,与原PR不要同时合并。

范围

空数组明确转换空字符串,避免越界与前一参数残留。重复标量仍取末值;列表仍按已有单个CSV字符串契约处理。
没有实现 repeated-query list 新语义;wayCodeStrs 现有文档是单个CSV,selectedIds是路径变量。新的列表参数语义需另行明确,不能通过全局拼接改变分页/金额。

验证

本地JDK17 core全量14项通过。新增测试对原PR实现出现1failure+1error(真实分页1,2转换异常),对dev基线出现空数组越界;最终补丁全部通过。覆盖真实AbstractCtrl分页、state/amount、嵌套参数、CSV、重复参数、空数组和null。
命令:mvn -B -pl jeepay-core -am test。git diff --check通过。远端CI待本head运行,未宣称全仓本地验证。

无配置、数据库、权限、生产或发布改动。

远端 CI 已核实(2026-10-27 14:48 UTC)

CI run 40524304470 在本草稿 head 077b6d5db227fa936ec2abccc239221382484885 已 completed/success;Shell 安装脚本断言与 Maven 编译(含 JDK17 编译/测试编译/单元及授权回归)均通过。以上替代前文创建时“待运行”的状态。远端文件已与本地已测试版本逐一比对一致。
保留草稿供维护者审核,CI 通过不消除上文业务/渠道边界,也不授权自动合并或发版。

参考 rwzdk 的社区 PR #150 问题分析及回归样例,采用空数组最小修复;保留末值及单值 CSV 契约,不全局拼接重复参数。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant