refatoração do código dos módulos RelatorioCompiler, RelatorioControl…

…ler e RelatorioHTML

lib/pescarte/modulo_pesquisa/relatorio_compiler.ex

@@ -6,32 +6,36 @@ defmodule Pescarte.RelatorioCompiler do
     relatorios = Enum.map(relatorios_selecionados, &find_relatorio/1)
     htmls = Enum.map(relatorios, &RelatorioHTML.content/1)
     pdfs = Enum.reduce(htmls, [], fn html, acc -> [{:html, html} | acc] end)
-    {:ok, pdf_compilado} = ChromicPDF.print_to_pdf(pdfs)
-    zip_file = criar_zip_file(pdf_compilado)
-    elem(zip_file, 1)
+
+    with {:ok, pdf_compilado} <- ChromicPDF.print_to_pdf(pdfs),
+         zip_file <- criar_zip_file(pdf_compilado) do
+      elem(zip_file, 1)
+    end
   end
 
-  def gerar_pdf(relatorio) do
-    {:ok, pdf} =
-      [content: RelatorioHTML.content(relatorio), size: :a4]
-      |> ChromicPDF.Template.source_and_options()
-      |> ChromicPDF.print_to_pdf()
+  def gerar_pdf(relatorio_id) do
+    relatorio = find_relatorio(relatorio_id)
 
-    Base.decode64!(pdf)
+    with {:ok, pdf} <-
+           [content: RelatorioHTML.content(relatorio), size: :a4]
+           |> ChromicPDF.Template.source_and_options()
+           |> ChromicPDF.print_to_pdf() do
+      Base.decode64(pdf)
+    end
   end
 
-  def find_relatorio(relatorio_id) do
+  defp find_relatorio(relatorio_id) do
     Repository.fetch_relatorio_pesquisa_by_id(relatorio_id)
   end
 
   defp criar_zip_file(pdf) do
-    {:ok, zip_file} =
-      :zip.create(
-        "relatorios_compilados.zip",
-        [{~c"relatorios_compilados.pdf", Base.decode64!(pdf)}],
-        [:memory]
-      )
-
-    zip_file
+    with {:ok, zip_file} <-
+           :zip.create(
+             "relatorios_compilados.zip",
+             [{~c"relatorios_compilados.pdf", Base.decode64!(pdf)}],
+             [:memory]
+           ) do
+      zip_file
+    end
   end
 end

lib/pescarte_web/controllers/relatorio_controller.ex

@@ -4,8 +4,7 @@ defmodule PescarteWeb.Pesquisa.RelatorioController do
   alias Pescarte.RelatorioCompiler
 
   def download_pdf(conn, %{"id" => id}) do
-    relatorio = RelatorioCompiler.find_relatorio(id)
-    pdf_binary = RelatorioCompiler.gerar_pdf(relatorio)
+    pdf_binary = RelatorioCompiler.gerar_pdf(id)
     enviar_pdf_response(conn, id, pdf_binary)
   end
 

lib/pescarte_web/templates/relatorio_html.ex

@@ -37,7 +37,6 @@ defmodule PescarteWeb.RelatorioHTML do
   end
 
   defp get_literal_mes(data) do
-    {:ok, mes} = Timex.lformat(data, "{Mfull}", "pt")
-    mes
+    Timex.lformat!(data, "{Mfull}", "pt")
   end
 end