3 # This file is part of Koha.
5 # Koha is free software; you can redistribute it and/or modify it
6 # under the terms of the GNU General Public License as published by
7 # the Free Software Foundation; either version 3 of the License, or
8 # (at your option) any later version.
10 # Koha is distributed in the hope that it will be useful, but
11 # WITHOUT ANY WARRANTY; without even the implied warranty of
12 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13 # GNU General Public License for more details.
15 # You should have received a copy of the GNU General Public License
16 # along with Koha; if not, see <http://www.gnu.org/licenses>.
20 use Test
::More tests
=> 47;
24 use Data
::Dumper qw
/Dumper/;
28 use Koha
::List
::Patron
;
32 use t
::lib
::TestBuilder
;
35 use_ok
('C4::Members');
38 my $schema = Koha
::Database
->schema;
39 $schema->storage->txn_begin;
40 my $builder = t
::lib
::TestBuilder
->new;
41 my $dbh = C4
::Context
->dbh;
43 # Remove invalid guarantorid's as long as we have no FK
44 $dbh->do("UPDATE borrowers b1 LEFT JOIN borrowers b2 ON b2.borrowernumber=b1.guarantorid SET b1.guarantorid=NULL where b1.guarantorid IS NOT NULL AND b2.borrowernumber IS NULL");
46 my $library1 = $builder->build({
49 my $library2 = $builder->build({
52 my $patron_category = $builder->build({ source
=> 'Category' });
53 my $CARDNUMBER = 'TESTCARD01';
54 my $FIRSTNAME = 'Marie';
55 my $SURNAME = 'Mcknight';
56 my $BRANCHCODE = $library1->{branchcode
};
58 my $CHANGED_FIRSTNAME = "Marry Ann";
59 my $EMAIL = "Marie\@email.com";
60 my $EMAILPRO = "Marie\@work.com";
61 my $PHONE = "555-12123";
63 # XXX should be randomised and checked against the database
64 my $IMPOSSIBLE_CARDNUMBER = "XYZZZ999";
66 #my ($usernum, $userid, $usercnum, $userfirstname, $usersurname, $userbranch, $branchname, $userflags, $emailaddress, $branchprinter)= @_;
79 C4
::Context
->_new_userenv ('DUMMY_SESSION_ID');
80 C4
::Context
->set_userenv ( @USERENV );
82 my $userenv = C4
::Context
->userenv
83 or BAIL_OUT
("No userenv");
85 # Make a borrower for testing
87 cardnumber
=> $CARDNUMBER,
88 firstname
=> $FIRSTNAME . q{ },
90 categorycode
=> $patron_category->{categorycode
},
91 branchcode
=> $BRANCHCODE,
93 dateexpiry
=> '9999-12-31',
97 my $addmem=Koha
::Patron
->new(\
%data)->store->borrowernumber;
98 ok
($addmem, "Koha::Patron->store()");
100 my $patron = Koha
::Patrons
->find( { cardnumber
=> $CARDNUMBER } )
101 or BAIL_OUT
("Cannot read member with card $CARDNUMBER");
102 my $member = $patron->unblessed;
104 ok
( $member->{firstname
} eq $FIRSTNAME &&
105 $member->{surname
} eq $SURNAME &&
106 $member->{categorycode
} eq $patron_category->{categorycode
} &&
107 $member->{branchcode
} eq $BRANCHCODE
109 or diag
("Mismatching member details: ".Dumper
(\
%data, $member));
111 is
($member->{dateofbirth
}, undef, "Empty dates handled correctly");
113 $member->{firstname
} = $CHANGED_FIRSTNAME . q{ };
114 $member->{email
} = $EMAIL;
115 $member->{phone
} = $PHONE;
116 $member->{emailpro
} = $EMAILPRO;
117 $patron->set($member)->store;
118 my $changedmember = Koha
::Patrons
->find( { cardnumber
=> $CARDNUMBER } )->unblessed;
119 ok
( $changedmember->{firstname
} eq $CHANGED_FIRSTNAME &&
120 $changedmember->{email
} eq $EMAIL &&
121 $changedmember->{phone
} eq $PHONE &&
122 $changedmember->{emailpro
} eq $EMAILPRO
124 or diag
("Mismatching member details: ".Dumper
($member, $changedmember));
126 t
::lib
::Mocks
::mock_preference
( 'CardnumberLength', '' );
127 C4
::Context
->clear_syspref_cache();
129 my $checkcardnum=C4
::Members
::checkcardnumber
($CARDNUMBER, "");
130 is
($checkcardnum, "1", "Card No. in use");
132 $checkcardnum=C4
::Members
::checkcardnumber
($IMPOSSIBLE_CARDNUMBER, "");
133 is
($checkcardnum, "0", "Card No. not used");
135 t
::lib
::Mocks
::mock_preference
( 'CardnumberLength', '4' );
136 C4
::Context
->clear_syspref_cache();
138 $checkcardnum=C4
::Members
::checkcardnumber
($IMPOSSIBLE_CARDNUMBER, "");
139 is
($checkcardnum, "2", "Card number is too long");
144 cardnumber
=> "123456789",
145 firstname
=> "Tomasito",
147 categorycode
=> $patron_category->{categorycode
},
148 branchcode
=> $library2->{branchcode
},
154 my $borrowernumber = Koha
::Patron
->new( \
%data )->store->borrowernumber;
155 $patron = Koha
::Patrons
->find( $borrowernumber );
156 my $borrower = $patron->unblessed;
157 is
( $borrower->{dateofbirth
}, undef, 'Koha::Patron->store should undef dateofbirth if empty string is given');
158 is
( $borrower->{debarred
}, undef, 'Koha::Patron->store should undef debarred if empty string is given');
159 isnt
( $borrower->{dateexpiry
}, '0000-00-00', 'Koha::Patron->store should not set dateexpiry to 0000-00-00 if empty string is given');
160 isnt
( $borrower->{dateenrolled
}, '0000-00-00', 'Koha::Patron->store should not set dateenrolled to 0000-00-00 if empty string is given');
162 $patron->set({ dateofbirth
=> '', debarred
=> '', dateexpiry
=> '', dateenrolled
=> '' })->store;
163 $borrower = Koha
::Patrons
->find( $borrowernumber )->unblessed;
164 is
( $borrower->{dateofbirth
}, undef, 'Koha::Patron->store should undef dateofbirth if empty string is given');
165 is
( $borrower->{debarred
}, undef, 'Koha::Patron->store should undef debarred if empty string is given');
166 isnt
( $borrower->{dateexpiry
}, '0000-00-00', 'Koha::Patron->store should not set dateexpiry to 0000-00-00 if empty string is given');
167 isnt
( $borrower->{dateenrolled
}, '0000-00-00', 'Koha::Patron->store should not set dateenrolled to 0000-00-00 if empty string is given');
169 $patron->set({ dateofbirth
=> '1970-01-01', debarred
=> '2042-01-01', dateexpiry
=> '9999-12-31', dateenrolled
=> '2015-09-06' })->store;
170 $borrower = Koha
::Patrons
->find( $borrowernumber )->unblessed;
171 is
( $borrower->{dateofbirth
}, '1970-01-01', 'Koha::Patron->store should correctly set dateofbirth if a valid date is given');
172 is
( $borrower->{debarred
}, '2042-01-01', 'Koha::Patron->store should correctly set debarred if a valid date is given');
173 is
( $borrower->{dateexpiry
}, '9999-12-31', 'Koha::Patron->store should correctly set dateexpiry if a valid date is given');
174 is
( $borrower->{dateenrolled
}, '2015-09-06', 'Koha::Patron->store should correctly set dateenrolled if a valid date is given');
176 subtest
'Koha::Patron->store should not update userid if not true' => sub {
179 # TODO Move this to t/db_dependent/Koha/Patrons.t subtest ->store
181 $data{ cardnumber
} = "234567890";
182 $data{userid
} = 'a_user_id';
183 $borrowernumber = Koha
::Patron
->new( \
%data )->store->borrowernumber;
184 my $patron = Koha
::Patrons
->find( $borrowernumber );
185 my $borrower = $patron->unblessed;
187 $patron->set( { firstname
=> 'Tomas', userid
=> '' } )->store;
188 $borrower = Koha
::Patrons
->find( $borrowernumber )->unblessed;
189 is
( $borrower->{userid
}, $data{userid
}, 'Koha::Patron->store should not update the userid with an empty string' );
190 $patron->set( { firstname
=> 'Tomas', userid
=> 0 } )->store;
191 $borrower = Koha
::Patrons
->find( $borrowernumber )->unblessed;
192 is
( $borrower->{userid
}, $data{userid
}, 'Koha::Patron->store should not update the userid with an 0');
193 $patron->set( { firstname
=> 'Tomas', userid
=> undef } )->store;
194 $borrower = Koha
::Patrons
->find( $borrowernumber )->unblessed;
195 is
( $borrower->{userid
}, $data{userid
}, 'Koha::Patron->store should not update the userid with an undefined value');
198 #Regression tests for bug 10612
199 my $library3 = $builder->build({
203 source
=> 'Category',
205 categorycode
=> 'STAFFER',
206 description
=> 'Staff dont batch del',
207 category_type
=> 'S',
212 source
=> 'Category',
214 categorycode
=> 'CIVILIAN',
215 description
=> 'Civilian batch del',
216 category_type
=> 'A',
221 source
=> 'Category',
223 categorycode
=> 'KIDclamp',
224 description
=> 'Kid to be guaranteed',
225 category_type
=> 'C',
229 my $borrower1 = $builder->build({
230 source
=> 'Borrower',
232 categorycode
=>'STAFFER',
233 branchcode
=> $library3->{branchcode
},
234 dateexpiry
=> '2015-01-01',
238 my $bor1inlist = $borrower1->{borrowernumber
};
239 my $borrower2 = $builder->build({
240 source
=> 'Borrower',
242 categorycode
=>'STAFFER',
243 branchcode
=> $library3->{branchcode
},
244 dateexpiry
=> '2015-01-01',
249 my $guarantee = $builder->build({
250 source
=> 'Borrower',
252 categorycode
=>'KIDclamp',
253 branchcode
=> $library3->{branchcode
},
254 dateexpiry
=> '2015-01-01',
255 guarantorid
=> undef, # will be filled later
259 my $bor2inlist = $borrower2->{borrowernumber
};
262 source
=> 'OldIssue',
264 borrowernumber
=> $bor2inlist,
265 timestamp
=> '2016-01-01',
269 # The following calls to GetBorrowersToExpunge are assuming that the pref
270 # IndependentBranches is off.
271 t
::lib
::Mocks
::mock_preference
('IndependentBranches', 0);
273 my $owner = Koha
::Patron
->new({ categorycode
=> 'STAFFER', branchcode
=> $library2->{branchcode
} })->store->borrowernumber;
274 my $list1 = AddPatronList
( { name
=> 'Test List 1', owner
=> $owner } );
275 my @listpatrons = ($bor1inlist, $bor2inlist);
276 AddPatronsToList
( { list
=> $list1, borrowernumbers
=> \
@listpatrons });
277 my $patstodel = GetBorrowersToExpunge
( {patron_list_id
=> $list1->patron_list_id() } );
278 is
( scalar(@
$patstodel),0,'No staff deleted from list of all staff');
279 Koha
::Patrons
->find($bor2inlist)->set({ categorycode
=> 'CIVILIAN' })->store;
280 $patstodel = GetBorrowersToExpunge
( {patron_list_id
=> $list1->patron_list_id()} );
281 ok
( scalar(@
$patstodel)== 1 && $patstodel->[0]->{'borrowernumber'} eq $bor2inlist,'Staff patron not deleted from list');
282 $patstodel = GetBorrowersToExpunge
( {branchcode
=> $library3->{branchcode
},patron_list_id
=> $list1->patron_list_id() } );
283 ok
( scalar(@
$patstodel) == 1 && $patstodel->[0]->{'borrowernumber'} eq $bor2inlist,'Staff patron not deleted by branchcode and list');
284 $patstodel = GetBorrowersToExpunge
( {expired_before
=> '2015-01-02', patron_list_id
=> $list1->patron_list_id() } );
285 ok
( scalar(@
$patstodel) == 1 && $patstodel->[0]->{'borrowernumber'} eq $bor2inlist,'Staff patron not deleted by expirationdate and list');
286 $patstodel = GetBorrowersToExpunge
( {not_borrowed_since
=> '2016-01-02', patron_list_id
=> $list1->patron_list_id() } );
287 ok
( scalar(@
$patstodel) == 1 && $patstodel->[0]->{'borrowernumber'} eq $bor2inlist,'Staff patron not deleted by last issue date');
289 Koha
::Patrons
->find($bor1inlist)->set({ categorycode
=> 'CIVILIAN' })->store;
290 Koha
::Patrons
->find($guarantee->{borrowernumber
})->set({ guarantorid
=> $bor1inlist })->store;
292 $patstodel = GetBorrowersToExpunge
( {patron_list_id
=> $list1->patron_list_id()} );
293 ok
( scalar(@
$patstodel)== 1 && $patstodel->[0]->{'borrowernumber'} eq $bor2inlist,'Guarantor patron not deleted from list');
294 $patstodel = GetBorrowersToExpunge
( {branchcode
=> $library3->{branchcode
},patron_list_id
=> $list1->patron_list_id() } );
295 ok
( scalar(@
$patstodel) == 1 && $patstodel->[0]->{'borrowernumber'} eq $bor2inlist,'Guarantor patron not deleted by branchcode and list');
296 $patstodel = GetBorrowersToExpunge
( {expired_before
=> '2015-01-02', patron_list_id
=> $list1->patron_list_id() } );
297 ok
( scalar(@
$patstodel) == 1 && $patstodel->[0]->{'borrowernumber'} eq $bor2inlist,'Guarantor patron not deleted by expirationdate and list');
298 $patstodel = GetBorrowersToExpunge
( {not_borrowed_since
=> '2016-01-02', patron_list_id
=> $list1->patron_list_id() } );
299 ok
( scalar(@
$patstodel) == 1 && $patstodel->[0]->{'borrowernumber'} eq $bor2inlist,'Guarantor patron not deleted by last issue date');
300 Koha
::Patrons
->find($guarantee->{borrowernumber
})->set({ guarantorid
=> undef })->store;
305 borrowernumber
=> $bor2inlist,
308 $patstodel = GetBorrowersToExpunge
( {patron_list_id
=> $list1->patron_list_id()} );
309 is
( scalar(@
$patstodel),1,'Borrower with issue not deleted from list');
310 $patstodel = GetBorrowersToExpunge
( {branchcode
=> $library3->{branchcode
},patron_list_id
=> $list1->patron_list_id() } );
311 is
( scalar(@
$patstodel),1,'Borrower with issue not deleted by branchcode and list');
312 $patstodel = GetBorrowersToExpunge
( {category_code
=> 'CIVILIAN',patron_list_id
=> $list1->patron_list_id() } );
313 is
( scalar(@
$patstodel),1,'Borrower with issue not deleted by category_code and list');
314 $patstodel = GetBorrowersToExpunge
( {expired_before
=> '2015-01-02',patron_list_id
=> $list1->patron_list_id() } );
315 is
( scalar(@
$patstodel),1,'Borrower with issue not deleted by expiration_date and list');
316 $builder->schema->resultset( 'Issue' )->delete_all;
317 $patstodel = GetBorrowersToExpunge
( {patron_list_id
=> $list1->patron_list_id()} );
318 ok
( scalar(@
$patstodel)== 2,'Borrowers without issue deleted from list');
319 $patstodel = GetBorrowersToExpunge
( {category_code
=> 'CIVILIAN',patron_list_id
=> $list1->patron_list_id() } );
320 is
( scalar(@
$patstodel),2,'Borrowers without issues deleted by category_code and list');
321 $patstodel = GetBorrowersToExpunge
( {expired_before
=> '2015-01-02',patron_list_id
=> $list1->patron_list_id() } );
322 is
( scalar(@
$patstodel),2,'Borrowers without issues deleted by expiration_date and list');
323 $patstodel = GetBorrowersToExpunge
( {not_borrowed_since
=> '2016-01-02', patron_list_id
=> $list1->patron_list_id() } );
324 is
( scalar(@
$patstodel),2,'Borrowers without issues deleted by last issue date');
326 # Test GetBorrowersToExpunge and TrackLastPatronActivity
327 $dbh->do(q
|UPDATE borrowers SET lastseen
=NULL
|);
328 $builder->build({ source
=> 'Borrower', value
=> { lastseen
=> '2016-01-01 01:01:01', categorycode
=> 'CIVILIAN', guarantorid
=> undef } } );
329 $builder->build({ source
=> 'Borrower', value
=> { lastseen
=> '2016-02-02 02:02:02', categorycode
=> 'CIVILIAN', guarantorid
=> undef } } );
330 $builder->build({ source
=> 'Borrower', value
=> { lastseen
=> '2016-03-03 03:03:03', categorycode
=> 'CIVILIAN', guarantorid
=> undef } } );
331 $patstodel = GetBorrowersToExpunge
( { last_seen
=> '1999-12-12' });
332 is
( scalar @
$patstodel, 0, 'TrackLastPatronActivity - 0 patrons must be deleted' );
333 $patstodel = GetBorrowersToExpunge
( { last_seen
=> '2016-02-15' });
334 is
( scalar @
$patstodel, 2, 'TrackLastPatronActivity - 2 patrons must be deleted' );
335 $patstodel = GetBorrowersToExpunge
( { last_seen
=> '2016-04-04' });
336 is
( scalar @
$patstodel, 3, 'TrackLastPatronActivity - 3 patrons must be deleted' );
337 my $patron2 = $builder->build({ source
=> 'Borrower', value
=> { lastseen
=> undef } });
338 t
::lib
::Mocks
::mock_preference
( 'TrackLastPatronActivity', '0' );
339 Koha
::Patrons
->find( $patron2->{borrowernumber
} )->track_login;
340 is
( Koha
::Patrons
->find( $patron2->{borrowernumber
} )->lastseen, undef, 'Lastseen should not be changed' );
341 Koha
::Patrons
->find( $patron2->{borrowernumber
} )->track_login({ force
=> 1 });
342 isnt
( Koha
::Patrons
->find( $patron2->{borrowernumber
} )->lastseen, undef, 'Lastseen should be changed now' );
344 # Regression tests for BZ13502
345 ## Remove all entries with userid='' (should be only 1 max)
346 $dbh->do(q
|DELETE FROM borrowers WHERE userid
= ''|);
347 ## And create a patron with a userid=''
348 $borrowernumber = Koha
::Patron
->new({ categorycode
=> $patron_category->{categorycode
}, branchcode
=> $library2->{branchcode
} })->store->borrowernumber;
349 $dbh->do(q
|UPDATE borrowers SET userid
= '' WHERE borrowernumber
= ?
|, undef, $borrowernumber);
350 # Create another patron and verify the userid has been generated
351 $borrowernumber = Koha
::Patron
->new({ categorycode
=> $patron_category->{categorycode
}, branchcode
=> $library2->{branchcode
} })->store->borrowernumber;
352 ok
( $borrowernumber > 0, 'Koha::Patron->store should have inserted the patron even if no userid is given' );
353 $borrower = Koha
::Patrons
->find( $borrowernumber )->unblessed;
354 ok
( $borrower->{userid
}, 'A userid should have been generated correctly' );
356 subtest
'purgeSelfRegistration' => sub {
361 C4
::Members
::DeleteUnverifiedOpacRegistrations
($d);
363 $dbh->do("INSERT INTO borrower_modifications (timestamp, borrowernumber, verification_token) VALUES ('2014-01-01 01:02:03',0,?)", undef, (scalar localtime)."_$_");
365 is
( C4
::Members
::DeleteUnverifiedOpacRegistrations
($d), 3, 'Test for DeleteUnverifiedOpacRegistrations' );
367 #purge members in temporary category
369 $dbh->do("INSERT IGNORE INTO categories (categorycode) VALUES ('$c')");
370 t
::lib
::Mocks
::mock_preference
('PatronSelfRegistrationDefaultCategory', $c );
371 t
::lib
::Mocks
::mock_preference
('PatronSelfRegistrationExpireTemporaryAccountsDelay', 360);
372 C4
::Members
::DeleteExpiredOpacRegistrations
();
373 $dbh->do("INSERT INTO borrowers (surname, address, city, branchcode, categorycode, dateenrolled) VALUES ('Testaabbcc', 'Street 1', 'CITY', ?, '$c', '2014-01-01 01:02:03')", undef, $library1->{branchcode
});
374 is
( C4
::Members
::DeleteExpiredOpacRegistrations
(), 1, 'Test for DeleteExpiredOpacRegistrations');
378 my ($resultset) = @_;
379 my $found = $resultset && grep( { $_->{cardnumber
} && $_->{cardnumber
} eq $CARDNUMBER } @
$resultset );
383 $schema->storage->txn_rollback;
385 subtest
'Koha::Patron->store (invalid categorycode) tests' => sub {
387 # TODO Move this to t/db_dependent/Koha/Patrons.t subtest ->store
389 $schema->storage->txn_begin;
391 my $category = $builder->build_object({ class => 'Koha::Patron::Categories' });
392 my $category_id = $category->id;
393 # Remove category to make sure the id is not on the DB
397 categorycode
=> $category_id
401 { Koha
::Patron
->new( $patron_data )->store; }
402 'Koha::Exceptions::Object::FKConstraint',
403 'AddMember raises an exception on invalid categorycode';
405 $schema->storage->txn_rollback;