Skip to content

Commit 39086c2

Browse files
authored
Merge pull request #14 from Yahav/patch-1
separate the validate function from verify function
2 parents 5f6203e + 2408648 commit 39086c2

File tree

1 file changed

+18
-6
lines changed

1 file changed

+18
-6
lines changed

src/Traits/VerifiesEmail.php

+18-6
Original file line numberDiff line numberDiff line change
@@ -17,21 +17,33 @@
1717
trait VerifiesEmail
1818
{
1919

20-
2120
/**
22-
* Verifies the given user's email.
21+
* Validate the request params
2322
*
2423
* @param \Illuminate\Http\Request|Request $request
25-
* @param EmailVerification $emailVerification
26-
* @return \Illuminate\Http\RedirectResponse
24+
* @return void
2725
*/
28-
public function verify(Request $request, EmailVerification $emailVerification)
26+
protected function validateVerificationRequest(Request $request)
2927
{
3028
$this->validate($request, [
3129
'token' => 'required',
3230
'email' => 'required|email',
3331
'expiration' => 'required|date_format:U'
3432
], []);
33+
34+
}
35+
36+
/**
37+
* Verifies the given user's email.
38+
*
39+
* @param \Illuminate\Http\Request|Request $request
40+
* @param EmailVerification $emailVerification
41+
* @return \Illuminate\Http\RedirectResponse
42+
*/
43+
public function verify(Request $request, EmailVerification $emailVerification)
44+
{
45+
$this->validateVerificationRequest($request);
46+
3547
// Here we will attempt to verify the user. If it is successful we
3648
// will update the verified on an actual user model and persist it to the
3749
// database. Otherwise we will parse the error and return the response.
@@ -130,4 +142,4 @@ protected function sendVerificationFailedResponse(Request $request, $response)
130142
}
131143

132144

133-
}
145+
}

0 commit comments

Comments
 (0)