Bugzilla – Attachment 2670 Details for
Bug 4305
Amazon book covers do not work with ISBN13
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Followup patch to bug 4305
0002-Followup-to-Bug-4305-multiple-ISBNS-in-biblioitems.patch (text/plain), 1.10 KB, created by
Ian Walls
on 2010-10-06 17:01:13 UTC
(
hide
)
Description:
Followup patch to bug 4305
Filename:
MIME Type:
Creator:
Ian Walls
Created:
2010-10-06 17:01:13 UTC
Size:
1.10 KB
patch
obsolete
>From f765ab6ad5c9e72e86d7ea5fc2a6ad2d0ba91356 Mon Sep 17 00:00:00 2001 >From: Ian Walls <ian.walls@bywatersolutions.com> >Date: Wed, 6 Oct 2010 12:36:21 -0400 >Subject: [PATCH] Followup to Bug 4305: multiple ISBNS in biblioitems.isbn breaks patron > login >Content-Type: text/plain; charset="utf-8" > >This patch adds a scrub for ' | ' when the ISBN comes from the >biblioitems table, in the event that Koha has attempted to store >multiple ISBNs in the field. The ISBN presented to Business::ISBN will >now be something it can handle. >--- > C4/Koha.pm | 3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > >diff --git a/C4/Koha.pm b/C4/Koha.pm >index fb27ee2..cbcd048 100644 >--- a/C4/Koha.pm >+++ b/C4/Koha.pm >@@ -1241,6 +1241,9 @@ sub GetNormalizedISBN { > my ($isbn,$record,$marcflavour) = @_; > my @fields; > if ($isbn) { >+ # Koha attempts to store multiple ISBNs in biblioitems.isbn, separated by " | " >+ # anything after " | " should be removed, along with the delimiter >+ $isbn =~ s/(.*)( \| )(.*)/$1/; > return _isbn_cleanup($isbn); > } > return undef unless $record; >-- >1.5.6.5 >
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 4305
:
2669
| 2670 |
2695
|
2737