Merge pull request 'Añadir Bibliografía #128' (#132) from issue/128-aadir-bibliografa into main

Reviewed-on: #132

src/components/asignaturas/detalle/BibliographyItem.tsx

@@ -1,6 +1,9 @@
+/* eslint-disable jsx-a11y/click-events-have-key-events */
+/* eslint-disable jsx-a11y/label-has-associated-control */
+/* eslint-disable jsx-a11y/no-static-element-interactions */
 import { useParams } from '@tanstack/react-router'
 import { Plus, Search, BookOpen, Trash2, Library, Edit3 } from 'lucide-react'
-import { useEffect, useState } from 'react'
+import { useState } from 'react'
 
 import {
   AlertDialog,
@@ -31,7 +34,12 @@ import {
   SelectValue,
 } from '@/components/ui/select'
 import { Textarea } from '@/components/ui/textarea'
-import { useSubjectBibliografia } from '@/data/hooks/useSubjects'
+import {
+  useCreateBibliografia,
+  useDeleteBibliografia,
+  useSubjectBibliografia,
+  useUpdateBibliografia,
+} from '@/data/hooks/useSubjects'
 import { cn } from '@/lib/utils'
 
 // --- Interfaces ---
@@ -50,9 +58,16 @@ export function BibliographyItem() {
     from: '/planes/$planId/asignaturas/$asignaturaId',
   })
 
-  const { data: bibliografia2, isLoading: loadinasignatura } =
+  // --- 1. Única fuente de verdad: La Query ---
+  const { data: bibliografia = [], isLoading } =
     useSubjectBibliografia(asignaturaId)
-  const [entries, setEntries] = useState<Array<BibliografiaEntry>>([])
+
+  // --- 2. Mutaciones ---
+  const { mutate: crearBibliografia } = useCreateBibliografia()
+  const { mutate: actualizarBibliografia } = useUpdateBibliografia(asignaturaId)
+  const { mutate: eliminarBibliografia } = useDeleteBibliografia(asignaturaId)
+
+  // --- 3. Estados de UI (Solo para diálogos y edición) ---
   const [isAddDialogOpen, setIsAddDialogOpen] = useState(false)
   const [isLibraryDialogOpen, setIsLibraryDialogOpen] = useState(false)
   const [deleteId, setDeleteId] = useState<string | null>(null)
@@ -61,29 +76,27 @@ export function BibliographyItem() {
     'BASICA',
   )
 
-  useEffect(() => {
+  console.log('Datos actuales en el front:', bibliografia)
-    console.log(entries)
+  // --- 4. Derivación de datos (Se calculan en cada render) ---
-
+  const basicaEntries = bibliografia.filter((e) => e.tipo === 'BASICA')
-    if (bibliografia2 && Array.isArray(bibliografia2)) {
+  const complementariaEntries = bibliografia.filter(
-      setEntries(bibliografia2)
-    }
-  }, [bibliografia2])
-
-  const basicaEntries = entries.filter((e) => e.tipo === 'BASICA')
-  const complementariaEntries = entries.filter(
     (e) => e.tipo === 'COMPLEMENTARIA',
   )
-  console.log(bibliografia2)
+
+  // --- Handlers Conectados a la Base de Datos ---
 
   const handleAddManual = (cita: string) => {
-    const newEntry: BibliografiaEntry = {
+    crearBibliografia(
-      id: `manual-${Date.now()}`,
+      {
+        asignatura_id: asignaturaId,
         tipo: newEntryType,
         cita,
-    }
+        tipo_fuente: 'MANUAL',
-    setEntries([...entries, newEntry])
+      },
-    setIsAddDialogOpen(false)
+      {
-    // toast.success('Referencia manual añadida');
+        onSuccess: () => setIsAddDialogOpen(false),
+      },
+    )
   }
 
   const handleAddFromLibrary = (
@@ -91,22 +104,43 @@ export function BibliographyItem() {
     tipo: 'BASICA' | 'COMPLEMENTARIA',
   ) => {
     const cita = `${resource.autor} (${resource.anio}). ${resource.titulo}. ${resource.editorial}.`
-    const newEntry: BibliografiaEntry = {
+    crearBibliografia(
-      id: `lib-ref-${Date.now()}`,
+      {
+        asignatura_id: asignaturaId,
         tipo,
         cita,
-      fuenteBibliotecaId: resource.id,
+        tipo_fuente: 'BIBLIOTECA',
-      fuenteBiblioteca: resource,
+        biblioteca_item_id: resource.id,
-    }
+      },
-    setEntries([...entries, newEntry])
+      {
-    setIsLibraryDialogOpen(false)
+        onSuccess: () => setIsLibraryDialogOpen(false),
-    // toast.success('Añadido desde biblioteca');
+      },
+    )
   }
 
-  const handleUpdateCita = (id: string, cita: string) => {
+  const handleUpdateCita = (id: string, nuevaCita: string) => {
-    setEntries(entries.map((e) => (e.id === id ? { ...e, cita } : e)))
+    actualizarBibliografia(
+      {
+        id,
+        updates: { cita: nuevaCita },
+      },
+      {
+        onSuccess: () => setEditingId(null),
+      },
+    )
   }
 
+  const onConfirmDelete = () => {
+    if (deleteId) {
+      eliminarBibliografia(deleteId, {
+        onSuccess: () => setDeleteId(null),
+      })
+    }
+  }
+
+  if (isLoading)
+    return <div className="p-10 text-center">Cargando bibliografía...</div>
+
   return (
     <div className="animate-in fade-in mx-auto max-w-5xl space-y-8 py-10 duration-500">
       <div className="flex items-center justify-between border-b pb-4">
@@ -134,9 +168,13 @@ export function BibliographyItem() {
             </DialogTrigger>
             <DialogContent className="max-w-2xl">
               <LibrarySearchDialog
-                resources={bibliografia2 || []}
+                // CORRECCIÓN: Usamos 'bibliografia' en lugar de 'bibliografia2'
+                resources={[]} // Aquí deberías pasar el catálogo general, no la bibliografía de la asignatura
                 onSelect={handleAddFromLibrary}
-                existingIds={entries.map((e) => e.fuenteBibliotecaId || '')}
+                // CORRECCIÓN: Usamos 'bibliografia' en lugar de 'entries'
+                existingIds={bibliografia.map(
+                  (e) => e.biblioteca_item_id || '',
+                )}
               />
             </DialogContent>
           </Dialog>
@@ -216,13 +254,7 @@ export function BibliographyItem() {
           </AlertDialogHeader>
           <AlertDialogFooter>
             <AlertDialogCancel>Cancelar</AlertDialogCancel>
-            <AlertDialogAction
+            <AlertDialogAction onClick={onConfirmDelete} className="bg-red-600">
-              onClick={() => {
-                setEntries(entries.filter((e) => e.id !== deleteId))
-                setDeleteId(null)
-              }}
-              className="bg-red-600"
-            >
               Eliminar
             </AlertDialogAction>
           </AlertDialogFooter>
@@ -412,7 +444,7 @@ function LibrarySearchDialog({ resources, onSelect, existingIds }: any) {
         </Select>
       </div>
       <div className="max-h-[300px] space-y-2 overflow-y-auto pr-2">
-        {filtered.map((res) => (
+        {filtered.map((res: any) => (
           <div
             key={res.id}
             onClick={() => onSelect(res, tipo)}

src/data/api/subjects.api.ts

@@ -490,3 +490,51 @@ export async function lineas_delete(lineaId: string) {
   if (error) throw error
   return lineaId
 }
+
+export async function bibliografia_insert(entry: {
+  asignatura_id: string
+  tipo: 'BASICA' | 'COMPLEMENTARIA'
+  cita: string
+  tipo_fuente: 'MANUAL' | 'BIBLIOTECA'
+  biblioteca_item_id?: string | null
+}) {
+  const supabase = supabaseBrowser()
+  const { data, error } = await supabase
+    .from('bibliografia_asignatura')
+    .insert([entry])
+    .select()
+    .single()
+
+  if (error) throw error
+  return data
+}
+
+export async function bibliografia_update(
+  id: string,
+  updates: {
+    cita?: string
+    tipo?: 'BASICA' | 'COMPLEMENTARIA'
+  },
+) {
+  const supabase = supabaseBrowser()
+  const { data, error } = await supabase
+    .from('bibliografia_asignatura')
+    .update(updates) // Ahora 'updates' es compatible con lo que espera Supabase
+    .eq('id', id)
+    .select()
+    .single()
+
+  if (error) throw error
+  return data
+}
+
+export async function bibliografia_delete(id: string) {
+  const supabase = supabaseBrowser()
+  const { error } = await supabase
+    .from('bibliografia_asignatura')
+    .delete()
+    .eq('id', id)
+
+  if (error) throw error
+  return id
+}

src/data/hooks/useSubjects.ts

@@ -3,6 +3,9 @@ import { useMutation, useQuery, useQueryClient } from '@tanstack/react-query'
 import {
   ai_generate_subject,
   asignaturas_update,
+  bibliografia_delete,
+  bibliografia_insert,
+  bibliografia_update,
   lineas_insert,
   lineas_update,
   subjects_bibliografia_list,
@@ -276,3 +279,41 @@ export function useUpdateLinea() {
     },
   })
 }
+
+export function useCreateBibliografia() {
+  const queryClient = useQueryClient()
+  return useMutation({
+    mutationFn: bibliografia_insert,
+    onSuccess: (data) => {
+      // USAR LA MISMA LLAVE QUE EL HOOK DE LECTURA
+      queryClient.invalidateQueries({
+        queryKey: qk.asignaturaBibliografia(data.asignatura_id),
+      })
+    },
+  })
+}
+
+export function useUpdateBibliografia(asignaturaId: string) {
+  const qc = useQueryClient()
+  return useMutation({
+    mutationFn: ({ id, updates }: { id: string; updates: any }) =>
+      bibliografia_update(id, updates),
+    onSuccess: () => {
+      qc.invalidateQueries({
+        queryKey: qk.asignaturaBibliografia(asignaturaId),
+      })
+    },
+  })
+}
+
+export function useDeleteBibliografia(asignaturaId: string) {
+  const queryClient = useQueryClient()
+  return useMutation({
+    mutationFn: (id: string) => bibliografia_delete(id),
+    onSuccess: () => {
+      queryClient.invalidateQueries({
+        queryKey: qk.asignaturaBibliografia(asignaturaId),
+      })
+    },
+  })
+}