Skip to content

Instantly share code, notes, and snippets.

@chrismccord
Last active November 14, 2018 10:56
Show Gist options
  • Save chrismccord/29100e16d3990469c47f851e3142f766 to your computer and use it in GitHub Desktop.
Save chrismccord/29100e16d3990469c47f851e3142f766 to your computer and use it in GitHub Desktop.

#Phoenix 1.1.x to 1.2.0 Upgrade Instructions

Project Generator

To generate new projects as 1.2.0, install the new mix archive:

mix archive.install https://github.com/phoenixframework/archives/raw/master/phoenix_new.ez

Deps

Update your phoenix dep for phoenix and include the extracted :phoenix_pubsub depdendency. If using Ecto, bump your :phoenix_ecto dependency as well:

# mix.exs
def deps do
  [{:phoenix, "~> 1.2.0"},
   {:phoenix_pubsub, "~> 1.0"},
   # if using Ecto:
   {:phoenix_ecto, "~> 3.0-rc"},
   ...]
end

Next, add :phoenix_pubsub to your :applications list in mix.exs:

  def application do
    [mod: {MyApp, []},
     applications: [:phoenix, :phoenix_pubsub, :phoenix_html, :cowboy, :logger, :gettext,
                    :phoenix_ecto, :postgrex]]
  end

Next, update your deps:

$ mix deps.update phoenix phoenix_pubsub phoenix_ecto phoenix_live_reload phoenix_html --unlock

Tests

The conn/0 helper imported into your tests has been deprecated. use build_conn/0 instead.

Ecto 2.0

If using Ecto, update your test_helper.exs:

ExUnit.start

- Mix.Task.run "ecto.create", ~w(-r Onetwo.Repo --quiet)
- Mix.Task.run "ecto.migrate", ~w(-r Onetwo.Repo --quiet)
- Ecto.Adapters.SQL.begin_test_transaction(Onetwo.Repo)
+ Ecto.Adapters.SQL.Sandbox.mode(MyAPp.Repo, :manual)

Next, add the following to config/config.exs:

config :my_app, ecto_repos: [MyApp.Repo]

Next, add a new test alias to your aliases in mix.exs:

  defp aliases do
    ["ecto.setup": ["ecto.create", "ecto.migrate", "run priv/repo/seeds.exs"],
     "ecto.reset": ["ecto.drop", "ecto.setup"],
+    "test": ["ecto.create --quiet", "ecto.migrate", "test"]]
  end

Next, update your setup blocks in test/support/model_case.ex and test/support/channel_case.ex:

  setup tags do
-   unless tags[:async] do
-     Ecto.Adapters.SQL.restart_test_transaction(MyApp.Repo, [])
-   end

+   :ok = Ecto.Adapters.SQL.Sandbox.checkout(MyApp.Repo)

+   unless tags[:async] do
+     Ecto.Adapters.SQL.Sandbox.mode(MyApp.Repo, {:shared, self()})
+   end

    :ok
  end

And update your test/support/conn_case.ex:

  setup tags do
-   unless tags[:async] do
-     Ecto.Adapters.SQL.restart_test_transaction(MyApp.Repo, [])
-   end

-   {:ok, conn: Phoenix.ConnTest.conn()}
+   :ok = Ecto.Adapters.SQL.Sandbox.checkout(MyApp.Repo)

+   unless tags[:async] do
+     Ecto.Adapters.SQL.Sandbox.mode(MyApp.Repo, {:shared, self()})
+   end

+   {:ok, conn: Phoenix.ConnTest.build_conn()}
  end

Passing :empty to cast has been deprecated. Pass an empty map in your changesets instead:

-  def changeset(struct, params \\ :empty) do
+  def changeset(struct, params \\ %{}) do

ErrorHelpers

Update your translate_error function in web/views/error_helpers.ex:

- @doc """
- Translates an error message using gettext.
- """
- def translate_error({msg, opts}) do
-   # Because error messages were defined within Ecto, we must
-   # call the Gettext module passing our Gettext backend. We
-   # also use the "errors" domain as translations are placed
-   # in the errors.po file. On your own code and templates,
-   # this could be written simply as:
-   #
-   #     dngettext "errors", "1 file", "%{count} files", count
-   #
-   Gettext.dngettext(MyApp.Gettext, "errors", msg, msg, opts[:count], opts)
- end

- def translate_error(msg) do
-   Gettext.dgettext(MyApp.Gettext, "errors", msg)
- end
+def translate_error({msg, opts}) do
+   # Because error messages were defined within Ecto, we must
+   # call the Gettext module passing our Gettext backend. We
+   # also use the "errors" domain as translations are placed
+   # in the errors.po file.
+   # Ecto will pass the :count keyword if the error message is
+   # meant to be pluralized.
+   # On your own code and templates, depending on whether you
+   # need the message to be pluralized or not, this could be
+   # written simply as:
+   #
+   #     dngettext "errors", "1 file", "%{count} files", count
+   #     dgettext "errors", "is invalid"
+   #
+   if count = opts[:count] do
+     Gettext.dngettext(MyApp.Gettext, "errors", msg, msg, count, opts)
+   else
+     Gettext.dgettext(MyApp.Gettext, "errors", msg, opts)
+   end
+ end

Next, update your test/support/model_case.ex's errors_on function:

-  def errors_on(model, data) do
-    model.__struct__.changeset(model, data).errors
-  end
  
+  def errors_on(struct, data) do
+    struct.__struct__.changeset(struct, data)
+    |> Ecto.Changeset.traverse_errors(&MyApp.ErrorHelpers.translate_error/1)
+    |> Enum.flat_map(fn {key, errors} -> for msg <- errors, do: {key, msg} end)
+  end

Configuration

Watcher

Using the :root endpoint configuration for watchers is deprecated and can be removed from config/config.exs. Instead, pass the :cd option at the end of your watcher argument list in config/dev.exs. For example:

watchers: [node: ["node_modules/brunch/bin/brunch", "watch", "--stdin",
           cd: Path.expand("../", __DIR__)]]

Namespace

Change app_namespace: in the application configuration to namespace:.

 config :my_app,
+  namespace: My.App
-  app_namespace: My.App

JavaScript Client

If using brunch, run npm update phoenix phoenix_html.

If vendoring the phoenix.js JavaScript client, grab a new copy: https://raw.githubusercontent.com/phoenixframework/phoenix/v1.2.0/web/static/js/phoenix.js

@kelvinst
Copy link

Update your phoenix dep for phoenix and include the extracted :phoenix_pubsub depdendency.

Just a little typo @chrismccord. :)

@silbermm
Copy link

silbermm commented Feb 4, 2017

I'm trying to update a working app from 1.1.x to 1.2.1 and after following these instructions I get

== Compilation error on file web/views/angular_view.ex==
** (CompileError) web/views/angular_view.ex:2: module CrossroadsInterface.Router.Helpers is not loaded and could not be found
    expanding macro: CrossroadsInterface.Web.__using__/1
    web/views/angular_view.ex:2: CrossroadsInterface.AngularView (module)
    (elixir) expanding macro: Kernel.use/2
    web/views/angular_view.ex:2: CrossroadsInterface.AngularView (module)
    (elixir) lib/kernel/parallel_compiler.ex:117: anonymous fn/4 in Kernel.ParallelCompiler.spawn_compilers/1

It's not always the same view that fails to compile, it will fail on the first view it trys to compile. My views are all bare bones, for instance, the AngularView in it's entirety is:

defmodule CrossroadsInterface.AngularView do
  use CrossroadsInterface.Web, :view
end

angular_view.ex
Any ideas what might be happening?

Elixir version: 1.4
Phoenix: 1.2.1
No Ecto

Thanks!!

@msp
Copy link

msp commented Mar 31, 2017

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment