schuwi/component-system
2
1
Fork 0
Code Issues 6 Pull Requests Actions Packages Projects Releases Wiki Activity

fix(elixir): edit modal of category parent issue

Browse Source
This commit is contained in:
Schuwi
2025-09-14 15:47:17 +02:00
parent 5717931ace
commit 4f1b273793
1 changed files with 14 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
lib/components_elixir_web/live/categories_live.ex
View File

@@ -44,7 +44,14 @@ defmodule ComponentsElixirWeb.CategoriesLive do
def handle_event("show_edit_form", %{"id" => id}, socket) do def handle_event("show_edit_form", %{"id" => id}, socket) do
category = Inventory.get_category!(id) category = Inventory.get_category!(id)
changeset = Inventory.change_category(category) # Create a changeset with current values forced into changes for proper form display
changeset = Inventory.change_category(category, %{
name: category.name,
description: category.description,
parent_id: category.parent_id
})
|> Ecto.Changeset.force_change(:parent_id, category.parent_id)
form = to_form(changeset) form = to_form(changeset)
{:noreply, {:noreply,
@@ -127,15 +134,15 @@ defmodule ComponentsElixirWeb.CategoriesLive do
[{"No parent (Root category)", nil}] ++ available_categories [{"No parent (Root category)", nil}] ++ available_categories
end end
defp is_descendant?(categories, potential_ancestor_id, category_id) do defp is_descendant?(categories, descendant_id, ancestor_id) do
# Prevent circular references by checking if the potential parent is a descendant # Check if descendant_id is a descendant of ancestor_id
category = Enum.find(categories, fn cat -> cat.id == category_id end) descendant = Enum.find(categories, fn cat -> cat.id == descendant_id end)
case category do case descendant do
nil -> false nil -> false
%{parent_id: nil} -> false %{parent_id: nil} -> false
%{parent_id: parent_id} when parent_id == potential_ancestor_id -> true %{parent_id: parent_id} when parent_id == ancestor_id -> true
%{parent_id: parent_id} -> is_descendant?(categories, potential_ancestor_id, parent_id) %{parent_id: parent_id} -> is_descendant?(categories, parent_id, ancestor_id)
end end
end end