已开启
fix: 补充close/open链卸载守卫与兜底;panresponderThreshold阈值生效;supportedOrientations朝向约束 #33
fix: 补充close/open链卸载守卫与兜底;panresponderThreshold阈值生效;supportedOrientations朝向约束 #33
已开启
mazheng创建于 7 天前
mazheng
7 天前

fix: 补充close/open链卸载守卫与兜底;panresponderThreshold阈值生效;supportedOrientations朝向约束

likedislike
合并受阻
openharmony_ci
openharmony_ci成员
7 天前 评论:

感谢提交 Pull Requests !此PR未通过DCO校验。
校验失败可能原因:

1. 未签署“DCO协议”(开发者原创声明协议),在线签署、查看签署状态

2. Commits 中未包含 Signed-off-by信息,参考FAQ处理

修复上述问题后,在PR的评论框输入“check dco” ,单击”评论”,系统将再次进行DCO校验。

当前检测到如下Commits 未包含Signed-off-by信息:


Thanks for submitting a pull request. This pull request has not passed the DCO check.
Possible causes:

1. You have not signed the Developer Certificate of Origin (DCO). Sign the DCO and check DCO status.

2. The commits do not contain the Signed-off-by information. To resolve this issue, see FAQs.

After resolving the preceding issues, enter check dco in the comment box of this pull request and click Comment. The system will check DCO status again.

The following commits do not contain the Signed-off-by information:

likedislike
openharmony_ciopenharmony_ci成员
7 天前 添加了label:dco检查失败
Mmazheng
7 天前 强制推送  1 个提交:0f34a0b3-fix: 补充close/open链卸载守卫与兜底;panresponderThreshold阈值生效;supportedOrientations朝向约束
openharmony_ci
openharmony_ci成员
7 天前 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
openharmony_ciopenharmony_ci成员
7 天前 删除了label:dco检查失败
openharmony_ciopenharmony_ci成员
7 天前 添加了label:dco检查成功
cpf-manager
cpf-manager7 天前进行代码检视1
src/modal.tsx
@@ -649,0 +745,4 @@
745+ // Content host unavailable (e.g. the Modal host dismissed the content
746+ // or the component is unmounting). Complete the open sequence without
747+ // animation so the modal does not stay stuck in isTransitioning state.
748+ this.completeOpenWithoutAnimation();
cpf-manager
cpf-manager7 天前评论:

【AI-Review】【一般】【基础代码问题】【代码逻辑错误】open() fallback 路径缺少 onModalWillShow 回调,违反生命周期契约

● 问题: open() 的 else 分支(contentRef 为 null 时,line 744-749)直接调用 this.completeOpenWithoutAnimation()(line 748),该方法在 isVisible 为 true 时直接触发 this.props.onModalShow()(line 704),但 else 分支未先调用 this.props.onModalWillShow。对比 close() 的 else 分支(line 792-798)在 this.finalizeClose()(line 797)之前正确补齐了 this.props.onModalWillHide && this.props.onModalWillHide()(line 796),open 与 close 的 fallback 路径存在不对称性。

触发路径: Modal host 提前 dismiss 内容导致 this.contentRef 为 null(如 RNOH Modal 原生窗口在内容挂载前被关闭,或组件处于卸载过程中但 isUnmounted 尚未置 true)→ 用户设置 isVisible=truecomponentDidUpdate(line 321)调用 open()isTransitioning 为 false,进入 → this.contentRef 为 null → 进入 else 分支(line 744)→ completeOpenWithoutAnimation()(line 748)→ isUnmounted 为 false → isVisible 为 true → this.props.onModalShow()(line 704)触发,但 onModalWillShow 从未触发。

● 影响: 一般。依赖 onModalWillShow 回调的调用方(如在该回调中准备 UI、启动并行动画、埋点上报)在 fallback 路径下会丢失该回调,仅收到 onModalShow,违反 onModalWillShow → onModalShow 的生命周期契约,可能导致调用方前置逻辑未执行。

● 建议: 在 else 分支的 this.completeOpenWithoutAnimation() 之前补齐 onModalWillShow 调用,与 close() 的 fallback 路径保持对称:
} else {
this.props.onModalWillShow && this.props.onModalWillShow();
this.completeOpenWithoutAnimation();
}

likedislike
Mmazheng
6 天前 关联了issue:[Bug] 关闭模态时崩溃 startAnimation 空引用及 hideModalContentWhileAnimating/panResponderThreshold/supportedOrientations 属性不生效
Mmazheng
5 天前 删除了关联的issue:[Bug][br_rnoh0.72] 关闭模态时崩溃 startAnimation 空引用及 hideModalContentWhileAnimating/panResponderThreshold/supportedOrientations 属性不生效
Mmazheng
5 天前 关联了issue:[Bug][br_rnoh0.61] 关闭模态时崩溃 startAnimation 空引用及 hideModalContentWhileAnimating/panResponderThreshold/supportedOrientations 属性不生效
Mmazheng
4 天前 修改了pull request 的描述
mazheng
23 小时前 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
23 小时前 评论:

首次触发
门禁构建开始,包含静态检查、代码编译【rntpc_br_rnoh0.61编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6a9e7d4864650f998bd104af/runlist

likedislike
openharmony_ciopenharmony_ci成员
23 小时前 添加了label:静态检查失败
openharmony_ci
openharmony_ci成员
23 小时前 评论:

代码门禁未通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6a9e7d4864650f998bd104af/runlist

静态检查:

# check type result report
1 codeCheck noPass >>>

编译测试:
# Device build result package
1 rntpc_br_rnoh0.61 pending NA

likedislike
Mmazheng
7 小时前 推送  1 个提交:fcfaaba0-fix: 滑动百分比改用真实窗口尺寸,修复方向约束下分母为负
openharmony_ciopenharmony_ci成员
7 小时前 删除了label:静态检查失败
openharmony_ci
openharmony_ci成员
7 小时前 评论:

代码有更新,重置PR验证状态

likedislike
openharmony_ci
openharmony_ci成员
7 小时前 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
mazheng
7 小时前 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
7 小时前 评论:

本地或库上代码有更新,全量重新构建,重置所有关联PR的验证状态
门禁构建开始,包含静态检查、代码编译【rntpc_br_rnoh0.61编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6a9f5f1064650f998b10bbed/runlist

likedislike
openharmony_ciopenharmony_ci成员
7 小时前 添加了label:静态检查失败
openharmony_ci
openharmony_ci成员
7 小时前 评论:

代码门禁未通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6a9f5f1064650f998b10bbed/runlist

静态检查:

# check type result report
1 codeCheck noPass >>>

编译测试:
# Device build result package
1 rntpc_br_rnoh0.61 pending NA

likedislike