Attachment 32801 Details for Bug 9580 – Bug 9580 Cover images from Coce, a remote image URL cache

[patch] Bug 9580 Cover images from Coce, a remote image URL cache

Bug-9580-Cover-images-from-Coce-a-remote-image-URL.patch (text/plain), 10.72 KB, created by Frédéric Demians on 2014-10-27 19:10:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Frédéric Demians

Created: 2014-10-27 19:10:32 UTC

Size: 10.72 KB

patch

obsolete

>From 44dcdb012247bae994d7c4ef36bcd8e6b4542a8c Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Demians?= <f.demians@tamil.fr>
>Date: Thu, 7 Feb 2013 09:55:29 +0100
>Subject: [PATCH] Bug 9580 Cover images from Coce, a remote image URL cache
>
>Add to Koha support for displaying Book cover requested to Coce, a
>remote image URL cache:
>
>  https://github.com/fredericd/coce
>
>With this enhancement, cover images are not fetched directly from
>Amazon, Google, and so on. Their URL are requested via a web service to
>Coce which manages a cache of URLs.
>
>Three cover image providers are now available: Google Books (gb), Amazon
>(aws), and Open Library (ol). Two system preferences enable this service:
>
>  - CoceHost - URL of Coce server
>  - CoceProviders - Ordered list of requested providers. For example
>    gb,aws,ol, will return the first available URL from those providers.
>
>Several advantages to this architecture:
>
>  - If a book cover is not available from a provider, but is available
>    from another one, Koha will be able to display a book cover, which
>    isn't the case now
>  - Since URLs are cached, it isn't necessary for each book cover to
>    request, again and again, the provider, and several of them if
>    necessary.
>  - Amazon book covers are retrieved with Amazon Product Advertising
>    API, which means that more covers are retrieved (ISBN13).
>
>Test plan:
>
>- Apply this patch, and test with 'Bootstrap' themes
>- You can verify that nothing has changed on OPAC result and detail
>  page, since new syspref haven't been filled
>- Install Code: https://github.com/fredericd/coce
>  Or ask me directly for the URL of a Coce server
>- In sysprefs, tab Enhanced content, modify:
>  CoceHost -- enter the URL of your Coce server
>  CoceProviders -- fill with: gb,aws,ol
>- Do a search. On result page, and detail page, you will see cover
>  images originating from the 3 providers: fly over the image to see its
>  URL. Try to compare with a Koha with just GoogleJacket or Amazon cover
>  to confirm that you have more images. Verify that it's quick, and even
>  quicker for cached images URLs.
>- The ID sent to Coce is EAN or ISBN. Try with various type of biblios. DVD
>  have often EAN, but no ISBN.
>---
> C4/Auth.pm                                         |  2 +
> .../admin/preferences/enhanced_content.pref        |  9 +++++
> .../bootstrap/en/includes/opac-bottom.inc          |  9 +++++
> .../opac-tmpl/bootstrap/en/modules/opac-detail.tt  |  7 ++++
> .../opac-tmpl/bootstrap/en/modules/opac-results.tt |  9 +++++
> koha-tmpl/opac-tmpl/bootstrap/js/coce.js           | 43 ++++++++++++++++++++++
> .../opac-tmpl/prog/en/includes/doc-head-close.inc  |  3 ++
> 7 files changed, 82 insertions(+)
> create mode 100644 koha-tmpl/opac-tmpl/bootstrap/js/coce.js
>
>diff --git a/C4/Auth.pm b/C4/Auth.pm
>index bb2f308..c5751fa 100644
>--- a/C4/Auth.pm
>+++ b/C4/Auth.pm
>@@ -351,6 +351,8 @@ sub get_template_and_user {
>             "BiblioDefaultView".C4::Context->preference("BiblioDefaultView")         => 1,
>             EnhancedMessagingPreferences => C4::Context->preference('EnhancedMessagingPreferences'),
>             GoogleJackets                => C4::Context->preference("GoogleJackets"),
>+            CoceHost                     => C4::Context->preference("CoceHost"),
>+            CoceProviders                => C4::Context->preference("CoceProviders"),
>             OpenLibraryCovers            => C4::Context->preference("OpenLibraryCovers"),
>             KohaAdminEmailAddress        => "" . C4::Context->preference("KohaAdminEmailAddress"),
>             LoginBranchcode              => (C4::Context->userenv?C4::Context->userenv->{"branch"}:undef),
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/enhanced_content.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/enhanced_content.pref
>index 627f095..cf639aa 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/enhanced_content.pref
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/enhanced_content.pref
>@@ -337,3 +337,12 @@ Enhanced Content:
>             - "Show items from the OverDrive catalog of library #"
>             - pref: OverDriveLibraryID
>             - .
>+    Coce Cover images cache:
>+        -
>+            - Coce server URL
>+            - pref: CoceHost
>+              class: url
>+        -
>+            - Providers
>+            - pref: CoceProviders
>+            - 'separated by comma, for example: aws,gb. Leave empty not to activate this service.'
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc
>index 596341a..7d91a03 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc
>@@ -166,6 +166,15 @@ $.widget.bridge('uitooltip', $.ui.tooltip);
>         //]]>
>     </script>
> [% END %]
>+[% IF CoceProviders %]
>+    <script type="text/javascript" src="[% interface %]/[% theme %]/js/coce.js"></script>
>+    <script type="text/javascript">
>+        //<![CDATA[
>+        var NO_COCE_JACKET = _("No cover image available");
>+        //]]>
>+    </script>
>+[% END %]
>+
> [% IF OpenLibraryCovers %]
>     <script type="text/javascript" src="[% interface %]/[% theme %]/js/openlibrary.js"></script>
>     <script type="text/javascript">
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>index e641d76..ff49d3d 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>@@ -79,6 +79,9 @@
>                     [% IF ( GoogleJackets ) %]
>                         <div title="[% biblionumber |url %]" class="[% normalized_isbn %]" id="gbs-thumbnail-preview"></div>
>                     [% END %]
>+                    [% IF ( CoceProviders ) %]
>+                      <div style="block" title="[% biblionumber |url %]" class="[% IF ( normalized_ean ) %][% normalized_ean %][% ELSE %][% normalized_isbn %][% END %]" id="coce-thumbnail-preview"></div>
>+                    [% END %]
>                     [% IF OpenLibraryCovers %]
>                         <div title="[% biblionumber |url %]" class="[% normalized_isbn %]" id="openlibrary-thumbnail-preview"></div>
>                     [% END %]
>@@ -1430,6 +1433,10 @@
>     [% IF ( GoogleJackets ) %]
>         KOHA.Google.GetCoverFromIsbn([% covernewwindow %]);
>     [% END %]
>+    [% IF ( CoceProviders ) %]
>+        KOHA.coce.getURL('[% CoceHost %]', '[% CoceProviders %]',[% covernewwindow %]);
>+    [% END %]
>+
>     [% IF OpenLibraryCovers %]
>         KOHA.OpenLibrary.GetCoverFromIsbn();
>     [% END %]
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>index 72a3a89..481d38c 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>@@ -257,6 +257,14 @@
>                                                             [% END %]
>                                                         [% END %]
> 
>+                                                        [% IF ( CoceProviders ) %]
>+                                                          [% IF ( SEARCH_RESULT.normalized_ean || SEARCH_RESULT.normalized_isbn ) %]
>+                                                            <span style="block" title="[% SEARCH_RESULT.biblionumber |url %]" class="[% IF ( SEARCH_RESULT.normalized_ean ) %][% SEARCH_RESULT.normalized_ean %][% ELSE %][% SEARCH_RESULT.normalized_isbn %][% END %]" id="coce-thumbnail[% loop.count %]"></span>
>+                                                          [% ELSE %]
>+                                                            <span class="no-image">No cover image available</span>
>+                                                          [% END %]
>+                                                        [% END %]
>+
>                                                         [% IF OpenLibraryCovers %]
>                                                             [% IF SEARCH_RESULT.normalized_isbn %]
>                                                                 <span style="block" title="[% SEARCH_RESULT.biblionumber %]" class="[% SEARCH_RESULT.normalized_isbn %]" id="openlibrary-thumbnail[% loop.count %]"></span>
>@@ -908,6 +916,7 @@ $(document).ready(function(){
> [% IF OpenLibraryCovers %]KOHA.OpenLibrary.GetCoverFromIsbn();[% END %]
> [% IF OPACLocalCoverImages %]KOHA.LocalCover.GetCoverFromBibnumber(false);[% END %]
> [% IF ( GoogleJackets ) %]KOHA.Google.GetCoverFromIsbn();[% END %]
>+[% IF ( CoceProviders ) %]KOHA.coce.getURL('[% CoceHost %]', '[% CoceProviders %]');[% END %]
> 
> [% IF ( DidYouMean ) %]
>     $("#didyoumean").load("/cgi-bin/koha/svc/suggestion?render=stub&q=[% querystring |uri %]",
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/js/coce.js b/koha-tmpl/opac-tmpl/bootstrap/js/coce.js
>new file mode 100644
>index 0000000..0105a82
>--- /dev/null
>+++ b/koha-tmpl/opac-tmpl/bootstrap/js/coce.js
>@@ -0,0 +1,43 @@
>+if (KOHA === undefined || !KOHA) { var KOHA = {}; }
>+
>+
>+/**
>+ * A namespace for Coce cover images cache
>+ */
>+KOHA.coce = {
>+
>+  /**
>+   * Search all:
>+   *    <div title="biblionumber" id="isbn" class="coce-thumbnail"></div>
>+   * or
>+   *    <div title="biblionumber" id="isbn" class="coce-thumbnail-preview"></div>
>+   * and run a search with all collected isbns to coce cover service.
>+   * The result is asynchronously returned, and used to append <img>.
>+   */
>+  getURL: function(host,provider,newWindow) {
>+    var ids = [];
>+    $("[id^=coce-thumbnail]").each(function(i) {
>+        var id = $(this).attr("class"); // id=isbn
>+        if ( id !== '' ) { ids.push(id); }
>+    });
>+    if (ids.length == 0) return;
>+    ids = ids.join(',');
>+    var coceURL = host + '/cover?id=' + ids + '&provider=' + provider;
>+    $.ajax({
>+      url: coceURL,
>+      dataType: 'jsonp',
>+      success: function(urlPerID){
>+        for (var id in urlPerID) {
>+          var url = urlPerID[id];
>+          $("[id^=coce-thumbnail]."+id).each(function() {
>+            var img = document.createElement("img");
>+            img.src = url;
>+            img.title = url; //FIXME: to delete
>+            $(this).append(img);
>+         });
>+        }
>+      }
>+    });
>+  }
>+  
>+};
>diff --git a/koha-tmpl/opac-tmpl/prog/en/includes/doc-head-close.inc b/koha-tmpl/opac-tmpl/prog/en/includes/doc-head-close.inc
>index 66666f4..ecb1b25 100644
>--- a/koha-tmpl/opac-tmpl/prog/en/includes/doc-head-close.inc
>+++ b/koha-tmpl/opac-tmpl/prog/en/includes/doc-head-close.inc
>@@ -112,6 +112,9 @@
> 	//]]>
> </script>
> [% END %]
>+[% IF CoceProviders %]
>+<script type="text/javascript" src="[% themelang %]/js/coce.js"></script>
>+[% END %]
> [% IF OpenLibraryCovers %]
> <script type="text/javascript" src="[% themelang %]/js/openlibrary.js"></script>
> <script type="text/javascript">
>-- 
>2.1.2

Actions: View | Diff | Splinter Review

Attachments on bug 9580: 15203 | 16778 | 21162 | 24071 | 24145 | 32794 | 32795 | 32801 | 32808 | 32809 | 32857 | 32891 | 32929 | 32942 | 32944 | 33201 | 34678 | 34679 | 37409 | 37410