From 882ba3780f660f115131e917d41b9b724bdff857 Mon Sep 17 00:00:00 2001
From: AdalbertoCV <34152734@uaz.edu.mx>
Date: Mon, 4 Nov 2024 12:43:51 -0600
Subject: [PATCH] =?UTF-8?q?Revisi=C3=B3n=20de=20detalles?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 cosiap_frontend/src/App.jsx                   |  2 +-
 .../components/admin/CrearAdministrador.jsx   |  4 +-
 .../components/admin/TablaAdministradores.jsx |  5 +-
 .../src/components/admin/TablaUsuarios.jsx    |  5 +-
 .../common/utility/RenderElemento.jsx         |  3 ++
 .../src/components/modalidades/Modalidad.jsx  | 10 +++-
 .../src/components/users/Login/LoginForm.jsx  | 50 ++++++++-----------
 7 files changed, 43 insertions(+), 36 deletions(-)

diff --git a/cosiap_frontend/src/App.jsx b/cosiap_frontend/src/App.jsx
index 7d6143c..ddd0e10 100644
--- a/cosiap_frontend/src/App.jsx
+++ b/cosiap_frontend/src/App.jsx
@@ -145,7 +145,7 @@ function RoutesApp({ setViewPageLoader }) {
             <Route path="/historial-solicitante/:id" element={<ListaSolicitudesSolicitante/>} />
             <Route path="/visualizar-solicitud/:id" element={<VisualizarSolicitud/>} />
             <Route path="/administradores" element={<ListaAdmins/>} />
-            <Route path="/nuevo-administrador" element={<CrearAdmin/>} />
+            <Route path="/nuevo-administrador" element={<CrearAdmin setViewPageLoader={setViewPageLoader}/>} />
             <Route path="/formatos" element={<ListaFormatos/>} />
             <Route path="/crear-formato" element={<CrearFormato/>} />
             <Route path="/editar-estilo" element={<EditarColorLogo/>} />
diff --git a/cosiap_frontend/src/components/admin/CrearAdministrador.jsx b/cosiap_frontend/src/components/admin/CrearAdministrador.jsx
index b4d72df..e13a3fa 100644
--- a/cosiap_frontend/src/components/admin/CrearAdministrador.jsx
+++ b/cosiap_frontend/src/components/admin/CrearAdministrador.jsx
@@ -4,7 +4,7 @@ import MainContainer from "../common/utility/MainContainer";
 import '@/App.css';
 import { useNavigate } from "react-router-dom";
 
-const CrearAdmin = () =>{
+const CrearAdmin = ({setViewPageLoader}) =>{
     const [nombre, setNombre] = useState('');
     const [curp, setCurp] = useState('');
     const [email, setEmail] = useState('');
@@ -25,6 +25,7 @@ const CrearAdmin = () =>{
 
     const handleSubmit = async (e) => {
         e.preventDefault();
+        setViewPageLoader(true)
     
         if (password !== passwordConfirm) {
           showAlert("Las contraseñas no coinciden.", false);
@@ -51,6 +52,7 @@ const CrearAdmin = () =>{
                 }
             }
         }
+        setViewPageLoader(false);
       };
 
     return (
diff --git a/cosiap_frontend/src/components/admin/TablaAdministradores.jsx b/cosiap_frontend/src/components/admin/TablaAdministradores.jsx
index 8e694e6..4e96b67 100644
--- a/cosiap_frontend/src/components/admin/TablaAdministradores.jsx
+++ b/cosiap_frontend/src/components/admin/TablaAdministradores.jsx
@@ -37,7 +37,8 @@ const ListaAdmins = () =>{
         fetchAdmins();
     }, []);
 
-    const handleSingleClick = (fila) => {
+    const handleSingleClick = (e,fila) => {
+        e.preventDefault();
         setSelectedRow(fila.pk);
     };
 
@@ -150,7 +151,7 @@ const ListaAdmins = () =>{
             />
         ) : (
             <span
-                onClick={() => handleSingleClick(fila)}  
+                onContextMenu={(e) => handleSingleClick(e,fila)}  
                 onDoubleClick={() => handleEditRow(fila.pk)} 
                 style={{ cursor: "pointer" }}  
             >
diff --git a/cosiap_frontend/src/components/admin/TablaUsuarios.jsx b/cosiap_frontend/src/components/admin/TablaUsuarios.jsx
index 2874608..18f7e22 100644
--- a/cosiap_frontend/src/components/admin/TablaUsuarios.jsx
+++ b/cosiap_frontend/src/components/admin/TablaUsuarios.jsx
@@ -37,7 +37,8 @@ const ListaUsuarios = () => {
         fetchUsuarios();
     }, []);
 
-    const handleSingleClick = (fila) => {
+    const handleSingleClick = (e,fila) => {
+        e.preventDefault();
         setSelectedRow(fila.id); // Muestra el menú para el registro seleccionado
     };
 
@@ -157,7 +158,7 @@ const ListaUsuarios = () => {
             />
         ) : (
             <span
-                onClick={() => handleSingleClick(fila)}  
+                onContextMenu={(e) => handleSingleClick(e,fila)}  
                 onDoubleClick={() => handleEditRow(fila.id)} 
                 style={{ cursor: "pointer" }}  
             >
diff --git a/cosiap_frontend/src/components/common/utility/RenderElemento.jsx b/cosiap_frontend/src/components/common/utility/RenderElemento.jsx
index 497fe5f..2f05c90 100644
--- a/cosiap_frontend/src/components/common/utility/RenderElemento.jsx
+++ b/cosiap_frontend/src/components/common/utility/RenderElemento.jsx
@@ -88,6 +88,9 @@ export const renderElemento = (seccionId, elemento, handleInputChange, handleChe
     case "desplegable":
       return renderInput(
         <select onChange={handleChange} key={id} name={nombre} required={obligatorio} className="select-class">
+          <option value="">
+            Seleccione...
+          </option>
           {Object.keys(opciones).map((opcionId) => (
             <option key={opcionId} value={opciones[opcionId].nombre}>
               {opciones[opcionId].nombre}
diff --git a/cosiap_frontend/src/components/modalidades/Modalidad.jsx b/cosiap_frontend/src/components/modalidades/Modalidad.jsx
index d3e1c3f..2419bea 100644
--- a/cosiap_frontend/src/components/modalidades/Modalidad.jsx
+++ b/cosiap_frontend/src/components/modalidades/Modalidad.jsx
@@ -206,8 +206,7 @@ const SolicitarModalidad = () => {
   };
 
   const handleSubmit = async () => {
-    console.log("Respuestas:", respuesta);
-
+  
     // Filtrar los elementos obligatorios y verificar si están contestados
     const elementosObligatoriosSinRespuesta = Object.keys(secciones).flatMap(seccionId => {
         return Object.values(secciones[seccionId].elementos)
@@ -230,6 +229,13 @@ const SolicitarModalidad = () => {
         return;
     }
 
+     // Validar que las respuestas de tipo archivo sean solo PDF
+    const invalidFiles = respuesta.filter(res => res.valor instanceof File && !res.valor.name.toLowerCase().endsWith('.pdf'));
+    if (invalidFiles.length > 0) {
+        showAlert('Por favor, asegúrate de que todos los documentos estén en formato PDF.', false);
+        return;
+    }
+
     // Crear FormData para enviar en formato multipart/form-data
     const formData = new FormData();
     formData.append('monto_solicitado', montoSolicitado);
diff --git a/cosiap_frontend/src/components/users/Login/LoginForm.jsx b/cosiap_frontend/src/components/users/Login/LoginForm.jsx
index d2efa3d..15268e2 100644
--- a/cosiap_frontend/src/components/users/Login/LoginForm.jsx
+++ b/cosiap_frontend/src/components/users/Login/LoginForm.jsx
@@ -1,71 +1,66 @@
 import { useForm } from "react-hook-form";
 import { yupResolver } from "@hookform/resolvers/yup";
-//Importa el objeto de validacion del formulario
 import { LoginValidationSchema } from "@/components/FormsValidations";
-
-import api from "@/api"; // Asegúrate de importar tu instancia de API
-import {LoginInputCURP} from '@/components/users/Login/LoginInputCURP'
+import api from "@/api";
+import { LoginInputCURP } from '@/components/users/Login/LoginInputCURP';
 import { LoginInputPassword } from "@/components/users/Login/LoginInputPassword";
 import { useState } from "react";
-import {ErrorDisplay} from '@/components/common/ui/ErrorDisplay'
-import {useAutenticacion} from "@/components/common/utility/Autenticador"
+import { ErrorDisplay } from '@/components/common/ui/ErrorDisplay';
+import { useAutenticacion } from "@/components/common/utility/Autenticador";
 import { useNavigate } from "react-router-dom";
 
-export function LoginForm( {setViewPageLoader} ) {
+export function LoginForm({ setViewPageLoader }) {
     const [loginError, setLoginError] = useState('');
-    const navigate = useNavigate()
+    const navigate = useNavigate();
     const { setToken, setIsAdmin, configurarInterceptors, setUid } = useAutenticacion();
-    const {
-        register,
-        handleSubmit,
-        formState: { errors, isSubmitting },
-    } = useForm({
+    const { register, handleSubmit, formState: { errors, isSubmitting } } = useForm({
         resolver: yupResolver(LoginValidationSchema),
     });
-    
+
     const onSubmit = async (data) => {
-        setViewPageLoader(true)
+        setViewPageLoader(true);
+
         try {
+            console.log("Datos del formulario antes del envío:", data);
             const response = await api.usuarios.token.login(data);
             console.log("Login successful:", response.data);    
             setToken(response.data.access);
-            //Configuramos inteceptores
             configurarInterceptors(response.data.access);
-            //Establecemos si es admin o no
+
             const responseAd = await api.usuarios.admin.is_admin();
             setIsAdmin(responseAd.data.user_is_admin);
-            //Extraemos el id
+
             const responseUid = await api.usuarios.getId();
             setUid(responseUid.data.user_id);
+
             navigate('/inicio');
         } catch (error) {
             console.error("Login failed:", error);
-            setLoginError(error.response.data.detail)
+            setLoginError(error.response?.data?.detail || "Error de inicio de sesión");
         }
-        setViewPageLoader(false)
+        setViewPageLoader(false);
     };
 
     return (
         <div className="mt-10 sm:mx-auto sm:w-full sm:max-w-sm">
             <form onSubmit={handleSubmit(onSubmit)} className="space-y-6">
-            <ErrorDisplay errors={loginError} />
+                <ErrorDisplay errors={loginError} />
 
                 <div className="mt-2 relative">
-                    <LoginInputCURP                    
+                    <LoginInputCURP
                         name="curp"
                         type="text"
                         placeholder="CURP"
-                        className="pl-10"                      
+                        className="pl-10"
                         register={register}
                         errors={errors.curp ? errors.curp.message : undefined}
                     />
-                    
                 </div>
                 <div className="mt-2 relative">
-                    <LoginInputPassword              
+                    <LoginInputPassword
                         name="password"
-                        placeholder="CONTRASEÑA" 
-                        className="pl-10"                   
+                        placeholder="CONTRASEÑA"
+                        className="pl-10"
                         register={register}
                         errors={errors.password ? errors.password.message : undefined}
                     />
@@ -80,7 +75,6 @@ export function LoginForm( {setViewPageLoader} ) {
                     </button>
                 </div>
             </form>
-            
         </div>
     );
 }
-- 
GitLab