Skip to content

Commit ff0d11e

Browse files
committed
security: refactor ticket creation validation into Form Request
1 parent 6568aa4 commit ff0d11e

2 files changed

Lines changed: 36 additions & 4 deletions

File tree

app/Http/Controllers/Agent/helpdesk/TicketController.php

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,9 @@
77
use App\Http\Controllers\Common\NotificationController as Notify;
88
use App\Http\Controllers\Common\PhpMailController;
99
use App\Http\Controllers\Controller;
10-
use App\Http\Requests\helpdesk\CreateTicketRequest;
11-
// requests
1210
use App\Http\Requests\helpdesk\TicketRequest;
11+
use App\Http\Requests\TicketStoreRequest;
1312
use App\Model\helpdesk\Agent\Department;
14-
// models
1513
use App\Model\helpdesk\Agent\Teams;
1614
use App\Model\helpdesk\Email\Emails;
1715
use App\Model\helpdesk\Form\Fields;
@@ -99,7 +97,7 @@ public function newticket(CountryCode $code)
9997
*
10098
* @return type response
10199
*/
102-
public function post_newticket(CreateTicketRequest $request, CountryCode $code, $api = false)
100+
public function post_newticket(TicketStoreRequest $request, CountryCode $code, $api = false)
103101
{
104102
try {
105103
if ($request->input('email')) {
Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
1+
<?php
2+
3+
namespace App\Http\Requests;
4+
5+
use Illuminate\Foundation\Http\FormRequest;
6+
7+
class TicketStoreRequest extends FormRequest
8+
{
9+
/**
10+
* Determine if the user is authorized to make this request.
11+
*
12+
* @return bool
13+
*/
14+
public function authorize()
15+
{
16+
return true;
17+
}
18+
19+
/**
20+
* Get the validation rules that apply to the request.
21+
*
22+
* @return array
23+
*/
24+
public function rules()
25+
{
26+
return [
27+
'subject' => 'required|string|max:255',
28+
'body' => 'required|string',
29+
'first_name' => 'required|string|max:255',
30+
'last_name' => 'required|string|max:255',
31+
'email' => 'required|email',
32+
];
33+
}
34+
}

0 commit comments

Comments
 (0)