Merge pull request #8 from SashaMGuerra/developer204

Arreglado cambio de contraseña close #6
SashaMGuerra · Dec 14, 2021 · 69385d2 · 69385d2
2 parents f9b4e0b + 7f6613a
commit 69385d2
Show file tree

Hide file tree

Showing 5 changed files with 33 additions and 19 deletions.
diff --git a/codigoPHP/cambiarPassword.php b/codigoPHP/cambiarPassword.php
@@ -82,13 +82,15 @@
      */
     if (!$oResultado) {
         $aErrores['passwordActual'] = 'Contraseña incorrecta.';
+        $bEntradaOK = false;
     }
     /*
      * Si la contraseña coincide con la introducida, valida la nueva y comprueba
      * si la introducida repetida es igual.
      */
     else{
         // Si la descripción no cumple con lo especificado, mostrará el error.
+        $aErrores['passwordActual'] = validacionFormularios::validarPassword($_REQUEST['passwordActual'], 8, 4, 1);
         $aErrores['passwordNueva'] = validacionFormularios::validarPassword($_REQUEST['passwordNueva'], 8, 4, 1);
 
         // Si la nueva contraseña no coincide con la repetida, añade el error.

diff --git a/codigoPHP/editarPerfil.php b/codigoPHP/editarPerfil.php
@@ -381,6 +381,14 @@
         </main>
 <?php include_once './elementoFooter.php'; //Footer          ?>
         <script>
+            /**
+             * Si el checkbox que ejecuta esta función (el de eliminar la imagen
+             * de usuario existente) se checa, elimina de la página el input de
+             * subida de imagen de usuario.
+             * Si el checkbox se desactiva, lo vuelve a mostrar. 
+             * 
+             * @param elemento checkbox Checkbox que ejecuta la función.
+             */
             function ocultarSubidaImagen(checkbox) {
                 if (checkbox.checked) {
                     document.getElementById('imagenUsuario').style.display = 'none';

diff --git a/codigoPHP/programa.php b/codigoPHP/programa.php
@@ -87,17 +87,6 @@
                 margin-top: 0;
             }
 
-            @media (max-width: 650px){
-                main{
-                    flex-flow: column;
-                    justify-content: flex-start;
-                    align-items: center;
-                }
-                section:last-of-type{
-                    order: -1;
-                }
-            }
-
             div.bienvenida{
                 text-align: justify;
             }
@@ -121,6 +110,21 @@
                 max-height: 500px;
                 border-radius: 5px;
             }
+
+            @media (max-width: 650px){
+                main{
+                    flex-flow: column;
+                    justify-content: flex-start;
+                    align-items: center;
+                }
+                section:last-of-type{
+                    order: -1;
+                }
+                section:last-of-type img{
+                    max-height: 250px;
+                    max-width: 500px;
+                }
+            }
         </style>
     </head>
     <body>
@@ -155,7 +159,7 @@
             <section>
                 <?php // Si el usuario tiene imagen de usuario, la muestra. Si no, muestra una de las de por defecto.
                 if ($oResultado->T01_ImagenUsuario) { ?>
-                    <img src="data:image/jpg;base64, <?php echo $oResultado->T01_ImagenUsuario ?>" alt="imagen de usuario">
+                    <img src="data:image/gif;base64, <?php echo $oResultado->T01_ImagenUsuario ?>" alt="imagen de usuario">
                 <?php } else{ ?>
                     <script>document.write(`<img src="../webroot/media/img/randomDefault/${Math.floor(Math.random()*5)}.jpg" alt="imagen de usuario"/>`);</script>
                 <?php } ?>

diff --git a/core/libreriaValidacion.php b/core/libreriaValidacion.php
@@ -498,14 +498,14 @@ public static function validarTelefono($tel, $obligatorio = 0) { //AÑADIDO PARA
      * @author Isabel Martínez Guerra.
      * @version 1.0 Creada la función.
      * @since 2021-12-13
-     * @param string $nombreArchivo Cadena que se va a comprobar.
-     * @param array $aExtensiones Tamaño máximo de la cadena.
-     * @param int $maxTamanio Tamaño mínimo de la cadena.
-     * @param int $minTamanio Valor booleano indicado mediante 1, si es obligatorio o 0 si no lo es.
-     * @param boolean $obligatorio
+     * @param string $nombreArchivo Cadena con el nombre del archivo que se va a comprobar.
+     * @param array $aExtensiones Array con las posibles extensiones que puede tener el archivo.
+     * @param int $maxTamanio Tamaño máximo del nombre del archivo (contando la extensión y el punto).
+     * @param int $minTamanio Tamaño mínimo del nombre del archivo (contando la extensión y el punto).
+     * @param boolean $obligatorio Valor booleano indicado mediante 1, si es obligatorio o 0 si no lo es.
      * @return null|string Devuelve null si es correcto o un mensaje de error en caso de que lo haya.
      */
-    public static function validarNombreArchivo($nombreArchivo, $aExtensiones, $maxTamanio = 150, $minTamanio = 1, $obligatorio = 0) {
+    public static function validarNombreArchivo($nombreArchivo, $aExtensiones, $maxTamanio = 150, $minTamanio = 4, $obligatorio = 0) {
         // Validación del texto del nombre del archivo.
         $mensajeError = self::comprobarAlfaNumerico($nombreArchivo, $maxTamanio, $minTamanio, $obligatorio);
 

diff --git a/todo b/todo
@@ -2,7 +2,7 @@ index ->    sin poner la ruta total, debe abrirse el index (LoginLogout/) (htacc
 
 login ->    botón ver password
 
-editarPerfil -> imagen de usuario: sólo jpg y png. Gif hace algo raro.
+editarPerfil -> imagen de usuario: sólo imágenes cuadradas.
 
 el nombre de la aplicación se tiene que ver siempre