merge request id	632
Titel	Feature/ll 1201 assignment via catalog inactive users are assigned via department
Beschreibung
Status	merged
Created at	2022-06-14T12:59:26.027Z
Updated at	2022-06-30T09:36:32.872Z
Source branch	feature/LL-1201-assignment-via-catalog-inactive-users-are-assigned-via-department
Target branch	develop
Author	Markus Gerlach
Assignee	Gregor Gabriel

Discussion notes

assigned to @gabrielg	Sascha Immig	2022-06-14T20:51:51.954Z
bitte rückgängig machen. Rest siehe unten.	Gregor Gabriel	2022-06-21T11:59:19.512Z
Dein Ansatz entfernt zwar die inaktiven User die der Abteilung direkt zugeordnet sind. Ich sehe aber nicht, wie sichergestellt wird, dass die inaktiven User aus Unterabteilungen herausgefiltert werden? Das Problem ist, dass die Methode `users()` zwei Queries mit `OR` verbindet aber das ganze am Ende nicht in eine Klammer gepackt wird. Damit werden später hinzugefügte Scopes nur auf die zweite Hälfte der OR-Bedingung angewendet. Lass uns hier mal telefonieren Man müsste sowas in der Art machen: `public function users() { return User::join('organization_unit_user','users.id','organization_unit_user.user_id') ->whereIn('organization_unit_user.organization_unit_id', $this->getDescendants(['id'])->push($this->id)) ->where('users.client_Id', $this->client_id) ->select('users.*', 'organization_unit_user.role'); }` Oben ist noch zu prüfen, ob das mit der Rolle so passt Dann müssen wir aber noch explizit die Methode `public function getUsersAttribute() {return $this->users()->get();}` bereitstellen, da sonst $unit->users knallen würde.	Gregor Gabriel	2022-06-21T11:59:57.021Z
added 1 commit 1cb5fe4d - fix: inactive users from other and suborganization units are now also taken into account Compare with previous version	Markus Gerlach	2022-06-23T11:44:49.095Z
added 1 commit 5b4e4813 - updated guzzlehttp/guzzle and dependencies due to vulnerability warning Compare with previous version	Markus Gerlach	2022-06-23T12:06:59.861Z
resolved all discussions	Markus Gerlach	2022-06-23T12:13:24.264Z
Ich habe jetzt eine einfach Lösung finden könen sihe unte.	Markus Gerlach	2022-06-23T12:14:02.342Z
resolved all discussions	Markus Gerlach	2022-06-23T12:14:21.912Z
Ich habe jetzt eine einfache Lösung finden könen (siehe neue Version).	Markus Gerlach	2022-06-23T12:15:26.645Z
resolved all discussions	Markus Gerlach	2022-06-23T12:15:26.771Z
resolved all discussions	Markus Gerlach	2022-06-24T12:39:49.086Z
added 1 commit cabcf727 - LL-1201 clean up test function Compare with previous version	Markus Gerlach	2022-06-24T12:39:49.293Z
merged	Markus Gerlach	2022-06-30T09:36:32.963Z

Merge Commits

Title	Id	Author	Created at	Message
fix: inactive users from other and suborganization units are now also taken into account	1cb5fe4d3ed8ba32dfe06e83b9cb4be0c53d4665	markusgerlach	2022-06-23T11:44:45.000Z	fix: inactive users from other and suborganization units are now also taken into account
updated guzzlehttp/guzzle and dependencies due to vulnerability warning	5b4e4813430cb57c5fc34cc55500a36c74f0c333	markusgerlach	2022-06-23T12:06:55.000Z	updated guzzlehttp/guzzle and dependencies due to vulnerability warning
LL-1201-inactive users of the departments are no longer assigned	b6c41a5f12e142d0491d04dc93ac2717bbce0f23	markusgerlach	2022-06-14T12:18:30.000Z	LL-1201-inactive users of the departments are no longer assigned
LL-1201 clean up test function	cabcf7273469667f5e23758daff9c675455aa73e	markusgerlach	2022-06-24T12:39:44.000Z	LL-1201 clean up test function
LL-1201 fix destroyed file	d16911b34d60347cd4f8c29aadeea7cfac0b18e5	markusgerlach	2022-06-14T12:41:34.000Z	LL-1201 fix destroyed file