From ac867a25265d07476967eef7d01516a92b9f2b73 Mon Sep 17 00:00:00 2001
From: Deon George <deon@leenooks.net>
Date: Fri, 13 Jul 2018 14:39:10 +1000
Subject: [PATCH] Enhancements to user switch

---
 resources/themes/adminlte/lang/en/message.php |  1 +
 .../adminlte/views/auth/register.blade.php    |  2 +-
 .../layouts/partials/mainheader.blade.php     | 26 ++++++++++++-------
 src/Controllers/AdminController.php           | 25 +++++++++++-------
 src/Traits/UserSwitch.php                     | 16 ++++++++++++
 5 files changed, 49 insertions(+), 21 deletions(-)
 create mode 100644 src/Traits/UserSwitch.php

diff --git a/resources/themes/adminlte/lang/en/message.php b/resources/themes/adminlte/lang/en/message.php
index 5defedf..fa6276e 100644
--- a/resources/themes/adminlte/lang/en/message.php
+++ b/resources/themes/adminlte/lang/en/message.php
@@ -136,4 +136,5 @@ return [
     'loggedin'                => 'Logged in!',
     'entering'                => 'Entering...',
     'registered'              => 'User Registered!',
+    'switchoff'               => 'Switch Back',
 ];
diff --git a/resources/themes/adminlte/views/auth/register.blade.php b/resources/themes/adminlte/views/auth/register.blade.php
index 088ddf0..eefe88f 100644
--- a/resources/themes/adminlte/views/auth/register.blade.php
+++ b/resources/themes/adminlte/views/auth/register.blade.php
@@ -10,7 +10,7 @@
     <div id="app" v-cloak>
         <div class="register-box">
             <div class="register-logo">
-                <a href="{{ url('/home') }}"><b>Pipeline</b>Management</a>
+                <a href="{{ url('/home') }}">{!! config('app.name_html_long') !!}</a>
             </div>
 
             @if (count($errors) > 0)
diff --git a/resources/themes/adminlte/views/layouts/partials/mainheader.blade.php b/resources/themes/adminlte/views/layouts/partials/mainheader.blade.php
index 58b6220..7574731 100644
--- a/resources/themes/adminlte/views/layouts/partials/mainheader.blade.php
+++ b/resources/themes/adminlte/views/layouts/partials/mainheader.blade.php
@@ -45,7 +45,7 @@
 					<!-- Top Menu Items -->
 					@include('adminlte::layouts.partials.topmenu')
 
-					<li class="dropdown user user-menu" id="user_menu">
+					<li class="dropdown user user-menu @if($user->switched()) bg-red @endif" id="user_menu">
 					<!-- Menu Toggle Button -->
 						<a href="#" class="dropdown-toggle" data-toggle="dropdown">
 							<!-- The user image in the navbar-->
@@ -69,16 +69,22 @@
 								</div>
 
 								<div class="pull-right">
-									<a href="{{ url('/logout') }}" class="btn btn-default btn-flat" id="logout"
-										onclick="event.preventDefault();
-										document.getElementById('logout-form').submit();">
-										{{ trans('adminlte_lang::message.signout') }}
-									</a>
+									@if ($user->switched())
+										<a href="{{ url('/admin/switch/stop') }}" class="btn btn-default btn-flat" id="switch">
+											{{ trans('adminlte_lang::message.switchoff') }}
+										</a>
+									@else
+										<a href="{{ url('/logout') }}" class="btn btn-default btn-flat" id="logout"
+										   onclick="event.preventDefault();
+													document.getElementById('logout-form').submit();">
+											{{ trans('adminlte_lang::message.signout') }}
+										</a>
 
-									<form id="logout-form" action="{{ url('/logout') }}" method="POST" style="display: none;">
-										{{ csrf_field() }}
-										<input type="submit" value="logout" style="display: none;">
-									</form>
+										<form id="logout-form" action="{{ url('/logout') }}" method="POST" style="display: none;">
+											{{ csrf_field() }}
+											<input type="submit" value="logout" style="display: none;">
+										</form>
+									@endif
 								</div>
 							</li>
 						</ul>
diff --git a/src/Controllers/AdminController.php b/src/Controllers/AdminController.php
index 49b1f0e..6dae1d4 100644
--- a/src/Controllers/AdminController.php
+++ b/src/Controllers/AdminController.php
@@ -2,12 +2,13 @@
 
 namespace Leenooks\Controllers;
 
+use Illuminate\Support\Facades\Auth;
 use App\Http\Controllers\Controller;
-use App\User;
-use Auth;
 use Redirect;
 use Session;
 
+use App\User;
+
 class AdminController extends Controller
 {
 	public function __construct()
@@ -15,12 +16,22 @@ class AdminController extends Controller
 		$this->middleware('auth');
 	}
 
+	public function switch_authorised()
+	{
+		return TRUE;
+	}
+
+	public function switch_session()
+	{
+		return ! Session::get('orig_user');
+	}
+
 	/**
 	 * @todo Change the background color (or something) so we know we are switched
 	 */
 	public function user_switch_start($id)
 	{
-		if ($this->switch_authorised())
+		if ($this->switch_session() AND $this->switch_authorised())
 		{
 			$uo = User::find($id);
 
@@ -43,10 +54,4 @@ class AdminController extends Controller
 
 		return Redirect::to('/home');
 	}
-
-	public function switch_authorised()
-	{
-		// @todo
-		return TRUE;
-	}
-}
+}
\ No newline at end of file
diff --git a/src/Traits/UserSwitch.php b/src/Traits/UserSwitch.php
new file mode 100644
index 0000000..a760421
--- /dev/null
+++ b/src/Traits/UserSwitch.php
@@ -0,0 +1,16 @@
+<?php
+
+/**
+ * Check if users have been switched
+ */
+namespace Leenooks\Traits;
+
+use Session;
+
+trait UserSwitch
+{
+	public function switched()
+	{
+		return Session::get('orig_user');
+	}
+}
\ No newline at end of file