Corrigindo migrations de post/tag (#246)

* Revert "Revert "Feat/core create post (#234)" (#245)"

This reverts commit 30beabd.

* correctly renames blog_posts column

* formatter

* minor fixes to migrations

* Update post.ex

* correctly rename N-N posts_tags

.formatter.exs

@@ -1,6 +1,7 @@
 # Used by "mix format"
 [
-  import_deps: [:phoenix, :ecto],
+  import_deps: [:phoenix, :ecto, :ecto_sql],
+  subdirectories: ["priv/*/migrations"],
   plugins: [Phoenix.LiveView.HTMLFormatter],
-  inputs: ["*.{ex,exs,heex}", "{lib,test}/**/*.{ex,exs,heex}"]
+  inputs: ["*.{ex,exs,heex}", "{config,lib,test}/**/*.{ex,exs,heex}", "priv/*/seeds.exs"]
 ]

lib/pescarte/blog/entity/tag.ex

@@ -5,6 +5,7 @@ defmodule Pescarte.Blog.Entity.Tag do
 
   use Pescarte, :model
 
+  alias Pescarte.Blog.Post
   alias Pescarte.Database.Types.PublicId
 
   @type t :: %Tag{nome: binary, id: binary}
@@ -14,6 +15,7 @@ defmodule Pescarte.Blog.Entity.Tag do
   @primary_key {:id, PublicId, autogenerate: true}
   schema "blog_tag" do
     field :nome, :string
+    many_to_many :blog_posts, Post, join_through: "posts_tags"
 
     timestamps()
   end

lib/pescarte/blog/post.ex

@@ -2,11 +2,14 @@ defmodule Pescarte.Blog.Post do
   @moduledoc """
   Módulo que define o schema e o changeset para os posts.
   """
+  use Pescarte, :model
+
+  alias Ecto.Multi
+  alias Pescarte.Blog.Entity.Tag
   alias Pescarte.Database
   alias Pescarte.Database.Repo
   alias Pescarte.Database.Types.PublicId
   alias Pescarte.Identidades.Models.Usuario
-  use Pescarte, :model
 
   @type t :: %Post{
           id: String.t(),
@@ -16,33 +19,32 @@ defmodule Pescarte.Blog.Post do
           published_at: NaiveDateTime.t(),
           inserted_at: NaiveDateTime.t(),
           updated_at: NaiveDateTime.t(),
-          usuario: Usuario.t(),
-          usuario_id: String.t()
+          usuario: Usuario.t()
         }
 
-  @required_params [:titulo, :conteudo, :link_imagem_capa, :published_at]
+  @required_params [:titulo, :conteudo, :link_imagem_capa, :published_at, :usuario_id]
 
   @primary_key {:id, PublicId, autogenerate: true}
-  schema "posts" do
+  schema "blog_posts" do
     field :titulo, :string
     field :conteudo, :binary
     field :link_imagem_capa, :string
     field :published_at, :naive_datetime
 
-    belongs_to :usuario, Usuario
-
-    # comentado enquanto o PR das tags não é aprovado
-    # many_to_many :tags, Tag, through: [:post_tags, :tag]
+    belongs_to :usuario, Usuario, type: :string
+    many_to_many :blog_tags, Tag, join_through: "posts_tags"
 
     timestamps()
   end
 
-  @spec changeset(t, map) :: changeset
+  @spec changeset(Post.t(), map) :: changeset
   def changeset(post \\ %Post{}, params) do
     post
+    |> Map.put(:published_at, NaiveDateTime.utc_now(:second))
     |> cast(params, @required_params)
     |> validate_required(@required_params)
     |> unique_constraint(:titulo)
+    |> foreign_key_constraint(:usuario_id)
   end
 
   @spec get_posts :: list(Post.t()) | Ecto.QueryError
@@ -55,11 +57,15 @@ defmodule Pescarte.Blog.Post do
     Database.fetch(Post, id)
   end
 
-  @spec create_post(Post.t()) :: {:ok, Post.t()} | {:error, Ecto.Changeset.t()}
-  def create_post(params) do
-    %Post{}
-    |> Post.changeset(params)
-    |> Repo.insert()
+  @spec create_post(Map) :: {:ok, Post.t()} | {:error, Ecto.Changeset.t()}
+  def create_post(%{blog_tags: tags} = params) do
+    Multi.new()
+    |> Multi.insert_all(:update_tags, Tag, tags,
+      on_conflict: :replace_all,
+      conflict_target: :nome
+    )
+    |> Multi.insert(:blog_posts, changeset(%Post{}, params))
+    |> Repo.transaction()
   end
 
   @spec delete_post(String.t()) :: {:ok, Post.t()} | {:error, :not_found}

lib/pescarte/identidades/models/usuario.ex

@@ -45,7 +45,7 @@ defmodule Pescarte.Identidades.Models.Usuario do
 
     belongs_to :contato, Contato, type: :string
 
-    has_many :posts, Post
+    has_many :blog_posts, Post
 
     timestamps()
   end

priv/repo/migrations/20240407235836_create_tag.exs

@@ -5,7 +5,7 @@ defmodule Pescarte.Database.Repo.Migrations.CreateTag do
     create table(:tag, primary_key: false) do
       add :id, :string, primary_key: true
       add :etiqueta, :string, null: false
-      add :categoria_id, references(:categoria,type: :string), null: false
+      add :categoria_id, references(:categoria, type: :string), null: false
 
       timestamps()
     end

priv/repo/migrations/20240424025541_retype-_external_customer_id_type.exs

@@ -1,6 +1,8 @@
-defmodule :"Elixir.Pescarte.Database.Repo.Migrations.Retype-ExternalCustomerIdType" do
+defmodule Elixir.Pescarte.Database.Repo.Migrations.RetypeExternalCustomerIdType do
   use Ecto.Migration
 
+  # ESSA MIGRATION É IRREVERSÍVEL
+
   def change do
     alter table(:usuario) do
       modify :external_customer_id, :text, from: :binary_id

priv/repo/migrations/20240814202453_create_blog_tag.exs

@@ -8,6 +8,7 @@ defmodule Pescarte.Database.Repo.Migrations.CreateBlogTag do
 
       timestamps()
     end
+
     create unique_index(:blog_tag, [:nome])
   end
 end

priv/repo/migrations/20241025203509_create_blog_posts.exs

@@ -4,16 +4,16 @@ defmodule Pescarte.Database.Repo.Migrations.CreatePost do
   def change do
     create table(:blog_posts, primary_key: false) do
       add :id, :string, primary_key: true
-      add :user_id, references(:usuario, type: :string), null: false
       add :titulo, :string
       add :conteudo, :binary
       add :link_imagem_capa, :string
       add :published_at, :naive_datetime
+      add :user_id, references(:usuario, type: :string)
 
       timestamps()
-  end
+    end
 
-  create unique_index(:blog_posts, :titulo)
-  create index(:blog_posts, :user_id)
-end
+    create unique_index(:blog_posts, :titulo)
+    create index(:blog_posts, :user_id)
+  end
 end

priv/repo/migrations/20241025203733_create_post_tag.exs → priv/repo/migrations/20241122115106_create_posts_tags_relation.exs

@@ -2,7 +2,7 @@ defmodule Pescarte.Database.Repo.Migrations.CreatePostTags do
   use Ecto.Migration
 
   def change do
-    create table(:post_tag) do
+    create table(:posts_tag) do
       add :tag_id, references(:blog_tag, type: :string), null: false
       add :post_id, references(:blog_posts, type: :string), null: false
       timestamps()

priv/repo/migrations/20241220102712_rename_blog_post_relation.exs

@@ -0,0 +1,9 @@
+defmodule Pescarte.Database.Repo.Migrations.RenameBlogPostRelation do
+  use Ecto.Migration
+
+  def change do
+    drop_if_exists index(:blog_posts, [:user_id])
+    rename table(:blog_posts), :user_id, to: :usuario_id
+    create index(:blog_posts, [:usuario_id])
+  end
+end

priv/repo/migrations/20241220110342_rename_post_tag_table.exs

@@ -0,0 +1,7 @@
+defmodule Pescarte.Database.Repo.Migrations.RenamePostTagTable do
+  use Ecto.Migration
+
+  def change do
+    rename table(:posts_tag), to: table(:posts_tags)
+  end
+end