Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing uniqueness in profession group in AutoristationsRegister #127

Open
jkiddo opened this issue May 6, 2024 · 14 comments
Open

Missing uniqueness in profession group in AutoristationsRegister #127

jkiddo opened this issue May 6, 2024 · 14 comments
Assignees
Milestone

Comments

@jkiddo
Copy link
Collaborator

jkiddo commented May 6, 2024

The code 5176 mentioned in

Description: "DK health ProfessionGroup codes as defined by https://www.retsinformation.dk/eli/lta/2019/731, and derived from http://autregwebservice.sst.dk/autregservice.asmx/GetAllProfessionGroups. Code (5176) was not uniqe, and have been converted to 5176a and 5176b"
is not unique when querying the autorisationsregister at http://autregwebservice.sst.dk/autregservice.asmx/GetAllProfessionGroups

This needs to be adressed to SDS.

@jkiddo jkiddo added this to the 3.3.0 milestone May 6, 2024
@AsgerSDSArkitekt
Copy link

Det er ikke helt trivielt i forhold til kode 5176, hvor der er “overlap” mellem kontaktlinseoptiker og optometristernes uddannelseskoder. Vi har haft fat i STPS (Styrelsen for patientsikkerhed), og man kunne oprindeligt åbenbart starte som uddannet optiker og derfra efterfølgende videreuddanne sig til kontaktlinseoptiker. Men nu kan man tilsyneladende kun blive optometrist, men det er så samme uddannelseskode. Så hvis man kalder STPS' service med kode 5176 så er det vanskeligt for den, at gætte på hvilken af de to, der tales om.

@jkiddo jkiddo assigned AsgerSDSArkitekt and unassigned tmsMedcom Jul 10, 2024
@jkiddo
Copy link
Collaborator Author

jkiddo commented Jul 10, 2024

@AsgerSDSArkitekt er det noget der prioriteres at skulle rettes af SDS eller STPS? Vi skal helst undgå fordums inkonsistens i nye standarder.

@AsgerSDSArkitekt
Copy link

@jkiddo Jeg er ved at undersøge, hvad vi i SDS vil kunne gøre overfor/sammen med STPS.

@tmsMedcom
Copy link
Collaborator

@AsgerSDSArkitekt er der nyt vedr. dette issue?

@AsgerSDSArkitekt
Copy link

Desværre ikke - den er druknet i en nedprioriteret gryde.

@jkiddo
Copy link
Collaborator Author

jkiddo commented Oct 11, 2024

Er der en ETA?

@tmsMedcom
Copy link
Collaborator

@AsgerSDSArkitekt Skal dette issue udsættes til næste release?

@jkiddo
Copy link
Collaborator Author

jkiddo commented Dec 13, 2024

@christiangasser-lakeside hvordan er det løst i OIOSAML-H 3.0.5 (https://www.nspop.dk/display/KLMIDNLIN3/Profiler) ? Der er problemet også.

@jkiddo
Copy link
Collaborator Author

jkiddo commented Dec 13, 2024

aka - uddannelseskoden er ikke unik.

@christiangasser-lakeside
Copy link
Collaborator

I OIOSAML-H profilerne og DGWS/SOSI for den sags skyld har det ikke rigtig været et issue.

I infrastrukturen (SEB, SOSI-STS) beriges tokens med oplsyningerne med autorisationsregisteret som den autoritative kilde, hvor et opslag på brugerens CPR returnerer null, et eller flere tripler på formen (autorisation ID, uddannelses kode, uddannelses navn). At der kan være forskellige udd. navne til samme kode har ingen betydning.

Når man laver en søgning på faggruppen 'Optiker og optometrist' (koder 4498 og 5176) i autorisationsregisteret via web (https://autregweb.sst.dk/authorizationsearch.aspx) fås kun 'Optometrist' og ikke 'Kontaktlinseoptiker' på koden 5176. Så spørgsmål er om 'Kontaktlinseoptiker' overhovedet anvendes?

@jkiddo
Copy link
Collaborator Author

jkiddo commented Dec 16, 2024

https://autregwebservice.sst.dk/autregservice.asmx/GetAllProfessionGroups returnerer 2 på samme kode. Det er bottomline. Hvordan andre klienter fortolker det, må de vel selv være ansvarlige for. Målet må være at der ikke er plads til fortolkning.

@jkiddo
Copy link
Collaborator Author

jkiddo commented Dec 16, 2024

@christiangasser-lakeside jeg kan ikke forstå hvorfor det ikke har betydning hvad angår udd.navne, for det indgår jo i tokenet, så en eller anden betydning har det jo, ellers var der vel ingen grund til at have det med.

@christiangasser-lakeside
Copy link
Collaborator

Min pointe var et infrastrukturen får indgangene fra aut. registeret som tripel (id, kode, navn) og ikke bare som par (id, kode). At der ikke er en én-til-én mapning mellem kode og navn er selvfølgelig ikke pænt, men infrastrukturen har ingen antagelse om at koden skal være unik. Navnet medføres primært for at spare anvenderne et opslag på 'GetAllProfessionGroups' (og nu også fordi der jo ikke er 1-1 mapning mellem kode/navn).

@jkiddo
Copy link
Collaborator Author

jkiddo commented Dec 16, 2024

Hvad er pointen med en kode fra et kodesystem som ikke er unik? Mig bekendt får aftagere ( som i "klienter") ikke noget id. De får parret (koden, navn).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants