fix(fiches): affiche les fiches partagees meme si aucune fiche de la collectivite

Author: dthib

apps/app/src/plans/fiches/list-all-fiches/data/use-get-fiches-total-count.ts

@@ -1,32 +1,29 @@
-import { useQuery } from '@tanstack/react-query';
-
 import { useCollectiviteId } from '@/api/collectivites';
-import { ficheActionCount } from '@/api/plan-actions/fiche-resumes.list';
-import { useSupabase } from '@/api/utils/supabase/use-supabase';
+import {
+  GetFichesOptions,
+  useListFiches,
+} from '@/app/plans/fiches/list-all-fiches/data/use-list-fiches';
 
 /**
- * @deprecated Basé sur supabase donc ne doit pas être utilisé
+ * TODO: not really optimized, because we are still fetching the fiche with another sql query
+ * We should have another trpc endpoint
  */
 export const useGetFichesTotalCount = () => {
   const collectiviteId = useCollectiviteId();
-  const supabase = useSupabase();
-
-  const { data } = useQuery({
-    queryKey: ['fiche_action_count', collectiviteId],
-
-    queryFn: async () => {
-      if (!collectiviteId) {
-        throw new Error('Aucune collectivité associée');
-      }
-
-      return await ficheActionCount({
-        dbClient: supabase,
-        collectiviteId,
-      });
+  const ficheResumesOptions: GetFichesOptions = {
+    filters: {},
+    queryOptions: {
+      page: 1,
+      limit: 1,
     },
-  });
+  };
+  const { data: ficheResumes, isLoading } = useListFiches(
+    collectiviteId,
+    ficheResumesOptions
+  );
 
   return {
-    count: typeof data === 'number' ? data : 0,
+    count: ficheResumes?.count,
+    isLoading,
   };
 };

apps/app/src/plans/fiches/list-all-fiches/data/use-get-fiches.ts

@@ -40,13 +40,14 @@ export const useGetFiches = (
     collectivite.collectiviteId,
     ficheResumesOptions
   );
-  const { count: fichesCount } = useGetFichesTotalCount();
-  const hasFiches = fichesCount > 0;
+  const { count: fichesCount, isLoading: isLoadingCount } =
+    useGetFichesTotalCount();
+  const hasFiches = fichesCount && fichesCount > 0;
   const countTotal = ficheResumes?.count || 0;
 
   return {
     ficheResumes,
-    isLoading,
+    isLoading: isLoading || isLoadingCount,
     hasFiches,
     countTotal,
     collectivite,

apps/app/src/plans/fiches/list-all-fiches/toutes-les-fiches.view.tsx

@@ -71,8 +71,8 @@ const ToutesLesFichesActionContent = () => {
   const fichesNonClasseesCount = useFichesNonClasseesCount();
   const fichesClasseesCount = useFichesClasseesCount();
   const totalCount =
-    fichesNonClasseesCount && fichesClasseesCount
-      ? fichesNonClasseesCount + fichesClasseesCount
+    fichesNonClasseesCount || fichesClasseesCount
+      ? (fichesNonClasseesCount || 0) + (fichesClasseesCount || 0)
       : undefined;
   const { mutate: createFicheAction } = useCreateFicheAction();
   const { filters, ficheType } = useFicheActionFilters();

apps/backend/src/plans/fiches/list-fiches/list-fiches.service.ts

@@ -1254,7 +1254,7 @@ export default class ListFichesService {
     if (filters.noServicePilote) {
       conditions.push(isNull(sql`service_tag_ids`));
     }
-    if (filters.noPlan) {
+    if (filters.noPlan === true) {
       if (!collectiviteId) {
         conditions.push(isNull(sql`plans`));
       } else {
@@ -1271,16 +1271,24 @@ export default class ListFichesService {
           )
         );
       }
+    } else if (filters.noPlan === false) {
+      if (!collectiviteId) {
+        conditions.push(isNotNull(sql`plans`));
+      } else {
+        const collectiviteIdSqlNumberArray = `{${collectiviteId}}`;
+        conditions.push(
+          arrayOverlaps(
+            sql`axes_collectivite_ids`,
+            sql`${collectiviteIdSqlNumberArray}`
+          )
+        );
+      }
     }
+
+    // TODO: make it work for shared collectivite, not simple
     if (filters.doesBelongToSeveralPlans) {
       conditions.push(gt(sql`array_length(plan_ids, 1)`, 1));
     }
-    if (filters.noPlan === false) {
-      conditions.push(isNotNull(sql`plans`));
-    }
-    if (filters.noPlan === true) {
-      conditions.push(isNull(sql`plans`));
-    }
 
     const piloteConditions: (SQLWrapper | SQL | undefined)[] = [];
     if (filters.noPilote) {