[Koha-patches] [PATCH] Enabling Bookseller + Publisher Reviews from syndetics

[Chris Cormack]

---
 C4/External/Syndetics.pm                           |    5 +++--
 .../opac-tmpl/prog/en/modules/opac-detail.tmpl     |   14 +++++++++-----
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/C4/External/Syndetics.pm b/C4/External/Syndetics.pm
index 4a49379..2bbd6f0 100644
--- a/C4/External/Syndetics.pm
+++ b/C4/External/Syndetics.pm
@@ -78,7 +78,7 @@ sub get_syndetics_index {
     ) unless !$content;
 
     my $syndetics_elements;
-    for my $available_type ('SUMMARY','TOC','FICTION','AWARDS1','SERIES1','SPSUMMARY','SPREVIEW', 'AVPROFILE', 'AVSUMMARY','DBCHAPTER','LJREVIEW','PWREVIEW','SLJREVIEW','CHREVIEW','BLREVIEW','HBREVIEW','KIREVIEW','CRITICASREVIEW','ANOTES') {
+    for my $available_type ('SUMMARY','TOC','FICTION','AWARDS1','SERIES1','SPSUMMARY','SPREVIEW', 'AVPROFILE', 'AVSUMMARY','DBCHAPTER','LJREVIEW','PWREVIEW','SLJREVIEW','CHREVIEW','BLREVIEW','HBREVIEW','KIREVIEW','CRITICASREVIEW','ANOTES','ABREVIEW') {
         if (exists $response->{$available_type} && $response->{$available_type} =~ /$available_type/) {
             $syndetics_elements->{$available_type} = $available_type;
             #warn "RESPONSE: $available_type : $response->{$available_type}";
@@ -163,13 +163,14 @@ sub get_syndetics_reviews {
     {title => 'Kirkus Book Review', file => 'KIREVIEW.XML', element => 'KIREVIEW'},
     {title => 'Criticas Review', file => 'CRITICASREVIEW.XML', element => 'CRITICASREVIEW'},
     {title => 'Spanish Review', file => 'SPREVIEW.XML', element => 'SPREVIEW'},
+    {title => 'Bookseller and Publisher Review', file=> 'ABREVIEW.XML', element=> 'ABREVIEW'},
     ];
 
     for my $source (@$review_sources) {
         if ($syndetics_elements->{$source->{element}} and $source->{element} =~ $syndetics_elements->{$source->{element}}) {
 
         } else {
-            #warn "Skipping $source->{element} doesn't match $syndetics_elements->{$source->{element}} \n";
+#            warn "Skipping $source->{element} doesn't match $syndetics_elements->{$source->{element}} \n";
             next;
         }
         my $response = _fetch_syndetics_content($source->{file}, $isbn, $upc, $oclc);
diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tmpl b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tmpl
index b1fdf84..1fb2162 100644
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tmpl
+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tmpl
@@ -321,13 +321,17 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
 <!-- TMPL_IF NAME="SyndeticsExcerpt" --><!-- TMPL_IF NAME="SYNDETICS_EXCERPT" -->
  <li> <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=<!-- TMPL_VAR NAME="biblionumber" -->#excerpt">Excerpt</a></li>
 <!-- /TMPL_IF --><!-- /TMPL_IF -->
-<!-- TMPL_IF NAME="SyndeticsReviews" --><!-- TMPL_IF NAME="SYNDETICS_REVIEWS" -->
+<!-- TMPL_IF NAME="SyndeticsEnabled" -->
+<!-- TMPL_IF NAME="SYNDETICS_REVIEWS" -->
  <li> <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=<!-- TMPL_VAR NAME="biblionumber" -->#reviews">Reviews</a></li>
-<li> <!-- TMPL_IF NAME="using_https" --><script src="https://www.librarything.com/forlibraries/widget.js?id=<!-- TMPL_VAR NAME="LibraryThingForLibrariesID" -->" type="text/javascript"></script><noscript>This page contains enriched content visible when JavaScript is enabled or by clicking <a href="https://www.librarything.com/forlibraries/noscript.php?id=99-274208589&amp;accessibility=1">here</a>.</noscript>
+ <li> 
+<!-- TMPL_IF NAME="using_https" -->
+   <script src="https://www.librarything.com/forlibraries/widget.js?id=<!-- TMPL_VAR NAME="LibraryThingForLibrariesID" -->" type="text/javascript"></script><noscript>This page contains enriched content visible when JavaScript is enabled or by clicking <a href="https://www.librarything.com/forlibraries/noscript.php?id=99-274208589&amp;accessibility=1">here</a>.</noscript>
 <!-- TMPL_ELSE -->
-<script src="http://www.librarything.com/forlibraries/widget.js?id=<!-- TMPL_VAR NAME="LibraryThingForLibrariesID" -->" type="text/javascript"></script><noscript>This page contains enriched content visible when JavaScript is enabled or by clicking <a href="http://www.librarything.com/forlibraries/noscript.php?id=99-274208589&amp;accessibility=1">here</a>.</noscript><!-- /TMPL_IF -->
-</li>
-<!-- /TMPL_IF --><!-- /TMPL_IF -->
+   <script src="http://www.librarything.com/forlibraries/widget.js?id=<!-- TMPL_VAR NAME="LibraryThingForLibrariesID" -->" type="text/javascript"></script><noscript>This page contains enriched content visible when JavaScript is enabled or by clicking <a href="http://www.librarything.com/forlibraries/noscript.php?id=99-274208589&amp;accessibility=1">here</a>.</noscript><!-- /TMPL_IF -->
+  </li>
+<!-- /TMPL_IF -->
+<!-- /TMPL_IF -->
 <!-- TMPL_IF NAME="SyndeticsAuthorNotes" --><!-- TMPL_IF NAME="SYNDETICS_ANOTES" -->
  <li> <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=<!-- TMPL_VAR NAME="biblionumber" -->#anotes">About the Author</a></li>
 <!-- /TMPL_IF --><!-- /TMPL_IF -->
-- 
1.6.3.3