-
-
Notifications
You must be signed in to change notification settings - Fork 999
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(module:PropertyReflector): handle null value of propertyInfo #2025
The head ref may contain hidden characters: "bugfix/fix(module\uFF1APropertyReflector)"
fix(module:PropertyReflector): handle null value of propertyInfo #2025
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2025 +/- ##
==========================================
+ Coverage 25.51% 27.40% +1.89%
==========================================
Files 488 511 +23
Lines 32193 24538 -7655
Branches 0 238 +238
==========================================
- Hits 8214 6725 -1489
+ Misses 23979 17777 -6202
- Partials 0 36 +36
Continue to review full report at Codecov.
|
@anddrzejb |
Please rebase to master branch. |
rebase done |
🤔 This is a ...
💡 Background and solution
When bind a Field (not Property) to control values, will prompt the following error:
So, we need to handle the situation that the
propertyInfo
is null.☑️ Self Check before Merge