Skip to content

Commit 0bdbf63

Browse files
committed
Update checking if old PIN/Password was valid during password reset process
1 parent 3c69259 commit 0bdbf63

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

code/web/Drivers/Polaris.php

+2-1
Original file line numberDiff line numberDiff line change
@@ -1668,7 +1668,7 @@ function updatePatronInfo($patron, $canUpdateContactInfo, $fromMasquerade): arra
16681668
}
16691669

16701670
function updatePin(User $patron, ?string $oldPin, string $newPin) {
1671-
if ($patron->cat_password != $oldPin) {
1671+
if ($patron->ils_password != $oldPin && $patron->cat_password != $oldPin) {
16721672
return [
16731673
'success' => false,
16741674
'message' => "The old PIN provided is incorrect.",
@@ -1694,6 +1694,7 @@ function updatePin(User $patron, ?string $oldPin, string $newPin) {
16941694
$result['success'] = true;
16951695
$result['message'] = 'Your password was updated successfully.';
16961696
$patron->cat_password = $newPin;
1697+
$patron->ils_password = $newPin;
16971698
$patron->update();
16981699
} else {
16991700
$result['message'] = "Error updating your password. (Error {$jsonResponse->PAPIErrorCode}).";

code/web/release_notes/24.05.00.MD

+1
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,7 @@
121121

122122
### Polaris Updates
123123
- Add additional checking for if an item has been moved between bibs when determining which bibs need to be updated. (Ticket 124477) (*MDN*)
124+
- Update checking if old PIN/Password was valid during password reset process. (Ticket 131557) (*MDN*)
124125

125126
### Sierra Updates
126127
- When prompting for an item to place a hold on. Treat item statuses of - with a due date as checked out. (Tickets 117611, 122886, 129418) (*MDN*)

0 commit comments

Comments
 (0)