search

UNIMOODLE / moodle-mod_certifygen

GNU General Public License v3.0
1 stars 1 forks source link

Falta la validación del comando #57

Closed tmas0 closed 3 weeks ago

tmas0 commented 1 month ago

Buenas,

Según el pliego:

  1. Este proceso ha de permitir configurar un comando del sistema operativo del servidor de Moodle pasándole los parámetros que se requiera. Este comando recibirá el documento PDF del certificado por la entrada estándar y se obtendrá el documento modificado de la salida estándar.

  2. Si el plugin no detecta el comando de sistema, no ha de ofrecer este proceso al usuario.

Este segundo punto no se cumple, ya que revisando la interfaz y el código no se valida. Entiendo que esto debería ser como los "path de sistema", dónde se valida de que exista. Luego en código deberá revisar este punto, a fin de no devolver errores.

Un saludo

xpazv commented 1 month ago

En revisión.

elena3ip commented 1 month ago

En el settings he añadio una validación, si la ruta no existe sale el siguiente mensaje image

Ya está en develop.

tmas0 commented 1 month ago

Buenas @elena3ip,

Aunque me parece bien lo implementado, lo estándar en Moodle es marcarlo de forma más común, como por ejemplo en la sección de backups o system paths. Adjunto capturas:

Selecció_1329 Selecció_1330

Un saludo

elena3ip commented 1 month ago

He visto que este elemento es de otro tipo de settings es de otro tipo e incluye la validación del path en cuestión. Lo cambio. Gracias por la aportación.

elena3ip commented 1 month ago

Ya está el cambio en develop

tmas0 commented 1 month ago

Buenas @elena3ip,

El primer field ya está bien, pero los dos otros no se validan.

Selecció_1331

Un saludo

elena3ip commented 3 weeks ago

Modificado! ya está el cambio en develop

jmerinogranizo commented 3 weeks ago

A mi me funcionan originalfilespath y validatedfilespath, pero aún cuando el settings path sea válido simpre me sale la aspa roja, aunque no me sale el texto de debajo de Path no válido.