Skip to content

Commit aeaddc5

Browse files
Added dedicated Request object to PasswordResetLinkController (#628)
* Added dedicated Request object to PasswordResetLinkController to allow extensibility * formatting --------- Co-authored-by: Taylor Otwell <[email protected]>
1 parent 8c539e1 commit aeaddc5

File tree

2 files changed

+33
-11
lines changed

2 files changed

+33
-11
lines changed

src/Http/Controllers/PasswordResetLinkController.php

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -12,14 +12,12 @@
1212
use Laravel\Fortify\Contracts\RequestPasswordResetLinkViewResponse;
1313
use Laravel\Fortify\Contracts\SuccessfulPasswordResetLinkRequestResponse;
1414
use Laravel\Fortify\Fortify;
15+
use Laravel\Fortify\Http\Requests\SendPasswordResetLinkRequest;
1516

1617
class PasswordResetLinkController extends Controller
1718
{
1819
/**
1920
* Show the reset password link request view.
20-
*
21-
* @param \Illuminate\Http\Request $request
22-
* @return \Laravel\Fortify\Contracts\RequestPasswordResetLinkViewResponse
2321
*/
2422
public function create(Request $request): RequestPasswordResetLinkViewResponse
2523
{
@@ -28,14 +26,9 @@ public function create(Request $request): RequestPasswordResetLinkViewResponse
2826

2927
/**
3028
* Send a reset link to the given user.
31-
*
32-
* @param \Illuminate\Http\Request $request
33-
* @return \Illuminate\Contracts\Support\Responsable
3429
*/
35-
public function store(Request $request): Responsable
30+
public function store(SendPasswordResetLinkRequest $request): Responsable
3631
{
37-
$request->validate([Fortify::email() => 'required|email']);
38-
3932
if (config('fortify.lowercase_usernames') && $request->has(Fortify::email())) {
4033
$request->merge([
4134
Fortify::email() => Str::lower($request->{Fortify::email()}),
@@ -56,8 +49,6 @@ public function store(Request $request): Responsable
5649

5750
/**
5851
* Get the broker to be used during password reset.
59-
*
60-
* @return \Illuminate\Contracts\Auth\PasswordBroker
6152
*/
6253
protected function broker(): PasswordBroker
6354
{
Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
<?php
2+
3+
namespace Laravel\Fortify\Http\Requests;
4+
5+
use Illuminate\Foundation\Http\FormRequest;
6+
use Laravel\Fortify\Fortify;
7+
8+
class SendPasswordResetLinkRequest extends FormRequest
9+
{
10+
/**
11+
* Determine if the user is authorized to make this request.
12+
*
13+
* @return bool
14+
*/
15+
public function authorize()
16+
{
17+
return true;
18+
}
19+
20+
/**
21+
* Get the validation rules that apply to the request.
22+
*
23+
* @return array
24+
*/
25+
public function rules()
26+
{
27+
return [
28+
Fortify::email() => 'required|email',
29+
];
30+
}
31+
}

0 commit comments

Comments
 (0)