Differential D1000 Diff 2497 src/app/Observers/UserObserver.php

Changeset 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
		vanmeeuwenUnsubmitted Not Done Inline Actions 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. 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