From aa0369a8c28f35bfb6fde9f0f290dd562358a954 Mon Sep 17 00:00:00 2001
From: sergiomarquez778 <Sergiomarquez778@gmail.com>
Date: Fri, 28 Mar 2025 13:29:44 -0600
Subject: [PATCH] cambios

solcuion de las validadaciones de marcas y tambien de prestamos
---
 app/Http/Controllers/MarcaController.php    |  5 ++
 app/Http/Controllers/PrestamoController.php | 52 +-------------
 resources/views/prestamos.blade.php         | 79 ++++++++++-----------
 3 files changed, 47 insertions(+), 89 deletions(-)

diff --git a/app/Http/Controllers/MarcaController.php b/app/Http/Controllers/MarcaController.php
index 3b25303..1ec46de 100644
--- a/app/Http/Controllers/MarcaController.php
+++ b/app/Http/Controllers/MarcaController.php
@@ -64,7 +64,9 @@ class MarcaController extends Controller
      * Show the form for editing the specified resource.
      */
     public function edit($id)
+
     {
+
         $marca = Marca::findOrFail($id); // Busca la marca por ID
         return view('marcasCrearEditar', ['marca' => $marca]); // Pasa la marca a la vista
     }
@@ -74,6 +76,9 @@ class MarcaController extends Controller
      */
     public function update(Request $request, $id)
     {
+        $request->validate([
+        'marca' => 'required|string|max:255|unique:marcas,marca', // Asegúrate de que 'marca' sea único
+        ]);
         $marca = Marca::findOrFail($id); // Encuentra la marca por ID
         $marca->marca = $request->marca; // Actualiza el nombre de la marca
         $marca->eliminado = 0; // Cambia el estado a activo si se está editando
diff --git a/app/Http/Controllers/PrestamoController.php b/app/Http/Controllers/PrestamoController.php
index e31dbdb..75f13b7 100644
--- a/app/Http/Controllers/PrestamoController.php
+++ b/app/Http/Controllers/PrestamoController.php
@@ -21,18 +21,15 @@ class PrestamoController extends Controller
             // Busca en la columna 'nombre_solicitante' de la tabla 'prestamos'
             $prestamos = Prestamo::where('nombre_solicitante', 'LIKE', "%{$busqueda}%")
                 ->where('eliminado', 0)
-                ->where('estado', 'pendiente')
                 ->get();
 
             if ($prestamos->isEmpty()) {
-                return redirect()->route('prestamos.index')
+                return redirect()->route('prestamo.index')
                     ->with('error', 'No existe ningún préstamo con el solicitante "' . $busqueda . '". Por favor, inténtalo de nuevo.');
             }
         } else {
-            // Si no hay búsqueda, mostrar todos los préstamos pendientes
-            $prestamos = Prestamo::where('eliminado', 0)
-                ->where('estado', 'pendiente')
-                ->get();
+            // Si no hay búsqueda, mostrar todos los préstamos
+            $prestamos = Prestamo::where('eliminado', 0)->get();
         }
 
         return view('prestamos', ['prestamos' => $prestamos]);
@@ -147,47 +144,4 @@ class PrestamoController extends Controller
         $pdf = PDF::loadView('exports.prestamos-pdf', ['prestamos' => $prestamos]);
         return $pdf->download('prestamos.pdf');
     }
-
-    public function aceptados(Request $request)
-    {
-        $busqueda = $request->busqueda;
-
-        if ($busqueda) {
-            $prestamos = Prestamo::where('nombre_solicitante', 'LIKE', "%{$busqueda}%")
-                ->where('eliminado', 0)
-                ->where('estado', 'aceptado')
-                ->get();
-
-            if ($prestamos->isEmpty()) {
-                return redirect()->route('prestamos.aceptados')
-                    ->with('error', 'No existe ningún préstamo aceptado con el solicitante "' . $busqueda . '". Por favor, inténtalo de nuevo.');
-            }
-        } else {
-            $prestamos = Prestamo::where('eliminado', 0)
-                ->where('estado', 'aceptado')
-                ->get();
-        }
-
-        return view('prestamos.aceptados', ['prestamos' => $prestamos]);
-    }
-
-    public function aceptar($id)
-    {
-        $prestamo = Prestamo::findOrFail($id);
-        $prestamo->estado = 'aceptado';
-        $prestamo->save();
-
-        return redirect()->route('prestamos.index')
-            ->with('success', 'Préstamo aceptado exitosamente.');
-    }
-
-    public function rechazar($id)
-    {
-        $prestamo = Prestamo::findOrFail($id);
-        $prestamo->estado = 'rechazado';
-        $prestamo->save();
-
-        return redirect()->route('prestamos.index')
-            ->with('success', 'Préstamo rechazado exitosamente.');
-    }
 }
diff --git a/resources/views/prestamos.blade.php b/resources/views/prestamos.blade.php
index dbc3f69..a262638 100644
--- a/resources/views/prestamos.blade.php
+++ b/resources/views/prestamos.blade.php
@@ -15,34 +15,44 @@
         </div>
     @endif
 
+    @if($errors->any())
+        <div class="mb-6 bg-red-50 border-l-4 border-red-500 p-4 rounded-r-lg">
+            <div class="flex items-center">
+                <div class="text-red-700">
+                    <ul>
+                        @foreach($errors->all() as $error)
+                            <li>{{ $error }}</li>
+                        @endforeach
+                    </ul>
+                </div>
+            </div>
+        </div>
+    @endif
+
     <div class="bg-white rounded-lg shadow-lg">
         <!-- Encabezado con título y botones de acción -->
         <div class="p-4 border-b border-gray-200 flex justify-between items-center">
             <h2 class="text-2xl font-bold">Gestión de Préstamos</h2>
             <div class="flex items-center space-x-6">
+                <!-- Exportar a Excel -->
+                <a href="{{ route('prestamos.excel') }}"
+                class="text-green-600 hover:text-green-700 transition-colors duration-200"
+                title="Exportar a Excel">
+                 <i class="fas fa-file-excel text-xl"></i>
+             </a>
 
-                <!-- Íconos de exportación -->
-                <div class="flex space-x-4">
-                    <!-- Exportar a Excel -->
-                    <a href="{{ route('prestamos.excel') }}"
-                       class="text-green-600 hover:text-green-700 transition-colors duration-200"
-                       title="Exportar a Excel">
-                        <i class="fas fa-file-excel text-xl"></i>
-                    </a>
-
-                    <!-- Exportar a PDF -->
-                    <a href="{{ route('prestamos.pdf') }}"
-                       class="text-red-600 hover:text-red-700 transition-colors duration-200"
-                       title="Exportar a PDF">
-                        <i class="fas fa-file-pdf text-xl"></i>
-                    </a>
-                    <!-- Agregar nuevo préstamo -->
-                    <a href="{{ route('prestamos.create') }}"
-                       class="text-blue-500 hover:text-blue-600 transition-colors duration-200"
-                       title="Agregar nuevo préstamo">
-                        <i class="fas fa-plus text-xl"></i>
-                    </a>
-                </div>
+             <!-- Exportar a PDF -->
+             <a href="{{ route('prestamos.pdf') }}"
+                class="text-red-600 hover:text-red-700 transition-colors duration-200"
+                title="Exportar a PDF">
+                 <i class="fas fa-file-pdf text-xl"></i>
+             </a>
+                <!-- Agregar nuevo préstamo -->
+                <a href="{{ route('prestamos.create') }}"
+                   class="text-blue-500 hover:text-blue-600 transition-colors duration-200"
+                   title="Agregar nuevo préstamo">
+                    <i class="fas fa-plus text-xl"></i>
+                </a>
             </div>
         </div>
 
@@ -151,28 +161,16 @@
                         </td>
                         <td class="px-6 py-4 whitespace-nowrap text-sm font-medium">
                             <div class="flex space-x-2">
-                                <a href="{{ route('prestamos.edit', $prestamo->id) }}" class="text-blue-600 hover:text-blue-900">
+                                <a href="{{ route('prestamos.edit', $prestamo->id) }}" class="text-yellow-600 hover:text-blue-900">
                                     <i class="fas fa-edit"></i>
                                 </a>
-                                <form action="{{ route('prestamos.destroy', $prestamo->id) }}" method="POST" class="inline">
+                                <form action="{{ route('prestamos.destroy', $prestamo->id) }}" method="POST" class="inline" onsubmit="return confirmarEliminacion(this);">
                                     @csrf
                                     @method('DELETE')
-                                    <button type="submit" class="text-red-600 hover:text-red-900" onclick="return confirm('¿Estás seguro de que deseas eliminar este préstamo?')">
+                                    <button type="submit" class="text-red-600 hover:text-red-900">
                                         <i class="fas fa-trash"></i>
                                     </button>
                                 </form>
-                                <form action="{{ route('prestamos.aceptar', $prestamo->id) }}" method="POST" class="inline">
-                                    @csrf
-                                    <button type="submit" class="text-green-600 hover:text-green-900" onclick="return confirm('¿Estás seguro de que deseas aceptar este préstamo?')">
-                                        <i class="fas fa-check"></i>
-                                    </button>
-                                </form>
-                                <form action="{{ route('prestamos.rechazar', $prestamo->id) }}" method="POST" class="inline">
-                                    @csrf
-                                    <button type="submit" class="text-yellow-600 hover:text-yellow-900" onclick="return confirm('¿Estás seguro de que deseas rechazar este préstamo?')">
-                                        <i class="fas fa-times"></i>
-                                    </button>
-                                </form>
                             </div>
                         </td>
                     </tr>
@@ -196,8 +194,8 @@
         }
     }, 3000);
 
-    function confirmarEliminacion(button) {
-        Swal.fire({
+    function confirmarEliminacion(form) {
+        return Swal.fire({
             title: '¿Estás seguro?',
             text: "Esta acción no se puede deshacer",
             icon: 'warning',
@@ -208,8 +206,9 @@
             cancelButtonText: 'Cancelar'
         }).then((result) => {
             if (result.isConfirmed) {
-                button.closest('form').submit();
+                form.submit();
             }
+            return false; // Evitar el envío del formulario por defecto
         });
     }
 </script>