-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(core): fix form fields query throw error #3992
Conversation
Size Change: +26 B (0%) Total Size: 516 kB
ℹ️ View Unchanged
|
@@ -45,6 +45,10 @@ export class Query { | |||
} | |||
} else { | |||
each(this.form.fields, (field, address) => { | |||
if (!field) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
虽然这样修复了这个问题, 但为什么不采用 https://github.com/alibaba/formily/pull/3987/files 在 undefined 产生的源头尽快修复的方案, 在这里修复有种头痛医头脚痛医脚的感觉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
痛哭流涕
Before submitting a pull request, please make sure the following is done...
master
orformily_next
.npm test
).npm run lint
) - we've done our best to make sure these rules match our internal linting guidelines.Please do not delete the above content
What have you changed?
Fixed #3932