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

Improved disabled inputs testing when binding values into fieldset object #7109

Merged
merged 1 commit into from
Jan 13, 2015

Conversation

sjuvonen
Copy link
Contributor

@sjuvonen sjuvonen commented Jan 8, 2015

It's not enough to test that 'disabled' attribute exists, should also check that its value is something else than null or false or empty.

It's not enough to test that 'disabled' attribute exists, should also check that its value is something else than null or false or empty.
@Ocramius
Copy link
Member

Ocramius commented Jan 8, 2015

Actually, this requires a test case as well.

weierophinney added a commit that referenced this pull request Jan 13, 2015
Improved disabled inputs testing when binding values into fieldset object
weierophinney added a commit that referenced this pull request Jan 13, 2015
- Added unit test for behavior added with this pull request
weierophinney added a commit that referenced this pull request Jan 13, 2015
@weierophinney weierophinney merged commit ff04baf into zendframework:master Jan 13, 2015
weierophinney added a commit that referenced this pull request Jan 13, 2015
@weierophinney
Copy link
Member

I've written and committed a test case to validate the change.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants