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.

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

Call object.dump in rendered for Dumpable instances #27

Merged
merged 2 commits into from
Feb 18, 2016

Conversation

blanquer
Copy link
Contributor

When fully dumping an object (i.e., when no subfields were selected), that is not a Blueprint, ensure we call object.dump if they have the Attributor::Dumpable module.

In other words, attributor types (custom or not) will need to include the Attributor::Dumpable module and properly implement the dump instance method to produce the right output with native ruby objects.

fixes #26

Signed-off-by: Josep M. Blanquer blanquer@rightscale.com

When fully dumping an object (i.e., when no subfields were selected), that is not a Blueprint, ensure we call `object.dump` if they have the Attributor::Dumpable module.


Signed-off-by: Josep M. Blanquer <blanquer@rightscale.com>
@blanquer blanquer force-pushed the 26_utilize_attributor_dumpable_while_rendering branch from 2a64bd0 to c1dd6cd Compare January 27, 2016 00:30
@careo
Copy link
Contributor

careo commented Jan 27, 2016

LGTM, pending rightscale/attributor#158

Signed-off-by: Dane Jensen <dane@rightscale.com>
@careo careo force-pushed the 26_utilize_attributor_dumpable_while_rendering branch from cff2489 to beec198 Compare February 18, 2016 22:20
careo pushed a commit that referenced this pull request Feb 18, 2016
…_while_rendering

Call object.dump in rendered for Dumpable instances
@careo careo merged commit 9c89d55 into master Feb 18, 2016
@blanquer blanquer deleted the 26_utilize_attributor_dumpable_while_rendering branch August 4, 2017 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Custom type using Attributor::Type not serialized properly
2 participants