Skip to content
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

getDerivedStateFromProp must return null if no change #11398

Merged
merged 1 commit into from Jul 22, 2018

Conversation

yoyo837
Copy link
Contributor

@yoyo837 yoyo837 commented Jul 22, 2018

A valid state object (or null) must be returned. You have returned undefined.

First of all, thank you for your contribution! :-)

Please makes sure that these checkboxes are checked before submitting your PR, thank you!

  • Make sure that you propose PR to right branch: bugfix for master, feature for latest active branch feature-x.x.
  • Make sure that you follow antd's code convention.
  • Run npm run lint and fix those errors before submitting in order to keep consistent code style.
  • Rebase before creating a PR to keep commit history clear.
  • Add some descriptions and refer relative issues for you PR.

Extra checklist:

if isBugFix :

  • Make sure that you add at least one unit test for the bug which you had fixed.

elif isNewFeature :

  • Update API docs for the component.
  • Update/Add demo to demonstrate new feature.
  • Update TypeScript definition for the component.
  • Add unit tests for the feature.

A valid state object (or null) must be returned. You have returned undefined.
@yoyo837
Copy link
Contributor Author

yoyo837 commented Jul 22, 2018

Related: #10309

@yoyo837 yoyo837 changed the title getDerivedStateFromProp return null if not change getDerivedStateFromProp return null if no change Jul 22, 2018
@yoyo837 yoyo837 changed the title getDerivedStateFromProp return null if no change getDerivedStateFromProp must return null if no change Jul 22, 2018
@ant-design-bot
Copy link
Contributor

Deploy preview for ant-design ready!

Built with commit 22c0a1c

https://deploy-preview-11398--ant-design.netlify.com

@codecov
Copy link

codecov bot commented Jul 22, 2018

Codecov Report

Merging #11398 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #11398      +/-   ##
==========================================
+ Coverage   91.76%   91.76%   +<.01%     
==========================================
  Files         199      199              
  Lines        5014     5015       +1     
  Branches     1403     1403              
==========================================
+ Hits         4601     4602       +1     
  Misses        408      408              
  Partials        5        5
Impacted Files Coverage Δ
components/date-picker/createPicker.tsx 96.87% <100%> (+0.04%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8c7dfe4...22c0a1c. Read the comment docs.

@afc163 afc163 merged commit f728fbe into ant-design:master Jul 22, 2018
@yoyo837 yoyo837 deleted the patch-2 branch July 22, 2018 06:41
bors bot added a commit to mozilla/delivery-console that referenced this pull request Jul 25, 2018
319: Update dependency antd to v3.7.2 r=magopian a=renovate[bot]

This Pull Request updates dependency [antd](https://github.com/ant-design/ant-design) from `v3.7.1` to `v3.7.2`



<details>
<summary>Release Notes</summary>

### [3.7.2](https://github.com/ant-design/ant-design/releases/3.7.2)
[Compare Source](ant-design/ant-design@3.7.1...3.7.2)
- DatePicker
  - 🐞 **Fix issue resulting in year and month can not be changed in control mode.** [b9992f4](ant-design/ant-design@b9992f4)
  - 🐞 Fix warning of `getDerivedStateFromProp`. [#&#8203;11398](`ant-design/ant-design#11398) [@&#8203;yoyo837]
- Drawer
  - 🐞 Fix close animation when setting `destroyOnClose`. [#&#8203;11307](`ant-design/ant-design#11307)
  - 🐞 Fix display issue when using a `vw` value as `width`. [#&#8203;11326](`ant-design/ant-design#11326)
  - 🐞 Fix `wrapClassName` now working.
- 🐞 Fix text overflow of Tooltip. [#&#8203;11402](`ant-design/ant-design#11402) [@&#8203;weidapao]
- 🐞 Fix style issue of dark theme Menu in Layout.Header. [#&#8203;11400](`ant-design/ant-design#11400) [@&#8203;hongxuWei]
- 🐞 Fix the arrow buttons of InputNumber showing wrong positon in a fixed table. [#&#8203;11408](`ant-design/ant-design#11408)
- 🐞 Fix issue resulting in Select.Option shows wrong border radius in Select.OptGroup. [6cb6f5c](ant-design/ant-design@6cb6f5c)
- 🐞 Fix issue resulting in `onChange` was trigged twice when click the filter icon of Table. [#&#8203;11164](`ant-design/ant-design#11164) [@&#8203;adybionka]
- 🐞 Fix issue resulting title of Model.confirm shows scrollbar on Firefox. [#&#8203;11432](`ant-design/ant-design#11432)
- TypeScript
  - 🐞 Fix type definition of Radio.Group. [#&#8203;11409](`ant-design/ant-design#11409) [@&#8203;eddiemoore]
  - 🐞 Fix type definition of TreeSelect. [#&#8203;11442](`ant-design/ant-design#11442) [@&#8203;JribiBelhassen]
  - 🐞 Fix type definition of Badge. [#&#8203;11421](`ant-design/ant-design#11421) [@&#8203;zongzi531]


---


- DatePicker
  - 🐞 **修复在受控模式下不能切换年月的问题。**[b9992f4](ant-design/ant-design@b9992f4)
  - 🐞 修复在 `getDerivedStateFromProp` 的警告。[#&#8203;11398](`ant-design/ant-design#11398) [@&#8203;yoyo837]
- Drawer
  - 🐞 修复使用 `destroyOnClose` 时没有关闭动画的问题。[#&#8203;11307](`ant-design/ant-design#11307)
  - 🐞 修复 `width` 以 `vw` 为单位时的显示错误。[#&#8203;11326](`ant-design/ant-design#11326)
  - 🐞 修复 `wrapClassName` 属性无效的问题。
- 🐞 修复 Tooltip 文字溢出的问题。[#&#8203;11402](`ant-design/ant-design#11402) [@&#8203;weidapao]
- 🐞 修复 Menu 在 `theme` 为 `dark` 是在 Layout.Header 里的样式问题。[#&#8203;11400](`ant-design/ant-design#11400) [@&#8203;hongxuWei]
- 🐞 修复 InputNumber 的箭头按钮在使用了固定列的 Table 里显示错位的问题。[#&#8203;11408](`ant-design/ant-design#11408)
- 🐞 修复 Select 使用分组时 Option 的圆角显示错误。[6cb6f5c](ant-design/ant-design@6cb6f5c)
- 🐞 修复 Table 第一次点击过滤按钮的时候 `onChange` 会被触发两次的问题。[#&#8203;11164](`ant-design/ant-design#11164) [@&#8203;adybionka]
- 🐞 修复 Model.confirm 的标题在 Firefox 下会显示滚动条的问题。[#&#8203;11432](`ant-design/ant-design#11432)
- TypeScript
  - 🐞 修复 Radio.Group 类型定义。[#&#8203;11409](`ant-design/ant-design#11409) [@&#8203;eddiemoore]
  - 🐞 修复 TreeSelect 类型定义。[#&#8203;11442](`ant-design/ant-design#11442) [@&#8203;JribiBelhassen]
  - 🐞 修复 Badge 类型定义。[#&#8203;11421](`ant-design/ant-design#11421) [@&#8203;zongzi531]

---

</details>




---

This PR has been generated by [Renovate Bot](https://renovatebot.com).

Co-authored-by: Renovate Bot <bot@renovateapp.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants