Changeset View
Changeset View
Standalone View
Standalone View
src/app/Observers/UserObserver.php
<?php | <?php | ||||
namespace App\Observers; | namespace App\Observers; | ||||
use App\Entitlement; | |||||
use App\Domain; | |||||
use App\User; | use App\User; | ||||
use Illuminate\Support\Facades\DB; | |||||
class UserObserver | class UserObserver | ||||
{ | { | ||||
/** | /** | ||||
* Handle the "creating" event. | * Handle the "creating" event. | ||||
* | * | ||||
* Ensure that the user is created with a random, large integer. | * Ensure that the user is created with a random, large integer. | ||||
* | * | ||||
▲ Show 20 Lines • Show All 60 Lines • ▼ Show 20 Lines | class UserObserver | ||||
* Handle the "deleting" event. | * Handle the "deleting" event. | ||||
* | * | ||||
* @param User $user The user that is being deleted. | * @param User $user The user that is being deleted. | ||||
* | * | ||||
* @return void | * @return void | ||||
*/ | */ | ||||
public function deleting(User $user) | public function deleting(User $user) | ||||
{ | { | ||||
// TODO: Especially in tests we're doing delete() on a already deleted user. | |||||
// Should we escape here - for performance reasons? | |||||
// TODO: I think all of this should use database transactions | |||||
// Entitlements do not have referential integrity on the entitled object, so this is our | // Entitlements do not have referential integrity on the entitled object, so this is our | ||||
// way of doing an onDelete('cascade') without the foreign key. | // way of doing an onDelete('cascade') without the foreign key. | ||||
$entitlements = \App\Entitlement::where('entitleable_id', $user->id) | Entitlement::where('entitleable_id', $user->id) | ||||
->where('entitleable_type', \App\User::class)->get(); | ->where('entitleable_type', User::class) | ||||
->delete(); | |||||
// Remove owned users/domains | |||||
vanmeeuwen: Think... "remove entitlements billed to wallets owned".
When john@kolab.org owns a wallet to… | |||||
$wallets = $user->wallets()->pluck('id')->all(); | |||||
$assignments = Entitlement::whereIn('wallet_id', $wallets)->get(); | |||||
$users = []; | |||||
$domains = []; | |||||
$entitlements = []; | |||||
foreach ($entitlements as $entitlement) { | foreach ($assignments as $entitlement) { | ||||
$entitlement->delete(); | if ($entitlement->entitleable_type == Domain::class) { | ||||
$domains[] = $entitlement->entitleable_id; | |||||
} elseif ($entitlement->entitleable_type == User::class && $entitlement->entitleable_id != $user->id) { | |||||
$users[] = $entitlement->entitleable_id; | |||||
} else { | |||||
$entitlements[] = $entitlement->id; | |||||
} | |||||
} | |||||
$users = array_unique($users); | |||||
$domains = array_unique($domains); | |||||
// Note: Domains/users need to be deleted one by one to make sure | |||||
// events are fired and observers can do the proper cleanup. | |||||
// Entitlements have no delete event handlers as for now. | |||||
if (!empty($users)) { | |||||
foreach (User::whereIn('id', $users)->get() as $_user) { | |||||
$_user->delete(); | |||||
} | |||||
} | } | ||||
if (!empty($domains)) { | |||||
foreach (Domain::whereIn('id', $domains)->get() as $_domain) { | |||||
$_domain->delete(); | |||||
} | |||||
} | |||||
if (!empty($entitlements)) { | |||||
Entitlement::whereIn('id', $entitlements)->delete(); | |||||
} | |||||
// FIXME: What do we do with user wallets? | |||||
\App\Jobs\UserDelete::dispatch($user->id); | \App\Jobs\UserDelete::dispatch($user->id); | ||||
} | } | ||||
/** | /** | ||||
* Handle the "retrieving" event. | * Handle the "retrieving" event. | ||||
* | * | ||||
* @param User $user The user that is being retrieved. | * @param User $user The user that is being retrieved. | ||||
* | * | ||||
Show All 21 Lines |
Think... "remove entitlements billed to wallets owned".
When john@kolab.org owns a wallet to which all entitlements associated with any @kolab.org-related entitlements are billed -- and how would he not?? --, then iterating over his wallets and entitlements billed to said wallets is a very predictable path.