DRCIT/snipe-it
1
0
Fork 0
mirror of https://github.com/snipe/snipe-it.git synced 2025-02-21 03:15:45 -08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Merge pull request #11107 from uberbrady/fix_bulk_user_reset_passwords

Browse source 
Fixed #11104 - bulk-user reset-password links returning 500
This commit is contained in:
snipe 2022-05-16 10:41:37 -07:00 committed by GitHub
parent 93e509bd79 b05d85ab0a
commit 63f8b0cad1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
app/Http/Controllers/Users/BulkUsersController.php
View file

@ -46,9 +46,9 @@ class BulkUsersController extends Controller
foreach ($users as $user) { foreach ($users as $user) {
if (($user->activated == '1') && ($user->email != '')) { if (($user->activated == '1') && ($user->email != '')) {
$credentials = ['email' => $user->email]; $credentials = ['email' => $user->email];
Password::sendResetLink($credentials, function (Message $message) { Password::sendResetLink($credentials/* , function (Message $message) {
$message->subject($this->getEmailSubject()); $message->subject($this->getEmailSubject()); // TODO - I'm not sure if we still need this, but this second parameter is no longer accepted in later Laravel versions.
}); } */ ); // TODO - so hopefully this doesn't give us generic password reset messages? But it at least _works_
} }
} }
return redirect()->back()->with('success', trans('admin/users/message.password_resets_sent')); return redirect()->back()->with('success', trans('admin/users/message.password_resets_sent'));