]> git.koha-community.org Git - koha.git/commit
Bug 7134: patron records getting corrupted birthdates
authorIan Walls <ian.walls@bywatersolutions.com>
Thu, 17 Nov 2011 15:27:47 +0000 (10:27 -0500)
committerChris Nighswonger <chris.nighswonger@gmail.com>
Sun, 20 Nov 2011 02:06:28 +0000 (21:06 -0500)
commit12f6201ea90fac634ea58dd950914d32060e68f4
treeab8d7a8a9d35824b889e4fc14d427d2d8ceaa830
parent35234b478a6017fb04cc8d0a64ddf177974b5727
Bug 7134: patron records getting corrupted birthdates

If no date of birth is provided in a new member form, the borrower gets the birthdate of "00/02/0", which
is obviously incorrect (the result of passing 0 to the database directly)

The fix is to remove a conditional check buried deep in C4::SQLHelper.  This is in a helper subroutine that was changed
for the Unified Member Search dev (bug 6253).  Reverting this small change to the conditional does not seem to break
search, but testing will be required.

To test:
1. apply patch, and create a new patron with blank birthdate.  The new patron should now have a blank birthdate instead of '00/02/0'
2. search for a borrower by 1 part surname, confirm search works
3. search for a borrower by 2 part surname, confirm search works
4. search for a borrower across firstname AND surname; confirm
5. search for a borrower and include categorycode/branchcode filters

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
I confirm all working tests 1-5
1. Created patron with blank birthdate - birthdate was blank
2. Searched for patron "henry james acevedo III" by using "acevedo" - result was correct
3. Searched for mr. acevedo using "Acevedo III" - result was correct
4. Searched for Henry James Acevedo III, Henry Acevedo and James Acevedo - results were correct
5. Searched for Mr Acevedo with the branch he was not in, with the branch he is in, with a category he was not, and a category he was. All results were correct.

also noted: inner searching seems to be working now? Example: Henry James (firstname) Acevedo III (surname) - searching James Acevedo brings back the correct patron.
Not complaining, just noting. :)

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
(cherry picked from commit 440c0f21ee59664bfbd2238c638f84a6af3d9d80)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
C4/SQLHelper.pm