Attachment #120940 for bug #27920

View | Details | Raw Unified | Return to bug 27920
Collapse All | Expand All

Lines 71-76 sub import_patrons { Link Here

(-)a/Koha/Patrons/Import.pm (+3 lines)
71	my $matchpoint = $params->{matchpoint};	71	my $matchpoint = $params->{matchpoint};
72	my $defaults = $params->{defaults};	72	my $defaults = $params->{defaults};
73	my $preserve_fields = $params->{preserve_fields};	73	my $preserve_fields = $params->{preserve_fields};
		74	my $update_dateexpiry = $params->{update_dateexpiry};
74	my $ext_preserve = $params->{preserve_extended_attributes};	75	my $ext_preserve = $params->{preserve_extended_attributes};
75	my $overwrite_cardnumber = $params->{overwrite_cardnumber};	76	my $overwrite_cardnumber = $params->{overwrite_cardnumber};
76	my $overwrite_passwords = $params->{overwrite_passwords};	77	my $overwrite_passwords = $params->{overwrite_passwords};
Lines 264-269 sub import_patrons { Link Here
264	}	265	}
265	}	266	}
266		267
		268	$borrower{dateexpiry} = Koha::Patron::Categories->find( $borrower{categorycode} )->get_expiry_date( $borrower{dateenrolled} ) if $update_dateexpiry;
		269
267	for my $col ( keys %borrower ) {	270	for my $col ( keys %borrower ) {
268		271
269	# use values from extant patron unless our csv file includes this column or we provided a default.	272	# use values from extant patron unless our csv file includes this column or we provided a default.

Lines 295-301 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/import_borrowers.tt (-1 / +7 lines)
295	<li>	295	<li>
296	<input class="overwrite_passwords" type="checkbox" id="overwrite_passwords" name="overwrite_passwords" disabled/>	296	<input class="overwrite_passwords" type="checkbox" id="overwrite_passwords" name="overwrite_passwords" disabled/>
297	<label class="overwrite_passwords" for="overwrite_passwords">Replace patron passwords with those in the file (blank passwords will be ignored)</label>	297	<label class="overwrite_passwords" for="overwrite_passwords">Replace patron passwords with those in the file (blank passwords will be ignored)</label>
298	</li	298	</li>
		299	<li>
		300	<input class="update_dateexpiry" type="checkbox" id="update_dateexpiry" name="update_dateexpiry" disabled/>
		301	<label class="update_dateexpiry" for="update_dateexpiry">Update patron's expiration date based on enrollment date</label>
		302	</li>
299	</ul>	303	</ul>
300	</li>	304	</li>
301	</ol>	305	</ol>
Lines 399-407 you can supply dates in ISO format (e.g., '2010-10-28'). Link Here
399		403
400	$("#overwrite_cardnumberno").click(function(){	404	$("#overwrite_cardnumberno").click(function(){
401	$("#overwrite_passwords").prop('checked',false).prop('disabled',true);	405	$("#overwrite_passwords").prop('checked',false).prop('disabled',true);
		406	$("#update_dateexpiry").prop('checked',false).prop('disabled',true);
402	});	407	});
403	$("#overwrite_cardnumberyes").click(function(){	408	$("#overwrite_cardnumberyes").click(function(){
404	$("#overwrite_passwords").prop('disabled',false);	409	$("#overwrite_passwords").prop('disabled',false);
		410	$("#update_dateexpiry").prop('disabled',false);
405	});	411	});
406	</script>	412	</script>
407	[% END %]	413	[% END %]




my $verbose      = 0;
my $help;
my @preserve_fields;
my $update_dateexpiry;

GetOptions(
    'c|confirm'                      => \$confirm,

    'd|default=s'                    => \%defaults,
    'o|overwrite'                    => \$overwrite_cardnumber,
    'op|overwrite_passwords'         => \$overwrite_passwords,
    'ue|update_expiration'           => \$update_dateexpiry,
    'p|preserve-extended-attributes' => \$ext_preserve,
    'pf|preserve-field=s'            => \@preserve_fields,
    'v|verbose+'                     => \$verbose,

        overwrite_passwords          => $overwrite_passwords,
        preserve_extended_attributes => $ext_preserve,
        preserve_fields              => \@preserve_fields,
        update_dateexpiry            => $update_dateexpiry
        dry_run                      => !$confirm,
    }
);


=head1 SYNOPSIS

import_patrons.pl --file /path/to/patrons.csv --matchpoint cardnumber --confirm [--default branchcode=MPL] [--overwrite] [--preserve_field <column>] [--preserve-extended-attributes] [--update-expiration] [--verbose]

=head1 OPTIONS



Retain extended patron attributes for existing patrons being overwritten

=item B<-p|--update-expiration>

If a matching patron is found, extend the expiration date of their account using the patron's enrollment date as the base

=item B<-v|--verbose>

Be verbose

Lines 18-24 Link Here

(-)a/t/db_dependent/Koha/Patrons/Import.t (-2 / +7 lines)
18	# along with Koha; if not, see <http://www.gnu.org/licenses>.	18	# along with Koha; if not, see <http://www.gnu.org/licenses>.
19		19
20	use Modern::Perl;	20	use Modern::Perl;
21	use Test::More tests => 172;	21	use Test::More tests => 174;
22	use Test::Warn;	22	use Test::Warn;
23	use Encode qw( encode_utf8 );	23	use Encode qw( encode_utf8 );
24	use utf8;	24	use utf8;
Lines 188-194 is($result_3a->{overwritten}, 1, 'Got the expected 1 overwritten result from imp Link Here
188	# overwrite but firstname is not	188	# overwrite but firstname is not
189	my $filename_3c = make_csv($temp_dir, $csv_headers, $csv_one_line_b);	189	my $filename_3c = make_csv($temp_dir, $csv_headers, $csv_one_line_b);
190	open(my $handle_3c, "<", $filename_3c) or die "cannot open < $filename_3: $!";	190	open(my $handle_3c, "<", $filename_3c) or die "cannot open < $filename_3: $!";
191	my $params_3c = { file => $handle_3c, matchpoint => 'cardnumber', overwrite_cardnumber => 1, preserve_fields => [ 'firstname' ] };	191	my $params_3c = { file => $handle_3c, matchpoint => 'cardnumber', overwrite_cardnumber => 1, preserve_fields => [ 'firstname' ], update_dateexpiry => 1 };
		192
		193	my $patron_3 = Koha::Patrons->find({ cardnumber => '1000' });
		194	is( $patron_3->dateexpiry, '2015-07-01', "Expiration date is correct with update_dateexpiry = false" );
192		195
193	# When ...	196	# When ...
194	my $result_3c;	197	my $result_3c;
Lines 210-215 is($result_3c->{invalid}, 0, 'Got the expected 0 invalid result from import_patr Link Here
210	is($result_3c->{overwritten}, 1, 'Got the expected 1 overwritten result from import_patrons that matched');	213	is($result_3c->{overwritten}, 1, 'Got the expected 1 overwritten result from import_patrons that matched');
211		214
212	my $patron_3c = Koha::Patrons->find({ cardnumber => '1000' });	215	my $patron_3c = Koha::Patrons->find({ cardnumber => '1000' });
		216	is( $patron_3c->dateexpiry, '2023-03-28', "Expiration date is correct with update_dateexpiry = true" );
		217
213	is( $patron_3c->surname, "Nancy2", "Surname field is preserved from original" );	218	is( $patron_3c->surname, "Nancy2", "Surname field is preserved from original" );
214	is( $patron_3c->firstname, "Jenkins", "Firstname field is overwritten" );	219	is( $patron_3c->firstname, "Jenkins", "Firstname field is overwritten" );
215		220

Lines 120-125 if ( $uploadborrowers && length($uploadborrowers) > 0 ) { Link Here

(-)a/tools/import_borrowers.pl (-1 / +2 lines)
120	my $handle = $input->upload('uploadborrowers');	120	my $handle = $input->upload('uploadborrowers');
121	my %defaults = $input->Vars;	121	my %defaults = $input->Vars;
122	my $overwrite_passwords = defined $input->param('overwrite_passwords') ? 1 : 0;	122	my $overwrite_passwords = defined $input->param('overwrite_passwords') ? 1 : 0;
		123	my $update_dateexpiry = defined $input->param('update_dateexpiry') ? 1 : 0;
123	my $return = $Import->import_patrons(	124	my $return = $Import->import_patrons(
124	{	125	{
125	file => $handle,	126	file => $handle,
Lines 129-134 if ( $uploadborrowers && length($uploadborrowers) > 0 ) { Link Here
129	overwrite_passwords => $overwrite_passwords,	130	overwrite_passwords => $overwrite_passwords,
130	preserve_extended_attributes => scalar $input->param( 'ext_preserve' ) \|\| 0,	131	preserve_extended_attributes => scalar $input->param( 'ext_preserve' ) \|\| 0,
131	preserve_fields => \@preserve_fields,	132	preserve_fields => \@preserve_fields,
		133	update_dateexpiry => $update_dateexpiry,
132	}	134	}
133	);	135	);
134		136
135	-

Return to bug 27920

Lines 36-41 my $confirm; Link Here

(-)a/misc/import_patrons.pl (-1 / +8 lines)
36	my $verbose = 0;	36	my $verbose = 0;
37	my $help;	37	my $help;
38	my @preserve_fields;	38	my @preserve_fields;
		39	my $update_dateexpiry;
39		40
40	GetOptions(	41	GetOptions(
41	'c\|confirm' => \$confirm,	42	'c\|confirm' => \$confirm,
Lines 44-49 GetOptions( Link Here
44	'd\|default=s' => \%defaults,	45	'd\|default=s' => \%defaults,
45	'o\|overwrite' => \$overwrite_cardnumber,	46	'o\|overwrite' => \$overwrite_cardnumber,
46	'op\|overwrite_passwords' => \$overwrite_passwords,	47	'op\|overwrite_passwords' => \$overwrite_passwords,
		48	'ue\|update_expiration' => \$update_dateexpiry,
47	'p\|preserve-extended-attributes' => \$ext_preserve,	49	'p\|preserve-extended-attributes' => \$ext_preserve,
48	'pf\|preserve-field=s' => \@preserve_fields,	50	'pf\|preserve-field=s' => \@preserve_fields,
49	'v\|verbose+' => \$verbose,	51	'v\|verbose+' => \$verbose,
Lines 68-73 my $return = $Import->import_patrons( Link Here
68	overwrite_passwords => $overwrite_passwords,	70	overwrite_passwords => $overwrite_passwords,
69	preserve_extended_attributes => $ext_preserve,	71	preserve_extended_attributes => $ext_preserve,
70	preserve_fields => \@preserve_fields,	72	preserve_fields => \@preserve_fields,
		73	update_dateexpiry => $update_dateexpiry
71	dry_run => !$confirm,	74	dry_run => !$confirm,
72	}	75	}
73	);	76	);
Lines 107-113 import_patrons.pl - CLI script to import patrons data into Koha Link Here
107		110
108	=head1 SYNOPSIS	111	=head1 SYNOPSIS
109		112
110	import_patrons.pl --file /path/to/patrons.csv --matchpoint cardnumber --confirm [--default branchcode=MPL] [--overwrite] [--preserve_field <column>] [--preserve-extended-attributes] [--verbose]	113	import_patrons.pl --file /path/to/patrons.csv --matchpoint cardnumber --confirm [--default branchcode=MPL] [--overwrite] [--preserve_field <column>] [--preserve-extended-attributes] [--update-expiration] [--verbose]
111		114
112	=head1 OPTIONS	115	=head1 OPTIONS
113		116
Lines 145-150 Overwrite existing patrons with new data if a match is found Link Here
145		148
146	Retain extended patron attributes for existing patrons being overwritten	149	Retain extended patron attributes for existing patrons being overwritten
147		150
		151	=item B<-p\|--update-expiration>
		152
		153	If a matching patron is found, extend the expiration date of their account using the patron's enrollment date as the base
		154
148	=item B<-v\|--verbose>	155	=item B<-v\|--verbose>
149		156
150	Be verbose	157	Be verbose