From b7b6a575bcd27634f44c6bfeb42fcb2bb32e21d1 Mon Sep 17 00:00:00 2001
From: Deon George <deon@leenooks.net>
Date: Fri, 17 Dec 2021 16:09:03 +1100
Subject: [PATCH] Site related updates

---
 app/Http/Controllers/AdminController.php      |  8 ++---
 app/Http/Controllers/Auth/LoginController.php |  2 +-
 app/Http/Controllers/OrderController.php      | 14 ++++-----
 app/Models/Invoice.php                        |  2 +-
 app/Models/Scopes/SiteScope.php               | 22 ++++++++++++++
 app/Models/User.php                           |  3 +-
 app/Traits/NextKey.php                        |  6 ++--
 app/Traits/OrderServiceOptions.php            |  2 +-
 app/Traits/SiteID.php                         | 29 +++++++++++++++++++
 .../theme/backend/adminlte/a/setup.blade.php  |  2 +-
 10 files changed, 71 insertions(+), 19 deletions(-)
 create mode 100644 app/Models/Scopes/SiteScope.php
 create mode 100644 app/Traits/SiteID.php

diff --git a/app/Http/Controllers/AdminController.php b/app/Http/Controllers/AdminController.php
index 066d756..9ec6a86 100644
--- a/app/Http/Controllers/AdminController.php
+++ b/app/Http/Controllers/AdminController.php
@@ -31,7 +31,7 @@ class AdminController extends Controller
 			]);
 
 			if (! $o->exists) {
-				$o->site_id = config('SITE')->site_id;
+				$o->site_id = config('site')->site_id;
 				$o->user_id = Auth::id();
 				$o->active = TRUE;
 			}
@@ -92,7 +92,7 @@ class AdminController extends Controller
 
 			if (! $o->exists) {
 				$o->forceFill($request->only(['account_id','payment_date','checkout_id','checkout_id','total_amt','fees_amt','source_id','pending','notes','ip']));
-				$o->site_id = config('SITE')->site_id;
+				$o->site_id = config('site')->site_id;
 				$o->save();
 			}
 
@@ -114,7 +114,7 @@ class AdminController extends Controller
 				}
 
 				$oo->alloc_amt = ($oo->invoice->due >= 0) && ($oo->invoice->due-$amount >= 0) ? $amount : 0;
-				$oo->site_id = config('SITE')->site_id;
+				$oo->site_id = config('site')->site_id;
 				$o->items()->save($oo);
 			}
 
@@ -178,7 +178,7 @@ class AdminController extends Controller
 				'email_logo' => 'nullable|image',
 			]);
 
-			$site = config('SITE');
+			$site = config('site');
 
 			// @todo - not currently rendered in the home page
 			$validated['social'] = [];
diff --git a/app/Http/Controllers/Auth/LoginController.php b/app/Http/Controllers/Auth/LoginController.php
index af9c0b8..3ea3eaf 100644
--- a/app/Http/Controllers/Auth/LoginController.php
+++ b/app/Http/Controllers/Auth/LoginController.php
@@ -46,7 +46,7 @@ class LoginController extends Controller
 	{
 		$this->validateLogin($request);
 
-		if (Auth::attempt(array_merge($this->credentials($request),['active'=>TRUE,'site_id'=>config('site')->site_id]))) {
+		if (Auth::attempt(array_merge($this->credentials($request),['active'=>TRUE]))) {
 			$request->session()->regenerate();
 
 			return $this->sendLoginResponse($request);
diff --git a/app/Http/Controllers/OrderController.php b/app/Http/Controllers/OrderController.php
index ec1135e..cbaa381 100644
--- a/app/Http/Controllers/OrderController.php
+++ b/app/Http/Controllers/OrderController.php
@@ -67,11 +67,11 @@ class OrderController extends Controller
 			// If this is a new client
 			if (! $uo->exists) {
 				// @todo Make this automatic
-				$uo->site_id = config('SITE')->site_id;
+				$uo->site_id = config('site')->site_id;
 				$uo->active = FALSE;
 				$uo->firstname = '';
 				$uo->lastname = '';
-				$uo->country_id = config('SITE')->country_id; // @todo This might be wrong
+				$uo->country_id = config('site')->country_id; // @todo This might be wrong
 				$uo->parent_id = Auth::id() ?: 1; // @todo This should be configured to a default user
 				$uo->active = 1;
 				$uo->save();
@@ -83,10 +83,10 @@ class OrderController extends Controller
 			$ao = new Account;
 			//$ao->id = Account::NextId();
 			// @todo Make this automatic
-			$ao->site_id = config('SITE')->site_id;
-			$ao->country_id = config('SITE')->country_id; // @todo This might be wrong
-			$ao->language_id = config('SITE')->language_id; // @todo This might be wrong
-			$ao->currency_id = config('SITE')->currency_id; // @todo This might be wrong
+			$ao->site_id = config('site')->site_id;
+			$ao->country_id = config('site')->country_id; // @todo This might be wrong
+			$ao->language_id = config('site')->language_id; // @todo This might be wrong
+			$ao->currency_id = config('site')->currency_id; // @todo This might be wrong
 			$ao->active = 1;
 			$uo->accounts()->save($ao);
 
@@ -97,7 +97,7 @@ class OrderController extends Controller
 		$so = new Service;
 
 		// @todo Make this automatic
-		$so->site_id = config('SITE')->site_id;
+		$so->site_id = config('site')->site_id;
 		$so->product_id = $request->input('product_id');
 		$so->order_status = 'ORDER-SUBMIT';
 		$so->orderby_id = Auth::id();
diff --git a/app/Models/Invoice.php b/app/Models/Invoice.php
index ab5d28c..2070e00 100644
--- a/app/Models/Invoice.php
+++ b/app/Models/Invoice.php
@@ -141,7 +141,7 @@ class Invoice extends Model implements IDs
 	// @todo Move this to a site configuration
 	public function getInvoiceTextAttribute()
 	{
-		return sprintf('Thank you for using %s for your Internet Services.',config('SITE')->site_name);
+		return sprintf('Thank you for using %s for your Internet Services.',config('site')->site_name);
 	}
 
 	/**
diff --git a/app/Models/Scopes/SiteScope.php b/app/Models/Scopes/SiteScope.php
new file mode 100644
index 0000000..549a88b
--- /dev/null
+++ b/app/Models/Scopes/SiteScope.php
@@ -0,0 +1,22 @@
+<?php
+
+namespace App\Models\Scopes;
+
+use Illuminate\Database\Eloquent\Builder;
+use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Scope;
+
+class SiteScope implements Scope
+{
+	/**
+	 * Apply the scope to a given Eloquent query builder.
+	 *
+	 * @param  \Illuminate\Database\Eloquent\Builder  $builder
+	 * @param  \Illuminate\Database\Eloquent\Model  $model
+	 * @return void
+	 */
+	public function apply(Builder $builder, Model $model)
+	{
+		$builder->where($model->getTable().'.site_id',config('site')->site_id);
+	}
+}
\ No newline at end of file
diff --git a/app/Models/User.php b/app/Models/User.php
index c2b4540..879c57e 100644
--- a/app/Models/User.php
+++ b/app/Models/User.php
@@ -14,10 +14,11 @@ use Leenooks\Traits\UserSwitch;
 use Spinen\QuickBooks\HasQuickBooksToken;
 
 use App\Notifications\ResetPassword as ResetPasswordNotification;
+use App\Traits\SiteID;
 
 class User extends Authenticatable
 {
-	use HasApiTokens,Notifiable,UserSwitch,HasQuickBooksToken;
+	use HasApiTokens,Notifiable,UserSwitch,HasQuickBooksToken,SiteID;
 
 	protected $appends = [
 		'active_display',
diff --git a/app/Traits/NextKey.php b/app/Traits/NextKey.php
index aaddde9..6343f4e 100644
--- a/app/Traits/NextKey.php
+++ b/app/Traits/NextKey.php
@@ -23,7 +23,7 @@ trait NextKey
 			$model->id = self::NextId();
 
 			if (! $model->site_id)
-				$model->site_id = config('SITE')->site_id;
+				$model->site_id = config('site')->site_id;
 		});
 
 		static::saved(function($model)
@@ -41,14 +41,14 @@ trait NextKey
 
 					$mo = new Module;
 					$mo->name = $model::RECORD_ID;
-					$mo->site_id = $model->site_id ?: config('SITE')->site_id;
+					$mo->site_id = $model->site_id ?: config('site')->site_id;
 					$mo->save();
 				}
 
 				if (! $mo->record) {
 					$mo->record = new Record;
 					$mo->record->module_id = $mo->id;
-					$mo->record->site_id = $model->site_id ?: config('SITE')->site_id;
+					$mo->record->site_id = $model->site_id ?: config('site')->site_id;
 				}
 
 				$mo->record->id = $model->id;
diff --git a/app/Traits/OrderServiceOptions.php b/app/Traits/OrderServiceOptions.php
index 3913f18..ae3cee6 100644
--- a/app/Traits/OrderServiceOptions.php
+++ b/app/Traits/OrderServiceOptions.php
@@ -39,7 +39,7 @@ trait OrderServiceOptions
 		$o->forceFill(array_undot($x));
 
 		// @todo Make this automatic
-		$o->site_id = config('SITE')->site_id;
+		$o->site_id = config('site')->site_id;
 
 		return $o;
 	}
diff --git a/app/Traits/SiteID.php b/app/Traits/SiteID.php
new file mode 100644
index 0000000..d28d4e0
--- /dev/null
+++ b/app/Traits/SiteID.php
@@ -0,0 +1,29 @@
+<?php
+
+/**
+ * Ensures we retrieve and save models with the correct site_id
+ */
+namespace App\Traits;
+
+use Illuminate\Support\Arr;
+
+use App\Models\Scopes\SiteScope;
+
+trait SiteID
+{
+	/**
+	 * This model is site scoped
+	 */
+	protected static function booted()
+	{
+		static::addGlobalScope(new SiteScope);
+	}
+
+	public function save(array $options = [])
+	{
+		if (! Arr::get($this->attributes,'site_id'))
+			$this->site_id = config('site')->site_id;
+
+		return parent::save();
+	}
+}
\ No newline at end of file
diff --git a/resources/views/theme/backend/adminlte/a/setup.blade.php b/resources/views/theme/backend/adminlte/a/setup.blade.php
index 2585004..b36949b 100644
--- a/resources/views/theme/backend/adminlte/a/setup.blade.php
+++ b/resources/views/theme/backend/adminlte/a/setup.blade.php
@@ -11,7 +11,7 @@
 	{{ $site->site_name }}
 @endsection
 @section('contentheader_description')
-	Setup
+	Setup #{{ $site->site_id }}
 @endsection
 
 @section('main-content')