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.

Sign up for GitHub

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

avoid blank hash in embeded parameter #321

Open

y00rb wants to merge 3 commits into remi:master from SeniorAdvisor:master

y00rb commented Mar 9, 2015

Hi Guys

improve the generate embedded attributes(has_many relation).if the has_many value is blank, will make errors in embeded_params, such as TypeError: no implicit conversion of nil into Hash, NoMethodError: undefined method[]=' for nil:NilClass`, so I have add method to make default in inject, set blank hash.
added has_one in to_params


avoid blank hash in embeded

bd429d7

if pass blank in embeded(has_many) will get error.

Author

y00rb commented Mar 9, 2015

@remiprev Could you checkout it? if it is merge, I can used new version in my project

xueshibing added 2 commits

March 10, 2015 15:56


to support has_one in embeded params

cc3ed1c


add test for has_one

eea10ee

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