GCstar forums

tetromino · 2007-01-24 00:43:24

Amazon recently upgraded to ISBN-13, at least for the books that I looked at, and now lists ISBN numbers like this:

ISBN-10: 0123456789
ISBN-13: 123-01234567890

However, GCAmazon.pm searches for "ISBN:", fails to find it, and therefore deletes your ISBN any time you fetch information. Very annoying.

One solution is to change

Code:

elsif ($self->{insideIsbn})
{
    $self->{curInfo}->{isbn} = $origtext;
    $self->{insideIsbn} = 0;
}
[...]
$self->{insideIsbn} = 1 if $origtext =~ /ISBN:/;

to

Code:

elsif ($self->{insideIsbn10})
{
    $self->{curInfo}->{isbn} = $origtext unless $self->{haveIsbn13};
    $self->{insideIsbn10} = 0;
}
elsif ($self->{insideIsbn13})
{
    $self->{curInfo}->{isbn} = $origtext;
    $self->{insideIsbn13} = 0;
    $self->{haveIsbn13} = 1;
}
[...]
$self->{insideIsbn10} = 1 if $origtext =~ /ISBN(-10)?:/;
$self->{insideIsbn13} = 1 if $origtext =~ /ISBN-13:/;

in GCAmazon.pm. (I am assuming that ISBN-13 is preferable over -10.)
I suspect that the changeover to ISBN-13 will very soon also affect all other sources of book information - if it hasn't already.

Tian · 2007-01-24 12:17:21

Hello,

Thank you very much for this patch. I will add it to the CVS repository ASAP.

Tian · 2007-01-26 19:26:07

It's on CVS now

GCstar forums - Amazon ISBN change

#1 2007-01-24 00:43:24

Amazon ISBN change

Code:

Code:

#2 2007-01-24 12:17:21

Re: Amazon ISBN change

#3 2007-01-26 19:26:07

Re: Amazon ISBN change