From 0137f1055c5cf4dcdf8d32587f9347dbac5738d1 Mon Sep 17 00:00:00 2001 From: Jesse Weaver Date: Wed, 2 Sep 2015 17:54:04 -0600 Subject: [PATCH] Bug 13757: Make patron attributes editable in the OPAC (1/2) This patch adds an opac_editable property of borrower attribute types that can be set in the interface. Test plan: 1. Repeat the following with a new and existing borrower attribute type: 2. Verify that "Editable in OPAC" can only be checked if "Display in OPAC" is checked. 3. Verify that this new property is correctly saved. Signed-off-by: Aleisha Signed-off-by: Tomas Cohen Arazi --- C4/Members/AttributeTypes.pm | 45 +++++++++++++++------- admin/patron-attr-types.pl | 8 ++++ ...d_opac_editable_to_borrower_attribute_types.sql | 1 + installer/data/mysql/kohastructure.sql | 1 + koha-tmpl/intranet-tmpl/prog/css/staff-global.css | 4 ++ .../prog/en/modules/admin/patron-attr-types.tt | 16 ++++++++ 6 files changed, 62 insertions(+), 13 deletions(-) create mode 100644 installer/data/mysql/atomicupdate/bug_13757-add_opac_editable_to_borrower_attribute_types.sql diff --git a/C4/Members/AttributeTypes.pm b/C4/Members/AttributeTypes.pm index d4927db..ad2ef22 100644 --- a/C4/Members/AttributeTypes.pm +++ b/C4/Members/AttributeTypes.pm @@ -37,6 +37,7 @@ C4::Members::AttributeTypes - mananage extended patron attribute types $attr_type->repeatable($repeatable); $attr_type->unique_id($unique_id); $attr_type->opac_display($opac_display); + $attr_type->opac_editable($opac_editable); $attr_type->staff_searchable($staff_searchable); $attr_type->authorised_value_category($authorised_value_category); $attr_type->store(); @@ -122,6 +123,7 @@ sub new { $self->{'repeatable'} = 0; $self->{'unique_id'} = 0; $self->{'opac_display'} = 0; + $self->{'opac_editable'} = 0; $self->{'staff_searchable'} = 0; $self->{'display_checkout'} = 0; $self->{'authorised_value_category'} = ''; @@ -163,6 +165,7 @@ sub fetch { $self->{'repeatable'} = $row->{'repeatable'}; $self->{'unique_id'} = $row->{'unique_id'}; $self->{'opac_display'} = $row->{'opac_display'}; + $self->{'opac_editable'} = $row->{'opac_editable'}; $self->{'staff_searchable'} = $row->{'staff_searchable'}; $self->{'display_checkout'} = $row->{'display_checkout'}; $self->{'authorised_value_category'} = $row->{'authorised_value_category'}; @@ -203,6 +206,7 @@ sub store { repeatable = ?, unique_id = ?, opac_display = ?, + opac_editable = ?, staff_searchable = ?, authorised_value_category = ?, display_checkout = ?, @@ -211,23 +215,23 @@ sub store { WHERE code = ?"); } else { $sth = $dbh->prepare_cached("INSERT INTO borrower_attribute_types - (description, repeatable, unique_id, opac_display, + (description, repeatable, unique_id, opac_display, opac_editable, staff_searchable, authorised_value_category, display_checkout, category_code, class, code) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)"); } - $sth->execute( - $self->{description}, - $self->{repeatable}, - $self->{unique_id}, - $self->{opac_display}, - $self->{staff_searchable}, - $self->{authorised_value_category}, - $self->{display_checkout}, - $self->{category_code} || undef, - $self->{class}, - $self->{code}, - ); + $sth->bind_param(1, $self->{'description'}); + $sth->bind_param(2, $self->{'repeatable'}); + $sth->bind_param(3, $self->{'unique_id'}); + $sth->bind_param(4, $self->{'opac_display'}); + $sth->bind_param(5, $self->{'opac_editable'}); + $sth->bind_param(7, $self->{'staff_searchable'}); + $sth->bind_param(8, $self->{'authorised_value_category'}); + $sth->bind_param(9, $self->{'display_checkout'}); + $sth->bind_param(10, $self->{'category_code'} || undef); + $sth->bind_param(11, $self->{'class'}); + $sth->bind_param(12, $self->{'code'}); + $sth->execute; if ( defined $$self{branches} ) { $sth = $dbh->prepare("DELETE FROM borrower_attribute_types_branches WHERE bat_code = ?"); @@ -335,6 +339,21 @@ sub opac_display { @_ ? $self->{'opac_display'} = ((shift) ? 1 : 0) : $self->{'opac_display'}; } +=head2 opac_editable + + my $opac_editable = $attr_type->opac_editable(); + $attr_type->opac_editable($opac_editable); + +Accessor. The C<$opac_editable> argument +is interpreted as a Perl boolean. + +=cut + +sub opac_editable { + my $self = shift; + @_ ? $self->{'opac_editable'} = ((shift) ? 1 : 0) : $self->{'opac_editable'}; +} + =head2 staff_searchable my $staff_searchable = $attr_type->staff_searchable(); diff --git a/admin/patron-attr-types.pl b/admin/patron-attr-types.pl index 74e075c..0cca3b5 100755 --- a/admin/patron-attr-types.pl +++ b/admin/patron-attr-types.pl @@ -118,6 +118,9 @@ sub error_add_attribute_type_form { if ($input->param('opac_display')) { $template->param(opac_display_checked => 1); } + if ($input->param('opac_editable')) { + $template->param(opac_editable_checked => 1); + } if ($input->param('staff_searchable')) { $template->param(staff_searchable_checked => 1); } @@ -162,6 +165,8 @@ sub add_update_attribute_type { my $opac_display = $input->param('opac_display'); $attr_type->opac_display($opac_display); + my $opac_editable = $input->param('opac_editable'); + $attr_type->opac_editable($opac_editable); my $staff_searchable = $input->param('staff_searchable'); $attr_type->staff_searchable($staff_searchable); my $authorised_value_category = $input->param('authorised_value_category'); @@ -242,6 +247,9 @@ sub edit_attribute_type_form { if ($attr_type->opac_display()) { $template->param(opac_display_checked => 1); } + if ($attr_type->opac_editable()) { + $template->param(opac_editable_checked => 1); + } if ($attr_type->staff_searchable()) { $template->param(staff_searchable_checked => 1); } diff --git a/installer/data/mysql/atomicupdate/bug_13757-add_opac_editable_to_borrower_attribute_types.sql b/installer/data/mysql/atomicupdate/bug_13757-add_opac_editable_to_borrower_attribute_types.sql new file mode 100644 index 0000000..76a989f --- /dev/null +++ b/installer/data/mysql/atomicupdate/bug_13757-add_opac_editable_to_borrower_attribute_types.sql @@ -0,0 +1 @@ +ALTER TABLE borrower_attribute_types ADD COLUMN `opac_editable` tinyint(1) NOT NULL default 0 AFTER `opac_display`; diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql index 6893d6b..fd04027 100644 --- a/installer/data/mysql/kohastructure.sql +++ b/installer/data/mysql/kohastructure.sql @@ -215,6 +215,7 @@ CREATE TABLE `borrower_attribute_types` ( -- definitions for custom patron field `repeatable` tinyint(1) NOT NULL default 0, -- defines whether one patron/borrower can have multiple values for this custom field (1 for yes, 0 for no) `unique_id` tinyint(1) NOT NULL default 0, -- defines if this value needs to be unique (1 for yes, 0 for no) `opac_display` tinyint(1) NOT NULL default 0, -- defines if this field is visible to patrons on their account in the OPAC (1 for yes, 0 for no) + `opac_editable` tinyint(1) NOT NULL default 0, -- defines if this field is editable by patrons on their account in the OPAC (1 for yes, 0 for no) `staff_searchable` tinyint(1) NOT NULL default 0, -- defines if this field is searchable via the patron search in the staff client (1 for yes, 0 for no) `authorised_value_category` varchar(32) default NULL, -- foreign key from authorised_values that links this custom field to an authorized value category `display_checkout` tinyint(1) NOT NULL default 0,-- defines if this field displays in checkout screens diff --git a/koha-tmpl/intranet-tmpl/prog/css/staff-global.css b/koha-tmpl/intranet-tmpl/prog/css/staff-global.css index c8921f9..f9a9895 100644 --- a/koha-tmpl/intranet-tmpl/prog/css/staff-global.css +++ b/koha-tmpl/intranet-tmpl/prog/css/staff-global.css @@ -701,6 +701,10 @@ fieldset.action, div.action { width: auto; } +fieldset.rows li[aria-disabled="true"] { + color: #999; +} + div.rows+div.rows { margin-top : .6em; } diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/patron-attr-types.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/patron-attr-types.tt index 6fb7c97..4f3af30 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/patron-attr-types.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/patron-attr-types.tt @@ -20,6 +20,14 @@ $(document).ready(function() { if ( $("#branches option:selected").length < 1 ) { $("#branches option:first").attr("selected", "selected"); } + + $("#opac_display").change( function() { + if ( this.checked ) { + $("#opac_editable").removeAttr('disabled').parent().removeAttr('aria-disabled'); + } else { + $("#opac_editable").attr('disabled', true).parent().attr('aria-disabled', 'true'); + } + } ).change(); } ); //]]> @@ -123,6 +131,14 @@ $(document).ready(function() { [% END %] Check to display this attribute on a patron's details page in the OPAC. +
  • + [% IF ( opac_editable_checked ) %] + + [% ELSE %] + + [% END %] + Check to allow patrons to edit this attribute from their details page in the OPAC. (Requires above.) +
  • [% IF ( staff_searchable_checked ) %] -- 2.7.4