Rename FlattenJson to Attributes (allow plural adapter names)

Author: bf4

README.md

@@ -10,7 +10,7 @@ AMS does this through two components: **serializers** and **adapters**.
 Serializers describe _which_ attributes and relationships should be serialized.
 Adapters describe _how_ attributes and relationships should be serialized.
 
-By default AMS will use the **Flatten Json Adapter**. But we strongly advise you to use **JsonApi Adapter** that follows 1.0 of the format specified in [jsonapi.org/format](http://jsonapi.org/format).
+By default AMS will use the **Attributes Adapter**. But we strongly advise you to use **JsonApi Adapter** that follows 1.0 of the format specified in [jsonapi.org/format](http://jsonapi.org/format).
 Check how to change the adapter in the sections bellow.
 
 # RELEASE CANDIDATE, PLEASE READ
@@ -68,7 +68,7 @@ ActiveModel::Serializer.config.adapter = :json_api
 You won't need to implement an adapter unless you wish to use a new format or
 media type with AMS.
 
-If you want to have a root key on your responses you should use the Json adapter, instead of the default FlattenJson:
+If you want to have a root key on your responses you should use the Json adapter, instead of the default Attributes:
 
 ```ruby
 ActiveModel::Serializer.config.adapter = :json
@@ -138,7 +138,7 @@ The key can be customized using `meta_key` option.
 render json: @post, meta: { total: 10 }, meta_key: "custom_meta"
 ```
 
-`meta` will only be included in your response if you are using an Adapter that supports `root`, as JsonAPI and Json adapters, the default adapter (FlattenJson) doesn't have `root`.
+`meta` will only be included in your response if you are using an Adapter that supports `root`, as JsonAPI and Json adapters, the default adapter (Attributes) doesn't have `root`.
 
 ### Overriding association methods
 
@@ -174,7 +174,7 @@ end
 
 ### Built in Adapters
 
-#### FlattenJSON
+#### Attributes
 
 It's the default adapter, it generates a json response without a root key.
 Doesn't follow any specifc convention.

docs/general/adapters.md

@@ -3,11 +3,11 @@
 AMS does this through two components: **serializers** and **adapters**.
 Serializers describe _which_ attributes and relationships should be serialized.
 Adapters describe _how_ attributes and relationships should be serialized.
-You can use one of the built-in adapters (```FlattenJSON``` is the default one) or create one by yourself, but you won't need to implement an adapter unless you wish to use a new format or media type with AMS.
+You can use one of the built-in adapters (```Attributes``` is the default one) or create one by yourself, but you won't need to implement an adapter unless you wish to use a new format or media type with AMS.
 
 ## Built in Adapters
 
-### FlattenJSON - Default
+### Attributes - Default
 
 It's the default adapter, it generates a json response without a root key.
 Doesn't follow any specifc convention.
@@ -44,7 +44,7 @@ or
 ActiveModel::Serializer.config.adapter = :json_api
 ```
 
-If you want to have a root key in your responses you should use the Json adapter, instead of the default FlattenJson:
+If you want to have a root key in your responses you should use the Json adapter, instead of the default Attributes:
 
 ```ruby
 ActiveModel::Serializer.config.adapter = :json

docs/howto/add_pagination_links.md

@@ -107,6 +107,6 @@ ex.
 }
 ```
 
-### FlattenJSON adapter
+### Attributes adapter
 
 This adapter does not allow us to use `meta` key, due to that it is not possible to add pagination links.

docs/howto/add_root_key.md

@@ -1,6 +1,6 @@
 # How to add root key
 
-Add the root key to your API is quite simple with AMS. The **Adapter** is what determines the format of your JSON response. The default adapter is the ```FlattenJSON``` which doesn't have the root key, so your response is something similar to:
+Add the root key to your API is quite simple with AMS. The **Adapter** is what determines the format of your JSON response. The default adapter is the ```Attributes``` which doesn't have the root key, so your response is something similar to:
 
 ```json
 {

lib/active_model/serializer/adapter.rb

@@ -5,7 +5,7 @@ class Adapter
       autoload :CacheCheck
       require 'active_model/serializer/adapter/json'
       require 'active_model/serializer/adapter/json_api'
-      autoload :FlattenJson
+      autoload :Attributes
       autoload :Null
       autoload :FragmentCache
 
@@ -19,7 +19,8 @@ def self.create(resource, options = {})
 
       def self.adapter_class(adapter)
         adapter_name = adapter.to_s.classify.sub("API", "Api")
-        "ActiveModel::Serializer::Adapter::#{adapter_name}".safe_constantize
+        "ActiveModel::Serializer::Adapter::#{adapter_name}".safe_constantize ||
+          "ActiveModel::Serializer::Adapter::#{adapter_name.pluralize}".safe_constantize
       end
 
       attr_reader :serializer

lib/active_model/serializer/adapter/flatten_json.rb renamed to lib/active_model/serializer/adapter/attributes.rb

@@ -1,12 +1,12 @@
 module ActiveModel
   class Serializer
     class Adapter
-      class FlattenJson < Adapter
+      class Attributes < Adapter
 
         def serializable_hash(options = nil)
           options ||= {}
           if serializer.respond_to?(:each)
-            @result = serializer.map { |s| FlattenJson.new(s).serializable_hash(options) }
+            @result = serializer.map { |s| Attributes.new(s).serializable_hash(options) }
           else
             @hash = {}
 
@@ -47,7 +47,7 @@ def fragment_cache(cached_hash, non_cached_hash)
 
         private
 
-        # no-op: FlattenJson adapter does not include meta data, because it does not support root.
+        # no-op: Attributes adapter does not include meta data, because it does not support root.
         def include_meta(json)
           json
         end

lib/active_model/serializer/adapter/json.rb

@@ -6,7 +6,7 @@ class Adapter
       class Json < Adapter
 
         def serializable_hash(options = {})
-          { root => FlattenJson.new(serializer).serializable_hash(options) }
+          { root => Adapter::Attributes.new(serializer).serializable_hash(options) }
         end
 
         private

lib/active_model/serializer/configuration.rb

@@ -6,7 +6,7 @@ module Configuration
 
       included do |base|
         base.config.array_serializer = ActiveModel::Serializer::ArraySerializer
-        base.config.adapter = :flatten_json
+        base.config.adapter = :attributes
         base.config.jsonapi_resource_type = :plural
       end
     end

test/adapter_test.rb

@@ -31,7 +31,7 @@ def test_adapter_class_for_unknown_adapter
 
       def test_create_adapter
         adapter = ActiveModel::Serializer::Adapter.create(@serializer)
-        assert_equal ActiveModel::Serializer::Adapter::FlattenJson, adapter.class
+        assert_equal ActiveModel::Serializer::Adapter::Attributes, adapter.class
       end
 
       def test_create_adapter_with_override

test/serializers/adapter_for_test.rb

@@ -11,7 +11,7 @@ def teardown
 
       def test_returns_default_adapter
         adapter = ActiveModel::Serializer.adapter
-        assert_equal ActiveModel::Serializer::Adapter::FlattenJson, adapter
+        assert_equal ActiveModel::Serializer::Adapter::Attributes, adapter
       end
 
       def test_overwrite_adapter_with_symbol

test/serializers/attribute_test.rb

@@ -21,7 +21,7 @@ def test_json_serializable_hash
       def test_attribute_inheritance_with_key
         inherited_klass = Class.new(AlternateBlogSerializer)
         blog_serializer = inherited_klass.new(@blog)
-        adapter = ActiveModel::Serializer::Adapter::FlattenJson.new(blog_serializer)
+        adapter = ActiveModel::Serializer::Adapter::Attributes.new(blog_serializer)
         assert_equal({:id=>1, :title=>"AMS Hints"}, adapter.serializable_hash)
       end
 

test/serializers/configuration_test.rb

@@ -8,7 +8,7 @@ def test_array_serializer
       end
 
       def test_default_adapter
-        assert_equal :flatten_json, ActiveModel::Serializer.config.adapter
+        assert_equal :attributes, ActiveModel::Serializer.config.adapter
       end
     end
   end

test/serializers/meta_test.rb

@@ -27,7 +27,7 @@ def test_meta_is_present_with_root
       end
 
       def test_meta_is_not_included_when_root_is_missing
-        # load_adapter uses FlattenJson Adapter
+        # load_adapter uses Attributes Adapter
         adapter = load_adapter(meta: {total: 10})
         expected = {
           id: 1,
@@ -67,8 +67,8 @@ def test_meta_key_is_used_with_json_api
 
       def test_meta_is_not_present_on_arrays_without_root
         serializer = ArraySerializer.new([@blog], meta: {total: 10})
-        # FlattenJSON doesn't have support to root
-        adapter = ActiveModel::Serializer::Adapter::FlattenJson.new(serializer)
+        # Attributes doesn't have support to root
+        adapter = ActiveModel::Serializer::Adapter::Attributes.new(serializer)
         expected = [{
           id: 1,
           name: "AMS Hints",
@@ -113,7 +113,7 @@ def test_meta_is_present_on_arrays_with_root
       private
 
       def load_adapter(options)
-        options = options.merge(adapter: :flatten_json, serializer: AlternateBlogSerializer)
+        options = options.merge(adapter: :attributes, serializer: AlternateBlogSerializer)
         ActiveModel::SerializableResource.new(@blog, options)
       end
     end