Remove dangerouslySetInnerHTML from ContactLink

[pnicolli]

No description provided.

[mamico]

Occhio che ho visto che c'è chi in quel campo ho visto mette un numero e un testo. Tipo "05123456 ore pasti" e il pattern di prima faceva qualcosa di diverso da quello che fai ora, mi sembra.

[pnicolli]

Mi ero posto il problema infatti che potesse essere qualcosa del genere, per questo motivo sono andato ad indagare dove venisse utilizzato questo componente e ho scoperto che è utilizzato in campi che non possono più essere compilati, mostrano solo dati da migrazioni vecchie. Farò una valutazione per capire se posso gestirla meglio al volo ma probabilmente non serve.

[pnicolli]

@mamico ho reimplementato in modo migliore.
Ho fatto però un altro giro nel codice per confermare la mia ipotesi che in realtà sto aggiustando una cosa inutile e l'ho quasi confermata. Il componente affetto (ContactLink) è attualmente usato in SmallVenue che è un componente apparentemente inutilizzato, stesso vale per UOTelephones. Li ho lasciati in caso siano usati da customizzazioni nei temi dei singoli comuni. In aggiunta è usato anche nel componente EventoContattiOrganizzatoreEsterno che mostra i contatto dell'organizzatore esterno, che però sono campi che nell'attuale versione non esistono più quindi al massimo restano da migrazioni vecchie. Credo addirittura che non restino nemmeno, perché in realtà potrebbero essere trasferiti dalla migrazione a v3 direttamente nei punti di contatto, ma anche lì per scrupolo non ho toccato nulla.

[mamico]

@mamico ho reimplementato in modo migliore. Ho fatto però un altro giro nel codice per confermare la mia ipotesi che in realtà sto aggiustando una cosa inutile e l'ho quasi confermata. Il componente affetto (ContactLink) è attualmente usato in SmallVenue che è un componente apparentemente inutilizzato, stesso vale per UOTelephones. Li ho lasciati in caso siano usati da customizzazioni nei temi dei singoli comuni.

Io nello specifico l'ho verificato in un V2 e il tuo codice ne faccio backport lì. Se però qui sono componenti che non servono possiamo sicuramente fare un lavoro per deprecare quei componenti, servirà però sicuramente anche fare degli upgrade step per riportare eventuali contatti scritti nei vecchi campi sulle nuove strutture dati (?)