Bug 3346: Fixes invalid XHTML in opac-shelves.tmpl

[koha.git] / C4 / XISBN.pm

diff --git a/C4/XISBN.pm b/C4/XISBN.pm
index 4229d0d..8e97b6a 100644 (file)
--- a/C4/XISBN.pm
+++ b/C4/XISBN.pm
@@ -21,47 +21,61 @@ use XML::Simple;
 #use LWP::Simple;
 use C4::Biblio;
 use C4::Items;
-
+use C4::Koha;
+use C4::External::Syndetics qw(get_syndetics_editions);
 use LWP::UserAgent;
 use HTTP::Request::Common;
 
 use strict;
-require Exporter;
-
 use vars qw($VERSION @ISA @EXPORT @EXPORT_OK %EXPORT_TAGS);
 
-$VERSION = 3.0;
+BEGIN {
+       require Exporter;
+       $VERSION = 3.01;
+       @ISA = qw(Exporter);
+       @EXPORT_OK = qw(
+               &get_xisbns
+        &get_biblionumber_from_isbn
+       );
+}
+
+sub get_biblionumber_from_isbn {
+    my $isbn = shift;
+       $isbn.='%';
+    my @biblionumbers;
+    my $dbh=C4::Context->dbh;
+    my $query = "SELECT biblionumber FROM biblioitems WHERE isbn LIKE ? LIMIT 10";
+    my $sth = $dbh->prepare($query);
+    $sth->execute($isbn);
+       return $sth->fetchall_arrayref({});
+}
 =head1 NAME
 
 C4::XISBN - Functions for retrieving XISBN content in Koha
 
 =head1 FUNCTIONS
 
-This module provides facilities for retrieving XISBN, ThingISBN and XISBN content in Koha
+This module provides facilities for retrieving ThingISBN and XISBN content in Koha
 
 =cut
 
-@ISA = qw(Exporter);
-@EXPORT_OK = qw(
-  &get_xisbns
-  &get_biblio_from_xisbn
-);
-
-sub get_biblio_from_xisbn {
+sub _get_biblio_from_xisbn {
     my $xisbn = shift;
+    $xisbn.='%';
     my $dbh = C4::Context->dbh;
-    my $query = "SELECT biblionumber FROM biblioitems WHERE isbn=?";
+    my $query = "SELECT biblionumber FROM biblioitems WHERE isbn LIKE ?";
     my $sth = $dbh->prepare($query);
     $sth->execute($xisbn);
     my $xbib_data =  $sth->fetchrow_hashref();
     my $xbiblio;
     if ($xbib_data->{biblionumber}) {
         $xbiblio = GetBiblioData($xbib_data->{biblionumber});
+        $xbiblio->{normalized_isbn} = GetNormalizedISBN($xbiblio->{isbn});
         $xbiblio->{items} = GetItemsByBiblioitemnumber($xbib_data->{biblionumber});
     }
     return ($xbiblio);
-
 }
+
 =head1 get_xisbns($isbn);
 
 =head2 $isbn is an ISBN string
@@ -70,14 +84,22 @@ sub get_biblio_from_xisbn {
 
 sub get_xisbns {
     my ( $isbn ) = @_;
-    my ($response,$thing_response,$xisbn_response,$gapines_response);
-
+    my ($response,$thing_response,$xisbn_response,$gapines_response,$syndetics_response);
     # THINGISBN
     if ( C4::Context->preference('ThingISBN') ) {
         my $url = "http://www.librarything.com/api/thingISBN/".$isbn;
         $thing_response = _get_url($url,'thingisbn');
     }
 
+       if ( C4::Context->preference("SyndeticsEnabled") && C4::Context->preference("SyndeticsEditions") ) {
+       my $syndetics_preresponse = &get_syndetics_editions($isbn);
+               my @syndetics_response;
+               for my $response (@$syndetics_preresponse) {
+                       push @syndetics_response, {content => $response->{a}};
+               }
+               $syndetics_response = {isbn => \@syndetics_response};
+       }
+
     # XISBN
     if ( C4::Context->preference('XISBN') ) {
         my $affiliate_id=C4::Context->preference('OCLCAffiliateID');
@@ -95,16 +117,18 @@ sub get_xisbns {
     #    my $url = "http://www.librarything.com/api/thingISBN/".$isbn;
     #    $gapines_response = _get_url($url,'thingisbn');
     #}
-    $response->{isbn} = [ @{ $xisbn_response->{isbn} or [] }, @{ $thing_response->{isbn} or [] }, @{ $gapines_response->{isbn} or [] } ];
+    $response->{isbn} = [ @{ $xisbn_response->{isbn} or [] },  @{ $syndetics_response->{isbn} or [] }, @{ $thing_response->{isbn} or [] }, @{ $gapines_response->{isbn} or [] } ];
     my @xisbns;
     my $unique_xisbns; # a hashref
 
     # loop through each ISBN and scope to the local collection
     for my $response_data( @{ $response->{ isbn } } ) {
+        next if $response_data->{'content'} eq $isbn;
+        next if $isbn eq $response_data;
         next if $unique_xisbns->{ $response_data->{content} };
         $unique_xisbns->{ $response_data->{content} }++;
-        my $xbiblio= get_biblio_from_xisbn($response_data->{content});
-        push @xisbns, $xbiblio if $xbiblio; #response_data->{xbiblio}; #->{biblionumber}; # if $xbiblionumber;
+        my $xbiblio= _get_biblio_from_xisbn($response_data->{content});
+        push @xisbns, $xbiblio if $xbiblio;
     }
     return \@xisbns;
 }
@@ -155,9 +179,14 @@ sub _service_throttle {
     return undef;
 }
 
+1;
+__END__
+
 =head1 NOTES
 
 =head1 AUTHOR
 
 Joshua Ferraro <jmf@liblime.com>
+
 =cut
+