Merge branch 'groyoh-follow_up_1535'

Needs followup
- way we deprecate classes to be generalized
  - initialize old json api #1543 (comment)
  - consider restoring ensure #1543 (comment)

Author: bf4

CHANGELOG.md

@@ -29,6 +29,9 @@ Fixes:
 - [#1488](https://github.com/rails-api/active_model_serializers/pull/1488) Require ActiveSupport's string inflections (@nate00)
 
 Misc:
+- [#1543](https://github.com/rails-api/active_model_serializers/pull/1543) Add the changes missing from #1535. (@groyoh)
+- [#1535](https://github.com/rails-api/active_model_serializers/pull/1535) Move the adapter and adapter folder to
+  active_model_serializers folder and changes the module namespace. (@domitian @bf4)
 - [#1497](https://github.com/rails-api/active_model_serializers/pull/1497) Add JRuby-9000 to appveyor.yml(@corainchicago)
 
 ### v0.10.0.rc4 (2016/01/27 11:00 +00:00)

lib/active_model/serializer.rb

@@ -18,13 +18,15 @@
 # reified when subclassed to decorate a resource.
 module ActiveModel
   class Serializer
+    extend ActiveSupport::Autoload
     include Configuration
     include Associations
     include Attributes
     include Caching
     include Links
     include Meta
     include Type
+    autoload :Adapter
 
     # @param resource [ActiveRecord::Base, ActiveModelSerializers::Model]
     # @return [ActiveModel::Serializer]
@@ -46,7 +48,7 @@ def self.serializer_for(resource, options = {})
     # @see ActiveModelSerializers::Adapter.lookup
     # Deprecated
     def self.adapter
-      warn 'Calling adapter method in Serializer, please use the ActiveModelSerializers::configured_adapter'
+      warn 'Calling adapter method in Serializer, please use the ActiveModelSerializers::Adapter.configured_adapter'
       ActiveModelSerializers::Adapter.lookup(config.adapter)
     end
 

lib/active_model/serializer/adapter.rb

@@ -0,0 +1,49 @@
+module ActiveModel
+  class Serializer
+    # @deprecated Use ActiveModelSerializers::Adapter instead
+    module Adapter
+      class << self
+        def create(resource, options = {})
+          warn_deprecation
+          ActiveModelSerializers::Adapter.create(resource, options)
+        end
+
+        def adapter_class(adapter)
+          warn_deprecation
+          ActiveModelSerializers::Adapter.adapter_class(adapter)
+        end
+
+        def adapter_map
+          warn_deprecation
+          ActiveModelSerializers::Adapter.adapter_map
+        end
+
+        def adapters
+          warn_deprecation
+          ActiveModelSerializers::Adapter.adapters
+        end
+
+        def register(name, klass = name)
+          warn_deprecation
+          ActiveModelSerializers::Adapter.register(name, klass)
+        end
+
+        def lookup(adapter)
+          warn_deprecation
+          ActiveModelSerializers::Adapter.lookup(adapter)
+        end
+
+        def warn_deprecation
+          warn "Calling deprecated #{name} (#{__FILE__}) from #{caller[1..3].join(', ')}. Please use ActiveModelSerializers::Adapter"
+        end
+        private :warn_deprecation
+      end
+
+      require 'active_model/serializer/adapter/base'
+      require 'active_model/serializer/adapter/null'
+      require 'active_model/serializer/adapter/attributes'
+      require 'active_model/serializer/adapter/json'
+      require 'active_model/serializer/adapter/json_api'
+    end
+  end
+end

lib/active_model/serializer/adapter/attributes.rb

@@ -0,0 +1,14 @@
+require 'active_model_serializers/adapter/attributes'
+
+module ActiveModel
+  class Serializer
+    module Adapter
+      class Attributes < DelegateClass(ActiveModelSerializers::Adapter::Attributes)
+        def initialize(serializer, options = {})
+          warn "Calling deprecated #{self.class.name} (#{__FILE__}) from #{caller[0..2].join(', ')}. Please use #{self.class.name.sub('ActiveModel::Serializer', 'ActiveModelSerializers')}"
+          super(ActiveModelSerializers::Adapter::Attributes.new(serializer, options))
+        end
+      end
+    end
+  end
+end

lib/active_model/serializer/adapter/base.rb

@@ -0,0 +1,16 @@
+module ActiveModel
+  class Serializer
+    module Adapter
+      class Base < DelegateClass(ActiveModelSerializers::Adapter::Base)
+        def self.inherited(base)
+          warn "Inheriting deprecated ActiveModel::Serializer::Adapter::Base in #{caller[0..2].join(', ')}. Please use ActiveModelSerializers::Adapter::Base"
+          super
+        end
+
+        def initialize(serializer, options = {})
+          super(ActiveModelSerializers::Adapter::Base.new(serializer, options))
+        end
+      end
+    end
+  end
+end

lib/active_model/serializer/adapter/json.rb

@@ -0,0 +1,14 @@
+require 'active_model_serializers/adapter/json'
+
+module ActiveModel
+  class Serializer
+    module Adapter
+      class Json < DelegateClass(ActiveModelSerializers::Adapter::Json)
+        def initialize(serializer, options = {})
+          warn "Calling deprecated #{self.class.name} (#{__FILE__}) from #{caller[0..2].join(', ')}. Please use #{self.class.name.sub('ActiveModel::Serializer', 'ActiveModelSerializers')}"
+          super(ActiveModelSerializers::Adapter::Json.new(serializer, options))
+        end
+      end
+    end
+  end
+end

lib/active_model/serializer/adapter/json_api.rb

@@ -0,0 +1,14 @@
+require 'active_model_serializers/adapter/json_api'
+
+module ActiveModel
+  class Serializer
+    module Adapter
+      class JsonApi < DelegateClass(ActiveModelSerializers::Adapter::JsonApi)
+        def initialize(serializer, options = {})
+          warn "Calling deprecated #{self.class.name} (#{__FILE__}) from #{caller[0..2].join(', ')}. Please use #{self.class.name.sub('ActiveModel::Serializer', 'ActiveModelSerializers')}"
+          super(ActiveModelSerializers::Adapter::JsonApi.new(serializer, options))
+        end
+      end
+    end
+  end
+end