Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

Make sure that schema#getAttributeProperties() always return an object #1718

Merged
merged 2 commits into from
Apr 3, 2019

Conversation

mlewand
Copy link
Contributor

@mlewand mlewand commented Apr 1, 2019

Suggested merge commit message (convention)

Fix: Make sure that schema#getAttributeProperties() always return an object. Closes ckeditor/ckeditor5#4503.


Additional information

This fixes a behavior that has not been officially released yet, so it might be reasonable to skip this in the changelog.

Without this there's a potential for making references to undefined value.

…object.

NOTE: This fixes a behavior that has not been officially released yet, so it might be reasonable to skip this in the changelog.
@coveralls
Copy link

coveralls commented Apr 1, 2019

Coverage Status

Coverage remained the same at 100.0% when pulling bd3d37d on t/1717 into c48f5a4 on master.

@jodator
Copy link
Contributor

jodator commented Apr 1, 2019

Two uncertain things:

@mlewand mlewand changed the title Make sure that schema#getAttributeProperties() always return an object. Make sure that schema#getAttributeProperties() always return an object Apr 3, 2019
It is no longer needed as schema#getAttributeProperties() guarantees returning proper value.
Copy link
Contributor

@jodator jodator left a comment

Choose a reason for hiding this comment

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

So after agreeing on returning {} this LGTM.

@jodator jodator merged commit b3f5da3 into master Apr 3, 2019
@jodator jodator deleted the t/1717 branch April 3, 2019 09:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Method schema#getAttributeProperties() should always return an object
3 participants