From e8138ebcb36faddb6f9a12ec2c52ac72d82229b2 Mon Sep 17 00:00:00 2001
From: Julien Froidefond <julien.froidefond@cdbdx.biz>
Date: Mon, 25 Aug 2025 22:00:40 +0200
Subject: [PATCH] fix: update delete button behavior in TreeCategoryHeader and
 TeamsList

- Removed unnecessary canDelete prop from TreeCategoryHeader's delete button condition.
- Added disabled state to the delete button based on canDelete logic.
- Updated canDelete prop in TeamsList to reflect whether the direction has members.
---
 components/admin/management/tree-category-header.tsx | 9 +++++++--
 components/admin/teams/teams-list.tsx                | 2 +-
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/components/admin/management/tree-category-header.tsx b/components/admin/management/tree-category-header.tsx
index 77af784..5db9097 100644
--- a/components/admin/management/tree-category-header.tsx
+++ b/components/admin/management/tree-category-header.tsx
@@ -66,7 +66,7 @@ export function TreeCategoryHeader({
             {itemCount} {itemLabel}
             {itemCount > 1 ? "s" : ""}
           </Badge>
-          {isDirection && onDelete && canDelete && (
+          {isDirection && onDelete && (
             <Button
               variant="ghost"
               size="sm"
@@ -75,7 +75,12 @@ export function TreeCategoryHeader({
                 onDelete();
               }}
               className="text-red-400 hover:text-red-300 hover:bg-red-500/20 h-6 w-6 p-0"
-              title={`Supprimer la direction "${category}" et toutes ses équipes`}
+              disabled={!canDelete}
+              title={
+                canDelete
+                  ? `Supprimer la direction "${category}" et toutes ses équipes`
+                  : `Impossible de supprimer la direction "${category}" car elle contient des membres`
+              }
             >
               <Trash2 className="w-3 h-3" />
             </Button>
diff --git a/components/admin/teams/teams-list.tsx b/components/admin/teams/teams-list.tsx
index 202757d..c143cf6 100644
--- a/components/admin/teams/teams-list.tsx
+++ b/components/admin/teams/teams-list.tsx
@@ -50,7 +50,7 @@ export function TeamsList({
               itemLabel="équipe"
               showSeparator={index > 0}
               onDelete={() => onDeleteDirection(direction)}
-              canDelete={true}
+              canDelete={!getDirectionStats(direction).hasMembers}
               isDirection={true}
               directionStats={getDirectionStats(direction)}
             />