[Koha] [Koha-devel] error in Amazon.pm script?

Joshua Ferraro jmf at liblime.com
Thu Nov 6 08:29:03 NZDT 2008


On Wed, Nov 5, 2008 at 11:51 AM, Rick Welykochy <rick at praxis.com.au> wrote:
> Emrys Minnig wrote:
>
>> Something that might be of interest to the development team...
>> I think there's a small oversight in C4/Amazon.pm.  On line 101 it reads:
>>
>> my $uri = "http://www.amazon.com/gp/reader/$isbn/ref=sib_dp_pt/002-7879865-0184864#reader-link";
>>
>> This line ignores the locale settings (in my case UK) which the majority of times breaks the 'Search Inside' capability.
>
> Which raises another question: why is there a URL such as the above
> hard-coded in the software?
>
> This kind of data should be treated as data ... and stored in the
> configuration database.
I see no advantage to treating the static portions of the URL as data,
especially since no other resource implements the Amazon.com API.

Josh

> cheers
> rickw
>
>
> _______________________________________________
> Koha-devel mailing list
> Koha-devel at lists.koha.org
> http://lists.koha.org/mailman/listinfo/koha-devel
>



-- 
Joshua Ferraro                       SUPPORT FOR OPEN-SOURCE SOFTWARE
CEO                         migration, training, maintenance, support
LibLime                                Featuring Koha Open-Source ILS
jmf at liblime.com |Full Demos at http://liblime.com/koha |1(888)KohaILS


More information about the Koha mailing list