Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improvement - General - Mejoras internas para el proceso de actualización #270

Draft
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

jordiSTIC
Copy link
Collaborator

@jordiSTIC jordiSTIC commented Jun 13, 2024

Descripción

Este PR implementa las siguientes mejoras internas:

  • Reparación para Updates: Se ha creado la clase SticRepairAndRebuild (en SticInclude/SticRepairAndRebuild.php) que realiza las acciones de reparación (eliminando acciones que se realizaban por duplicado) separadas por funciones:
    • sticUpdateInstances: Para el 'scope' updateInstances
    • sticOnlyDatabase: Para el 'scope' onlyDatabase
    • sticClearAll: Para el 'scope' vacío
  • SticRunScripts: Se ha añadido la opción de poder indicar que se ejecuten todos los scripts que estén en una carpeta indicada como parámetro (folder).
  • SticUpdates/Migrations/BeforeUpdate: Se ha creado una carpeta donde almacenar aquellos scripts (sql o php) que se deban ejecutar en las actualizaciones antes de la reparación de la instancia.

Pruebas

  1. Crear un fichero .php y otro .sql en la carpeta SticUpdates/Migrations/BeforeUpdate o: descargar testPR270.zip, descomprimir y copiar en la carpeta SticUpdates/Migrations/BeforeUpdate
  2. En el navegador, acceder a la url: <INSTANCIA>/SticRunScripts.php?folder=SticUpdates/Migrations/BeforeUpdate
  3. Verificar que se han ejecutado correctamente los scripts de la carpeta

@jordiSTIC jordiSTIC added General Improvement ... that do not affect user functionality not urgent labels Jun 13, 2024
@jordiSTIC jordiSTIC self-assigned this Jun 13, 2024
Copy link

github-actions bot commented Jun 13, 2024

Actions executed at: 2024-06-18 14:57:48.

@jordiSTIC jordiSTIC marked this pull request as ready for review June 13, 2024 17:15
@AlbertoSTIC AlbertoSTIC marked this pull request as draft August 28, 2024 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
General Improvement ... that do not affect user functionality not urgent
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant