From dae5607fd184c2e4b19b197cca313aca92975e47 Mon Sep 17 00:00:00 2001
From: emlam <emily.lamancusa@montgomerycountymd.gov>
Date: Thu, 27 Jul 2023 16:36:34 +0000
Subject: [PATCH] Bug 34435: Remove side effect from get_password_expiry_date

If get_password_expiry_date is passed a DateTime object
as a parameter, it modifies and returns the original
object. This can create possible side effects.

This patch modifies get_password_expiry_date to clone the
DateTime object that it receives as a parameter and
return the modified clone, so that object references can
be passed in safely.

To test:
prove t/db_dependent/Koha/Patron/Category.t

Signed-off-by: Sam Lau <samalau@gmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
---
 Koha/Patron/Category.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Koha/Patron/Category.pm b/Koha/Patron/Category.pm
index 335367b825..e24c8d6e79 100644
--- a/Koha/Patron/Category.pm
+++ b/Koha/Patron/Category.pm
@@ -126,7 +126,7 @@ sub get_password_expiry_date {
     my ($self, $date ) = @_;
     if ( $self->password_expiry_days ) {
         $date ||= dt_from_string;
-        $date = dt_from_string( $date ) unless ref $date;
+        $date = ref $date ? $date->clone() : dt_from_string( $date );
         return $date->add( days => $self->password_expiry_days )->ymd;
     } else {
         return;
-- 
2.39.5