Skip to content

fix:修复 Modal 中使用两个 Form 一个设置disabled为true 另外一个为false 会造成死循环的问题#1675

Open
leehaoChen wants to merge 1 commit intomasterfrom
fix-modal-mutilForm-status
Open

fix:修复 Modal 中使用两个 Form 一个设置disabled为true 另外一个为false 会造成死循环的问题#1675
leehaoChen wants to merge 1 commit intomasterfrom
fix-modal-mutilForm-status

Conversation

@leehaoChen
Copy link
Contributor

问题描述: Modal 中使用两个 Form 一个设置disabled为true 另外一个为false 会造成死循环的问题
问题原因:两个Form 同时在didUpdate 的时候设置了context 中传入的status
解决办法: 判断存在多个Form 的时候 进行兼容处理防止页面死循环,并给出报错提示。

@leehaoChen leehaoChen added the bug fixed a bug label Jun 27, 2022
@KMS-Bismarck KMS-Bismarck changed the base branch from master to dev October 21, 2022 09:53
@KMS-Bismarck KMS-Bismarck changed the base branch from dev to master October 21, 2022 09:53
@leehaoChen leehaoChen changed the title 修复 Modal 中使用两个 Form 一个设置disabled为true 另外一个为false 会造成死循环的问题 fix:修复 Modal 中使用两个 Form 一个设置disabled为true 另外一个为false 会造成死循环的问题 Dec 19, 2022
@renaesop renaesop requested a review from Copilot December 13, 2024 06:36
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

src/Card/Submit.js:19

  • The behavior introduced by the check method should be covered by tests to ensure it works as expected.
check() {

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug fixed a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants