# Patch review requested for #9677: Add :include to ActiveRecord::Base#to\_json

**URL:** <https://discuss.rubyonrails.org/t/patch-review-requested-for-9677-add-include-to-activerecord-base-to-json/18539>\
**Category:** rubyonrails-core\
**Created:** [September 25, 2007, 4:27pm UTC](https://discuss.rubyonrails.org/t/patch-review-requested-for-9677-add-include-to-activerecord-base-to-json/18539 "2007-09-25T16:27:27Z")\
**Posts on this page:** 2\
**Page:** 1

<div class="post-metadata">

**Author:** ![Cheah\_Chu\_Yeow](https://avatars.discourse-cdn.com/v4/letter/c/9de0a6/32.png) [@Cheah\_Chu\_Yeow](https://discuss.rubyonrails.org/u/Cheah_Chu_Yeow)\
**Post date:** [September 25, 2007, 4:27pm UTC](https://discuss.rubyonrails.org/t/patch-review-requested-for-9677-add-include-to-activerecord-base-to-json/18539/1 "2007-09-25T16:27:27Z")

</div>

Simple patch really, that adds :include option to ActiveRecord::Base#to\_json like for ActiveRecord::Base#to\_xml:

[http://dev.rubyonrails.org/ticket/9677](http://dev.rubyonrails.org/ticket/9677)

Allows you to do:

json = @david.to\_json(:include =\> :posts) # or even... json = @david.to\_json( &nbsp;&nbsp;:include =\> { &nbsp;&nbsp;&nbsp;&nbsp;:posts =\> { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;:include =\> { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;:taggings =\> { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;:include =\> { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;:tag =\> { :only =\> :name } &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp;} })

Please use the 2nd diff ([http://dev.rubyonrails.org/attachment/ticket/](http://dev.rubyonrails.org/attachment/ticket/) 9677/add\_includes\_to\_to\_json.2.diff) - I forgot to replace the existing diff when uploading a small fix.

Thanks!

Cheers, Chu Yeow

---

<div class="post-metadata">

**Author:** ![Cheah\_Chu\_Yeow](https://avatars.discourse-cdn.com/v4/letter/c/9de0a6/32.png) [@Cheah\_Chu\_Yeow](https://discuss.rubyonrails.org/u/Cheah_Chu_Yeow)\
**Post date:** [September 26, 2007, 4:31am UTC](https://discuss.rubyonrails.org/t/patch-review-requested-for-9677-add-include-to-activerecord-base-to-json/18539/2 "2007-09-26T04:31:15Z")

</div>

I've put up a new patch (see [http://dev.rubyonrails.org/ticket/9677](http://dev.rubyonrails.org/ticket/9677)) on David's suggestion to move add\_includes up into the Serializer superclass. Please help test. Thanks!

Cheers, Chu Yeow