Repository navigation
Visa hela organisationsträdet + enhetsdetaljsida med anställda - #5
Merged
Merged
Conversation
…llda
Fixar att /organisation bara visade ett fåtal avdelningar och att man inte
kunde klicka sig fram till en enhet.
- HamtaOrganisationAsync laddade bara roten + en nivå barn → bara ~9
förvaltningar syntes trots att DB har 134 enheter i 4 nivåer. Laddar nu
ALLA enheter platt i en fråga och bygger hierarkin i minnet
(ByggHierarki, testad) — alla 134 enheter nås. Trädet är expanderbart
(region + förvaltning utfällt som standard), varje nod visar antal
underenheter och antal anställda (upprullat per subträd).
- Ny detaljsida /organisation/{id} (Detalj.razor): visar enhetens info
(typ, kostnadsställe, CFAR, HSA-id, giltighet, överordnad enhet), direkta
underenheter (klickbara), fördelning per befattning, och de anställda som
jobbar i enheten i en PAGINERAD tabell (filtrerad på EnhetId i SQL, aldrig
hela registret). Anställd-rader klickbara till /anstallda/{id}.
- Behörighet: /organisation/{guid} ärver ChefHrAdmin via prefixmatch
(anställd nekas, fail-closed bevaras) — policyn orörd.
- Nytt test OrgTreeBuilderTests (5 fall: 4-nivåkoppling, dinglande förälder,
flera rötter, uppräkning, självreferens). Bygge grönt, tester gröna.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0114SipfUPpV6WRPh9jMRfUL
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixar att /organisation bara visade ett fåtal avdelningar och att man inte kunde klicka sig fram till en enhet. Live-verifierat på hr.404haven.se (134 enheter, 11 010 anställda).
Problem
Databasen har 134 enheter i 4 nivåer, men HamtaOrganisationAsync laddade bara roten + en nivå barn (
Include(Underenheter).Where(OverordnadEnhetId == null)) → bara ~9 förvaltningar syntes; sjukhusens kliniker och vårdcentralerna på nivå 3–4 hämtades aldrig. Ingen detaljsida fanns.Fix
/organisation/{id}(Detalj.razor): enhetens info (typ, kostnadsställe, CFAR, HSA-id, giltighet, överordnad enhet), direkta underenheter (klickbara, med antal), fördelning per befattning, och de anställda i enheten i en PAGINERAD tabell (filtrerad på EnhetId i SQL, count via CountAsync — laddar aldrig hela registret). Anställd-rader klickbara till /anstallda/{id}.🤖 Generated with Claude Code
https://claude.ai/code/session_0114SipfUPpV6WRPh9jMRfUL