Permalink
Browse files

`ActiveRecord::Base.include_root_in_json` is `false` by default.

Closes #9459.

The PR #6597 unified the configuration for `include_root_in_json`
in AM and AR to `false`.
Later on with the refactoring commit: e030f26 the value in AR was
set to `true` but I think this was not on purpose.

With this commit both AM and AR will have the same configuration
for `include_root_in_json`, which is `false`.
  • Loading branch information...
1 parent eeba679 commit 8c7d4018e0ddec462075027defb3575c47952c3e @senny senny committed Mar 4, 2013
@@ -5,7 +5,7 @@ module Serialization
include ActiveModel::Serializers::JSON
included do
- self.include_root_in_json = true
+ self.include_root_in_json = false
end
def serializable_hash(options = nil)
@@ -18,6 +18,10 @@ def setup
}
end
+ def test_include_root_in_json_is_false_by_default
+ assert_equal false, ActiveRecord::Base.include_root_in_json, "include_root_in_json should be false by default but was not"
+ end
+
def test_serialize_should_be_reversible
FORMATS.each do |format|
@serialized = Contact.new.send("to_#{format}")

0 comments on commit 8c7d401

Please sign in to comment.