Soft or hard deleted ticket creator: View ticket card and ticket in list

Author: xaviqv

src/Controllers/TicketsController.php

@@ -58,17 +58,20 @@ public function data($ticketList = 'active')
 			->leftJoin('panichd_members', function ($join2) {
 				$join2->on('panichd_members.id', '=', 'panichd_tickets.user_id');
 			})
+			->leftJoin('panichd_members as creator', function ($join3){
+				$join3->on('creator.id', '=', 'panichd_tickets.creator_id');
+			})
 			->join('panichd_statuses', 'panichd_statuses.id', '=', 'panichd_tickets.status_id')
-            ->leftJoin('panichd_members as agent', function ($join3){
-				$join3->on('agent.id', '=', 'panichd_tickets.agent_id');
+            ->leftJoin('panichd_members as agent', function ($join4){
+				$join4->on('agent.id', '=', 'panichd_tickets.agent_id');
 			})
 			->join('panichd_priorities', 'panichd_priorities.id', '=', 'panichd_tickets.priority_id')
             ->join('panichd_categories', 'panichd_categories.id', '=', 'panichd_tickets.category_id')			
 
 
 			// Tags joins
-			->leftJoin('panichd_taggables', function ($join4) {
-                $join4->on('panichd_tickets.id', '=', 'panichd_taggables.taggable_id')
+			->leftJoin('panichd_taggables', function ($join5) {
+                $join5->on('panichd_tickets.id', '=', 'panichd_taggables.taggable_id')
                     ->where('panichd_taggables.taggable_type', '=', 'PanicHD\\PanicHD\\Models\\Ticket');
             })
             ->leftJoin('panichd_tags', 'panichd_taggables.tag_id', '=', 'panichd_tags.id');
@@ -106,6 +109,7 @@ public function data($ticketList = 'active')
 			'panichd_priorities.name AS priority',
 			'panichd_priorities.magnitude AS priority_magnitude',
 			'panichd_members.name AS owner_name',
+			'creator.name as creator_name',
 			'panichd_tickets.user_id',
 			'panichd_tickets.creator_id',		
 			'panichd_tickets.category_id',
@@ -318,7 +322,7 @@ public function renderTicketTable($collection)
 			}
 
 			if ($ticket->user_id != $ticket->creator_id){
-				$return .="&nbsp;<span class=\"glyphicon glyphicon-user tooltip-info\" title=\"".trans('panichd::lang.show-ticket-creator').trans('panichd::lang.colon').$ticket->creator->name."\" data-toggle=\"tooltip\" data-placement=\"auto bottom\" style=\"color: #aaa;\"></span>";				
+				$return .="&nbsp;<span class=\"glyphicon glyphicon-user tooltip-info\" title=\"".trans('panichd::lang.show-ticket-creator').trans('panichd::lang.colon'). ($ticket->creator_name == "" ? trans('panichd::lang.deleted-member') : (is_null($ticket->creator) ? $ticket->creator_name : $ticket->creator->name)) ."\" data-toggle=\"tooltip\" data-placement=\"auto bottom\" style=\"color: #aaa;\"></span>";				
 			}
 
 			return $return;
@@ -1043,12 +1047,16 @@ public function show($id)
 
 		$ticket = $this->tickets
 			->with('owner')
+			->with('creator')
 			->with('agent')
 			->with('category.closingReasons')
 			->with('tags')
 			->join('panichd_members', 'panichd_members.id', '=', 'panichd_tickets.user_id')
-			->leftJoin('panichd_members as agent', function($join1){
-				$join1->on('agent.id', '=', 'panichd_tickets.agent_id');
+			->leftJoin('panichd_members as creator', function($join1){
+				$join1->on('creator.id', '=', 'panichd_tickets.creator_id');
+			})
+			->leftJoin('panichd_members as agent', function($join2){
+				$join2->on('agent.id', '=', 'panichd_tickets.agent_id');
 			});
 
 		if (Setting::grab('departments_feature')){
@@ -1058,6 +1066,7 @@ public function show($id)
 		$a_select = [
 			'panichd_tickets.*',
 			'panichd_members.name as owner_name',
+			'creator.name as creator_name',
 			'agent.name as agent_name',
 			'panichd_members.email as owner_email'
 		];

src/Views/tickets/show/body.blade.php

@@ -67,7 +67,14 @@ class="btn btn-default"
 				<strong>{{ trans('panichd::lang.ticket') }}</strong>{{ trans('panichd::lang.colon') . trans('panichd::lang.table-id') . $ticket->id }}
 				@if ($u->currentLevel() > 1)
 					@if ($ticket->user_id != $ticket->creator_id)
-						<br /><strong>{{ trans('panichd::lang.show-ticket-creator') }}</strong>{{ trans('panichd::lang.colon') . $ticket->creator->name }}<br />
+						<?php $creator_name = $ticket->creator_name == "" ? trans('panichd::lang.deleted-member') : (is_null($ticket->creator) ? $ticket->creator_name : $ticket->creator->name); ?>
+						<br /><strong>{{ trans('panichd::lang.show-ticket-creator') }}</strong>{{ trans('panichd::lang.colon') }}
+						@if ($ticket->creator_name == "" || is_null($ticket->creator))
+							<span class="glyphicon glyphicon-exclamation-sign tooltip-info text-danger"  data-toggle="tooltip" data-placement="auto bottom" title="{{ trans('panichd::lang.deleted-member') }}"> {{ $creator_name }}</span>
+						@else
+							{{ $creator_name }}
+						@endif
+						<br />
 					@endif
 
 					<br /><strong>{{ trans('panichd::lang.owner') }}</strong>{{ trans('panichd::lang.colon') }}