Skip to content
This repository has been archived by the owner on Jul 16, 2023. It is now read-only.

Adding doc on how to auto hydrate fields when updating a record #78

Closed
wants to merge 1 commit into from

Conversation

warrenca
Copy link

This resolves #77

@igorsantos07
Copy link
Member

Thanks for the PR!

However, I don't think I'll accept it.
You did a great job documenting the property, but I don't think it should
even exist, at least from a user point of view.
I'll take a closer look at this later and see if I can remove it (merging
the functionality of both properties into one).

Any idea is welcomed (:
On Jul 25, 2013 11:31 AM, "wcabundo" [email protected] wrote:

This resolves #77 #77

You can merge this Pull Request by running

git pull https://github.com/wcabundo/ardent master

Or view, comment on, or merge it at:

#78
Commit Summary

  • Adding doc on how to auto hydrate fields when updating a record.

File Changes

Patch Links:

@igorsantos07
Copy link
Member

@wcabundo I re-read the source code and I will add documentation about this property, just like you suggested!
I will, however, change the wording since it's actually a feature to prevent overwrite of properties, and thus I'll close this PR and write it myself.

Anyway, thank you for the PR!

@warrenca
Copy link
Author

warrenca commented Aug 2, 2013

Thanks!

igorsantos07 added a commit that referenced this pull request Aug 2, 2013
The old method relied on validation rules, meaning that properties that were not validated would not be populated when the model is configured to be automatically hydrated.
Documentation was changed to reflect this behavior, and also to explicitly explain the differences between forced and automatic hydration.
This references #77 and #78 and closes #51
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.

autoHydrateEntityFromInput not working on update?
2 participants