Skip to content

Commit 8e7dc1f

Browse files
committed
Added returns back in to keep the operations code consistent with the other operations that need the returns.
1 parent a10814d commit 8e7dc1f

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

lib/jsonapi/operation.rb

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ def apply
214214
resource = @resource_klass.find_by_key(@resource_id, context: @context)
215215
result = resource.replace_fields(data)
216216

217-
JSONAPI::ResourceOperationResult.new(result == :completed ? :ok : :accepted, resource)
217+
return JSONAPI::ResourceOperationResult.new(result == :completed ? :ok : :accepted, resource)
218218
end
219219
end
220220

@@ -232,7 +232,7 @@ def apply
232232
resource = @resource_klass.find_by_key(@resource_id, context: @context)
233233
result = resource.replace_has_one_link(@association_type, @key_value)
234234

235-
JSONAPI::OperationResult.new(result == :completed ? :no_content : :accepted)
235+
return JSONAPI::OperationResult.new(result == :completed ? :no_content : :accepted)
236236
end
237237
end
238238

@@ -250,7 +250,7 @@ def apply
250250
resource = @resource_klass.find_by_key(@resource_id, context: @context)
251251
result = resource.create_has_many_links(@association_type, @data)
252252

253-
JSONAPI::OperationResult.new(result == :completed ? :no_content : :accepted)
253+
return JSONAPI::OperationResult.new(result == :completed ? :no_content : :accepted)
254254
end
255255
end
256256

@@ -268,7 +268,7 @@ def apply
268268
resource = @resource_klass.find_by_key(@resource_id, context: @context)
269269
result = resource.replace_has_many_links(@association_type, @data)
270270

271-
JSONAPI::OperationResult.new(result == :completed ? :no_content : :accepted)
271+
return JSONAPI::OperationResult.new(result == :completed ? :no_content : :accepted)
272272
end
273273
end
274274

@@ -286,7 +286,7 @@ def apply
286286
resource = @resource_klass.find_by_key(@resource_id, context: @context)
287287
result = resource.remove_has_many_link(@association_type, @associated_key)
288288

289-
JSONAPI::OperationResult.new(result == :completed ? :no_content : :accepted)
289+
return JSONAPI::OperationResult.new(result == :completed ? :no_content : :accepted)
290290
end
291291
end
292292

@@ -303,7 +303,7 @@ def apply
303303
resource = @resource_klass.find_by_key(@resource_id, context: @context)
304304
result = resource.remove_has_one_link(@association_type)
305305

306-
JSONAPI::OperationResult.new(result == :completed ? :no_content : :accepted)
306+
return JSONAPI::OperationResult.new(result == :completed ? :no_content : :accepted)
307307
end
308308
end
309309
end

0 commit comments

Comments
 (0)