diff --git a/lib/cannery/tags.ex b/lib/cannery/tags.ex index 4a74f06..8246608 100644 --- a/lib/cannery/tags.ex +++ b/lib/cannery/tags.ex @@ -4,7 +4,7 @@ defmodule Cannery.Tags do """ import Ecto.Query, warn: false - alias Cannery.Repo + alias Cannery.{Accounts, Repo} alias Cannery.Tags.Tag @@ -17,8 +17,15 @@ defmodule Cannery.Tags do [%Tag{}, ...] """ - def list_tags do - Repo.all(Tag) + @spec list_tags(Accounts.User.t()) :: [Tag.t()] + def list_tags(%{id: user_id}) do + list_tags(user_id) + end + + def list_tags(user_id) do + Repo.all( + from t in Tag, where: t.user_id == ^user_id + ) end @doc """ @@ -35,6 +42,7 @@ defmodule Cannery.Tags do ** (Ecto.NoResultsError) """ + @spec get_tag!(Ecto.UUID.t()) :: Tag.t() def get_tag!(id), do: Repo.get!(Tag, id) @doc """ @@ -49,10 +57,9 @@ defmodule Cannery.Tags do {:error, %Ecto.Changeset{}} """ - def create_tag(attrs \\ %{}) do - %Tag{} - |> Tag.changeset(attrs) - |> Repo.insert() + @spec create_tag(map()) :: {:ok, Tag.t()} | {:error, Ecto.Changeset.t()} + def create_tag(attrs) do + %Tag{} |> Tag.changeset(attrs) |> Repo.insert() end @doc """ @@ -67,10 +74,9 @@ defmodule Cannery.Tags do {:error, %Ecto.Changeset{}} """ - def update_tag(%Tag{} = tag, attrs) do - tag - |> Tag.changeset(attrs) - |> Repo.update() + @spec update_tag(Tag.t(), map()) :: {:ok, Tag.t()} | {:error, Ecto.Changeset.t()} + def update_tag(tag, attrs) do + tag |> Tag.changeset(attrs) |> Repo.update() end @doc """ @@ -85,7 +91,8 @@ defmodule Cannery.Tags do {:error, %Ecto.Changeset{}} """ - def delete_tag(%Tag{} = tag) do + @spec delete_tag(Tag.t()) :: {:ok, Tag.t()} | {:error, Ecto.Changeset.t()} + def delete_tag(tag) do Repo.delete(tag) end @@ -98,7 +105,9 @@ defmodule Cannery.Tags do %Ecto.Changeset{data: %Tag{}} """ - def change_tag(%Tag{} = tag, attrs \\ %{}) do + @spec change_tag(Tag.t()) :: Ecto.Changeset.t() + @spec change_tag(Tag.t(), map()) :: Ecto.Changeset.t() + def change_tag(tag, attrs \\ %{}) do Tag.changeset(tag, attrs) end end diff --git a/lib/cannery/tags/tag.ex b/lib/cannery/tags/tag.ex index 521ca2f..abd564c 100644 --- a/lib/cannery/tags/tag.ex +++ b/lib/cannery/tags/tag.ex @@ -16,7 +16,7 @@ defmodule Cannery.Tags.Tag do @doc false def changeset(tag, attrs) do tag - |> cast(attrs, [:name, :bg_color, :text_color]) - |> validate_required([:name, :bg_color, :text_color]) + |> cast(attrs, [:name, :bg_color, :text_color, :user_id]) + |> validate_required([:name, :bg_color, :text_color, :user_id]) end end diff --git a/lib/cannery_web/live/tag_live/form_component.ex b/lib/cannery_web/live/tag_live/form_component.ex index 60e8879..5eeeb36 100644 --- a/lib/cannery_web/live/tag_live/form_component.ex +++ b/lib/cannery_web/live/tag_live/form_component.ex @@ -15,6 +15,8 @@ defmodule CanneryWeb.TagLive.FormComponent do @impl true def handle_event("validate", %{"tag" => tag_params}, socket) do + tag_params = tag_params |> Map.put("user_id", socket.assigns.current_user.id) + changeset = socket.assigns.tag |> Tags.change_tag(tag_params) @@ -24,6 +26,7 @@ defmodule CanneryWeb.TagLive.FormComponent do end def handle_event("save", %{"tag" => tag_params}, socket) do + tag_params = tag_params |> Map.put("user_id", socket.assigns.current_user.id) save_tag(socket, socket.assigns.action, tag_params) end diff --git a/lib/cannery_web/live/tag_live/form_component.html.leex b/lib/cannery_web/live/tag_live/form_component.html.leex index ae331c6..0c4f448 100644 --- a/lib/cannery_web/live/tag_live/form_component.html.leex +++ b/lib/cannery_web/live/tag_live/form_component.html.leex @@ -1,22 +1,32 @@ -

<%= @title %>

+

+ <%= @title %> +

<%= f = form_for @changeset, "#", id: "tag-form", + class: "grid grid-cols-3 justify-center items-center space-y-4", phx_target: @myself, phx_change: "validate", phx_submit: "save" %> <%= label f, :name, class: "title text-lg text-primary-500" %> - <%= text_input f, :name, class: "input input-primary" %> - <%= error_tag f, :name %> + <%= text_input f, :name, class: "input input-primary col-span-2" %> + + <%= error_tag f, :name %> + <%= label f, :bg_color, class: "title text-lg text-primary-500" %> - <%= text_input f, :bg_color %> - <%= error_tag f, :bg_color %> + <%= color_input f, :bg_color, class: "mx-auto col-span-2" %> + + <%= error_tag f, :bg_color %> + <%= label f, :text_color, class: "title text-lg text-primary-500" %> - <%= text_input f, :text_color %> - <%= error_tag f, :text_color %> + <%= color_input f, :text_color, class: "mx-auto col-span-2" %> + + <%= error_tag f, :text_color %> + - <%= submit "Save", phx_disable_with: "Saving..." %> + <%= submit "Save", class: "mx-auto btn btn-primary col-span-3", + phx_disable_with: "Saving..." %> diff --git a/lib/cannery_web/live/tag_live/index.ex b/lib/cannery_web/live/tag_live/index.ex index f6f5b35..b5dbe37 100644 --- a/lib/cannery_web/live/tag_live/index.ex +++ b/lib/cannery_web/live/tag_live/index.ex @@ -6,7 +6,7 @@ defmodule CanneryWeb.TagLive.Index do @impl true def mount(_params, session, socket) do - {:ok, socket |> assign_defaults(session) |> assign(:tags, list_tags())} + {:ok, socket |> assign_defaults(session) |> display_tags()} end @impl true @@ -36,11 +36,12 @@ defmodule CanneryWeb.TagLive.Index do def handle_event("delete", %{"id" => id}, socket) do tag = Tags.get_tag!(id) {:ok, _} = Tags.delete_tag(tag) - - {:noreply, socket |> assign(:tags, list_tags())} + socket = socket |> put_flash(:info, "Tag deleted succesfully") + {:noreply, socket |> display_tags()} end - defp list_tags do - Tags.list_tags() + defp display_tags(socket) do + tags = Tags.list_tags(socket.assigns.current_user) + socket |> assign(tags: tags) end end diff --git a/lib/cannery_web/live/tag_live/index.html.leex b/lib/cannery_web/live/tag_live/index.html.leex index 766ff11..c0f3a8e 100644 --- a/lib/cannery_web/live/tag_live/index.html.leex +++ b/lib/cannery_web/live/tag_live/index.html.leex @@ -1,4 +1,55 @@ -

Listing Tags

+
+

+ Listing Tags +

+ +

+ Tags can be added to your containers to help you organize +

+ + <%= if @tags |> Enum.empty?() do %> +
+

+ No tags +

+ + <%= live_patch to: Routes.tag_index_path(@socket, :new), + class: "btn btn-primary" do %> + Create your first tag! + <% end %> +
+ <% else %> + <%= live_patch to: Routes.tag_index_path(@socket, :new), + class: "btn btn-primary" do %> + New Tag + <% end %> + <% end %> + +
+ <%= for tag <- @tags do %> +
+ +

+ <%= tag.name %> +

+ +
+ <%= live_patch "Edit", to: Routes.tag_index_path(@socket, :edit, tag), + class: "text-primary-500 link" %> + + <%= link "Delete", to: "#", + class: "text-primary-500 link", + phx_click: "delete", + phx_value_id: tag.id, + data: [confirm: "Are you sure you want to delete #{tag.name}?"] %> +
+
+ <% end %> +
+
<%= if @live_action in [:new, :edit] do %> <%= live_modal CanneryWeb.TagLive.FormComponent, @@ -6,33 +57,6 @@ title: @page_title, action: @live_action, tag: @tag, - return_to: Routes.tag_index_path(@socket, :index) %> + return_to: Routes.tag_index_path(@socket, :index), + current_user: @current_user %> <% end %> - - - - - - - - - - - - - <%= for tag <- @tags do %> - - - - - - - - <% end %> - -
NameBg-colorText-color
<%= tag.name %><%= tag.bg_color %><%= tag.text_color %> - <%= live_patch "Edit", to: Routes.tag_index_path(@socket, :edit, tag) %> - <%= link "Delete", to: "#", phx_click: "delete", phx_value_id: tag.id, data: [confirm: "Are you sure?"] %> -
- -<%= live_patch "New Tag", to: Routes.tag_index_path(@socket, :new) %>