Disable member deletion when has any related ticket

Author: xaviqv

src/Controllers/MembersController.php

@@ -86,12 +86,19 @@ public function update(Request $request, $id)
 
 	public function destroy(Request $request, $id)
 	{
-		$member = User::findOrFail($id);
+		$user = User::findOrFail($id);
+		$member = Models\Member::findOrFail($id);
 		if (auth()->user()->id == $id){
-			\Session::flash('warning', 'You cannot delete your own user account');
+			\Session::flash('warning', trans('panichd::admin.member-delete-own-user-error'));
 			return redirect()->back();
 		}
-		$member->delete();
+		
+		if ($member->userTickets()->count() > 0 or $member->agentTotalTickets()->count() > 0){
+			\Session::flash('warning', trans('panichd::admin.member-with-tickets-delete'));
+			return redirect()->back();
+		}
+		
+		$user->delete();
 
 		\Session::flash('status', trans('panichd::admin.member-deleted', ['name' => $member->name]));
 

src/Translations/ca/admin.php

@@ -187,6 +187,8 @@
   'member-added-ok'               => 'El membre ":name" s\'ha creat correctament',
   'member-updated-ok'             => 'El membre ":name" s\'ha actualitzat correctament',
   'member-deleted'                => 'El membre ":name" s\'ha ELIMINAT',
+  'member-delete-own-user-error'  => 'No pots eliminar el teu propi compte d\'usuari',
+  'member-with-tickets-delete'    => 'No pots eliminar un membre amb tiquets relacionats',
 
 // $admin_route_path/priority/____
   'priority-index-title'          => 'Gestió de prioritats',

src/Translations/en/admin.php

@@ -188,6 +188,8 @@
   'member-added-ok'               => 'Member ":name" has been created correctly',
   'member-updated-ok'             => 'Member ":name" has been updated correctly',
   'member-deleted'                => 'Member ":name" has been DELETED',
+  'member-delete-own-user-error'  => 'You cannot delete your own user account',
+  'member-with-tickets-delete'    => 'You cannot delete a member with related tickets',
 
 // $admin_route_path/priority/____
   'priority-index-title'          => 'Priorities Management',

src/Views/admin/member/index.blade.php

@@ -56,25 +56,30 @@
 						<td>
                             <button type="button" class="btn btn-default btn_member_modal" data-member_id="{{ $member->id }}" data-member_name="{{ $member->name }}" data-member_email="{{ $member->email }}" data-route="update" data-form_action="{{ route($setting->grab('admin_route').'.member.update', ['id' => $member->id ]) }}">{{ trans('panichd::admin.btn-edit') }}</button>
 							@if ($member->panichd_admin != '1')
-								{!! link_to_route(
-								$setting->grab('admin_route').'.member.destroy', trans('panichd::admin.btn-delete'), $member->id,
-								[
-								'class' => 'btn btn-default deleteit',
-								'form' => "delete-$member->id",
-								"user" => $member->name
-								])
+								@if ($member->user_tickets_count == 0 && $member->agent_total_tickets_count == 0)
+									{!! link_to_route(
+									$setting->grab('admin_route').'.member.destroy', trans('panichd::admin.btn-delete'), $member->id,
+									[
+									'class' => 'btn btn-default deleteit',
+									'form' => "delete-$member->id",
+									"user" => $member->name
+									])
+										!!}
+									
+									{!! CollectiveForm::open([
+													'method' => 'DELETE',
+													'route' => [
+																$setting->grab('admin_route').'.member.destroy',
+																$member->id
+																],
+													'id' => "delete-$member->id"
+													])
 									!!}
+									{!! CollectiveForm::close() !!}
+								@else
+									<button type="button" class="btn btn-default"  disabled="disabled" title="{{ trans('panichd::admin.member-with-tickets-delete') }}"><strike>{{ trans('panichd::admin.btn-delete') }}</strike></button>
+								@endif
 
-								{!! CollectiveForm::open([
-												'method' => 'DELETE',
-												'route' => [
-															$setting->grab('admin_route').'.member.destroy',
-															$member->id
-															],
-												'id' => "delete-$member->id"
-												])
-								!!}
-								{!! CollectiveForm::close() !!}
 							@endif
                         </td>
                     </tr>