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

Compatibilità MySQL #355

Closed
Dasc3er opened this issue Nov 16, 2018 · 0 comments
Closed

Compatibilità MySQL #355

Dasc3er opened this issue Nov 16, 2018 · 0 comments
Labels
da definire Modifiche da pianficare e approfondire discussione Discussioni sulle funzionalità e i loro utilizzo
Milestone

Comments

@Dasc3er
Copy link
Contributor

Dasc3er commented Nov 16, 2018

Comportamento richiesto

Con l'introduzione della gestione delle componenti del gestionale tramite Eloquent, inizia a verificarsi un problema di retro-compatibilità dovuto al fatto che le installazioni con versioni di MySQL precedenti alla 5.6.5 non presentano il campo updated_at.
Questo è causato dal fatto che, per progettazione iniziale, il campo updated_at deve aggiornarsi in automatico alla data di modifica del record perchè altrimenti sarebbe stato impossibile modificare di conseguenza tutte le query UPDATE.

L'introduzione di Eloquent permette però una semplificazione notevole: utilizzando gli oggetti relativi, i campi updated_at e created_at vengono aggiornati automaticamente da Eloquent, e quindi in futuro il requisito di MySQL potrebbe essere nuovamente rimosso. Questo però sembra aver portato ad alcune problematiche in alcuni aggiornamenti (https://www.openstamanager.com/forum/viewtopic.php?f=4&t=91527&p=98362#p98362, https://www.openstamanager.com/forum/viewtopic.php?f=2&t=91523).

Possiamo quindi procedere in due modi:

  • Bloccare il supporto a installazioni con MySQL <= 5.6.5
  • Bloccare Eloquent nell'aggiornamento del campo updated_at

La seconda scelta permetterebbe al sistema di continuare a funzionare in ambienti non aggiornati, ma rimanderebbe il problema alla prossima incompatibilità che troveremo; il primo punto permetterebbe invece di avere una maggiore conformità tra tutte le installazioni, e garantirebbe il funzionamento dell'API.

Comportamento attuale

L'aggiornamento automatico di Eloquent per il campo updated_at viene bloccato, ma dipende da una sezione di codice aggiuntivo che dobbiamo implementare di volta in volta attraverso la classe personalizzata Common\Model.
Il fix per le classi di base è nel commit 1b6f186.

@Dasc3er Dasc3er added this to the OSM 2.5 milestone Nov 16, 2018
@Dasc3er Dasc3er added da definire Modifiche da pianficare e approfondire discussione Discussioni sulle funzionalità e i loro utilizzo labels Nov 16, 2018
@Dasc3er Dasc3er pinned this issue May 24, 2019
@Dasc3er Dasc3er unpinned this issue Dec 17, 2019
@Pek5892 Pek5892 closed this as completed Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
da definire Modifiche da pianficare e approfondire discussione Discussioni sulle funzionalità e i loro utilizzo
Projects
None yet
Development

No branches or pull requests

2 participants