X-Git-Url: http://koha-dev.rot13.org:8081/gitweb/?a=blobdiff_plain;f=C4%2FXISBN.pm;h=bf1f85770af9e74d42b98c2dba51cd06cda9d8b2;hb=36135f2d5646d91a46dcebed947fdcca8f336dfd;hp=ab934a1e2805d81371100de209ceff158254893a;hpb=377db43117584e2f90e772d27e3810aa7fd582d2;p=koha_fer diff --git a/C4/XISBN.pm b/C4/XISBN.pm index ab934a1e28..bf1f85770a 100644 --- a/C4/XISBN.pm +++ b/C4/XISBN.pm @@ -13,54 +13,70 @@ package C4::XISBN; # 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, write to the Free Software Foundation, Inc., +# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. use XML::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 warnings; FIXME - Bug 2505 use vars qw($VERSION @ISA @EXPORT @EXPORT_OK %EXPORT_TAGS); -$VERSION = 3.0; +BEGIN { + require Exporter; + $VERSION = 3.07.00.049; + @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 { - my $xisbn = shift; - my $dbh = C4::Context->dbh; - my $query = "SELECT biblionumber FROM biblioitems WHERE isbn=?"; - 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->{items} = GetItemsByBiblioitemnumber($xbib_data->{biblionumber}); - } - return ($xbiblio); +sub _get_biblio_from_xisbn { + my $xisbn = shift; + my $dbh = C4::Context->dbh; + my ( $errors, $results, $total_hits ) = C4::Search::SimpleSearch( "nb=$xisbn", 0, 1 ); + return unless ( !$errors && scalar @$results ); + + my $record = C4::Search::new_record_from_zebra( 'biblioserver', $results->[0] ); + my $biblionumber = C4::Biblio::get_koha_field_from_marc('biblio', 'biblionumber', $record, ''); + return unless $biblionumber; + + my $xbiblio = GetBiblioData($biblionumber); + return unless $xbiblio; + $xbiblio->{normalized_isbn} = GetNormalizedISBN($xbiblio->{isbn}); + return $xbiblio; } + =head1 get_xisbns($isbn); =head2 $isbn is an ISBN string @@ -69,65 +85,69 @@ sub get_biblio_from_xisbn { sub get_xisbns { my ( $isbn ) = @_; - - my ($response,$thing_response,$xisbn_response,$gapines_response); + my ($response,$thing_response,$xisbn_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'); - my $limit = C4::Context->preference('XISBNDailyLimit') || 499; + my $limit = C4::Context->preference('XISBNDailyLimit') || 999; 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'); - } + $url.="&ai=".$affiliate_id if $affiliate_id; + unless ($reached_limit) { + $xisbn_response = _get_url($url,'xisbn'); + } } - # 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 [] }, @{ $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; } - $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->{content}); - push @xisbns, $xbiblio if $xbiblio; #response_data->{xbiblio}; #->{biblionumber}; # if $xbiblionumber; - - } - return \@xisbns; - + 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"; } @@ -137,13 +157,17 @@ sub _get_url { # 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=?"); + my $dbh = C4::Context->dbh; + my $sth = $dbh->prepare(q{ 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}; + my $count = 0; + + if ($sth->rows == 0) { + # initialize services throttle + my $sth2 = $dbh->prepare(q{ INSERT INTO services_throttle (service_type, service_count) VALUES (?, ?) }); + $sth2->execute($service_type, $count); + } else { + $count = $sth->fetchrow_array; } # we're over the limit @@ -151,13 +175,22 @@ sub _service_throttle { # not over the limit $count++; - $sth = $dbh->do("UPDATE services_throttle SET service_count=$count WHERE service_type='xisbn'"); + my $sth3 = $dbh->prepare(q{ UPDATE services_throttle SET service_count=? WHERE service_type=? }); + $sth3->execute($count, $service_type); + return undef; } +1; +__END__ + =head1 NOTES +=cut + =head1 AUTHOR Joshua Ferraro + =cut +