X-Git-Url: http://koha-dev.rot13.org:8081/gitweb/?a=blobdiff_plain;ds=sidebyside;f=tools%2Fmodborrowers.pl;h=46f79d70eb2f6ac424905986a869517a1c0c2fe8;hb=bb5d95749bd597953fad17ba4cc3f5262d538ff2;hp=a2c27c6a02ad5dd3ff6a791d1fd9ce910810e490;hpb=715c52977a8d4a25cc91aabd22224c66956c6f33;p=koha-ffzg.git diff --git a/tools/modborrowers.pl b/tools/modborrowers.pl index a2c27c6a02..46f79d70eb 100755 --- a/tools/modborrowers.pl +++ b/tools/modborrowers.pl @@ -4,117 +4,159 @@ # # This file is part of Koha. # -# Koha is free software; you can redistribute it and/or modify it under the -# terms of the GNU General Public License as published by the Free Software -# Foundation; either version 2 of the License, or (at your option) any later -# version. +# Koha is free software; you can redistribute it and/or modify it +# under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. # -# Koha is distributed in the hope that it will be useful, but WITHOUT ANY -# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR -# A PARTICULAR PURPOSE. See the GNU General Public License for more details. +# Koha is distributed in the hope that it will be useful, but +# WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. # -# You should have received a copy of the GNU General Public License along -# with Koha; if not, write to the Free Software Foundation, Inc., -# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. +# You should have received a copy of the GNU General Public License +# along with Koha; if not, see . # modborrowers.pl # # Batch Edit Patrons # Modification for patron's fields: -# surname firstname branchcode categorycode sort1 sort2 dateenrolled dateexpiry debarred debarredcomment borrowernotes +# surname firstname branchcode categorycode city state zipcode country sort1 +# sort2 dateenrolled dateexpiry borrowernotes # And for patron attributes. use Modern::Perl; -use CGI; -use C4::Auth; -use C4::Branch; -use C4::Koha; +use CGI qw ( -utf8 ); +use C4::Auth qw( get_template_and_user ); +use C4::Koha qw( GetAuthorisedValues ); use C4::Members; -use C4::Members::Attributes; -use C4::Members::AttributeTypes qw/GetAttributeTypes_hashref/; -use C4::Output; -use List::MoreUtils qw /any uniq/; +use C4::Output qw( output_html_with_http_headers ); +use Koha::DateUtils qw( dt_from_string ); +use Koha::List::Patron qw( GetPatronLists ); +use Koha::Libraries; +use Koha::Patron::Categories; +use Koha::Patron::Debarments qw( AddDebarment DelDebarment ); +use Koha::Patrons; +use List::MoreUtils qw(uniq); -my $input = new CGI; +my $input = CGI->new; my $op = $input->param('op') || 'show_form'; my ( $template, $loggedinuser, $cookie ) = get_template_and_user( - { template_name => "tools/modborrowers.tmpl", + { template_name => "tools/modborrowers.tt", query => $input, type => "intranet", - authnotrequired => 0, flagsrequired => { tools => "edit_patrons" }, } ); -my %cookies = parse CGI::Cookie($cookie); -my $sessionID = $cookies{'CGISESSID'}->value; -my $dbh = C4::Context->dbh; +my $logged_in_user = Koha::Patrons->find( $loggedinuser ); +$template->param( CanUpdatePasswordExpiration => 1 ) if $logged_in_user->is_superlibrarian; +my $dbh = C4::Context->dbh; # Show borrower informations if ( $op eq 'show' ) { - my $filefh = $input->upload('uploadfile'); - my $filecontent = $input->param('filecontent'); my @borrowers; - my @cardnumbers; + my @patronidnumbers; my @notfoundcardnumbers; + my $useborrowernumbers = 0; # Get cardnumbers from a file or the input area - my @contentlist; - if ($filefh) { - while ( my $content = <$filefh> ) { - $content =~ s/[\r\n]*$//g; - push @cardnumbers, $content if $content; + if( my $cardnumberlist = $input->param('cardnumberlist') ){ + # User submitted a list of card numbers + push @patronidnumbers, split( /\s\n/, $cardnumberlist ); + } elsif ( my $cardnumberuploadfile = $input->param('cardnumberuploadfile') ){ + # User uploaded a file of card numbers + binmode $cardnumberuploadfile, ':encoding(UTF-8)'; + while ( my $content = <$cardnumberuploadfile> ) { + next unless $content; + $content =~ s/[\r\n]*$//; + push @patronidnumbers, $content if $content; } - } else { - if ( my $list = $input->param('cardnumberlist') ) { - push @cardnumbers, split( /\s\n/, $list ); + } elsif ( my $borrowernumberlist = $input->param('borrowernumberlist') ){ + # User submitted a list of borrowernumbers + $useborrowernumbers = 1; + push @patronidnumbers, split( /\s\n/, $borrowernumberlist ); + } elsif ( my $borrowernumberuploadfile = $input->param('borrowernumberuploadfile') ){ + # User uploaded a file of borrowernumbers + $useborrowernumbers = 1; + binmode $borrowernumberuploadfile, ':encoding(UTF-8)'; + while ( my $content = <$borrowernumberuploadfile> ) { + next unless $content; + $content =~ s/[\r\n]*$//; + push @patronidnumbers, $content if $content; } + } elsif ( my $patron_list_id = $input->param('patron_list_id') ){ + # User selected a patron list + my ($list) = GetPatronLists( { patron_list_id => $patron_list_id } ); + @patronidnumbers = + $list->patron_list_patrons()->search_related('borrowernumber') + ->get_column('cardnumber')->all(); } my $max_nb_attr = 0; - for my $cardnumber ( @cardnumbers ) { - my $borrower = GetBorrowerInfos( cardnumber => $cardnumber ); - if ( $borrower ) { - $max_nb_attr = scalar( @{ $borrower->{patron_attributes} } ) - if scalar( @{ $borrower->{patron_attributes} } ) > $max_nb_attr; - push @borrowers, $borrower; + + # Make sure there is only one of each patron id number + @patronidnumbers = uniq( @patronidnumbers ); + + for my $patronidnumber ( @patronidnumbers ) { + my $patron; + if( $useborrowernumbers == 1 ){ + $patron = Koha::Patrons->find( { borrowernumber => $patronidnumber } ); } else { - push @notfoundcardnumbers, $cardnumber; + $patron = Koha::Patrons->find( { cardnumber => $patronidnumber } ); + } + if ( $patron ) { + if ( $logged_in_user->can_see_patron_infos( $patron ) ) { + my $borrower = $patron->unblessed; + my $attributes = $patron->extended_attributes; + $borrower->{patron_attributes} = $attributes->as_list; + $borrower->{patron_attributes_count} = $attributes->count; + $max_nb_attr = $borrower->{patron_attributes_count} if $borrower->{patron_attributes_count} > $max_nb_attr; + push @borrowers, $borrower; + } else { + push @notfoundcardnumbers, $patronidnumber; + } + } else { + push @notfoundcardnumbers, $patronidnumber; } } # Just for a correct display for my $borrower ( @borrowers ) { - my $length = scalar( @{ $borrower->{patron_attributes} } ); + my $length = $borrower->{patron_attributes_count}; push @{ $borrower->{patron_attributes} }, {} for ( $length .. $max_nb_attr - 1); } # Construct the patron attributes list my @patron_attributes_values; my @patron_attributes_codes; - my $patron_attribute_types = C4::Members::AttributeTypes::GetAttributeTypes_hashref('all'); - my $patron_categories = C4::Members::GetBorrowercategoryList; - for ( values %$patron_attribute_types ) { - my $attr_type = C4::Members::AttributeTypes->fetch( $_->{code} ); + my $library_id = C4::Context->userenv ? C4::Context->userenv->{'branch'} : undef; + my $patron_attribute_types = Koha::Patron::Attribute::Types->search_with_library_limits({}, {}, $library_id); + my @patron_categories = Koha::Patron::Categories->search_with_library_limits({}, {order_by => ['description']})->as_list; + while ( my $attr_type = $patron_attribute_types->next ) { + # TODO Repeatable attributes are not correctly managed and can cause data lost. + # This should be implemented. + next if $attr_type->repeatable; + next if $attr_type->unique_id; # Don't display patron attributes that must be unqiue my $options = $attr_type->authorised_value_category ? GetAuthorisedValues( $attr_type->authorised_value_category ) : undef; push @patron_attributes_values, { - attribute_code => $_->{code}, + attribute_code => $attr_type->code, options => $options, }; - my $category_code = $_->{category_code}; + my $category_code = $attr_type->category_code; my ( $category_lib ) = map { - ( defined $category_code and $_->{categorycode} eq $category_code ) ? $_->{description} : () - } @$patron_categories; + ( defined $category_code and $attr_type->category_code eq $category_code ) ? $attr_type->description : () + } @patron_categories; push @patron_attributes_codes, { - attribute_code => $_->{code}, - attribute_lib => $_->{description}, + attribute_code => $attr_type->code, + attribute_lib => $attr_type->description, category_lib => $category_lib, type => $attr_type->authorised_value_category ? 'select' : 'text', }; @@ -129,15 +171,15 @@ if ( $op eq 'show' ) { @notfoundcardnumbers = map { { cardnumber => $_ } } @notfoundcardnumbers; $template->param( notfoundcardnumbers => \@notfoundcardnumbers ) if @notfoundcardnumbers; + $template->param( useborrowernumbers => $useborrowernumbers ); # Construct drop-down list values - my $branches = GetBranchesLoop; + my $branches = Koha::Libraries->search({}, { order_by => ['branchname'] })->unblessed; my @branches_option; - push @branches_option, { value => $_->{value}, lib => $_->{branchname} } for @$branches; + push @branches_option, { value => $_->{branchcode}, lib => $_->{branchname} } for @$branches; unshift @branches_option, { value => "", lib => "" }; - my $categories = GetBorrowercategoryList; my @categories_option; - push @categories_option, { value => $_->{categorycode}, lib => $_->{description} } for @$categories; + push @categories_option, { value => $_->categorycode, lib => $_->description } for @patron_categories; unshift @categories_option, { value => "", lib => "" }; my $bsort1 = GetAuthorisedValues("Bsort1"); my @sort1_option; @@ -162,7 +204,7 @@ if ( $op eq 'show' ) { { name => "firstname", type => "text", - mandatory => ( grep /surname/, @mandatoryFields ) ? 1 : 0, + mandatory => ( grep /firstname/, @mandatoryFields ) ? 1 : 0, } , { @@ -180,6 +222,66 @@ if ( $op eq 'show' ) { } , { + name => "streetnumber", + type => "text", + mandatory => ( grep /streetnumber/, @mandatoryFields ) ? 1 : 0, + } + , + { + name => "address", + type => "text", + mandatory => ( grep /address/, @mandatoryFields ) ? 1 : 0, + } + , + { + name => "address2", + type => "text", + mandatory => ( grep /address2/, @mandatoryFields ) ? 1 : 0, + } + , + { + name => "city", + type => "text", + mandatory => ( grep /city/, @mandatoryFields ) ? 1 : 0, + } + , + { + name => "state", + type => "text", + mandatory => ( grep /state/, @mandatoryFields ) ? 1 : 0, + } + , + { + name => "zipcode", + type => "text", + mandatory => ( grep /zipcode/, @mandatoryFields ) ? 1 : 0, + } + , + { + name => "country", + type => "text", + mandatory => ( grep /country/, @mandatoryFields ) ? 1 : 0, + } + , + { + name => "email", + type => "text", + mandatory => ( grep /email/, @mandatoryFields ) ? 1 : 0, + } + , + { + name => "phone", + type => "text", + mandatory => ( grep /phone/, @mandatoryFields ) ? 1 : 0, + } + , + { + name => "mobile", + type => "text", + mandatory => ( grep /mobile/, @mandatoryFields ) ? 1 : 0, + } + , + { name => "sort1", type => @sort1_option ? "select" : "text", option => \@sort1_option, @@ -206,6 +308,18 @@ if ( $op eq 'show' ) { } , { + name => "borrowernotes", + type => "text", + mandatory => ( grep /borrowernotes/, @mandatoryFields ) ? 1 : 0, + } + , + { + name => "opacnote", + type => "text", + mandatory => ( grep /opacnote/, @mandatoryFields ) ? 1 : 0, + } + , + { name => "debarred", type => "date", mandatory => ( grep /debarred/, @mandatoryFields ) ? 1 : 0, @@ -215,70 +329,98 @@ if ( $op eq 'show' ) { name => "debarredcomment", type => "text", mandatory => ( grep /debarredcomment/, @mandatoryFields ) ? 1 : 0, - } - , - { - name => "borrowernotes", - type => "text", - mandatory => ( grep /borrowernotes/, @mandatoryFields ) ? 1 : 0, - } + }, ); + push @fields, { name => "password_expiration_date", type => "date" } if $logged_in_user->is_superlibrarian; + $template->param('patron_attributes_codes', \@patron_attributes_codes); $template->param('patron_attributes_values', \@patron_attributes_values); $template->param( fields => \@fields ); - $template->param( DHTMLcalendar_dateformat => C4::Dates->DHTMLcalendar() ); } # Process modifications if ( $op eq 'do' ) { - my @disabled = $input->param('disable_input'); + my @disabled = $input->multi_param('disable_input'); my $infos; - for my $field ( qw/surname firstname branchcode categorycode sort1 sort2 dateenrolled dateexpiry debarred debarredcomment borrowernotes/ ) { + for my $field ( qw/surname firstname branchcode categorycode streetnumber address address2 city state zipcode country email phone mobile sort1 sort2 dateenrolled dateexpiry password_expiration_date borrowernotes opacnote debarred debarredcomment/ ) { my $value = $input->param($field); $infos->{$field} = $value if $value; - $infos->{$field} = "" if grep { /^$field$/ } @disabled; + $infos->{$field} = "" if grep { $_ eq $field } @disabled; + } + + for my $field ( qw( dateenrolled dateexpiry debarred password_expiration_date ) ) { + $infos->{$field} = dt_from_string($infos->{$field}) if $infos->{$field}; } - my @attributes = $input->param('patron_attributes'); - my @attr_values = $input->param('patron_attributes_value'); + delete $infos->{password_expiration_date} unless $logged_in_user->is_superlibrarian; + + my @attributes = $input->multi_param('patron_attributes'); + my @attr_values = $input->multi_param('patron_attributes_value'); my @errors; - my @borrowernumbers = $input->param('borrowernumber'); + my @borrowernumbers = $input->multi_param('borrowernumber'); # For each borrower selected for my $borrowernumber ( @borrowernumbers ) { + # If at least one field are filled, we want to modify the borrower if ( defined $infos ) { + # If a debarred date or debarred comment has been submitted make a new debarment + if ( $infos->{debarred} || $infos->{debarredcomment} ) { + AddDebarment( + { + borrowernumber => $borrowernumber, + type => 'MANUAL', + comment => $infos->{debarredcomment}, + expiration => $infos->{debarred}, + }); + } + + # If debarment date or debarment comment are disabled then remove all debarrments + my $patron = Koha::Patrons->find( $borrowernumber ); + if ( grep { /debarred/ } @disabled ) { + eval { + my $debarrments = $patron->restrictions; + while( my $debarment = $debarrments->next ) { + DelDebarment( $debarment->borrower_debarment_id ); + } + }; + } + $infos->{borrowernumber} = $borrowernumber; - my $success = ModMember(%$infos); - push @errors, { error => "can_not_update", borrowernumber => $infos->{borrowernumber} } if not $success; + eval { $patron->set($infos)->store; }; + if ( $@ ) { # FIXME We could provide better error handling here + $infos->{cardnumber} = $patron ? $patron->cardnumber || '' : ''; + push @errors, { error => "can_not_update", borrowernumber => $infos->{borrowernumber}, cardnumber => $infos->{cardnumber} }; + } } - # - my $borrower_categorycode = GetBorrowerCategorycode $borrowernumber; + my $patron = Koha::Patrons->find( $borrowernumber ); my $i=0; for ( @attributes ) { + next unless $_; my $attribute; $attribute->{code} = $_; $attribute->{attribute} = $attr_values[$i]; - my $attr_type = C4::Members::AttributeTypes->fetch( $_ ); + my $attr_type = Koha::Patron::Attribute::Types->find($_); # If this borrower is not in the category of this attribute, we don't want to modify this attribute - ++$i and next if $attr_type->{category_code} and $attr_type->{category_code} ne $borrower_categorycode; + ++$i and next if $attr_type->category_code and $attr_type->category_code ne $patron->categorycode; my $valuename = "attr" . $i . "_value"; - if ( grep { /^$valuename$/ } @disabled ) { + if ( grep { $_ eq $valuename } @disabled ) { # The attribute is disabled, we remove it for this borrower ! eval { - C4::Members::Attributes::DeleteBorrowerAttribute( $borrowernumber, $attribute ); + $patron->get_extended_attribute($attribute->{code})->delete; }; push @errors, { error => $@ } if $@; } else { - # Attribute's value is empty, we don't want to modify it - ++$i and next if not $attribute->{attribute}; - eval { - C4::Members::Attributes::UpdateBorrowerAttribute( $borrowernumber, $attribute ); + # Note: + # We should not need to filter by branch, but stay on the safe side + # Repeatable are not supported so we can do that - TODO + $patron->extended_attributes->search({'me.code' => $attribute->{code}})->filter_by_branch_limitations->delete; + $patron->add_extended_attribute($attribute); }; push @errors, { error => $@ } if $@; } @@ -291,10 +433,14 @@ if ( $op eq 'do' ) { my @borrowers; my $max_nb_attr = 0; for my $borrowernumber ( @borrowernumbers ) { - my $borrower = GetBorrowerInfos( borrowernumber => $borrowernumber ); - if ( $borrower ) { - $max_nb_attr = scalar( @{ $borrower->{patron_attributes} } ) - if scalar( @{ $borrower->{patron_attributes} } ) > $max_nb_attr; + my $patron = Koha::Patrons->find( $borrowernumber ); + if ( $patron ) { + my $category_description = $patron->category->description; + my $borrower = $patron->unblessed; + $borrower->{category_description} = $category_description; + my $attributes = $patron->extended_attributes; + $borrower->{patron_attributes} = $attributes->as_list; + $max_nb_attr = $attributes->count if $attributes->count > $max_nb_attr; push @borrowers, $borrower; } } @@ -313,8 +459,10 @@ if ( $op eq 'do' ) { $template->param( borrowers => \@borrowers ); $template->param( attributes_header => \@attributes_header ); - $template->param( borrowers => \@borrowers ); $template->param( errors => \@errors ); +} else { + + $template->param( patron_lists => [ GetPatronLists() ] ); } $template->param( @@ -322,23 +470,3 @@ $template->param( ); output_html_with_http_headers $input, $cookie, $template->output; exit; - -sub GetBorrowerInfos { - my ( %info ) = @_; - my $borrower = GetMember( %info ); - if ( $borrower ) { - $borrower->{branchname} = GetBranchName( $borrower->{branchcode} ); - for ( qw(dateenrolled dateexpiry debarred) ) { - my $userdate = $borrower->{$_}; - unless ($userdate && $userdate ne "0000-00-00" and $userdate ne "9999-12-31") { - $borrower->{$_} = ''; - next; - } - $borrower->{$_} = $userdate || ''; - } - $borrower->{category_description} = GetBorrowercategory( $borrower->{categorycode} )->{description}; - my $attr_loop = C4::Members::Attributes::GetBorrowerAttributes( $borrower->{borrowernumber} ); - $borrower->{patron_attributes} = $attr_loop; - } - return $borrower; -}