Bug 11240: ensure that date of restriction is saved when using patron edit screen

To test:

1/ Edit a patron
2/ Add a restriction with a date (in any format other than iso)
3/ Note the date is not saved
4/ Apply patch
5/ Add restriction again, note it is saved

The problem is output_pref was being called with the wrong parameters

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Note: The issue in on the memberentry form only (not the restriction
form on the check out page).

Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

diff --git a/members/memberentry.pl b/members/memberentry.pl
index 4b410a7..e7f07a9 100755 (executable)
--- a/members/memberentry.pl
+++ b/members/memberentry.pl
@@ -104,7 +104,11 @@ foreach my $d ( @debarments_to_remove ) {
 if ( $input->param('add_debarment') ) {
 
     my $expiration = $input->param('debarred_expiration');
-    $expiration = $expiration ? output_pref( dt_from_string($expiration), 'iso' ) : undef;
+    $expiration =
+      $expiration
+      ? output_pref(
+        { 'dt' => dt_from_string($expiration), 'dateformat' => 'iso' } )
+      : undef;
 
     AddUniqueDebarment(
         {