Fix rogue ) being displayed, and other updates from osb

composer.json

@@ -1,37 +1,37 @@
 {
-    "name": "leenooks/laravel",
-    "description": "Leenooks standard templates and tools.",
-    "keywords": ["laravel", "leenooks"],
-    "license": "MIT",
-    "authors": [
-        {
-            "name": "Deon George",
-            "email": "deon@leenooks.net"
-        }
-    ],
-    "require": {
-        "creativeorange/gravatar": "^1.0",
-        "orchestra/asset": "^6.0"
-    },
-    "require-dev": {
-    },
-    "autoload": {
-        "psr-4": {
-            "Leenooks\\": "src"
-        },
-        "files": [
-            "src/helpers.php"
-        ]
-    },
-    "extra": {
-        "laravel": {
-            "providers": [
-                "Orchestra\\Asset\\AssetServiceProvider",
-                "Collective\\Html\\HtmlServiceProvider",
-                "Leenooks\\Providers\\LeenooksServiceProvider",
-                "Leenooks\\Providers\\CustomBladeServiceProvider"
-            ]
-        }
-    },
-    "minimum-stability": "dev"
+	"name": "laravel/leenooks",
+	"description": "Leenooks standard templates and tools.",
+	"keywords": ["laravel","leenooks"],
+	"license": "MIT",
+	"authors": [
+		{
+			"name": "Deon George",
+			"email": "deon@dege.au"
+		}
+	],
+	"require": {
+		"creativeorange/gravatar": "^1.0",
+		"orchestra/asset": "^6.0"
+	},
+	"require-dev": {
+	},
+	"autoload": {
+		"psr-4": {
+			"Leenooks\\": "src"
+		},
+		"files": [
+			"src/helpers.php"
+		]
+	},
+	"extra": {
+		"laravel": {
+			"providers": [
+				"Orchestra\\Asset\\AssetServiceProvider",
+				"Collective\\Html\\HtmlServiceProvider",
+				"Leenooks\\Providers\\LeenooksServiceProvider",
+				"Leenooks\\Providers\\CustomBladeServiceProvider"
+			]
+		}
+	},
+	"minimum-stability": "dev"
 }

resources/themes/adminlte/views/layouts/partials/htmlheader.blade.php

@@ -6,16 +6,19 @@
 	<!-- CSRF Token -->
 	<meta name="csrf-token" content="{{ csrf_token() }}">
 
-	<link rel="stylesheet" href="//cdn.jsdelivr.net/npm/bootstrap@4.6.0/dist/css/bootstrap.min.css" integrity="sha384-B0vP5xmATw1+K9KRQjQERJvTumQW0nPEzvF6L/Z6nronJ3oUOFUFpCjEUQouq2+l" crossorigin="anonymous">
+	{{--
+	<!-- Included in adminlte -->
+	<link rel="stylesheet" href="//cdn.jsdelivr.net/npm/bootstrap@4.6.1/dist/css/bootstrap.min.css" integrity="sha384-B0vP5xmATw1+K9KRQjQERJvTumQW0nPEzvF6L/Z6nronJ3oUOFUFpCjEUQouq2+l" crossorigin="anonymous">
+	--}}
 
 	<!-- Font Awesome Icons -->
-	<link rel="stylesheet" href="//use.fontawesome.com/releases/v5.15.3/css/all.css">
+	<link rel="stylesheet" href="//use.fontawesome.com/releases/v5.15.4/css/all.css">
 	{{--
-	<link rel="stylesheet" href="//use.fontawesome.com/releases/v5.15.3/css/v4-shims.css">
+	<link rel="stylesheet" href="//use.fontawesome.com/releases/v5.15.4/css/v4-shims.css">
 	--}}
 
 	<!-- Theme style -->
-	<link rel="stylesheet" href="//cdn.jsdelivr.net/npm/admin-lte@3.1.0/dist/css/adminlte.min.css">
+	<link rel="stylesheet" href="//cdn.jsdelivr.net/npm/admin-lte@3.2.0/dist/css/adminlte.min.css">
 
 	<!-- Google Font: Source Sans Pro -->
 	<link href="https://fonts.googleapis.com/css2?family={{ str_replace(' ','+',config('app.font') ?: 'IBM Plex Sans') }}:wght@300&display=swap" rel="stylesheet">

resources/themes/adminlte/views/layouts/partials/mainheader.blade.php

@@ -83,7 +83,7 @@
 
 	<script type="text/javascript">
 		$(document).ready(function() {
-			$("input[name=q]").typeahead({
+			$('input[name=q]').typeahead({
 				autoSelect: false,
 				scrollHeight: 10,
 				theme: 'bootstrap4',
@@ -149,6 +149,11 @@
 							$('span[name=searching]').addClass("d-none");
 						}
 					}
+				},
+				statusCode: {
+					401: function() {
+						window.parent.location.href = '{{ route('login') }}';
+					}
 				}
 			})
 		}, 500);

resources/themes/adminlte/views/layouts/partials/scripts.blade.php

@@ -2,9 +2,9 @@
 <!-- Bootstrap & Jquery App -->
 <script type="text/javascript" src="//cdnjs.cloudflare.com/ajax/libs/jquery/3.6.0/jquery.min.js" integrity="sha512-894YE6QWD5I59HgZOGReFYm4dnWc1Qt5NtvYSaNcOP+u1T9qYdvdihz0PPSiiqn/+/3e7Jo4EaG7TubfWGUrMQ==" crossorigin="anonymous"></script>
 <script type="text/javascript" src="//cdnjs.cloudflare.com/ajax/libs/lodash.js/4.17.21/lodash.min.js" integrity="sha512-WFN04846sdKMIP5LKNphMaWzU7YpMyCU245etK3g/2ARYbPK9Ub18eG+ljU96qKRCWh+quCY7yefSmlkQw1ANQ==" crossorigin="anonymous"></script>
-<script type="text/javascript" src="//cdn.jsdelivr.net/npm/bootstrap@4.6.0/dist/js/bootstrap.bundle.min.js" integrity="sha384-Piv4xVNRyMGpqkS2by6br4gNJ7DXjqk09RmUpJ8jgGtD7zP9yug3goQfGII0yAns" crossorigin="anonymous"></script>
+<script type="text/javascript" src="//cdn.jsdelivr.net/npm/bootstrap@4.6.1/dist/js/bootstrap.bundle.min.js" integrity="sha384-fQybjgWLrvvRgtW6bFlB7jaZrFsaBXjsOMm/tB9LTS58ONXgqbR9W8oWht/amnpF" crossorigin="anonymous"></script>
 <!-- AdminLTE -->
-<script type="text/javascript" src="//cdn.jsdelivr.net/npm/admin-lte@3.1.0/dist/js/adminlte.min.js"></script>
+<script type="text/javascript" src="//cdn.jsdelivr.net/npm/admin-lte@3.2.0/dist/js/adminlte.min.js"></script>
 
 <!-- Additional Utilities -->
 <script src="{{ asset('plugin/bootstrap3-typeahead/js/bootstrap3-typeahead.min.js') }}"></script>
@@ -44,7 +44,7 @@
 		$.AdminLTESidebarTweak.Open = state;
 
 		if (addclass !== 'undefined')
-			$("body")
+			$('body')
 				.addClass(state ? 'sidebar-open' : 'sidebar-collapse');
 
 		$(this).delay($.AdminLTESidebarTweak.ResizeDelay).queue(function() {
@@ -56,7 +56,7 @@
 	$(function () {
 		"use strict";
 
-		$("body").on("collapsed.lte.pushmenu", function(){
+		$('body').on('collapsed.lte.pushmenu', function(){
 			if($.AdminLTESidebarTweak.options.EnableRemember) {
 				document.cookie = "toggleState=closed;path=/";
 
@@ -64,7 +64,7 @@
 			}
 		});
 
-		$("body").on("shown.lte.pushmenu", function(){
+		$('body').on('shown.lte.pushmenu', function(){
 			if($.AdminLTESidebarTweak.options.EnableRemember){
 				document.cookie = "toggleState=opened;path=/";
 

resources/themes/adminlte/views/widget/errors.blade.php

@@ -0,0 +1,12 @@
+<!-- Errors -->
+@if ($errors->any())
+	<div class="alert alert-danger alert-dismissible">
+		<button type="button" class="close" data-dismiss="alert" aria-hidden="true">&times;</button>
+		<h5><i class="icon fas fa-ban"></i> Whoops!</h5>
+		<ul>
+			@foreach ($errors->all() as $error)
+				<li>{!! $error !!}</li>
+			@endforeach
+		</ul>
+	</div>
+@endif

resources/themes/adminlte/views/widget/form_date.blade.php

@@ -0,0 +1,17 @@
+<div class="form-group">
+	<label for="{{ $id }}">{{ $label }}</label>
+	<div class="input-group has-validation">
+		@empty($icon)
+		@else
+			<div class="input-group-prepend">
+				<span class="input-group-text"><i class="fa-fw {{ $icon }}"></i></span>
+			</div>
+		@endempty()
+		<input type="date" class="form-control @error($id) is-invalid @enderror" id="{{ $id }}" name="{{ $name ?? $id}}" value="{{ old($old,$value) }}">
+		<span class="invalid-feedback" role="alert">
+			@error($id)
+				{{ $message }}
+			@enderror
+		</span>
+	</div>
+</div>

resources/themes/adminlte/views/widget/form_file.blade.php

@@ -0,0 +1,39 @@
+<div class="form-group">
+	<label for="{{ $id }}">{{ $label }}</label>
+	<div class="input-group has-validation">
+		@empty($icon)
+		@else
+			<div class="input-group-prepend">
+				<span class="input-group-text"><i class="fa-fw {{ $icon }}"></i></span>
+			</div>
+		@endempty()
+		<div class="custom-file">
+			<input type="file" class="custom-file-input @error($id) is-invalid @enderror" id="{{ $id }}" name="{{ $name ?? $id}}">
+			<label class="custom-file-label input-group-text" for="{{ $id }}">Choose file</label>
+		</div>
+
+		{{--
+		<div class="input-group-append">
+			<button type="submit" class="input-group-text btn btn-success" id="">Upload</button>
+		</div>
+		--}}
+		<span class="invalid-feedback @error($id) d-block @enderror" role="alert">
+			@error($id)
+				{{ $message }}
+			@enderror
+		</span>
+	</div>
+</div>
+
+@section('page-scripts')
+	<script>
+		$(document).ready(function() {
+			$('#{{ $id }}').on('change',function(){
+				//get the file name
+				var fileName = $(this).val();
+				//replace the "Choose a file" label
+				$(this).next('.custom-file-label').html(fileName);
+			})
+		});
+	</script>
+@append

resources/themes/adminlte/views/widget/form_select.blade.php

@@ -0,0 +1,57 @@
+<div class="form-group">
+	<label for="{{ $id }}">{{ $label }}</label>
+	<div class="input-group has-validation">
+		@empty($icon)
+		@else
+			<div class="input-group-prepend">
+				<span class="input-group-text"><i class="fa-fw {{ $icon }}"></i></span>
+			</div>
+		@endempty()
+
+		<select class="form-control @error($id) is-invalid @enderror" id="{{ $id }}" name="{{ $name ?? $id}}">
+			@if(! isset($addvalues) || ! $addvalues)
+				<option id="new"></option>
+			@endif
+			@empty($groupby)
+				@foreach($options as $option)
+					@empty(Arr::get($option,'value')) @continue @endempty
+					<option value="{{ Arr::get($option,'id') }}" @if(Arr::get($option,'id') == old($old,$value))selected @endif>{{ Arr::get($option,'value') }}</option>
+				@endforeach
+			@else
+				@foreach($options->groupBy($groupby) as $group)
+					<optgroup label="{{ $groupby == 'active' ? (Arr::get($group->first(),$groupby) ? 'Active' : 'Not Active') : Arr::get($group->first(),$groupby) }}">
+						@foreach($group as $option)
+							@empty(Arr::get($option,'value')) @continue @endempty
+							<option value="{{ Arr::get($option,'id') }}" @if(Arr::get($option,'id') == old($old,$value))selected @endif>{{ Arr::get($option,'value') }}</option>
+						@endforeach
+					</optgroup>
+				@endforeach
+			@endempty
+		</select>
+
+		<span class="invalid-feedback" role="alert">
+			@error($id)
+				{{ $message }}
+			@enderror
+		</span>
+	</div>
+</div>
+
+@section('page-scripts')
+	@css(select2)
+	@js(select2,autofocus)
+
+	<script type="text/javascript">
+		$(document).ready(function() {
+			$('#{{ $id }}').select2({
+				dropdownAutoWidth: true,
+				width: 'style',
+				@isset($addvalues)
+					tags: true,
+					placeholder: '',
+					allowClear: true,
+				@endisset
+			});
+		});
+	</script>
+@append

resources/themes/adminlte/views/widget/form_text.blade.php

@@ -0,0 +1,17 @@
+<div class="form-group">
+	<label for="{{ $id }}">{{ $label }}</label>
+	<div class="input-group has-validation">
+		@empty($icon)
+		@else
+			<div class="input-group-prepend">
+				<span class="input-group-text"><i class="fa-fw {{ $icon }}"></i></span>
+			</div>
+		@endempty()
+		<input type="text" class="form-control {{ $classes ?? ''}}@error($id) is-invalid @enderror" id="{{ $id }}" name="{{ $name ?: $id }}" value="{{ old($old ?? $name,$value) }}">
+		<span class="invalid-feedback" role="alert">
+			@error($id)
+				{{ $message }}
+			@enderror
+		</span>
+	</div>
+</div>

resources/themes/adminlte/views/widget/form_toggle.blade.php

@@ -0,0 +1,11 @@
+<div class="form-group">
+	<div class="custom-control custom-switch custom-switch-off-danger custom-switch-on-success">
+		<input type="checkbox" class="custom-control-input" id="{{ $id }}" name="{{ $name ?: $id }}" {{ old($old ?? $name,$value) ? 'checked' : '' }}>
+		<label class="custom-control-label" for="{{ $id }}">{{ $label }}</label>
+	</div>
+	<span class="invalid-feedback @error($id) d-block @enderror" role="alert">
+		@error($id)
+			{{ $message }}
+		@enderror
+	</span>
+</div>

resources/themes/adminlte/views/widget/status.blade.php

@@ -1,23 +1,3 @@
-<!-- Success -->
-@if(session()->has('success'))
-	<div class="alert alert-success alert-dismissible">
-		<button type="button" class="close" data-dismiss="alert" aria-hidden="true">&times;</button>
-		<h5><i class="icon fas fa-check"></i> Success!</h5>
-		<ul>
-			<li>{{ session()->get('success') }}</li>
-		</ul>
-	</div>
-@endif
+@include('adminlte::widget.success')
 
-<!-- Errors -->
-@if ($errors->any())
-	<div class="alert alert-danger alert-dismissible">
-		<button type="button" class="close" data-dismiss="alert" aria-hidden="true">&times;</button>
-		<h5><i class="icon fas fa-ban"></i> Whoops!</h5>
-		<ul>
-			@foreach ($errors->all() as $error)
-				<li>{!! $error !!}</li>
-			@endforeach
-		</ul>
-	</div>
-@endif
+@include('adminlte::widget.errors')

resources/themes/adminlte/views/widget/success.blade.php

@@ -0,0 +1,10 @@
+<!-- Success -->
+@if(session()->has('success'))
+	<div class="alert alert-success alert-dismissible">
+		<button type="button" class="close" data-dismiss="alert" aria-hidden="true">&times;</button>
+		<h5><i class="icon fas fa-check"></i> Success!</h5>
+		<ul>
+			<li>{{ session()->get('success') }}</li>
+		</ul>
+	</div>
+@endif

resources/themes/adminlte/views/widget/success_button.blade.php

@@ -0,0 +1 @@
+@if(session()->has('success'))<span class="ml-3 pt-0 pb-0 pr-1 pl-1 btn btn-outline-success float-right"><small>{{ session()->get('success') }}</small></span>@endif

src/Controllers/AdminController.php

@@ -1,54 +0,0 @@
-<?php
-
-namespace Leenooks\Controllers;
-
-use Illuminate\Support\Facades\Auth;
-use App\Http\Controllers\Controller;
-use Redirect;
-use Session;
-
-use App\Models\User;
-
-class AdminController extends Controller
-{
-	public function __construct()
-	{
-		$this->middleware('auth');
-	}
-
-	public function switch_authorised($id)
-	{
-		return (method_exists(Auth::user(),'isAdmin') && Auth::user()->isAdmin($id)) ? TRUE : FALSE;
-	}
-
-	public function switch_session()
-	{
-		return ! Session::get('orig_user');
-	}
-
-	public function user_switch_start($id)
-	{
-		if ($this->switch_session() AND $this->switch_authorised($id))
-		{
-			$uo = User::find($id);
-
-			if (! $uo)
-				abort(404,'User not found');
-
-			Session::put('orig_user',Auth::id());
-			Auth::login($uo);
-		}
-
-		return Redirect::to('/home');
-	}
-
-	public function user_switch_stop()
-	{
-		if ($id = Session::pull('orig_user')) {
-			$uo = User::find($id);
-			Auth::login($uo);
-		}
-
-		return Redirect::to('/home');
-	}
-}

src/Controllers/SwitchUserController.php

@@ -0,0 +1,62 @@
+<?php
+
+namespace Leenooks\Controllers;
+
+use App\Http\Controllers\Controller;
+use App\Providers\RouteServiceProvider;
+use Illuminate\Support\Facades\Auth;
+use Redirect;
+use Session;
+
+use App\Models\User;
+
+class SwitchUserController extends Controller
+{
+	public function __construct()
+	{
+		$this->middleware('auth');
+	}
+
+	/**
+	 * Determine if the user is authorised to switch to another user
+	 *
+	 * @param User $user
+	 * @return bool
+	 */
+	public function switch_authorised(User $user): bool
+	{
+		return (method_exists(Auth::user(),'isAdmin') && Auth::user()->isAdmin($user)) ? TRUE : FALSE;
+	}
+
+	/**
+	 * Switch to a different user
+	 *
+	 * @param User $user
+	 * @return mixed
+	 */
+	public function switch_start(User $user)
+	{
+		if ($user->switched)
+			abort(403,'User already switched');
+
+		if ($this->switch_authorised($user)) {
+			Session::put('orig_user',Auth::user());
+			Auth::login($user);
+		}
+
+		return Redirect::to('/home');
+	}
+
+	/**
+	 * Return back from the switch users
+	 *
+	 * @return mixed
+	 */
+	public function switch_stop()
+	{
+		if ($user = Session::pull('orig_user'))
+			Auth::login($user);
+
+		return Redirect::to(RouteServiceProvider::HOME);
+	}
+}

src/Traits/UserSwitch.php

@@ -7,14 +7,27 @@ namespace Leenooks\Traits;
 
 use Session;
 
+use App\Models\User;
+
 trait UserSwitch
 {
+	/**
+	 * Return if this is a switched user
+	 *
+	 * @return mixed
+	 */
 	public function getSwitchedAttribute()
 	{
 		return Session::get('orig_user');
 	}
 
-	public function isAdmin($id)
+	/**
+	 * If the user record has an admin attribute, we'll return that
+	 *
+	 * @param User|null $user
+	 * @return false|mixed
+	 */
+	public function isAdmin(User $user=NULL)
 	{
 		return isset($this->admin) ? $this->admin : FALSE;
 	}