next-l / enju_leaf

Next-L Enju Leaf 統合図書館管理システム / integrated library system https://enju.next-l.jp/
https://www.next-l.jp
MIT License
118 stars 24 forks source link

バグ:資料のTSVインポートした場合に書影が表示されない #280

Closed yegusa closed 10 years ago

yegusa commented 10 years ago

デモサーバー(Next-L Enju Leaf 1.1.0.rc13 + Next-L Enju Flower 0.1.0.pre16)で現象を確認しました。

例:http://enju.next-l.jp/resource_import_files/25

↑のファイルをインポートしたときに、 書影があるはずの書誌ばかりなのに、書影が表示されません。

nabeta commented 10 years ago

表示されているものもあるようです。 http://enju.next-l.jp/manifestations/4462

yegusa commented 10 years ago

へえ。でも、大半は表示されてませんよね。

nabeta commented 10 years ago

「書影があるはず」というのは、なにをもってそう判断されていますでしょうか。

yegusa commented 10 years ago

一回、全て書影があることをEnjuでインポートして確かめ済みのデータだからです。

yegusa commented 10 years ago

https://docs.google.com/spreadsheet/ccc?key=0ApH7F2ejX_26dDhuRHo4ZTdOaUxRMkhXVjJaYnJlT0E&usp=drive_web

これの上から50件ほどをインポートしました。

yegusa commented 10 years ago

Enjuの問題というよりは、以前は、書影がGooglebookにあったデータでも、 現在はなくなっちゃってるってことなのでしょうか?

nabeta commented 10 years ago

書影表示は、Google BooksのWebAPIをその都度呼んでいます。imageLinks属性が取得できた場合、書影を表示します。

書影があるもの https://www.googleapis.com/books/v1/volumes?q=isbn:9784764904279&callback=addTheCover

書影がないもの https://www.googleapis.com/books/v1/volumes?q=isbn:9784946553196&callback=addTheCover

nabeta commented 10 years ago

ですので、Google Books側のデータが変わると、以前取得できていた書影が取得できなくなる、ということはありえます。imageLinks属性が取得できているのに書影が表示されない場合はEnju側の問題になりますが、現状ではimageLinks属性が取得できていないため、Google Books側の問題だと思います。

yegusa commented 10 years ago

たしかに、表示されないのはimageLinksがないやつですねえ。 ところで、GoogleBooksやCiniiBooksの表示では表示されてるけど、これはなにをつかってるんだろ。

Enjuデモサーバー(書影がない例): Webを支える技術 http://enju.next-l.jp/manifestations/4458

GooglebooksAPI imageLinksがない: https://www.googleapis.com/books/v1/volumes?q=isbn:9784774142043&callback=addTheCover

Googlebooksの画面では書影が表示される: http://books.google.co.jp/books?id=ejWgngEACAAJ&dq=9784774142043&hl=ja&sa=X&ei=Pg_qU6PfIMvs8AWZ7ICQDQ&ved=0CB0Q6AEwAA

CiNiiBooksでは書影が表示される: http://ci.nii.ac.jp/ncid/BB01707380

nabeta commented 10 years ago

WebAPIのレスポンスにimageLinksがなくても、以下のリンクで書影を取得できるようですね。 http://books.google.co.jp/books?vid=ISBN9784774142043&printsec=frontcover&img=1&zoom=1 http://books.google.co.jp/books?id=ejWgngEACAAJ&printsec=frontcover&img=1&zoom=1&hl=ja

ただ、いずれもhttpsに対応していないようです。

yegusa commented 10 years ago

とりあえず、TSVインポートの不具合ではなかったのですね。 なので、一端クローズしますが、できれば、imageLinksがなくても表示できるようにはしてほしいなあとおもいます。こちらは別issueを要望としてたてておきますね。

yegusa commented 10 years ago

(しかし、データ作ったときは書影あったのに、、、サンプルデータという意味では、残念です。)