Skip to content
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

Jump to bottom

Hydrate attribute with NULL #28

Closed
qpautrat opened this issue Jun 30, 2016 · 2 comments
Closed

Hydrate attribute with NULL #28

qpautrat opened this issue Jun 30, 2016 · 2 comments

Comments

@qpautrat
Copy link
Contributor

Hi fellows,

I need your POV on this one.
Correct me if i'm wrong but I should be able to set an attribute to my domain object to null if want to.
But as we can see there hydration will be forgotten.
Thereby I'm not able to delete my field description for example.

Thx.
@kocsismate
Copy link
Member

kocsismate commented Jul 5, 2016 •
edited
Loading

Yes, you are absolutely right! Really important observation, so thank you very much!

Do you agree with using the array_key_exists() function instead of isset()? I think this is the correct fix for it.

@qpautrat qpautrat mentioned this issue Jul 5, 2016
Merged
@qpautrat
Copy link
Contributor Author

qpautrat commented Jul 5, 2016

PR open @kocsismate !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@qpautrat @kocsismate