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

[DateRangePicker] 如果赋值格式不符合 会报错 且点击组件 页面直接奔溃 #1451

Closed
Jiachang-Cai opened this issue Sep 2, 2022 · 7 comments · Fixed by #1453
Labels
to be published fixed, not be published

Comments

@Jiachang-Cai
Copy link

tdesign-vue 版本

0.46.3

重现链接

https://stackblitz.com/edit/angular-tmf6ug?file=src/demo.vue

重现步骤

可能存在 业务数据为Null 或空的情况
格式错误赋值会报错
Uploading image.png…
会报错 的情况下 点击组件 页面直接奔溃 卡死

期望结果

报错不影响组件使用 以及不要导致整个页面奔溃

实际结果

报错后点击 导致整个页面奔溃

框架版本

No response

浏览器版本

No response

系统版本

No response

Node版本

No response

补充说明

No response

@github-actions
Copy link
Contributor

github-actions bot commented Sep 2, 2022

👋 @Jiachang-Cai,感谢给 TDesign 提出了 issue。
请根据 issue 模版确保背景信息的完善,我们将调查并尽快回复你。

@Jiachang-Cai
Copy link
Author

Jiachang-Cai commented Sep 2, 2022

报错可以理解 但是组件无法使用 导致页面奔溃不理解

@honkinglin
Copy link
Collaborator

组件内部需要对数据格式化处理,dayjs并不支持空字符串的格式化,后续很多逻辑都依赖正确的时间格式做计算
我先兼容下空字符串卡死问题吧,但这种情况直接设置空数组就可以了,不明白为何还要穿空字符串

@Jiachang-Cai
Copy link
Author

Jiachang-Cai commented Sep 2, 2022

组件内部需要对数据格式化处理,dayjs并不支持空字符串的格式化,后续很多逻辑都依赖正确的时间格式做计算 我先兼容下空字符串卡死问题吧,但这种情况直接设置空数组就可以了,不明白为何还要穿空字符串

你要解决的是 不符合 格式的的赋值 直接重置为 空数组?(反正就是不要奔溃) 有时候返回的可能是 null 等参数 不仅仅为字符串 我只是写个示例

@uyarn uyarn added the to be published fixed, not be published label Sep 4, 2022
@Jiachang-Cai
Copy link
Author

没有解决就关闭吗

@xiaosansiji
Copy link
Collaborator

没有解决就关闭吗

对应 PR 里有写 fix #xx 就会被 GitHub 自动关闭,见:#1453

PR 合并后我们会把 issue label 置为 to be published,每周的迭代版本发布后在 issue 里评论提示升级到哪个版本解决

@xiaosansiji
Copy link
Collaborator

已在 0.46.4 版本修复

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
to be published fixed, not be published
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants