Update checking if old PIN/Password was valid during password reset p…

…rocess
PTFS-Europe · May 6, 2024 · 0bdbf63 · 0bdbf63
1 parent 3c69259
commit 0bdbf63
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/code/web/Drivers/Polaris.php b/code/web/Drivers/Polaris.php
@@ -1668,7 +1668,7 @@ function updatePatronInfo($patron, $canUpdateContactInfo, $fromMasquerade): arra
 	}
 
 	function updatePin(User $patron, ?string $oldPin, string $newPin) {
-		if ($patron->cat_password != $oldPin) {
+		if ($patron->ils_password != $oldPin && $patron->cat_password != $oldPin) {
 			return [
 				'success' => false,
 				'message' => "The old PIN provided is incorrect.",
@@ -1694,6 +1694,7 @@ function updatePin(User $patron, ?string $oldPin, string $newPin) {
 				$result['success'] = true;
 				$result['message'] = 'Your password was updated successfully.';
 				$patron->cat_password = $newPin;
+				$patron->ils_password = $newPin;
 				$patron->update();
 			} else {
 				$result['message'] = "Error updating your password. (Error {$jsonResponse->PAPIErrorCode}).";

diff --git a/code/web/release_notes/24.05.00.MD b/code/web/release_notes/24.05.00.MD
@@ -121,6 +121,7 @@
 
 ### Polaris Updates
 - Add additional checking for if an item has been moved between bibs when determining which bibs need to be updated. (Ticket 124477) (*MDN*)
+- Update checking if old PIN/Password was valid during password reset process. (Ticket 131557) (*MDN*) 
 
 ### Sierra Updates
 - 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*)