fix delete group bug

src/components/DisclosureWidget.vue

@@ -237,6 +237,7 @@ const actions = computed(() => {
             label-visible
             :aria-invalid="inputErrors.length ? true : undefined"
             :description-id="inputErrors.length ? 'errors-name' : undefined"
+            @keypress.prevent.enter="onValidateEdit"
           />
           <div v-if="inputErrors.length" id="errors-name" class="error">
             <p v-for="(error, index) in inputErrors" :key="index">

src/components/bouquets/BouquetDatasetList.vue

@@ -6,15 +6,15 @@ import DatasetEditModal, {
   type DatasetEditModalType
 } from '@/components/forms/dataset/DatasetEditModal.vue'
 import config from '@/config'
-import { type DatasetProperties, type DatasetsGroups } from '@/model/topic'
+import { type DatasetProperties } from '@/model/topic'
 import { useDatasetStore } from '@/store/DatasetStore'
 import { fromMarkdown } from '@/utils'
 import { isAvailable } from '@/utils/bouquet'
 import { useTopicsConf } from '@/utils/config'
 import { toastHttpError } from '@/utils/error'
 import { isNotFoundError } from '@/utils/http'
 
-import { noGroup, useGroups } from '@/utils/bouquetGroups'
+import { useGroups } from '@/utils/bouquetGroups'
 import DisclosureWidget from '../DisclosureWidget.vue'
 import BouquetDatasetCard from './BouquetDatasetCard.vue'
 
@@ -49,31 +49,6 @@ const {
   deleteGroup
 } = useGroups(datasetsProperties)
 
-const sortedGroupedDatasets = computed(() => {
-  // get all entries within nogroup if any exists
-  const noGroupEntries: DatasetProperties[] | null = groupedDatasets.value.has(
-    noGroup
-  )
-    ? groupedDatasets.value.get(noGroup)
-    : null
-  // create a new Map without nogroup
-  const filteredGroups: DatasetsGroups = new Map(
-    [...groupedDatasets.value].filter(([key]) => key !== noGroup)
-  )
-  // Alphabetically sort the keys with nogroup at the end
-  const entries = Array.from(filteredGroups.entries()).sort(
-    ([keyA], [keyB]) => {
-      return keyA.toString().localeCompare(keyB.toString())
-    }
-  )
-  // Add the nogroup to the end of the sorted array
-  if (noGroupEntries) {
-    entries.push([noGroup, noGroupEntries])
-  }
-  // Convert back the sorted groups into a Map
-  return new Map(entries)
-})
-
 const handleRemoveDataset = (group: string, index: number) => {
   removeDatasetFromGroup(group, index)
   emits('updateDatasets')
@@ -159,7 +134,7 @@ onMounted(() => {
   <template v-else>
     <div v-if="datasetEditorialization" class="fr-mt-10v">
       <ul role="list" class="groups fr-m-0 fr-p-0">
-        <li v-for="[group, datasets] in sortedGroupedDatasets" :key="group">
+        <li v-for="[group, datasets] in groupedDatasets" :key="group">
           <DisclosureWidget
             v-if="datasets.length"
             :group-name="group"

src/utils/bouquetGroups.ts

@@ -33,7 +33,30 @@ export function useGroups(datasetsProperties: Ref<DatasetProperties[]>): {
       }
     })
 
-    return datasetsGroups.value
+    // get all entries within nogroup if any exists
+    const noGroupEntries: DatasetProperties[] | undefined =
+      datasetsGroups.value.has(noGroup)
+        ? datasetsGroups.value.get(noGroup)
+        : undefined
+
+    // create a new Map without nogroup
+    const filteredGroups: DatasetsGroups = new Map(
+      [...datasetsGroups.value].filter(([key]) => key !== noGroup)
+    )
+
+    // Alphabetically sort the keys with nogroup at the end
+    const sortedEntries = Array.from(filteredGroups.entries()).sort(
+      ([keyA], [keyB]) => {
+        return keyA.toString().localeCompare(keyB.toString())
+      }
+    )
+
+    // Add the nogroup to the end of the sorted array
+    if (noGroupEntries) {
+      sortedEntries.push([noGroup, noGroupEntries])
+    }
+
+    return new Map(sortedEntries)
   })
   const getDatasetIndex = (group: string, indexInGroup: number) => {
     // get all datasets from group
@@ -76,7 +99,8 @@ export function useGroups(datasetsProperties: Ref<DatasetProperties[]>): {
       const matchingDatasets = datasetsProperties.value.filter((dataset) => {
         return oldGroupItems.some(
           (groupItem) =>
-            groupItem.id === dataset.id && groupItem.group === dataset.group
+            groupItem.title === dataset.title &&
+            groupItem.group === dataset.group
         )
       })
       if (matchingDatasets) {
@@ -97,7 +121,8 @@ export function useGroups(datasetsProperties: Ref<DatasetProperties[]>): {
         (dataset) => {
           return groupItems.find(
             (groupItem) =>
-              groupItem.id !== dataset.id && groupItem.group !== dataset.group
+              groupItem.title !== dataset.title &&
+              groupItem.group !== dataset.group
           )
         }
       )