From 2a33dca76a60031c30124e55e9f037e306b26590 Mon Sep 17 00:00:00 2001 From: Katrin Fischer Date: Sat, 18 Mar 2023 10:13:09 +0000 Subject: [PATCH] Bug 32610: Add date extended patron attributes to patron module This makes the necessary changes in the patron module of the staff interface, so the new patron attribute appers and behaves correctly when editing a patron record. To test: * You will need to test different configuration options for extended patron attributes (PA) in combination with the date option: * PA is a date and not mandatory * Patron form should have the calendar widget to let you set the date. * PA is a date and mandatory * Patron form shoudl have calendar widget and check that the date is set for allowing you to save the record. * PA is a date and unique * For this set the date in one patron record and try to set the same date in another. You should not be able to save. * PA displays in brief patron information * Make sure the date displays on the left formatted correctly * When the date PAs are saved, they should display nicely formatted on the details tab. Signed-off-by: Philip Orr --- .../intranet-tmpl/prog/en/includes/circ-menu.inc | 7 ++++++- .../prog/en/modules/members/memberentrygen.tt | 16 +++++++++++++--- .../prog/en/modules/members/moremember.tt | 6 +++++- members/memberentry.pl | 1 + 4 files changed, 25 insertions(+), 5 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc index ead8c18fbe..94ace19a73 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc @@ -110,7 +110,12 @@ [% IF ( extendedattribute.type.display_checkout ) %] [%# FIXME We should filter in the line above %] [% IF ( extendedattribute.attribute ) %] [%# FIXME Why that? why not if == 0? %]
  • - [% extendedattribute.type.description | html %]: [% extendedattribute.description | html %] + [% extendedattribute.type.description | html %]: + [% IF extendedattribute.type.is_date %] + [% extendedattribute.description | $KohaDates %] + [% ELSE %] + [% extendedattribute.description | html %] + [% END %]
  • [% END %] [% END %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt index 8374ee944e..e461ca039b 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt @@ -1562,13 +1562,23 @@ legend:hover { [% ELSE %] [% IF patron_attribute.mandatory %] - + [% IF patron_attribute.is_date %] + + [% ELSE %] + + [% END %] [% ELSE %] - + [% IF patron_attribute.is_date %] + + [% ELSE %] + + [% END %] [% END %] [% END # /IF ( patron_attribute.use_dropdown ) %] - Clear + [% IF ( !patron_attribute.is_date ) %] + Clear + [% END %] [% IF ( patron_attribute.repeatable ) %] New [% END %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt index f070c3eb97..aa063e2fdd 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt @@ -326,7 +326,11 @@ [% FOREACH item IN attribute.items %]
  • [% item.type.description | html %]: - [% item.description | html_line_break %] + [% IF item.type.is_date %] + [% item.description | $KohaDates %] + [% ELSE %] + [% item.description | html_line_break %] + [% END %]
  • [% END %] diff --git a/members/memberentry.pl b/members/memberentry.pl index 360fd2a6b3..52ef74c490 100755 --- a/members/memberentry.pl +++ b/members/memberentry.pl @@ -853,6 +853,7 @@ sub patron_attributes_form { category => $attr_type->authorised_value_category(), category_code => $attr_type->category_code(), mandatory => $attr_type->mandatory(), + is_date => $attr_type->is_date(), }; if (exists $attr_hash{$attr_type->code()}) { foreach my $attr (@{ $attr_hash{$attr_type->code()} }) { -- 2.34.1