UNIMOODLE / moodle-local_coursetransfer

Lote P3.2 - Restauración de cursos entre plataformas
2 stars 2 forks source link

Cambio de nomenclatura "destiny" debe ser "target" #11

Closed juacas closed 1 month ago

juacas commented 3 months ago

El término destiny se está usando como "target". Realmente "destiny" significa "futuro", "predestinación".

De cara a la publicación del software, es recomendable refactorizar ese término para usarlo correctamente y que no manche la reputación de los programadores.

xpazv commented 3 months ago

En revisión, pues supone cambio de impacto en código.

xpazv commented 3 months ago

En proceso de refactorización.

xpazv commented 3 months ago

Indicar @juacas que el alcance del cambio requiere hacer upgrade con modificación de tablas y columnas ya creadas, respecto a la última versión disponible.

juacas commented 3 months ago

Correcto. Hay que implementar el upgrade.php para que el cambio sea transparente entre versiones

xpazv commented 3 months ago

Asi lo estamos afrontando @juacas era solo para hacer notar el alcance de la refactorización, ya que en vuestro caso, disponeis de una versión antigua, y un upgrade tan complejo puede ser bastante problemático.

juacas commented 3 months ago

Un cambio de nombre de una columna de BBDD y el renombrado de variables en el código debería ser totalmente transparente para las actualizaciones. Nunca me he encontrado con problemas en estas operaciones. El único punto problemático podría ser la comunicación entre nodos con versiones distintas del plugin debido a la codificación de los mensajes (si es que aparece "destiny" ahí).

No obstante, avisad a las universidades que están probando el módulo para identificar esos problemas.

xpazv commented 3 months ago

Aparece en diferentes ámbitos, de ahí el riesgo si en un mismo entorno, conviven diferentes versiones.

Avisaremos a las universidades, tal y como indicas.

xpazv commented 2 months ago

Ya se ha implementado. Estamos realizando pruebas de todo el flujo completo, de nuevo, antes de subir la version correspondiente al repositorio @juacas