Skip to content

Commit 298fe50

Browse files
authored
Merge pull request #44 from richmolj/update-jsonapi-serializable
Update jsonapi serializable
2 parents d5fdfbf + 6b071ec commit 298fe50

File tree

6 files changed

+16
-8
lines changed

6 files changed

+16
-8
lines changed

Appraisals

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
appraise "rails-4" do
22
gem "rails", "~> 4.1"
3-
gem 'jsonapi-rails', '~> 0.1.0', require: 'jsonapi/rails'
3+
gem 'jsonapi-rails', '~> 0.2.0', require: 'jsonapi/rails'
44
gem 'rspec-rails'
55
end
66

77
appraise "rails-5" do
88
gem "rails", "~> 5.0"
9-
gem 'jsonapi-rails', '~> 0.1.0', require: 'jsonapi/rails'
9+
gem 'jsonapi-rails', '~> 0.2.0', require: 'jsonapi/rails'
1010
gem 'rspec-rails'
1111
end

gemfiles/rails_4.gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
source "https://rubygems.org"
44

55
gem "rails", "~> 4.1"
6-
gem "jsonapi-rails", "~> 0.1.0", :require => "jsonapi/rails"
6+
gem "jsonapi-rails", "~> 0.2.0", :require => "jsonapi/rails"
77
gem "rspec-rails"
88

99
group :test do

gemfiles/rails_5.gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
source "https://rubygems.org"
44

55
gem "rails", "~> 5.0"
6-
gem "jsonapi-rails", "~> 0.1.0", :require => "jsonapi/rails"
6+
gem "jsonapi-rails", "~> 0.2.0", :require => "jsonapi/rails"
77
gem "rspec-rails"
88

99
group :test do

lib/jsonapi_compliable/base.rb

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,7 @@ def jsonapi_update
190190
# this, pass +scope: false+
191191
#
192192
# This builds relevant options and sends them to
193-
# +JSONAPI::Serializable::Renderer.render+from
193+
# +JSONAPI::Serializable::SuccessRenderer#render+from
194194
# {http://jsonapi-rb.org jsonapi-rb}
195195
#
196196
# @example Build Scope by Default
@@ -247,7 +247,9 @@ def force_includes?
247247
end
248248

249249
def perform_render_jsonapi(opts)
250-
JSONAPI::Serializable::Renderer.render(opts.delete(:jsonapi), opts)
250+
# TODO(beauby): Reuse renderer.
251+
JSONAPI::Serializable::SuccessRenderer.new
252+
.render(opts.delete(:jsonapi), opts).to_json
251253
end
252254

253255
def _persist

lib/jsonapi_compliable/extensions/extra_attribute.rb

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,12 @@ def extra_attribute(name, options = {}, &blk)
5757
end
5858

5959
JSONAPI::Serializable::Resource.class_eval do
60-
prepend JSONAPI::Serializable::Resource::ConditionalFields
60+
def self.inherited(klass)
61+
super
62+
klass.class_eval do
63+
extend JSONAPI::Serializable::Resource::ConditionalFields
64+
end
65+
end
66+
6167
include JsonapiCompliable::Extensions::ExtraAttribute
6268
end

spec/support/scope_helper.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ def render(object, opts = {})
1717
opts[:expose] = { context: resource.context }
1818
opts = JsonapiCompliable::Util::RenderOptions.generate(object, query.to_hash[:authors], opts)
1919
resolved = opts.delete(:jsonapi)
20-
raw_json = JSONAPI::Serializable::Renderer.render(resolved, opts)
20+
raw_json = JSONAPI::Serializable::SuccessRenderer.new.render(resolved, opts).to_json
2121
JSON.parse(raw_json)
2222
end
2323
end

0 commit comments

Comments
 (0)