daf-ontologie-vocabolari-controllati
daf-ontologie-vocabolari-controllati copied to clipboard
italia
Fix: #172. Malformed key contains blank space.
This PR
- fixes wrong space in URI
- according to https://datatracker.ietf.org/doc/html/rfc3986 a blank char, either in ascii or %-encoded as %20 in an URI path segment terminates the URI.
Reading the taxonomy, the correct value is the one used in the PR since the other was already defined.
@marco-deligios puoi darci un feedback qui please? La modifica in un modo o nell'altro va fatta perché sintatticamente l'URL non è fungibile da un tool conforme alle specifiche.
Sorry but that URI is valid and I do not understand the point. %20 is added to encode the space. So the issue here is to understand whether that identifier contains two or only one value. I will check this, but changing a persistent URI is a critical operation to avoid as much as possible.
è passato molto tempo, non ricordo la discussione e non la trovo... mi aiutate?
@marco-deligios c'è una entry fatta così: la chiave contiene due identificativi separati da spazio
040102 040201,Obbligo di pubblicazione concernente i titolari di incarichi dirigenziali,Comuni,204
Poiché 04102 è già assegnato, l'ipotesi è che ci sia stato un typo in fase di editing
040201,Obbligo di pubblicazione concernente i titolari di incarichi dirigenziali,Comuni,204
La PR sostituisce l'id 04102 040201 con 040201 . Che tu sappia, l'id 040201 è quello corretto ?
Ci sono altri id con spazi in mezzo (mi risulta di no).
Ciao Roberto, mi devi perdonare, ma mi è difficile rientrare dopo così tanto tempo. Riguardando i miei appunti ho parecchi dubbi sulla corrispondenza tra la tabella ANAC e [transparency-obligation] Se vuoi possiamo fare una call per discuterne
@giorgialodi @marco-deligios sappiamo chi sta utilizzando quei codici?
Non lo so ma guardate quello che ho scritto sopra. Ho trovato come modificare l'URI. Attenzione, il problema non è il %20 nell'URI che è ammesso in presenza di spazi, ma il problema è che un pezzo di quel numero finale non esiste nel vocabolario in questione. Detta in altri termini, cambiare l'URI si può perché non è definito da nessuna parte e quindi è appeso, ma il cambio di URI è come vi ho scritto sopra e non come suggerito nella PR: quell'URI della PR non esiste nel vocabolario transparency-obligation, se ho ricostruito bene il tutto.
Se non ricordo male questa cosa è stata fixata in altro modo per via dell'errore che è stato rilevato a suo tempo e qui documentato. Chiuderei questa PR senza fare merge @bfabio @mfortini @Clou-dia