don't generate broadcast multiple times

Author: SteffenDE

lib/mix/tasks/phx.gen.context.ex

@@ -190,6 +190,13 @@ defmodule Mix.Tasks.Phx.Gen.Context do
   defp inject_schema_access(%Context{file: file} = context, paths, binding) do
     ensure_context_file_exists(context, paths, binding)
 
+    binding =
+      if File.exists?(file) and File.read!(file) =~ "defp broadcast" do
+        [{:add_broadcast, false} | binding]
+      else
+        [{:add_broadcast, true} | binding]
+      end
+
     paths
     |> Mix.Phoenix.eval_from(
       "priv/templates/phx.gen.context/#{schema_access_template(context)}",

priv/templates/phx.gen.context/schema_access_scope.ex

@@ -16,13 +16,13 @@
     key = scope.<%= Enum.join(scope.access_path, ".") %>
 
     Phoenix.PubSub.subscribe(<%= inspect context.base_module %>.PubSub, "<%= scope.name %>:#{key}:<%= schema.plural %>")
-  end
+  end<%= if add_broadcast do %>
 
   defp broadcast(%<%= inspect scope.alias %>{} = scope, message) do
     key = scope.<%= Enum.join(scope.access_path, ".") %>
 
     Phoenix.PubSub.broadcast(<%= inspect context.base_module %>.PubSub, "<%= scope.name %>:#{key}:<%= schema.plural %>", message)
-  end
+  end<% end %>
 
   @doc """
   Returns the list of <%= schema.plural %>.