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

Fixed notice on binding entity to form #5928

Closed
wants to merge 4 commits into from
Closed

Fixed notice on binding entity to form #5928

wants to merge 4 commits into from

Conversation

Ponsjuh
Copy link
Contributor

@Ponsjuh Ponsjuh commented Mar 6, 2014

Notice: Undefined index: year
Notice: Undefined index: month
Notice: Undefined index: day
Notice: Undefined index: hour
Notice: Undefined index: minute

When you bind a Entity, with a property set to NULL, to a form it then sets the element to NULL. This will break the generation of the DateTimeSelect

A notice is thrown when you bind a Entity to a form that has a default value of NULL
@weierophinney weierophinney added this to the 2.3.0 milestone Mar 7, 2014
@weierophinney weierophinney self-assigned this Mar 7, 2014
weierophinney added a commit that referenced this pull request Mar 7, 2014
Fixed notice on binding entity to form
weierophinney added a commit that referenced this pull request Mar 7, 2014
- Added unit test for patch; failed prior to applying patch, passes afterwards.
weierophinney added a commit that referenced this pull request Mar 7, 2014
@weierophinney
Copy link
Member

Merged to develop for release with 2.3.0; unit tests added as well.

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.

2 participants