Skip to content

Aggiunge cache per le configurazioni a database#463

Merged
Kasui92 merged 9 commits intodev57from
dev57_feat_config_cache
Apr 20, 2026
Merged

Aggiunge cache per le configurazioni a database#463
Kasui92 merged 9 commits intodev57from
dev57_feat_config_cache

Conversation

@blancks
Copy link
Copy Markdown
Member

@blancks blancks commented Apr 17, 2026

Questa PR introduce un sistema di caching su file per quanto riguarda la feature delle configurazioni a database.

Questa funzionalità si è resa necessaria a causa del fatto che il richiamo di ogni configurazione produce una query a database e punta ad evitare scenari di sovraffollamento di queries nel caso in cui occorra recuperare tante config durante una stessa esecuzione.

In particolare:

  • la funzionalità delle configurazioni a database sono state isolate in un loro functions dedicato: functions.configuration.inc.php
  • la funzione gdrcd_configuration_get è stata modificata affinché generi la cache quando non presente, e recuperi i dati leggendo dalle informazioni in cache invece che con una query.
  • la cache viene rigenerata ad ogni salvataggio all'interno del pannello gestione_config

La cache viene salvata all'interno di un nuovo path definito per questo genere di files: /cache.

La cartella viene generata automaticamente se non trovata.

@blancks blancks requested a review from Kasui92 April 17, 2026 23:20
@blancks blancks self-assigned this Apr 17, 2026
@blancks blancks added the v5.7 Indica che fa parte di GDRCD v5.7 label Apr 17, 2026
@blancks blancks added this to the 5.7.0 milestone Apr 17, 2026
@Kasui92
Copy link
Copy Markdown
Member

Kasui92 commented Apr 20, 2026

Tutto funziona alla grande! 🚀

@Kasui92 Kasui92 merged commit ac29b49 into dev57 Apr 20, 2026
@Kasui92 Kasui92 deleted the dev57_feat_config_cache branch April 20, 2026 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v5.7 Indica che fa parte di GDRCD v5.7

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants