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

Select 选择器 多选搜索框(使用nzKeepUnListOptions属性的情况下)数据重复BUG #590

Closed
ferqx opened this issue Nov 20, 2017 · 4 comments · Fixed by #609
Labels

Comments

@ferqx
Copy link

ferqx commented Nov 20, 2017

  1. 版本:自0.5.0-rc.0以来就一直存在,在此十分抱歉没有及时的反馈

  2. 代码地址:https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/src/components/select/nz-select.component.ts

  3. 重现步骤:

使用nz-select组件添加nzKeepUnListOptions属性时,并存在默认数据的情况下,如以下情况:
1b5b557c-20ea-4b4e-a7be-c122ce73c7bd

这时我再次输入与默认数据相关的文字时
qq20171120-173657

并回车
Uploading QQ20171120-173722.png…
这时搜索相关的数据时数据再次出现,并增加相同数据

  1. 原因:

qq20171120-174910
在执行 updateSelectedOption 方法时,该红框内的 Set 对象 has 方法无法正确的过滤对象进行判断

  1. 个人解决方案:

qq20171120-180339

@ng-zorro-bot
Copy link
Collaborator

Hello @ferqx, your issue has been closed because it does not conform to our issue requirements. Please use the Issue Helper to create an issue, thank you!

@ng-zorro-bot
Copy link
Collaborator

It will be better to write your issue/comment in English, so more people can understand you.
And this means that more people can help you or benefit from your issue/comment.

@vthinkxie
Copy link
Member

thanks for your feedback, will be fixed in next version.

@lock
Copy link

lock bot commented Feb 18, 2019

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked as resolved and limited conversation to collaborators Feb 18, 2019
hsuanxyz pushed a commit to hsuanxyz/ng-zorro-antd that referenced this issue Aug 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants