Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was 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
weierophinney merged 1 commit into
zendframework:masterfrom
sjuvonen:master
Jan 13, 2015
Merged

Improved disabled inputs testing when binding values into fieldset object#7109
weierophinney merged 1 commit into
zendframework:masterfrom
sjuvonen:master

Conversation

@sjuvonen
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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