Skip to content

fix issue related to Object.keys and Dropdown issue#26291

Merged
Johann-S merged 1 commit intov4-devfrom
v4-fix-issues-johann
Apr 13, 2018
Merged

fix issue related to Object.keys and Dropdown issue#26291
Johann-S merged 1 commit intov4-devfrom
v4-fix-issues-johann

Conversation

@Johann-S
Copy link
Copy Markdown
Member

Fixes: #26266

/CC @XhmikosR

Comment thread js/src/collapse.js
...Default,
...$this.data(),
...typeof config === 'object' && config
...typeof config === 'object' && config ? config : {}
Copy link
Copy Markdown
Contributor

@vsn4ik vsn4ik Apr 13, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not

typeof config === 'object' ? config : {}

?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep that can be enough thanks 👍

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Finally I won't make this change because it'll decrease our code coverage

@Johann-S Johann-S merged commit ba10b63 into v4-dev Apr 13, 2018
@Johann-S Johann-S deleted the v4-fix-issues-johann branch April 13, 2018 16:59
@mdo mdo mentioned this pull request Apr 16, 2018
sesemaya added a commit to Daemonite/material that referenced this pull request Apr 18, 2018
@Johann-S Johann-S mentioned this pull request Apr 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants