AVSDev/sprinkle-organisations
1
0 Fork 0
Code Issues 2 Pull Requests Projects Releases Wiki Activity

Fixed users being duplicated if they already existed in a target group during a merge

Browse Source
This commit is contained in:
bwilliams committed 2022-02-10 10:30:31 +00:00
1 parent 711968df08
commit 9d84dcec67
1 file changed
+9 -5
Download Patch File Download Diff File Expand all files Collapse all files
src/Database/Models/Organisation.php
+9 -5
View File
@@ -199,14 +199,18 @@ class Organisation extends Model implements OrganisationInterface
};
// Move all the users from this organisation to the target
$this->members()->each(function ($user) use ($target, $activityLogger, $currentUser, $logEntry) {
$this->members()->detach($user); // NOTE: Should this record be retained? Or is the activity log enough of an audit?
$target->members()->attach($user, ['flag_admin' => false]);
$logEntry($activityLogger, $currentUser, $user, $target);
});
$this->administrators()->each(function ($user) use ($target, $activityLogger, $currentUser, $logEntry) {
$this->administrators()->detach($user); // NOTE: Should this record be retained? Or is the activity log enough of an audit?
if (!$target->administrators()->find($user->id) && !$target->members()->find($user->id)) {
$target->administrators()->attach($user, ['flag_admin' => true]);
}
$logEntry($activityLogger, $currentUser, $user, $target);
});
$this->members()->each(function ($user) use ($target, $activityLogger, $currentUser, $logEntry) {
$this->members()->detach($user); // NOTE: Should this record be retained? Or is the activity log enough of an audit?
if (!$target->administrators()->find($user->id) && !$target->members()->find($user->id)) {
$target->members()->attach($user, ['flag_admin' => false]);
}
$logEntry($activityLogger, $currentUser, $user, $target);
});
$this->save();