User optimisation and code cleanup

app/Models/Policies/AccountPolicy.php

@@ -20,13 +20,7 @@ class AccountPolicy
 	public function view(User $uo,Account $ao): bool
 	{
 		// If this is a service for an account managed by a user.
-		return ($uo->accounts->pluck('id')->search($ao->id) !== FALSE)
-
-			// The user is the wholesaler
-			OR $uo->isWholesaler()
-
-			// The user has this as one of their accounts
-			OR $uo->accounts->pluck('id')->contains($ao->id);
+		return $uo->accounts_all->pluck('id')->contains($ao->id) || $uo->isWholesaler();
 	}
 
 	/**

app/Models/Policies/InvoicePolicy.php

@@ -19,14 +19,7 @@ class InvoicePolicy
 	 */
 	public function view(User $uo,Invoice $io): bool
 	{
-		// If this is a service for an account managed by a user.
-		return ($uo->invoices->pluck('id')->search($io->id) !== FALSE)
-
-			// The user is the wholesaler
-			OR $uo->isWholesaler()
-
-			// The user has this as one of their accounts
-			OR $uo->accounts->pluck('id')->contains($io->account_id);
+		return $uo->accounts_all->pluck('id')->contains($io->account_id) || $uo->isWholesaler();
 	}
 
 	/**

app/Models/Policies/ServicePolicy.php

@@ -20,13 +20,7 @@ class ServicePolicy
 	public function view(User $uo, Service $so): bool
 	{
 		// If this is a service for an account managed by a user.
-		return ($uo->services->pluck('id')->search($so->id) !== FALSE)
-
-			// The user is the wholesaler
-			OR $uo->isWholesaler()
-
-			// The user has this as one of their accounts
-			OR $uo->accounts->pluck('id')->contains($so->account_id);
+		return $uo->accounts_all->pluck('id')->contains($so->account_id) || $uo->isWholesaler();
 	}
 
 	/**

app/Models/Policies/UserPolicy.php

@@ -15,11 +15,11 @@ class UserPolicy
 	 *
 	 * @param User $uo
 	 * @param string $ability
-	 * @return bool|null
+	 * @return null|bool
 	 */
-	public function before(User $uo,string $ability): bool
+	public function before(User $uo,string $ability): bool|NULL
 	{
-		return $uo->isWholesaler() ?: FALSE;
+		return $uo->isWholesaler() ?: NULL;
 	}
 
 	/**
@@ -44,9 +44,6 @@ class UserPolicy
 	public function view(User $uo,User $o): bool
 	{
 		// If this is a service for an account managed by a user.
-		return ($uo->id == $o->id)
-
-			// The user has this as one of their accounts
-			OR $uo->accounts->pluck('user')->pluck('id')->unique()->contains($o->id);
+		return ($uo->id == $o->id) || $uo->accounts_all->pluck('user_id')->contains($o->id) || $uo->isWholesaler();
 	}
 }

app/Models/Rtm.php

@@ -4,6 +4,7 @@ namespace App\Models;
 
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Support\Collection;
 
 class Rtm extends Model
 {
@@ -23,4 +24,20 @@ class Rtm extends Model
 	{
 		return $this->hasMany(self::class,'parent_id');
 	}
+
+	/**
+	 * Return all the children RTM records that this record is parent of
+	 *
+	 * @return Collection
+	 */
+	public function children_all(): Collection
+	{
+		$result = collect();
+		$result->push($this->withoutRelations());
+
+		foreach ($this->children as $o)
+			$result = $result->merge($o->children_all());
+
+		return $result;
+	}
 }

app/Models/Service.php

@@ -317,6 +317,18 @@ class Service extends Model implements IDs
 		return sprintf('%02s-%04s.%s',$this->site_id,$this->account_id,$this->getLIDattribute());
 	}
 
+	/* STATIC */
+
+	public static function movements(User $uo): Collection
+	{
+		return (new self)
+			->active()
+			->serviceUserAuthorised($uo)
+			->where('order_status','!=','ACTIVE')
+			->with(['account','product'])
+			->get();
+	}
+
 	/* RELATIONS */
 
 	/**

app/Models/User.php

@@ -7,17 +7,12 @@ use Illuminate\Notifications\Notifiable;
 use Illuminate\Foundation\Auth\User as Authenticatable;
 use Illuminate\Support\Collection;
 use Illuminate\Database\Eloquent\Collection as DatabaseCollection;
-use Illuminate\Support\Facades\Cache;
-use Illuminate\Support\Facades\DB;
-use Illuminate\Support\Facades\Session;
 use Laravel\Passport\HasApiTokens;
 use Leenooks\Traits\ScopeActive;
 use Leenooks\Traits\UserSwitch;
 
 use App\Interfaces\IDs;
-use App\Models\Scopes\SiteScope;
 use App\Notifications\ResetPassword as ResetPasswordNotification;
-use App\Traits\SiteID;
 
 /**
  * Class User
@@ -27,14 +22,12 @@ use App\Traits\SiteID;
  */
 class User extends Authenticatable implements IDs
 {
-	use HasFactory,HasApiTokens,Notifiable,UserSwitch,SiteID,ScopeActive;
+	use HasFactory,HasApiTokens,Notifiable,UserSwitch,ScopeActive;
 
 	private const CACHE_TIME = 3600;
 
-	protected $dates = [
-		'created_at',
-		'updated_at',
-		'last_access'
+	protected $casts = [
+		'last_access' => 'datetime:Y-m-d H:i:s',
 	];
 
 	/**
@@ -96,27 +89,25 @@ class User extends Authenticatable implements IDs
 	/* RELATIONS */
 
 	/**
-	 * The accounts that this user manages
-	 *
-	 * @return \Illuminate\Database\Eloquent\Relations\HasMany
-	 * @note This cannot be loaded with "with"?
+	 * This user's accounts
 	 */
 	public function accounts()
 	{
 		return $this->hasMany(Account::class)
-			->orWhereIn('id',$this->rtm_accounts()->pluck('id'))
 			->active();
 	}
 
 	/**
-	 * This users invoices
+	 * The accounts that this user manages
 	 *
-	 * @return \Illuminate\Database\Eloquent\Relations\HasManyThrough
-	 * @deprecated Accounts have invoices, not users
+	 * @return \Illuminate\Database\Eloquent\Relations\HasMany
+	 * @note This cannot be preloaded with load() or with() - $this is a blank object
 	 */
-	public function invoices()
+	public function accounts_all()
 	{
-		return $this->hasManyThrough(Invoice::class,Account::class)
+		return $this->hasMany(Account::class)
+			->when((! is_null($this->rtm) && $this->id),
+				fn($query)=>$query->orWhereIn('rtm_id',$this->rtm?->children_all()->pluck('id')))
 			->active();
 	}
 
@@ -132,37 +123,20 @@ class User extends Authenticatable implements IDs
 
 	/**
 	 * Return the routes to market account for this user
-	 *
-	 * @return \Illuminate\Database\Eloquent\Relations\HasOneThrough
 	 */
 	public function rtm()
 	{
 		return $this->hasOneThrough(Rtm::class,Account::class);
 	}
 
-	/**
-	 * The services this user has
-	 *
-	 * @return \Illuminate\Database\Eloquent\Relations\HasManyThrough
-	 * @deprecated Accounts have services, not users
-	 */
-	public function services()
-	{
-		return $this->hasManyThrough(Service::class,Account::class)
-			->with(['product.type'])
-			->active();
-	}
-
 	/**
 	 * Supplier configuration for this user
 	 *
-	 * @return \Illuminate\Database\Eloquent\Relations\BelongsToMany
-	 * @deprecated Move to account->suppliers()
+	 * @deprecated To move to account->suppliers()
 	 */
 	public function suppliers()
 	{
 		return $this->belongsToMany(Supplier::class)
-			->where('supplier_user.site_id',$this->site_id)
 			->withPivot('id','created_at');
 	}
 
@@ -179,7 +153,7 @@ class User extends Authenticatable implements IDs
 	}
 
 	/**
-	 * Logged in users full name
+	 * Logged in user's full name
 	 *
 	 * @return string
 	 */
@@ -188,25 +162,9 @@ class User extends Authenticatable implements IDs
 		return sprintf('%s %s',$this->firstname,$this->lastname);
 	}
 
-	/**
-	 * Return my accounts, but only those accounts with the same group_id
-	 *
-	 * @note Users can only manage accounts with the same group ID, thereby ensuring they dont see different
-	 *		pricing options - since prices can be controlled by groups
-	 * @return Collection
-	 */
-	public function getMyAccountsAttribute(): Collection
-	{
-		$result = $this->accounts->where('user_id',$this->id);
-
-		if (! $result->count())
-			return $result;
-
-		return $this->isReseller() ? $result : $result->groupBy('group.id')->first();
-	}
-
 	/**
 	 * Return a friendly string of this persons role
+	 *
 	 * @return string
 	 */
 	public function getRoleAttribute(): string
@@ -259,22 +217,6 @@ class User extends Authenticatable implements IDs
 
 	/* METHODS */
 
-	/**
-	 * Show this user's clients with service movements
-	 *
-	 * A service movement, is an active service where the status is not ACTIVE
-	 *
-	 * @return DatabaseCollection
-	 */
-	public function client_service_movements(): DatabaseCollection
-	{
-		return Service::active()
-			->serviceUserAuthorised($this)
-			->where('order_status','!=','ACTIVE')
-			->with(['account','product'])
-			->get();
-	}
-
 	/**
 	 * Determine if the user is an admin of the user with $id
 	 *
@@ -283,7 +225,11 @@ class User extends Authenticatable implements IDs
 	 */
 	public function isAdmin(User $user=NULL): bool
 	{
-		return $user->exists AND $this->isReseller() AND $this->accounts->pluck('user_id')->contains($user->id);
+		return $user->exists
+			&& $this->isReseller()
+			&& $this->accounts_all
+				->pluck('user_id')
+				->contains($user->id);
 	}
 
 	/**
@@ -311,9 +257,11 @@ class User extends Authenticatable implements IDs
 	 *
 	 * @param bool $future
 	 * @return DatabaseCollection
+	 * @deprecated This should be done in accounts
 	 */
-	public function next_invoice_items(bool $future=FALSE): DatabaseCollection
+	public function next_invoice_items(bool $future=FALSE): Collection
 	{
+		return collect();
 		$result = new DatabaseCollection;
 		$this->services->load(['invoice_items.taxes']);
 
@@ -339,117 +287,6 @@ class User extends Authenticatable implements IDs
 		return $result;
 	}
 
-	/**
-	 * Return an SQL query that will return a list of invoices
-	 *
-	 * @return \Illuminate\Database\Query\Builder
-	 */
-	private function query_invoice_items()
-	{
-		return DB::table('invoice_items')
-			->select([
-				'invoice_id',
-				DB::raw('invoice_items.id AS invoice_item_id'),
-				DB::raw('COALESCE(invoice_items.discount_amt,0) AS discount'),
-				DB::raw('quantity*price_base AS base'),
-				DB::raw('invoice_item_taxes.amount AS tax'),
-			])
-			->leftjoin('invoice_item_taxes',['invoice_item_taxes.invoice_item_id'=>'invoice_items.id'])
-			->where('invoice_items.active',TRUE);
-	}
-
-	/**
-	 * Return an SQL query that will return payment summaries by invoices.
-	 *
-	 * @return \Illuminate\Database\Query\Builder
-	 */
-	private function query_payment_items()
-	{
-		return DB::table('payment_items')
-			->select([
-				'payment_id',
-				'invoice_id',
-				DB::raw('SUM(amount) AS allocate'),
-			])
-			->where('amount','>',0)
-			->groupBy(['invoice_id','payment_id']);
-	}
-
-	/**
-	 * Return an SQL query that will summarise invoices with payments
-	 *
-	 * @return \Illuminate\Database\Query\Builder
-	 * @todo change this to just return outstanding invoices as a collection.
-	 * @deprecated Use account->invoiceSummary
-	 */
-	public function query_invoice_summary()
-	{
-		$invoices = (new Invoice)
-			->select([
-				'invoice_id',
-				DB::raw('SUM(discount) AS discount'),
-				DB::raw('SUM(base) AS base'),
-				DB::raw('SUM(tax) AS tax'),
-				DB::raw('base+tax-discount AS total'),
-				DB::raw('0 AS payments'),
-				DB::raw('0 AS payment_fees'),
-			])
-			->from($this->query_invoice_items(),'II')
-			->join('invoices',['invoices.id'=>'II.invoice_id'])
-			->whereIN('account_id',$this->accounts->pluck('id'))
-			->where('invoices.active',TRUE)
-			->groupBy(['invoice_id','base','tax','discount']);
-
-		$payments = (new Payment)
-			->select([
-				'invoice_id',
-				DB::raw('0 AS discount'),
-				DB::raw('0 AS base'),
-				DB::raw('0 AS tax'),
-				DB::raw('0 AS total'),
-				DB::raw('SUM(allocate) AS payments'),
-				DB::raw('SUM(fees_amt) AS payment_fees'),
-			])
-			->from($this->query_payment_items(),'PI')
-			->join('payments',['payments.id'=>'PI.payment_id'])
-			->whereIN('account_id',$this->accounts->pluck('id'))
-			//->where('payments.active',TRUE) // @todo To implement
-			->groupBy(['invoice_id']);
-
-		$summary = (new Invoice)
-			->withoutGlobalScope(SiteScope::class)
-			->select([
-				'invoice_id',
-				DB::raw('SUM(discount) AS discount'),
-				DB::raw('SUM(base) AS invoice_base'),
-				DB::raw('SUM(tax) AS invoice_tax'),
-				DB::raw('SUM(total) AS invoice_total'),
-				DB::raw('SUM(payments) AS payments'),
-				DB::raw('SUM(payment_fees) AS payment_fees'),
-			])
-			->from($invoices->unionAll($payments),'invoices')
-			->groupBy(['invoice_id']);
-
-		return (new Invoice)
-			->select([
-				'account_id',
-				'id',
-				'due_at',
-				'created_at',
-				'discount',
-				'invoice_base',
-				'invoice_tax',
-				'invoice_total',
-				'payments',
-				'payment_fees',
-				DB::raw('invoice_total-payments AS balance'),
-			])
-			->join('invoices',['invoices.id'=>'invoice_id'])
-			->with(['items.taxes'])
-			->from($summary,'summary')
-			->groupBy(['id','account_id','discount','invoice_base','invoice_tax','invoice_total','payments','payment_fees']);
-	}
-
 	/**
 	 * Determine what the logged in user's role is
 	 * + Wholesaler - aka Super User
@@ -460,67 +297,8 @@ class User extends Authenticatable implements IDs
 	 */
 	public function role()
 	{
-		// Cache our role for this session
-		$cache_key = sprintf('%s:%s:%s',$this->id,__METHOD__,Session::getId());
-
-		return Cache::remember($cache_key,self::CACHE_TIME,function() {
-			// Get the RTM for our accounts
-			$rtms = Rtm::whereIn('account_id',$this->accounts->pluck('id'))->get();
-
-			// If I have no parent, I am the wholesaler
-			if ($rtms->whereNull('parent_id')->count())
-				return 'wholesaler';
-
-			// If I exist in the RTM table, I'm a reseller
-			else if ($rtms->count())
-				return 'reseller';
-
-			// Otherwise a client
-			else
-				return 'customer';
-		});
-	}
-
-	/**
-	 * Return the accounts that this user can manage
-	 * This method is a helper to User::accounts() - use $user->accounts instead
-	 *
-	 * @return Collection
-	 */
-	private function rtm_accounts(): Collection
-	{
-		return Account::whereIn('rtm_id',$this->rtm_list()->pluck('id'))
-			->get();
-	}
-
-	/**
-	 * Return the RTM hierarchy that this user can manage
-	 *
-	 * @param Rtm|null $rtm
-	 * @return Collection
-	 */
-	public function rtm_list(Rtm $rtm=NULL): Collection
-	{
-		// If this user doesnt manage any accounts
-		if (! $this->exists || ! $this->rtm)
-			return collect();
-
-		$list = collect();
-
-		// Add this RTM to the list
-		if (! $rtm) {
-			$list->push($this->rtm);
-			$children = $this->rtm->children;
-
-		} else {
-			$list->push($rtm);
-			$children =$rtm->children;
-		}
-
-		// Capture any children
-		foreach ($children as $child)
-			$list->push($this->rtm_list($child));
-
-		return $rtm ? $list : $list->flatten()->unique(function($item) { return $item->id; });
+		return $this->rtm
+			? ($this->rtm->parent_id ? 'reseller' : 'wholesaler')
+			: 'customer';
 	}
 }

app/Traits/ScopeServiceUserAuthorised.php

@@ -16,6 +16,6 @@ trait ScopeServiceUserAuthorised
 	public function scopeServiceUserAuthorised($query,User $uo)
 	{
 		return $query
-			->whereIN('services.account_id',$uo->accounts->pluck('id'));
+			->whereIN('services.account_id',$uo->accounts_all->pluck('id'));
 	}
 }

resources/views/theme/backend/adminlte/account/widget/summary_boxes.blade.php

@@ -1,27 +1,25 @@
-@if(Auth::user()->isReseller() && $o->my_accounts->count() <= 2 && $o->my_accounts->pluck('providers')->flatten()->count())
+@if($user->isReseller() && ($o->accounts->count() <= 2) && ($x=$o->accounts->pluck('providers')->flatten())->count())
 	<div class="col-12 col-sm-4 col-md-2">
 		<div class="info-box">
 			<span class="info-box-icon bg-dark elevation-1"><i class="fas fa-file-invoice"></i></span>
 			<div class="info-box-content">
 				<span class="info-box-text">Accounting</span>
-				@foreach ($o->my_accounts as $ao)
-					@foreach($ao->providers as $po)
-						<span class="info-box-number"><a href="{{ url(($po->api_class())::url().'/customerdetail?nameId='.$po->pivot->ref) }}" target="{{ $po->name }}">{{ ucfirst($po->name) }}</a></span>
-					@endforeach
+				@foreach($x as $po)
+					<span class="info-box-number"><a href="{{ url(($po->api_class())::url().'/customerdetail?nameId='.$po->pivot->ref) }}" target="{{ $po->name }}">{{ ucfirst($po->name) }}</a></span>
 				@endforeach
 			</div>
 		</div>
 	</div>
 @endif
 
-@if ($o->accounts->count() > 1)
+@if ($o->accounts_all->count() > 1)
 	<div class="col-12 col-sm-4 col-md-2">
 		<div class="info-box">
 			<span class="info-box-icon bg-primary elevation-1"><i class="fas fa-user"></i></span>
 
 			<div class="info-box-content">
 				<span class="info-box-text">Linked Accounts</span>
-				<span class="info-box-number">{{ number_format($o->my_accounts->count()) }}</span>
+				<span class="info-box-number">{{ number_format($o->accounts_all->count()) }}</span>
 			</div>
 		</div>
 	</div>
@@ -33,8 +31,7 @@
 
 		<div class="info-box-content">
 			<span class="info-box-text">Active Services</span>
-			<!-- @todo This should count of inactive services too -->
-			<span class="info-box-number">{{ $o->services->count() }} <small>/{{ $o->services->count() }}</small></span>
+			<span class="info-box-number">{{ $o->accounts_all->map(fn($item)=>$item->services->where('active',TRUE)->count())->sum() }} <small>/{{ $o->accounts_all->map(fn($item)=>$item->services->count())->sum() }}</small></span>
 		</div>
 	</div>
 </div>
@@ -45,7 +42,7 @@
 
 		<div class="info-box-content">
 			<span class="info-box-text">Account Balance</span>
-			<span class="info-box-number"><small>$</small> {{ number_format(($x=$o->accounts->map(fn($item)=>$item->invoiceSummaryDue()->get()->pluck('_balance'))->flatten())->sum(),2) }}</span>
+			<span class="info-box-number"><small>$</small> {{ number_format(($x=$o->accounts_all->map(fn($item)=>$item->invoiceSummaryDue()->get()->pluck('_balance'))->flatten())->sum(),2) }}</span>
 		</div>
 	</div>
 </div>

resources/views/theme/backend/adminlte/home.blade.php

@@ -24,7 +24,7 @@
 		<div class="card-header p-0 pt-1">
 			<ul class="nav nav-tabs" id="accounts-tab" role="tablist">
 				<li class="pt-2 px-3"><h3 class="card-title">Accounts</h3></li>
-				@foreach($o->my_accounts as $ao)
+				@foreach($o->accounts as $ao)
 					<li class="nav-item">
 						<a class="nav-link @if(! $loop->index)active @endif" href="#account_{{ $ao->id }}" data-toggle="tab" aria-controls="account_{{ $ao->id }}" aria-selected="true">{{ $ao->name }}</a>
 					</li>
@@ -42,7 +42,7 @@
 
 		<div class="card-body">
 			<div class="tab-content" id="accounts-tab-content">
-				@foreach($o->my_accounts as $ao)
+				@foreach($o->accounts as $ao)
 					<div class="tab-pane fade @if(! $loop->index)show active @endif" id="account_{{ $ao->id }}" role="tabpanel" aria-labelledby="account_{{ $ao->id }}">
 						<div class="row">
 							<div class="col-12">
@@ -93,7 +93,7 @@
 					</div>
 				@endforeach
 
-				@if ($o == $user)
+				@if($o==$user)
 					@canany('reseller','wholesaler')
 						<div class="tab-pane" id="tab-reseller" role="tabpanel">
 							@include('theme.backend.adminlte.r.home.widget.home')

resources/views/theme/backend/adminlte/r/account/widgets/list.blade.php

@@ -4,7 +4,7 @@
 	</div>
 
 	<div class="card-body">
-		@if ($user->accounts->count())
+		@if ($x=$user->accounts_all->count())
 			<table class="table table-striped table-hover" id="accounts">
 				<thead>
 				<tr>
@@ -14,7 +14,7 @@
 				</tr>
 				</thead>
 				<tbody>
-				@foreach ($user->accounts as $ao)
+				@foreach ($user->accounts_all as $ao)
 					<tr>
 						<td><a href="{{ url('r/switch/start',$ao->user_id) }}"><i class="fas fa-external-link-alt"></i></a></td>
 						<td>{{ $ao->name }}</td>
@@ -25,7 +25,7 @@
 
 				<tfoot>
 				<tr>
-					<th>Count {{ $user->accounts->count() }}</th>
+					<th>Count {{ $x }}</th>
 					<th colspan="2">&nbsp;</th>
 				</tr>
 				</tfoot>

resources/views/theme/backend/adminlte/r/service/widget/movement.blade.php

@@ -1,3 +1,7 @@
+@php
+use App\Models\Service;
+@endphp
+
 <!-- Show client movements -->
 <div class="card card-dark">
 	<div class="card-header">
@@ -5,7 +9,7 @@
 	</div>
 
 	<div class="card-body">
-		@if (($x=$user->client_service_movements())->count())
+		@if (($x=Service::movements($user))->count())
 			<table class="table table-striped table-hover" id="service_movements">
 				<thead>
 				<tr>