Bug 18736: (QA follow-up) Cosmetic changes
[koha.git] / C4 / XISBN.pm
old mode 100755 (executable)
new mode 100644 (file)
index ac5a2b7..22b4502
@@ -4,65 +4,74 @@ package C4::XISBN;
 #
 # This file is part of Koha.
 #
-# Koha is free software; you can redistribute it and/or modify it under the
-# terms of the GNU General Public License as published by the Free Software
-# Foundation; either version 2 of the License, or (at your option) any later
-# version.
+# Koha is free software; you can redistribute it and/or modify it
+# under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
 #
-# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
-# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
-# A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
+# Koha is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
 #
-# You should have received a copy of the GNU General Public License along with
-# Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
-# Suite 330, Boston, MA  02111-1307 USA
+# You should have received a copy of the GNU General Public License
+# along with Koha; if not, see <http://www.gnu.org/licenses>.
 
 use XML::Simple;
-use LWP::Simple;
+#use LWP::Simple;
 use C4::Biblio;
-
+use C4::Koha;
+use C4::Search;
+use C4::External::Syndetics qw(get_syndetics_editions);
 use LWP::UserAgent;
 use HTTP::Request::Common;
 
-use strict;
-require Exporter;
+use Koha::Biblios;
+use Koha::SearchEngine;
+use Koha::SearchEngine::Search;
 
-use vars qw($VERSION @ISA @EXPORT @EXPORT_OK %EXPORT_TAGS);
+use strict;
+#use warnings; FIXME - Bug 2505
+use vars qw(@ISA @EXPORT @EXPORT_OK %EXPORT_TAGS);
+
+BEGIN {
+       require Exporter;
+       @ISA = qw(Exporter);
+       @EXPORT_OK = qw(
+               &get_xisbns
+       );
+}
 
-$VERSION = 3.0;
 =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 {
-       my $xisbn_data = shift;
-       my $dbh = C4::Context->dbh;
-       my $query = "SELECT biblionumber FROM biblioitems WHERE isbn=?";
-       my $sth = $dbh->prepare($query);
-       $sth->execute($xisbn_data->{content});
-       my $xbib_data =  $sth->fetchrow_hashref();
-       my $xbiblio;
-       if ($xbib_data->{biblionumber}) {
-               $xbiblio = GetBiblioData($xbib_data->{biblionumber});
-               $xbiblio->{items} = GetItemsByBiblioitemnumber($xbib_data->{biblionumber});
-               use Data::Dumper;
-               warn Dumper($xbiblio->{items});
-       }
-       return ($xbiblio);
+sub _get_biblio_from_xisbn {
+    my $xisbn = shift;
+    my $dbh = C4::Context->dbh;
+
+    my $searcher = Koha::SearchEngine::Search->new({index => $Koha::SearchEngine::BIBLIOS_INDEX});
+    my ( $errors, $results, $total_hits ) = $searcher->simple_search_compat( "nb=$xisbn", 0, 1 );
+    return unless ( !$errors && scalar @$results );
+
+    my $record = C4::Search::new_record_from_zebra( 'biblioserver', $results->[0] );
+    my $biblionumber = C4::Biblio::TransformMarcToKohaOneField( 'biblio.biblionumber', $record );
+    return unless $biblionumber;
 
+    my $biblio = Koha::Biblios->find( $biblionumber );
+    return unless $biblio;
+    my $isbn = $biblio->biblioitem->isbn;
+    $biblio = $biblio->unblessed;
+    $biblio->{normalized_isbn} = GetNormalizedISBN($isbn);
+    return $biblio;
 }
+
 =head1 get_xisbns($isbn);
 
 =head2 $isbn is an ISBN string
@@ -71,95 +80,77 @@ sub get_biblio_from_xisbn {
 
 sub get_xisbns {
     my ( $isbn ) = @_;
-
-    my ($response,$thing_response,$xisbn_response,$gapines_response);
+    my ($response,$thing_response,$syndetics_response,$errors);
     # THINGISBN
     if ( C4::Context->preference('ThingISBN') ) {
         my $url = "http://www.librarything.com/api/thingISBN/".$isbn;
         $thing_response = _get_url($url,'thingisbn');
     }
 
-    # XISBN
-    if ( C4::Context->preference('XISBN') ) {
-        my $affiliate_id=C4::Context->preference('OCLCAffiliateID');
-               my $limit = C4::Context->preference('XISBNDailyLimit') || 499;
-        my $reached_limit = _service_throttle('xisbn',$limit);
-        my $url = "http://xisbn.worldcat.org/webservices/xid/isbn/".$isbn."?method=getEditions&format=xml&fl=form,year,lang,ed";
-               $url.="&ai=".$affiliate_id if $affiliate_id;
-               unless ($reached_limit) {
-               $xisbn_response = _get_url($url,'xisbn');
+       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}};
                }
-    }
-
-    # PINES ISBN
-    if ( C4::Context->preference('PINESISBN') ) {
-        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 [] } ];
-       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 $unique_xisbns->{ $response_data->{content} };
-               $unique_xisbns->{ $response_data->{content} }++;
-               my $xbiblio= get_biblio_from_xisbn($response_data);
-               push @xisbns, $xbiblio if $xbiblio; #response_data->{xbiblio}; #->{biblionumber}; # if $xbiblionumber;
-        
+               $syndetics_response = {isbn => \@syndetics_response};
        }
-       return \@xisbns;
 
+    $response->{isbn} = [ @{ $syndetics_response->{isbn} or [] }, @{ $thing_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;
+    }
+    if ( wantarray ) {
+        return (\@xisbns, $errors);
+    }
+    else {
+        return \@xisbns;
+    }
 }
 
 sub _get_url {
     my ($url,$service_type) = @_;
-       my $ua = LWP::UserAgent->new(
-               timeout => 2
-               );
-
-       my $response = $ua->get($url);
-       if ($response->is_success) {
-       warn "WARNING could not retrieve $service_type $url" unless $response;
-       if ($response) {
-               my $xmlsimple = XML::Simple->new();
-               my $content = $xmlsimple->XMLin(
+    my $ua = LWP::UserAgent->new(
+        timeout => 2
+        );
+
+    my $response = $ua->get($url);
+    if ($response->is_success) {
+        warn "WARNING could not retrieve $service_type $url" unless $response;
+        if ($response) {
+            my $xmlsimple = XML::Simple->new();
+            my $content = $xmlsimple->XMLin(
             $response->content,
             ForceArray => [ qw(isbn) ],
             ForceContent => 1,
             );
-                       return $content;
-       }
-       } else {
+            return $content;
+        }
+    } else {
         warn "WARNING: URL Request Failed " . $response->status_line . "\n";
     }
 
 }
 
-
-# Throttle services to the specified amount
-sub _service_throttle {
-    my ($service_type,$daily_limit) = @_;
-       my $dbh = C4::Context->dbh;
-    my $sth = $dbh->prepare("SELECT service_count FROM services_throttle WHERE service_type=?");
-    $sth->execute($service_type);
-    my $count = 1;
-
-    while (my $counter = $sth->fetchrow_hashref()) {
-        $count = $counter->{service_count} if $counter->{service_count};
-    }
-
-    # we're over the limit
-    return 1 if $count >= $daily_limit;
-
-    # not over the limit
-    $count++;
-    $sth = $dbh->do("UPDATE services_throttle SET service_count=$count WHERE service_type='xisbn'");
-    return undef;
-}
+1;
+__END__
 
 =head1 NOTES
 
+=cut
+
 =head1 AUTHOR
 
 Joshua Ferraro <jmf@liblime.com>
+
 =cut
+