Den Bereich einer Einheit ueber das Etikett suchen, nicht ueber die Stellung
divisionOf nahm "die oberste Einheit unterhalb der Gesellschaft" -- eine Aussage ueber die Stellung im Baum statt ueber unit_type. Im Altmodell fiel beides zusammen, weil unter der Gesellschaft genau die Bereiche hingen. Bei Manner liegt dort allein "CEO", darunter erst die sechs C-Level-Bereiche: die Funktion gab fuer alle 784 Personen "CEO" zurueck. In der Mitarbeiterliste stand es in jeder Zeile, auf der Uebersicht lag die ganze Belegschaft im Balken "CEO", waehrend die sechs uebrigen Bereiche auf null standen. Gesucht wird jetzt der naechste Vorfahre mit unit_type = 'Bereich', die Einheit selbst eingeschlossen. Der naechste und nicht der oberste: "CEO" ist selbst ein Bereich und liegt ueber den anderen, sonst stuende er wieder ueberall. Damit meinen Liste, Uebersicht, Sortierung (lib/employee-sort.ts) und Berichte (lib/reports-data.ts) denselben Bereich -- vorher sortierte die Liste bereits richtig, zeigte aber in der Spalte etwas anderes an.
This commit is contained in:
26
lib/org.ts
26
lib/org.ts
@@ -136,9 +136,31 @@ export function breadcrumbLabel(maps: OrgMaps, unitId: string | null | undefined
|
||||
return chain.map((u) => u.name).join(" › ") || "–";
|
||||
}
|
||||
|
||||
/** Die oberste Einheit unterhalb der Gesellschaft — das, was früher „Bereich" hiess. */
|
||||
/**
|
||||
* Der Bereich über einer Einheit — der nächste Vorfahre, der als Bereich
|
||||
* geführt wird, die Einheit selbst eingeschlossen.
|
||||
*
|
||||
* Hier stand „die oberste Einheit unterhalb der Gesellschaft", also eine
|
||||
* Aussage über die *Stellung* im Baum statt über das Etikett. Im Altmodell
|
||||
* fiel beides zusammen: unter der Gesellschaft hingen genau die Bereiche.
|
||||
* Sobald eine Einheit dazwischenliegt, fällt es auseinander — bei Manner
|
||||
* hängt unter der Gesellschaft allein „CEO", und die sechs C-Level-Bereiche
|
||||
* darunter. Die Funktion gab damit für alle 784 Personen „CEO" zurück: in
|
||||
* der Mitarbeiterliste stand es in jeder Zeile, und auf der Übersicht lagen
|
||||
* alle im Balken „CEO", während die sechs übrigen Bereiche auf null standen.
|
||||
*
|
||||
* Gesucht wird der **nächste** Bereich, nicht der oberste. „CEO" ist selbst
|
||||
* einer und liegt über den anderen; ohne diese Wahl stünde er wieder überall.
|
||||
* Dieselbe Regel gilt beim Sortieren (lib/employee-sort.ts) und in den
|
||||
* Berichten (lib/reports-data.ts) — alle drei müssen denselben Bereich
|
||||
* meinen, sonst sortiert die Liste nach etwas anderem, als sie anzeigt.
|
||||
*/
|
||||
export function divisionOf(maps: OrgMaps, unitId: string | null | undefined): OrgUnit | undefined {
|
||||
return ancestorsOf(maps, unitId).find((u) => u.unit_type !== "Gesellschaft");
|
||||
const kette = ancestorsOf(maps, unitId);
|
||||
for (let i = kette.length - 1; i >= 0; i--) {
|
||||
if (kette[i].unit_type === "Bereich") return kette[i];
|
||||
}
|
||||
return undefined;
|
||||
}
|
||||
|
||||
/** Die Einheit selbst, wenn sie nicht die Gesellschaft ist. */
|
||||
|
||||
Reference in New Issue
Block a user