Only emit deprecation warning for EEx if handle_text/3 is missing

josevalim · josevalim · commit a64f28c8dd81 · 2024-05-28T21:12:09.000+02:00
diff --git a/lib/eex/lib/eex/compiler.ex b/lib/eex/lib/eex/compiler.ex
@@ -306,7 +306,8 @@ defmodule EEx.Compiler do
 
     init = state.engine.init(opts)
 
-    if function_exported?(state.engine, :handle_text, 2) do
+    if function_exported?(state.engine, :handle_text, 2) and
+         not function_exported?(state.engine, :handle_text, 3) do
       IO.warn(
         "#{inspect(state.engine)}.handle_text/2 is deprecated, implement handle_text/3 instead"
       )

Original file line number	Diff line number	Diff line change
`@@ -306,7 +306,8 @@ defmodule EEx.Compiler do`
`306`	`306`
`307`	`307`	`init = state.engine.init(opts)`
`308`	`308`
`309`		`- if function_exported?(state.engine, :handle_text, 2) do`
	`309`	`+ if function_exported?(state.engine, :handle_text, 2) and`
	`310`	`+ not function_exported?(state.engine, :handle_text, 3) do`
`310`	`311`	`IO.warn(`
`311`	`312`	`"#{inspect(state.engine)}.handle_text/2 is deprecated, implement handle_text/3 instead"`
`312`	`313`	`)`