New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecated default_value key from entity_value #724

Merged
merged 1 commit into from Jul 2, 2015

Conversation

Projects
None yet
2 participants
@artfulrobot
Copy link

artfulrobot commented Jul 1, 2015

The correct way to set a default value on an entity is to use setDefaultValue() and not to pass a default_value key in the base table defs.

See related patch to examples module.

Nb. I think it is useful to keep this code (which sets a zero-length string as the default) in there as an example of how to provide a default.

Rich Lott

@jmolivas jmolivas modified the milestone: 0.7.15-release Jul 1, 2015

jmolivas added a commit that referenced this pull request Jul 2, 2015

Merge pull request #724 from artfulrobot/master
Remove deprecated default_value key from entity_value

@jmolivas jmolivas merged commit e9fbfbc into hechoendrupal:master Jul 2, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@jmolivas

This comment has been minimized.

Copy link
Member

jmolivas commented Jul 2, 2015

Thanks for the PR @artfulrobot

@jmolivas jmolivas added the enhancement label Jul 2, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment