[koha-commits] main Koha release repository branch master updated. v3.16.00-beta-41-ga18ce86
Git repo owner
gitmaster at git.koha-community.org
Mon May 19 17:18:08 CEST 2014
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".
The branch, master has been updated
via a18ce86e83740a58e9c10a2f99354a8b84d9a8bd (commit)
via 59c569ce92a72dc6d60a16e76fb8c4eebc741fcc (commit)
from 15e78498ac14dc28b3adf73ffda7938d1d47f9cb (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit a18ce86e83740a58e9c10a2f99354a8b84d9a8bd
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date: Tue May 13 07:20:39 2014 -0400
Bug 12243: ensure that records with invalid ISBNs do not cause searches to fail
Some types of invalid ISBNs, when run through C4::Koha::NormalizeISBN
can produce ISBN objects that contain invalid ISBNs. This can be
reproduced with an ISBN that has an invalid prefix, group code or
publisher code. An example ISBN would be "0788893777 (2 DVD 45th ed)".
When attempting to look up a record with such an ISBN, you will get an
error along the lines of: Can't call method "as_string" on an undefined
value
Instead of checking for the BAD_ISBN state, we should be checking for
the GOOD_ISBN state via the method is_valid.
Test Plan:
1) Edit a record, add the following ISBN to your record:
0788893777 (2 DVD 45th ed)
2) When Koha redirects to the record, you should see the error message
described
3) Apply this patch
4) Reload the page, you should now see your record
Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
commit 59c569ce92a72dc6d60a16e76fb8c4eebc741fcc
Author: Galen Charlton <gmc at esilibrary.com>
Date: Mon May 19 15:37:34 2014 +0000
Bug 12243: add regression test
This patch adds a regression test for C4::Koha::NormalizeISBN
throwing an exception when trying to parse an invalid ISBN.
To test:
[1] Run prove -v t/Koha.t. The last test should fail.
[2] Apply the main patch for this bug.
[3] Run prove -v t/Koha.t again. This time, the test
should pass.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
-----------------------------------------------------------------------
Summary of changes:
C4/Koha.pm | 2 +-
t/Koha.t | 8 +++++++-
2 files changed, 8 insertions(+), 2 deletions(-)
hooks/post-receive
--
main Koha release repository
More information about the koha-commits
mailing list