Merge pull request #223 from jayjun/2.0

Add parameters to DELETE requests (#222)

.gitignore

@@ -4,3 +4,4 @@ erl_crash.dump
 *.ez
 config/config.secret.exs
 /doc
+.DS_Store

lib/stripe/card.ex

@@ -139,6 +139,6 @@ defmodule Stripe.Card do
   @spec delete(source, String.t, String.t, Keyword.t) :: :ok | {:error, Stripe.api_error_struct}
   def delete(owner_type, owner_id, card_id, opts \\ []) do
     endpoint = endpoint_for_owner(owner_type, owner_id) <> "/" <> card_id
-    Stripe.Request.delete(endpoint, opts)
+    Stripe.Request.delete(endpoint, %{}, opts)
   end
 end

lib/stripe/customer.ex

@@ -97,6 +97,6 @@ defmodule Stripe.Customer do
   @spec delete(binary, list) :: :ok | {:error, Stripe.api_error_struct}
   def delete(id, opts \\ []) do
     endpoint = @plural_endpoint <> "/" <> id
-    Stripe.Request.delete(endpoint, opts)
+    Stripe.Request.delete(endpoint, %{}, opts)
   end
 end

lib/stripe/external_account.ex

@@ -86,6 +86,6 @@ defmodule Stripe.ExternalAccount do
   @spec delete(binary, list) :: :ok | {:error, Stripe.api_error_struct}
   def delete(id, opts = [connect_account: managed_account_id]) do
     endpoint = endpoint(managed_account_id) <> "/" <> id
-    Stripe.Request.delete(endpoint, opts)
+    Stripe.Request.delete(endpoint, %{}, opts)
   end
 end

lib/stripe/plan.ex

@@ -77,6 +77,6 @@ defmodule Stripe.Plan do
   @spec delete(binary, list) :: :ok | {:error, Stripe.api_error_struct}
   def delete(id, opts \\ []) do
     endpoint = @plural_endpoint <> "/" <> id
-    Stripe.Request.delete(endpoint, opts)
+    Stripe.Request.delete(endpoint, %{}, opts)
   end
 end

lib/stripe/request.ex

@@ -41,9 +41,9 @@ defmodule Stripe.Request do
     |> handle_result
   end
 
-  @spec delete(String.t, Keyword.t) :: :ok | {:error, Stripe.api_error_struct}
-  def delete(endpoint, opts) do
-    %{}
+  @spec delete(String.t, map, Keyword.t) :: :ok | {:error, Stripe.api_error_struct}
+  def delete(endpoint, params, opts) do
+    params
     |> Stripe.request(:delete, endpoint, %{}, opts)
     |> handle_result
   end

lib/stripe/subscription.ex

@@ -87,10 +87,12 @@ defmodule Stripe.Subscription do
 
   @doc """
   Delete a subscription.
+
+  Takes the `id` and an optional map of `params`.
   """
-  @spec delete(binary, list) :: :ok | {:error, Stripe.api_error_struct}
-  def delete(id, opts \\ []) do
+  @spec delete(binary, map, list) :: :ok | {:error, Stripe.api_error_struct}
+  def delete(id, params \\ %{}, opts \\ []) do
     endpoint = @plural_endpoint <> "/" <> id
-    Stripe.Request.delete(endpoint, opts)
+    Stripe.Request.delete(endpoint, params, opts)
   end
 end

mix.exs

@@ -15,7 +15,7 @@ defmodule Stripe.Mixfile do
         "coveralls.html": :test
       ],
       test_coverage: [tool: ExCoveralls],
-      version: "2.0.0-alpha.6"
+      version: "2.0.0-alpha.7"
     ]
   end