# Piano di Refactoring: Migrazione a FusionCache in `MpDataService.cs`

Stiamo lavorando sul progetto MP-SPEC.sln, dentro la cartella MP-SPEC (ed i progetti da cui dipende).

Voglio ottimizzare il file Data\MpDataService.cs

## Obiettivo
Migrare la logica di caching manuale (Redis + DB) verso l'utilizzo di `IFusionCache` per implementare un approccio multi-layer (Memory + Redis + DB), standardizzando l'accesso ai dati.

## Strategia di Migrazione
- **Metodo Standard**: `GetOrFetchAsync<T>(string operationName, string cacheKey, Func<Task<T>> fetchFunc, TimeSpan expiration, params string[] tagList)`.
- **Invalidazione**: Utilizzare i tag tramite `FlushCacheByTagsAsync`.

## Stato Avanzamento

### Fase 1: Analisi e Mapping (Completata)
- Analisi di `MpDataService.cs` effettuata.
- Identificati i metodi con caching manuale (Redis/DB) e quelli già migrati a `GetOrFetchAsync`.

### Fase 2: Refactoring Metodi di Lettura (Cache-aside) (In corso)

#### ✅ Metodi Migrati (Usano già `GetOrFetchAsync` o `FusionCache.GetOrSet`)
- `AnagEventiGeneralAsync`
- `AnagStatiCommAsync`
- `AnagTipoArtLvAsync`
- `ArticleWithDossierAsync`
- `ArticoliCountAsync`
- `ArticoliCountSearchAsync`
- `ArticoliGetByTipoAsync`
- `ArticoliGetSearchAsync`
- `ArticoliInKitAsync`
- `ConfigGetAllAsync`
- `DossiersGetLastFiltAsync`
- `ElencoAziendeAsync`
- `ElencoGruppiFaseAsync`
- `ElencoLinkAsync`
- `IstKitFiltAsync`
- `ListGiacenzeAsync`
- `MacchineGetFiltAsync`
- `MacchineRecipeArchiveAsync`
- `MacchineRecipeConfAsync`
- `MacchineWithFluxAsync`
- `MachineWithOdlAsync`
- `MachIobConfAsync`
- `OdlByBatchAsync` (Migrato e rinominato)
- `OdlListGetFiltAsync`
- `OperatoriGetFiltAsync`
- `ParametriGetFiltAsync`
- `PODL_getDictOdlPodlAsync` (Migrato con gestione manuale L1/L2)
- `POdlGetByOdlAsync`
- `POdlToKitListGetFiltAsync`
- `StatoMacchinaAsync`
- `TksScoreAsync`
- `Traduci` (Migrato con `GetOrSet` su dizionario lingua)
- `WipKitFiltAsync`
- `MseGetAllAsync` (Migrato e rinominato)
- `OdlByKeyAsync` (Migrato e rinominato)
- `TemplateKitDeleteAsync` (Migrato con tag invalidazione)
- `TemplateKitUpsertAsync` (Migrato con tag invalidazione)
- `WipKitDeleteAsync` (Migrato con tag invalidazione)
- `WipKitUpsertAsync` (Migrato con tag invalidazione)

#### 🛠️ Metodi da Migrare (Usano ancora Redis/DB manuale)
- [ ] Migrazione di `ActionGetReq` (linea 110: usa `redisDb.StringGetAsync`).
- [ ] Migrazione di `ActionSetReq` (linea 136: usa `BroadastMsgPipe.saveAndSendMessage`).
- [ ] Migrazione di `AnagGruppiDelete`/`Upsert` (linea 189/208: usa `ExecFlushRedisPattern`).
- [ ] Migrazione di `ArticoliDeleteRecord`/`UpdateRecord` (linea 296/372: usa `resetCacheArticoli`).
- [ ] Migrazione di `DbDedupStats` (linea 516: usa `redisDb.StringGet`).
- [ ] Migrazione di `DossiersDeleteRecord` (linea 554: usa `ExecFlushRedisPatternAsync`).
- [ ] Migrazione di `DossiersTakeParamsSnapshotLast` (linea 613: usa `ExecFlushRedisPatternAsync`).
- [ ] Migrazione di `ElencoRepartiDTO` (linea 697: usa `redisDb.StringGet` e `StringSet`).
- [ ] Migrazione di `PodlIstKitDelete` (linea 1842: usa `ExecFlushRedisPattern`).
- [ ] Migrazione di `POdlListByKitParent` (linea 1863: usa `redisDb.StringGet` e `StringSet`).
- [ ] Migrazione di `ProcFLStats` (linea 1992: usa `redisDb.StringSet`).
- [ ] Migrazione di `RecDbMaintStat` (linea 2451: usa `redisDb.StringSet`).

*(Nota: Il vecchio metodo `VocabolarioGetAll` e la gestione manuale del dizionario sono stati rimossi in favore di `Traduci` con FusionCache).*

### Fase 4: Verifica
- [ ] Verificare la compilazione della soluzione tramite script PowerShell.
- [ ] Controllare che i log (NLog) continuano a riflettere correttamente le operazioni.

## Rischi e Mitigazioni
- **Rischio**: Discrepanza nelle chiavi di cache tra vecchio e nuovo sistema.
  - *Mitigazione*: Utilizzare rigorosamente le costanti in `Utils.redis...` per garantire che le chiavi siano identiche o gestite dal nuovo sistema.
- **Rischio**: Errori di serializzazione.
  - *Mitigazione*: `FusionCache` gestisce la serializzazione, ma è necessario assicurarsi che i tipi di ritorno siano compatibili con le aspettative dei chiamanti.