flexirest 1.4.5 → 1.4.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b57d06388df46647b1e69e7f1673ae5a6dd36a5f
-  data.tar.gz: 669373fad9d9592266b3e362e0c565a46245c942
+  metadata.gz: f0179b1032c7c9c28be5895b6dc2f6268924469b
+  data.tar.gz: b2096625168b34e7dbe4509363546426cafd1899
 SHA512:
-  metadata.gz: '05659745e284dd34aaeed585cb19648e503df0851f6d680b687d9dc32322307b1ce560e07782cc15f532f2d7eaa9b265eed27d59a21b814a9ee90ec79ba0699f'
-  data.tar.gz: 7f55f9a8db580d9371b59e27b77fd8835228cbefd29e1384faf077a56bbe7f83026b52ef90c75de4df25b6eb13988616bdd1eee64ccbf913c0d0aabfa934d65b
+  metadata.gz: 6dd4e5f49b2ee856235101eb2a4039fb3200cbdcde7e5880346724a541c72452f3174a071684291decc91c16fa2c541b1086af984735ae634ff981849c4384a4
+  data.tar.gz: aa4c442cb8646e480249122168c50a8bf81748e4273d225526d08b8e39bf883d29a8a674e27b7e65f7f85183c4caa5fce605ffe2cf8ba861f150bae746c44a22

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 1.4.6
+
+Feature:
+
+- You can define how to handle the body of `delete` requests now with the mapping option `send_delete_body`.
+
 ## 1.4.5
 
 Bugfix:

data/README.md

@@ -94,6 +94,7 @@ class Person < Flexirest::Base
   get :find, "/people/:id"
   put :save, "/people/:id"
   post :create, "/people"
+  delete :remove, "/people/:id"
 end
 ```
 
@@ -129,6 +130,12 @@ id = @person.id
 end
 ```
 
+For `delete` requests whether an API can handle a body or not is undefined. The default is to ignore any parameters not sent in the URL named parameters, but you can optionally specify `send_delete_body` and it will encode them in your chosen way into the body.
+
+```
+  delete :remove, "/people/:id", send_delete_body: true
+```
+
 If an API returns an array of results and you have [will_paginate](https://rubygems.org/gems/will_paginate) installed then you can call the paginate method to return a particular page of the results (note: this doesn't reduce the load on the server, but it can help with pagination if you have a cached response).
 
 ```ruby

data/lib/flexirest/request.rb

@@ -249,6 +249,9 @@ module Flexirest
       if http_method == :get
         @get_params = default_params.merge(params || {})
         @post_params = nil
+      elsif http_method == :delete && @method[:options][:send_delete_body]
+        @post_params = default_params.merge(params || {})
+        @get_params = {}
       else
         @post_params = default_params.merge(params || {})
         @get_params = {}
@@ -335,7 +338,7 @@ module Flexirest
 
     def prepare_request_body(params = nil)
       if proxy == :json_api
-        if http_method == :get || http_method == :delete
+        if http_method == :get || (http_method == :delete && !@method[:options][:send_delete_body])
           @body = ""
         else
           headers["Content-Type"] ||= "application/vnd.api+json"
@@ -344,7 +347,7 @@ module Flexirest
 
         headers["Accept"] ||= "application/vnd.api+json"
         JsonAPIProxy::Headers.save(headers)
-      elsif http_method == :get || http_method == :delete
+      elsif http_method == :get || (http_method == :delete && !@method[:options][:send_delete_body])
         @body = ""
       elsif request_body_type == :form_encoded
         @body ||= (params || @post_params || {}).to_query

data/lib/flexirest/version.rb

@@ -1,3 +1,3 @@
 module Flexirest
-  VERSION = "1.4.5"
+  VERSION = "1.4.6"
 end

data/spec/lib/request_spec.rb

@@ -37,6 +37,7 @@ describe Flexirest::Request do
       put :update, "/put/:id"
       put :conversion, "/put/:id", parse_fields: [:converted]
       delete :remove, "/remove/:id"
+      delete :remove_body, "/remove/:id", send_delete_body: true
       get :hal, "/hal", fake:"{\"_links\":{\"child\": {\"href\": \"/child/1\"}, \"other\": {\"href\": \"/other/1\"}, \"cars\":[{\"href\": \"/car/1\", \"name\":\"car1\"}, {\"href\": \"/car/2\", \"name\":\"car2\"}, {\"href\": \"/car/not-embed\", \"name\":\"car_not_embed\"} ], \"lazy\": {\"href\": \"/lazy/load\"}, \"invalid\": [{\"href\": \"/invalid/1\"}]}, \"_embedded\":{\"other\":{\"name\":\"Jane\"},\"child\":{\"name\":\"Billy\"}, \"cars\":[{\"_links\": {\"self\": {\"href\": \"/car/1\"} }, \"make\": \"Bugatti\", \"model\": \"Veyron\"}, {\"_links\": {\"self\": {\"href\": \"/car/2\"} }, \"make\": \"Ferrari\", \"model\": \"F458 Italia\"} ], \"invalid\": [{\"present\":true, \"_links\": {} } ] } }", has_many:{other:ExampleOtherClient}
       get :fake, "/fake", fake:"{\"result\":true, \"list\":[1,2,3,{\"test\":true}], \"child\":{\"grandchild\":{\"test\":true}}}"
       get :fake_proc, "/fake", fake:->(request) { "{\"result\":#{request.get_params[:id]}}" }
@@ -150,6 +151,16 @@ describe Flexirest::Request do
     ExampleClient.remove(id:1)
   end
 
+  it "should get an HTTP connection when called and call delete with a body if send_delete_body is specified" do
+    expect_any_instance_of(Flexirest::Connection).to receive(:delete).with("/remove/1", "something=else", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:'{"result":true}', response_headers:{})))
+    ExampleClient.remove_body(id:1, something: "else")
+  end
+
+  it "should get an HTTP connection when called and call delete without a body if send_delete_body is not specified" do
+    expect_any_instance_of(Flexirest::Connection).to receive(:delete).with("/remove/1", "", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:'{"result":true}', response_headers:{})))
+    ExampleClient.remove(id:1, something: "else")
+  end
+
   it "should work with faraday response objects" do
     response = Faraday::Response.new
     allow(response).to receive(:body).and_return({}.to_json)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flexirest
 version: !ruby/object:Gem::Version
-  version: 1.4.5
+  version: 1.4.6
 platform: ruby
 authors:
 - Andy Jeffries
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-31 00:00:00.000000000 Z
+date: 2017-09-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler