bug #166

#166 supression de masse
correction page export qui ne s affichait pas

Author: alainabbas

apps/api/src/core/backends/backends.service.ts

@@ -315,46 +315,65 @@ export class BackendsService extends AbstractQueueProcessor {
 
     if (!payload.length) throw new BadRequestException('No identities to disable');
 
+    const result = {};
+
     for (const key of payload) {
-      const identity = await this.identitiesService.findById<any>(key);
-      if (identity.primaryEmployeeNumber !== null && identity.primaryEmployeeNumber !== '') {
-        identity.inetOrgPerson.employeeNumber = identity.primaryEmployeeNumber;
-      } else {
-        //on prend la premiere pour envoyer une chaine et non un tableau pour la compatibilité ldap
-        identity.inetOrgPerson.employeeNumber = identity.inetOrgPerson.employeeNumber[0];
-      }
-      if (!identity.lastBackendSync) {
-        // l identité n'a jamais été symchronisée on la soft delete
-        await this.identitiesService.model.findByIdAndUpdate(key, {
-          $set: {
-            state: IdentityState.DONT_SYNC,
-            deletedFlag: true,
-          },
+      try {
+        const identity = await this.identitiesService.findById<any>(key);
+        if (!identity) {
+          result[key] = { error: `Identity ${key} not found` };
+          continue;
+        }
+        if (identity.primaryEmployeeNumber !== null && identity.primaryEmployeeNumber !== '') {
+          identity.inetOrgPerson.employeeNumber = identity.primaryEmployeeNumber;
+        } else if (Array.isArray(identity.inetOrgPerson?.employeeNumber)) {
+          //on prend la premiere pour envoyer une chaine et non un tableau pour la compatibilité ldap
+          identity.inetOrgPerson.employeeNumber = identity.inetOrgPerson.employeeNumber[0];
+        }
+        if (!identity.lastBackendSync) {
+          // l identité n'a jamais été symchronisée on la soft delete
+          // puis on poursuit avec le reste de la sélection (suppression en masse)
+          await this.identitiesService.model.findByIdAndUpdate(key, {
+            $set: {
+              state: IdentityState.DONT_SYNC,
+              deletedFlag: true,
+            },
+          });
+          result[key] = { softDeleted: true };
+          continue;
+        }
+        identities.push({
+          action: ActionType.IDENTITY_DELETE,
+          identity,
         });
-        return [];
+      } catch (error) {
+        // une identité en erreur ne doit pas interrompre la suppression des autres
+        this.logger.error(`Unable to prepare deletion of identity ${key}: ${error?.message}`, error?.stack);
+        result[key] = { error: error?.message ?? 'Unknown error' };
       }
-      identities.push({
-        action: ActionType.IDENTITY_DELETE,
-        identity,
-      });
     }
 
+    if (!identities.length) return result;
+
     const task: Document<Tasks> = await this.tasksService.create<Tasks>({
       jobs: identities.map((identity) => identity.identity._id),
     });
 
-    const result = {};
     for (const identity of identities) {
-      const [executedJob] = await this.executeJob(identity.action, identity.identity._id, identity.identity, {
-        ...options,
-        updateStatus: true,
-        switchToProcessing: false,
-        targetState: IdentityState.DONT_SYNC,
-        dataState: DataStatusEnum.DELETED,
-        task: task._id as unknown as Types.ObjectId,
-      });
-      result[identity.identity._id] = executedJob;
-      // console.log(res);
+      try {
+        const [executedJob] = await this.executeJob(identity.action, identity.identity._id, identity.identity, {
+          ...options,
+          updateStatus: true,
+          switchToProcessing: false,
+          targetState: IdentityState.DONT_SYNC,
+          dataState: DataStatusEnum.DELETED,
+          task: task._id as unknown as Types.ObjectId,
+        });
+        result[`${identity.identity._id}`] = executedJob;
+      } catch (error) {
+        this.logger.error(`Unable to delete identity ${identity.identity._id}: ${error?.message}`, error?.stack);
+        result[`${identity.identity._id}`] = { error: error?.message ?? 'Unknown error' };
+      }
     }
     return result;
   }

apps/web/src/components/core/twopan.vue

@@ -250,8 +250,23 @@ export default defineNuxtComponent({
         this.syncSelectionWithRows()
       },
     },
+    // La sélection survit à un changement de page, mais pas à un changement de
+    // filtre / recherche : les lignes conservées ne feraient plus partie du jeu affiché.
+    filtersSignature: {
+      handler(next: string, previous: string) {
+        if (next !== previous) this.clearSelection()
+      },
+    },
   },
   computed: {
+    /** Empreinte des filtres actifs, hors pagination / tri / route de détail. */
+    filtersSignature(): string {
+      const query = this.$route?.query || {}
+      const entries = Object.entries(query)
+        .filter(([key]) => !['limit', 'skip', 'page', 'read'].includes(key) && !key.startsWith('sort'))
+        .sort(([a], [b]) => a.localeCompare(b))
+      return JSON.stringify(entries)
+    },
     visibleColumnsSelected(): QTableProps['visibleColumns'] {
       if (!this.visibleColumns || this.visibleColumns.length === 0) {
         return (this.columns || []).map((column) => column.name)
@@ -290,21 +305,29 @@ export default defineNuxtComponent({
     },
   },
   methods: {
-    /**
-     * Garde uniquement les lignes encore présentes dans `rows` (même clé `rowKey`)
-     * et réaligne sur les objets courants. Sinon la sélection peut rester bloquée
-     * sur d’anciennes lignes après changement de route / filtres / rechargement.
-     */
     tableRowKey(row: Record<string, unknown>): string {
       const rk = this.rowKey
       return typeof rk === 'function' ? String(rk(row)) : String(row?.[rk as string] ?? '')
     },
+    /**
+     * Réaligne la sélection sur les objets de la page courante (sinon Quasar ne
+     * reconnaît plus les lignes cochées après un rechargement) tout en conservant
+     * les lignes sélectionnées sur les autres pages : la sélection doit survivre
+     * à un changement de page. Les doublons de clé sont éliminés.
+     */
     syncSelectionWithRows() {
       if (!this.selection || this.selection === 'none') return
       if (!Array.isArray(this.selected) || this.selected.length === 0) return
       const rows = this.rows || []
       const rowByKey = new Map(rows.map((r) => [this.tableRowKey(r), r]))
-      const next = this.selected.map((s) => rowByKey.get(this.tableRowKey(s))).filter((r) => r !== undefined)
+      const seen = new Set<string>()
+      const next = [] as typeof this.selected
+      for (const s of this.selected) {
+        const key = this.tableRowKey(s)
+        if (seen.has(key)) continue
+        seen.add(key)
+        next.push(rowByKey.get(key) ?? s)
+      }
       if (next.length !== this.selected.length || next.some((r, i) => r !== this.selected[i])) {
         this.selected = next
       }

apps/web/src/components/pages/identities/modals/delete-many.vue

@@ -38,6 +38,14 @@ q-dialog(
             q-item-section
               q-item-label.text-weight-medium(lines="2") {{ item.label }}
               q-item-label.text-caption.text-grey-6(lines="1" style="font-family: ui-monospace, monospace") {{ item.idShort }}
+      .identity-modal-body-footer(v-if="allIdentitiesCount > selectedRows.length")
+        q-separator.q-my-md
+        q-checkbox(
+          dense
+          color="negative"
+          v-model="deleteAllIdentities",
+          :label="checkboxLabel",
+        )
     q-card-actions.identity-modal-actions(align="right")
       q-btn.identity-modal-btn-cancel(
         outline
@@ -54,20 +62,24 @@ q-dialog(
         color="negative"
         icon-right="mdi-check"
         label="Supprimer"
-        :disable="selectedRows.length === 0"
+        :disable="selectedRows.length === 0 && !deleteAllIdentities"
         @click="syncIdentities"
       )
 </template>
 
 <script lang="ts" setup>
-import { computed } from 'vue'
+import { computed, ref } from 'vue'
 import { useDialogPluginComponent, useQuasar } from 'quasar'
 
 const props = defineProps({
   selectedIdentities: {
     type: Array,
     default: () => [],
   },
+  allIdentitiesCount: {
+    type: Number,
+    default: 0,
+  },
 })
 
 defineEmits([...useDialogPluginComponent.emits])
@@ -159,15 +171,24 @@ const identityListItems = computed(() => {
   })
 })
 
-const mainText = computed(
-  () =>
-    `Vous allez supprimer ${selectedRows.value.length} identité${selectedRows.value.length > 1 ? 's' : ''}. Cette action est irréversible. Vérifiez la liste puis confirmez.`,
+const deleteAllIdentities = ref(false)
+
+const mainText = computed(() => {
+  if (deleteAllIdentities.value) {
+    return `Vous allez supprimer les ${props.allIdentitiesCount} identité${props.allIdentitiesCount > 1 ? 's' : ''} correspondant au filtre actuel, et pas seulement la sélection ci-dessous. Cette action est irréversible.`
+  }
+
+  return `Vous allez supprimer ${selectedRows.value.length} identité${selectedRows.value.length > 1 ? 's' : ''}. Cette action est irréversible. Vérifiez la liste puis confirmez.`
+})
+
+const checkboxLabel = computed(
+  () => `Supprimer toutes les identités du filtre actuel (${props.allIdentitiesCount} identité${props.allIdentitiesCount > 1 ? 's' : ''})`,
 )
 
 const { dialogRef, onDialogHide, onDialogOK, onDialogCancel } = useDialogPluginComponent()
 
 const syncIdentities = () => {
-  onDialogOK({ success: true })
+  onDialogOK({ success: true, deleteAllIdentities: deleteAllIdentities.value })
 }
 
 const cancelSync = () => {
@@ -210,6 +231,10 @@ const cancelSync = () => {
   flex-shrink: 0;
 }
 
+.identity-modal-body-footer {
+  flex-shrink: 0;
+}
+
 .identity-modal-lede {
   line-height: 1.55;
   margin-bottom: 1rem;

apps/web/src/pages/identities/export.vue

@@ -65,7 +65,7 @@ q-page.container.q-pa-sm
     q-table(
         :rows-per-page-options="[20,50,0]"
         :columns="exportColumns"
-        :rows="identities"
+        :rows="identities || []"
         row-key="_id"
         flat
         dense
@@ -74,6 +74,8 @@ q-page.container.q-pa-sm
 
 <script lang="ts">
 import type { LocationQueryValue } from 'vue-router'
+// Import explicite : l'auto-import d'unimport ne détecte pas cette constante dans ce fichier.
+import { DEFAULT_IDENTITY_FILTER_FIELD_PATHS } from '~/composables/useFilterFieldOptions'
 
 export default defineNuxtComponent({
   name: 'IdentitiesExportPage',
@@ -95,11 +97,13 @@ export default defineNuxtComponent({
 
     let rowsData = null
     const queryWithoutRead = computed(() => {
-      const { read, ...rest } = route.query
+      // L'export porte sur tout le résultat filtré : la pagination de la table
+      // (`limit`/`page`, présents dans l'URL au retour de /identities/table) ne doit pas être reprise.
+      const { read, limit, page, ...rest } = route.query
       return {
-        limit: 9999,
         ...getSearchFieldsQuery(),
         ...rest,
+        limit: 9999,
       }
     })
 
@@ -110,8 +114,11 @@ export default defineNuxtComponent({
     } = await useHttp('/management/identities/validation', {
       method: 'GET',
       transform: (result: any) => {
-        const allFields = result.data.flatMap((enr) => {
-          return Object.keys(enr[enr.name].properties)
+        const schemas = Array.isArray(result?.data) ? result.data : []
+        const allFields = schemas.flatMap((enr) => {
+          // Un schéma YAML vide ou sans `properties` ne doit pas casser la page.
+          const properties = enr?.[enr?.name]?.properties
+          return properties ? Object.keys(properties) : []
         })
 
         const columns = allFields.map((enr) => {