search

italia / daf-ontologie-vocabolari-controllati

Elenco di ontologie e vocabolari controllati. Per maggiori informazioni, si veda il readme principale e quello di singoli vocabolari/ontologie, ove presente, e la seguente documentazione
https://github.com/italia/daf-ontologie-vocabolari-controllati/wiki
Creative Commons Attribution 4.0 International
80 stars 52 forks source link

Fix: #172. Malformed key contains blank space. #183

Closed ioggstream closed 1 year ago

ioggstream commented 2 years ago

This PR

Reading the taxonomy, the correct value is the one used in the PR since the other was already defined.

ioggstream commented 2 years ago

@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.

giorgialodi commented 2 years ago

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.

marco-deligios commented 2 years ago

è passato molto tempo, non ricordo la discussione e non la trovo... mi aiutate?

ioggstream commented 2 years ago

@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).

marco-deligios commented 2 years ago

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

ioggstream commented 2 years ago

@giorgialodi @marco-deligios sappiamo chi sta utilizzando quei codici?

giorgialodi commented 2 years ago

@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.

giorgialodi commented 1 year ago

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