Multi example in the Contexts doc might be outdated

[karloescota]

Environment

• Elixir version (elixir -v): Elixir 1.8.2 (compiled with Erlang/OTP 20)
• Phoenix version (mix deps): locked at 1.4.6 (phoenix)
• NodeJS version (node -v):
• NPM version (npm -v):
• Operating system:

Code in the documentation for Contexts under Strategies for cross-context workflows possibly outdated.

Was getting a failing test case at the latter part of the doc, the one with the Multi implementation.

Error:
** (FunctionClauseError) no function clause matching in Ecto.Multi.run/3

I checked the Multi.run doc and it says

receives the repo as the first argument, and the changes so far as the second argument

I updated the run code to Multi.run(:user, fn _, _ -> Accounts.create_user(params) end)
and it works.

Expected behavior

def register_account(attrs) do
  Multi.new()
  |> Multi.run(:account, fn _, _ ->
    IO.puts "do something here"
  end)
end

Actual behavior

def register_account(attrs) do
  Multi.new()
  |> Multi.run(:account, fn _ ->
    IO.puts "do something here"
  end)
end

Thank you!

[wojtekmach]

Good catch. Passing a 1-arity function to Multi.run was deprecated in Ecto 3.0 and removed in Ecto 3.1.