合并受阻
感谢提交 Pull Requests！如果您提交的PR已经开发完毕，请评论 "start build" 触发门禁，更多交互操作，请访问OpenHarmony社区支持命令清单。如果需要调整订阅PR、Issue的变更状态，请访问订阅链接。
Thanks for submitting the pull request. If your Pull Request has already been developed, you can leave a "start build" comment to trigger the gated system. For more commands, please visit OpenHarmony Command List. If you need to change the subscription of a Pull Request or Issue, please visit the link.
start build
首次触发
门禁构建开始,包含静态检查、代码编译和测试【master_inner_build编译, hispark_pegasus编译, hispark_pegasus_3863编译, dayu200测试, hispark_pegasus测试, dayu600_7885编译, hispark_taurus_Linux编译, hispark_pegasus_3863测试, dayu200_tdd编译, x86_64_virt编译, dayu600_7885测试, dayu200编译, ohos-host_mini_tdd编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6a840dfc64650f998b137c7e/runlist
PR 审查报告：#5556 - portal版本修复
仓库：OpenHarmony/communication_wifi
PR 地址：https://gitcode.com/openharmony/communication_wifi/merge_requests/5556
作者：Aki (daweq)
目标分支：master
PR SHA：9109c1a2a9ed7018b7d4b26699e783de9bc5e5f1
状态：open
标签：waiting_on_author、dco检查成功
CI 状态：
- ✅ DCO 检查成功
- ✅ 冲突检查通过
变更概要
本 PR 递增 Portal 登录应用的 versionCode（2 个文件，+1/-1）：
| 文件 | 变更 |
|---|---|
wifi/application/portal_login/AppScope/app.json |
versionCode 1000011 → 1000012 |
wifi/application/portal_login/AppScope/app.json5 |
versionCode 1000011 → 1000012 |
versionName 保持 1.0.0 不变。
问题详情
⚠️主要
1. app.json 与 app.json5 双份配置重复维护
位置：wifi/application/portal_login/AppScope/
描述：同一目录下同时存在 app.json 和 app.json5，内容重复（bundleName/vendor/versionCode/versionName 均一致）。OpenHarmony 新工程规范推荐使用 app.json5（支持注释、类型更严格），app.json 通常为旧工程残留。本次虽同步修改了两份，但双文件并存必然导致后续版本号维护不同步的风险。
影响：版本号漂移、构建产物版本不一致、维护成本翻倍。
修复建议：确认构建实际使用哪份配置（查看 entry/build-profile.json5 或工程结构），删除冗余文件：
# 若 app.json5 为实际生效配置：
rm wifi/application/portal_login/AppScope/app.json
若两份均为构建所需（不同构建链路），建议在 PR 描述中说明原因。
2. versionCode 递增但 versionName 未同步
位置：同上
描述：
"versionCode": 1000012,
"versionName": "1.0.0",
versionCode 递增（1000011 → 1000012）而 versionName 仍为 1.0.0。若本次为修复后的正式发版，versionName 应同步升级（如 1.0.1）；若 versionCode 仅为内部构建号，则需在说明中注明。
影响：用户侧可见版本号不变，问题修复无法通过版本号区分；可能影响 OTA/升级判断逻辑。
修复建议：按发布策略同步更新：
"versionCode": 1000012,
"versionName": "1.0.1",
💡建议
3. 变更说明缺失
- PR 标题"portal版本修复"未说明修复内容与版本号递增的对应关系（哪个修复、对应哪个版本）；
- 描述为空模板。
建议补充：本次版本号递增对应的修复内容清单。
4. 提交粒度
仅版本号递增的提交建议在 PR 描述中注明是否伴随功能/修复代码（如与 #5555 等 Portal 修复 PR 的发布关系），便于追溯。
总体评分
| 维度 | 评分 | 说明 |
|---|---|---|
| 功能正确性 | 7/10 | 版本号递增正确 |
| 代码质量 | 5/10 | 双份配置重复维护 |
| 安全性 | 8/10 | 无安全影响 |
| 可维护性 | 5/10 | 配置冗余、说明缺失 |
| 测试充分性 | 6/10 | 配置变更，构建验证即可 |
| 综合评分 | 6/10 |
审查结论
建议操作：✅ 可以合并（建议顺手清理）
建议修复：
- 问题 1：确认并清理
app.json/app.json5双份配置（建议保留 json5）； - 问题 2：确认
versionName是否需要同步递增； - 问题 3：补充版本号递增对应的修复内容说明。
优点：
- 变更简单明确（2 文件，各 1 行）
- 两份配置文件同步修改，无遗漏
- DCO 检查通过
代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6a840dfc64650f998b137c7e/runlist
静态检查:
| # | check type | result | report |
|---|---|---|---|
| 1 | codeCheck | pass | >>> |
编译测试:
| # | Device | build result | test result | package |
|---|---|---|---|---|
| 1 | hispark_taurus_Linux | success | NA | >>> |
| 2 | dayu200 | success | success | >>> |
| 3 | dayu200_tdd | success | NA | >>> |
| 4 | hispark_pegasus | success | success | >>> |
| 5 | master_inner_build | (IGNORE) | NA | >>> |
| 6 | ohos-host_mini_tdd | success | NA | >>> |
| 7 | dayu600_7885 | success | success | >>> |
| 8 | x86_64_virt | success | NA | >>> |
| 9 | hispark_pegasus_3863 | success | success | >>> |
您好，Committer @wshixjr @bjtu_wujun_transportation @f00651013 @chengguohong ，请分配检视人员检视该PR，可以通过命令"assign [@someone_id]"分配检视人员，也可以直接评论"assign"分配给自己进行检视。
Hello, Committer @wshixjr @bjtu_wujun_transportation @f00651013 @chengguohong . Please assign someone to review the PR. You can assign a reviewer by using the command "assign [@someone_id]", or you can comment "assign" to review the PR by yourself.
一、内容说明（相关的Issue）
二、建议测试周期和提测地址
建议测试完成时间：xxxx.xx.xx
投产上线时间：xxxx.xx.xx
提测地址：CI环境/压测环境
测试账号：
三、变更内容
3.1 关联PR列表
3.2 数据库和部署说明
bundle exec xxx RAILS_ENV = production；没有写 "无"3.4 其他技术优化内容（做了什么，变更了什么）
3.5 废弃通知（什么字段、方法弃用？）
3.6 后向不兼容变更（是否有无法向后兼容的变更？）
四、研发自测点（自测哪些？冒烟用例全部自测？）
自测测试结论：
五、测试关注点（需要提醒QA重点关注的、可能会忽略的地方）
检查点：
接口测试：
性能测试：
并发测试：
其他：