Skip to content

Commit b1a4038

Browse files
authored
Merge pull request #12 from rickwest/ensure_session_has_id
Ensure session has confirmation_user_id
2 parents fac09fc + e9a17bf commit b1a4038

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

src/Traits/SendsPasswordResetEmails.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,9 @@ public function sendResetLinkEmail(Request $request)
3131
// we will throw a validation exception for this error.
3232
// A user can not request a password reset link if they are not confirmed.
3333
if (is_null($user->confirmed_at)) {
34+
35+
session(['confirmation_user_id' => $user->getKey()]);
36+
3437
throw ValidationException::withMessages([
3538
'confirmation' => [
3639
__('confirmation::confirmation.not_confirmed_reset_password', [

tests/ConfirmationTest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -135,6 +135,8 @@ public function it_does_not_allow_reset_password_request_for_unconfirmed_users()
135135
'email' => '[email protected]',
136136
]);
137137

138+
$response->assertSessionHas('confirmation_user_id', $user->getKey());
139+
138140
$response->assertSessionHasErrors('confirmation');
139141

140142
Notification::assertNotSentTo($user, ResetPassword::class);

0 commit comments

Comments
 (0)