blacklight 4.0.1 → 4.1.0

data/app/helpers/blacklight/catalog_helper_behavior.rb

@@ -104,7 +104,7 @@ module Blacklight::CatalogHelperBehavior
 
   def show_sort_and_per_page? response = nil
     response ||= @response
-    response.response['numFound'] > 1
+    response.total > 1
   end
 
 end

data/app/helpers/blacklight/search_history_constraints_helper_behavior.rb

@@ -32,7 +32,7 @@ module Blacklight::SearchHistoryConstraintsHelperBehavior
       render_search_to_s_element(blacklight_config.facet_fields[facet_field].label,
         value_list.collect do |value|
           render_filter_value(value)
-        end.join(content_tag(:span, t('blacklight.and'), :class =>'label')).html_safe
+        end.join(content_tag(:span, " #{t('blacklight.and')} ", :class =>'filterSeparator')).html_safe
       )    
     end.join(" \n ").html_safe    
   end

data/app/views/catalog/_document_header.html.erb

@@ -3,7 +3,7 @@
       <div class="documentHeader clearfix">
         
         <% # main title container for doc partial view -%>
-          <h5 class="index_title"><%= t('blacklight.search.documents.counter', :counter => (document_counter + 1 + @response.params[:start].to_i)) %><%= link_to_document document, :label=>document_show_link_field, :counter => (document_counter + 1 + @response.params[:start].to_i) %></h5>
+          <h5 class="index_title"><%= t('blacklight.search.documents.counter', :counter => (document_counter + 1 + @response.params[:start].to_i)) %><%= link_to_document document, :label=>document_show_link_field(document), :counter => (document_counter + 1 + @response.params[:start].to_i) %></h5>
 
         
         <% # bookmark functions for items/docs -%>

data/app/views/catalog/_index_default.html.erb

@@ -1,10 +1,10 @@
 <%# default partial to display solr document fields in catalog index view -%>
 <dl class="dl-horizontal dl-invert">
   
-  <% index_fields.each do |solr_fname, field| -%>
+  <% index_fields(document).each do |solr_fname, field| -%>
     <% if should_render_index_field? document, field %>
-	    <dt class="blacklight-<%= solr_fname.parameterize %>"><%= render_index_field_label :field => solr_fname %></dt>
-	    <dd class="blacklight-<%= solr_fname.parameterize %>"><%= render_index_field_value :document => document, :field => solr_fname %></dd>
+	    <dt class="blacklight-<%= solr_fname.parameterize %>"><%= render_index_field_label document, :field => solr_fname %></dt>
+	    <dd class="blacklight-<%= solr_fname.parameterize %>"><%= render_index_field_value document, :field => solr_fname %></dd>
     <% end -%>
   <% end -%>
 

data/app/views/catalog/_per_page_widget.html.erb

@@ -6,7 +6,7 @@
       <%= link_to(t(:'blacklight.search.per_page.button_label', :count => current_per_page), "#") %> <span class="caret"></span>
       <ul>
         <%- blacklight_config.per_page.each do |count| %>
-          <li><%= link_to(t(:'blacklight.search.per_page.label', :count => count).html_safe, url_for(params_for_search.merge(:per_page => count))) %></li>
+          <li><%= link_to(t(:'blacklight.search.per_page.label', :count => count).html_safe, url_for(params_for_search(:per_page => count))) %></li>
         <%- end -%>
       </ul>
     </li>

data/app/views/catalog/_show_default.html.erb

@@ -1,9 +1,9 @@
 <%# default partial to display solr document fields in catalog show view -%>
 <dl class="dl-horizontal  dl-invert">
-  <% document_show_fields.each do |solr_fname, field| -%>
+  <% document_show_fields(document).each do |solr_fname, field| -%>
     <% if should_render_show_field? document, field %>
-	    <dt class="blacklight-<%= solr_fname.parameterize %>"><%= render_document_show_field_label :field => solr_fname %></dt>
-	    <dd class="blacklight-<%= solr_fname.parameterize %>"><%= render_document_show_field_value :document => document, :field => solr_fname %></dd>
+	    <dt class="blacklight-<%= solr_fname.parameterize %>"><%= render_document_show_field_label document, :field => solr_fname %></dt>
+	    <dd class="blacklight-<%= solr_fname.parameterize %>"><%= render_document_show_field_value document, :field => solr_fname %></dd>
     <% end -%>
   <% end -%>
 </dl>

data/app/views/catalog/_sort_widget.html.erb

@@ -5,7 +5,7 @@
       <%= link_to(t('blacklight.search.sort.label', :field =>current_sort_field.label), "#") %> <span class="caret"></span>
       <ul>
         <%- blacklight_config.sort_fields.each do |sort_key, field| %>
-          <li><%= link_to(field.label, url_for(params_for_search.merge(:sort => sort_key))) %></li>
+          <li><%= link_to(field.label, url_for(params_for_search(:sort => sort_key))) %></li>
         <%- end -%>
       </ul>
     </li>

data/app/views/catalog/show.html.erb

@@ -1,44 +1,41 @@
-<div class="row">
+<div id="content" class="span9 show-document">
 
-  <div id="content" class="span9 show-document">
+  <%= render 'previous_next_doc' %>
 
-    <%= render 'previous_next_doc' %>
+   
+<% @page_title = t('blacklight.search.show.title', :document_title => document_show_html_title, :application_name => application_name) %>
+<% extra_head_content << render_link_rel_alternates %>
+<% sidebar_items << render_document_sidebar_partial %>
 
-     
-  <% @page_title = t('blacklight.search.show.title', :document_title => document_show_html_title, :application_name => application_name) %>
-  <% extra_head_content << render_link_rel_alternates %>
-  <% sidebar_items << render_document_sidebar_partial %>
+<%# this should be in a partial -%>
+<div id="document" class="<%= render_document_class %>">
+  <div id="doc_<%= @document.id.to_s.parameterize %>">
+  
 
-  <%# this should be in a partial -%>
-  <div id="document" class="<%= render_document_class %>">
-    <div id="doc_<%= @document.id.to_s.parameterize %>">
-    
+    <% # bookmark/folder functions -%>
+    <%= render_document_heading(:h1).html_safe %>
 
-      <% # bookmark/folder functions -%>
-      <%= render_document_heading(:h1).html_safe %>
-
-      <div class="document">
-        <%= render_document_partial @document, :show %>
-      </div>
-
-     
+    <div class="document">
+      <%= render_document_partial @document, :show %>
     </div>
-  </div>
 
+   
+  </div>
+</div>
 
 
-    <% if @document.respond_to?(:export_as_openurl_ctx_kev) %>
-      <!-- 
-           // COinS, for Zotero among others. 
-           // This document_partial_name(@document) business is not quite right,
-           // but has been there for a while. 
-      -->
-      <span class="Z3988" title="<%= @document.export_as_openurl_ctx_kev(document_partial_name(@document)) %>"></span>
-    <% end %>
 
-  </div>
+  <% if @document.respond_to?(:export_as_openurl_ctx_kev) %>
+    <!-- 
+         // COinS, for Zotero among others. 
+         // This document_partial_name(@document) business is not quite right,
+         // but has been there for a while. 
+    -->
+    <span class="Z3988" title="<%= @document.export_as_openurl_ctx_kev(document_partial_name(@document)) %>"></span>
+  <% end %>
 
-  <div id="sidebar" class="span3">
-     <%= render_document_sidebar_partial  %>
-  </div><!--/span -->
 </div>
+
+<div id="sidebar" class="span3">
+   <%= render_document_sidebar_partial  %>
+</div><!--/span -->

data/app/views/layouts/blacklight.html.erb

@@ -1,5 +1,5 @@
 <!DOCTYPE html>
-<html lang="en">
+<html lang="en" class="no-js">
   <head>
     <meta charset="utf-8">
     <meta http-equiv="Content-Type" content="text/html; charset=utf-8">

data/blacklight.gemspec

@@ -20,7 +20,7 @@ Gem::Specification.new do |s|
 
   # PRODUCTION GEM REQUIREMENTS
   # ---------------------------------------   
-  s.add_dependency "rails",     "~> 3.2"
+  s.add_dependency "rails",     ">= 3.2"
   s.add_dependency "nokogiri",  "~>1.5"     # XML Parser
   # Let's allow future versions of marc, count on
   # them to be backwards compat until 1.1

data/config/locales/blacklight.fr.yml

@@ -0,0 +1,236 @@
+fr:
+  active_record:
+    models:
+      feedback: 'Vos réactions'
+    attributes:
+      feedback:
+        name: 'Votre nom'
+        email: 'Votre courriel'
+        message: 'Votre message'
+  views:
+    pagination:
+      first: '« Première'
+      last: 'Dernière »'
+      previous: '« Précédente'
+      next: 'Suivante »'
+      truncate: '…'
+
+    pagination_compact:
+      previous: '« Précédente'
+      next: 'Suivante »'
+
+  blacklight:
+    application_name: 'Blacklight BCU'
+
+    header_links:
+      login: 'Mon compte'
+      logout: 'Me déconnecter'
+      bookmarks: 'Mon panier'
+      saved_searches: 'Mes recherches'
+      search_history: 'Historique de recherche'
+
+    welcome: 'Welcome!'
+    and: 'et'
+    or: 'ou'
+
+    bookmarks:
+      collection:
+        add:
+          button: 'Panier'
+      title: 'Favoris'
+      no_bookmarks: 'Accédez à votre compte pour conserver des références dans votre panier'
+      add:
+        button: 'Panier'
+        success:
+          one: 'Favori bien enregistré.'
+          other: 'Favoris bien enregistrés.'
+        failure:  'Pardon, un problème est survenu lors de l''ajout.'
+      remove:
+        button: 'Supprimer'
+        success: 'Favori supprimé.'
+        failure: 'Pardon, un problème est survenu lors de la suppression.'
+        action_confirm: 'Supprimer ce favori ?'
+      clear:
+        action_title: 'Effacer tout'
+        action_confirm: 'Effacer tous vos favoris ?'
+        success: 'Favoris effacés.'
+        failure: 'Pardon, un problème est survenu lors de la suppression.'
+      need_login: 'Veuillez vous connecter pour accéder à vos favoris.'
+      list_title: 'Vos favoris'
+      delete: 'Supprimer'
+
+    feedback:
+      valid_name: 'Vous devez saisir votre nom.'
+      valid_email: 'Vous devez saisir votre courriel.'
+      need_message: 'Vous devez saisir un message.'
+      submit: 'Envoyer'
+      errors:
+        title: 'Veuillez corriger les erreurs suivantes :'
+      complete:
+        title: 'Merci pour vos remarques.'
+
+    folder:
+      title: 'Panier'
+      add:
+        success: '%{title} a bien été sélectionné.'
+      remove:
+        success: '%{title} a bien été supprimé.'
+      clear:
+        success: 'Panier vide.'
+      no_folder: 'Votre panier est vide'
+
+    saved_searches:
+      add:
+        success: 'Votre recherche a bien été sauvegardée.'
+        failure: 'Un problème est survenu lors de l''enregistrement.'
+      remove:
+        success: 'Votre recherche a bien été supprimée.'
+        failure: 'Un problème est survenu lors de la suppression.'
+      clear:
+        action_title: 'Effacer tout'
+        action_confirm: 'Effacer toutes vos recherches ?'
+        success: 'Recherches effacées.'
+        failure: 'Un problème est survenu lors de la suppression.'
+      title: 'Recherches sauvegardées'
+      need_login: 'Veuillez vous connecter pour accéder à vos recherches.'
+      no_searches: 'Vous n''avez sauvegardé aucune recherche.'
+      list_title: 'Vos recherches'
+      delete: 'Supprimer'
+
+    search_history:
+      clear:
+        action_title: 'Effacer l''historique'
+        action_confirm: 'Effacer votre historique de recherche ?'
+        success: 'Votre historique de recherche a bien été effacé.'
+        failure: 'Un problème est survenu lors de la suppression.'
+      title: 'Historique de recherche'
+      no_history: 'Il n''y a aucun historique en cours.'
+      recent: 'Dernières recherches'
+      forget: 'Supprimer'
+      save: 'Sauvegarder'
+
+    tools:
+      title: 'Outils'
+      cite: 'Citer'
+      endnote: 'Exporter vers EndNote ou Zotero'
+      refworks: 'Exporter vers Refworks'
+      email: 'Envoyer'
+      sms: 'SMS'
+      librarian_view: 'Vue UNIMARC'
+      clear: 'Effacer'
+
+    citation:
+      mla: 'MLA'
+      apa: 'APA'
+      chicago: 'Chicago'
+
+    email:
+      form:
+        title: 'Envoyer par courriel'
+        to: 'Courriel :'
+        message: 'Message :'
+        submit: 'Envoyer'
+      text:
+        title: 'Titre : %{value}'
+        author: 'Auteur : %{value}'
+        format: 'Format : %{value}'
+        language: 'Langue : %{value}'
+        subject:
+          one: 'Une référence : %{title}'
+          other: 'Des références bibliographiques'
+        url: 'URL : %{url}'
+        message: 'Message : %{message}'
+
+      errors:
+        to:
+          invalid: 'Vous devez saisir une adresse correcte.'
+          blank: 'Vous devez saisir l''adresse de votre destinataire.'
+    sms:
+      form:
+        title: 'Envoyer par SMS'
+        to: 'Numéro de téléphone :'
+        carrier: 'Opérateur téléphonique'
+        carrier_prompt: 'Veuillez choisir votre opérateur'
+        submit: 'Envoyer'
+      text:
+        title: '%{value}'
+        author: ' by %{value}'
+        url: 'Lien : %{url}'
+      errors:
+        to:
+          invalid: 'Vous devez saisir un numéro de téléphone à 10 chiffres.'
+          blank: "Vous devez saisir le numéro de téléphone de votre destinataire."
+        carrier:
+          blank: 'Vous devez choisir un opérateur.'
+
+    back_to_search: 'Retour aux résultats'
+    back_to_bookmarks: 'Retour aux favoris'
+
+    search:
+      title: '%{application_name} Résultats de recherche'
+      search_results_header: 'Recherche'
+      search_results: 'Résultats de recherche'
+      errors:
+        request_error: "Pardon, nous n'avons pas compris votre recherche."
+        invalid_solr_id: "Vous avez demandé une notice qui n'existe pas."
+      per_page:
+        label: '%{count}<span class="hide-text"> par page</span>'
+        button_label: '%{count} par page'
+        title: 'Nombre de résultats à afficher par page'
+        submit: 'mettre à jour'
+      sort:
+        label: 'Trier par %{field}'
+        submit: 'trier les résultats'
+      form:
+        search_field:
+          label: 'dans'
+          title: 'Options de recherche'
+        post_label: 'pour'
+        q: 'Rechercher '
+        submit: 'Chercher'
+      pagination:
+        title: 'Navigation dans les résultats'
+      pagination_info:
+        no_items_found: 'Pas de %{entry_name}'
+        single_item_found: '<b>1</b> de <b>1</b> sur <b>1</b>'
+        pages:
+          one: '<b>%{start_num}</b> - <b>%{end_num}</b> sur <b>%{total_num}</b>'
+          other: '<b>%{start_num}</b> - <b>%{end_num}</b> sur <b>%{total_num}</b>'
+      entry_pagination_info:
+        one: '<b>1 sur 1</b>'
+        other: '<b>%{current}</b> sur <b>%{total}</b>'
+      documents:
+        counter: '%{counter}. '
+      folder:
+        select: 'Panier'
+        unselect: 'Panier'
+      facets:
+        title: 'Limiter votre recherche'
+        sort:
+          count: 'Du + au - fréquent'
+          index: 'Tri de A à Z'
+        count: '%{number}'
+        more: 'plus »'
+        selected:
+          remove: '[ X ]'
+      filters:
+        title: 'Vous avez demandé :'
+        label: '%{label}:'
+        remove:
+          value: 'Supprimer la restriction %{value}'
+          label_value: 'Supprimer la restriction %{label}: %{value}'
+      librarian_view:
+        title: 'Vue UNIMARC'
+        leader: 'LABEL %{leader}'
+      start_over: 'Accueil'
+      show:
+        title: '%{document_title} - %{application_name}'
+      rss_feed: 'RSS pour les résultats'
+      atom_feed: 'Atom pour les résultats'
+      fields:
+        default: 'Keyword'
+
+    entry_name:
+      default: 'résultat'
+
+    did_you_mean: 'Essayez peut-être avec : %{options}'

data/gemfiles/rails3.gemfile

@@ -0,0 +1,9 @@
+source 'http://rubygems.org'
+
+# Please see blacklight.gemspec for dependency information.
+gemspec :path=>"../"
+
+gem 'simplecov', :platform => :mri
+gem 'simplecov-rcov', :platform => :mri
+
+gem 'rails', "~> 3.2"

data/gemfiles/rails4.gemfile

@@ -0,0 +1,10 @@
+source 'http://rubygems.org'
+
+# Please see blacklight.gemspec for dependency information.
+gemspec :path=>"../"
+
+gem 'simplecov', :platform => :mri
+gem 'simplecov-rcov', :platform => :mri
+
+gem 'rails', '4.0.0.beta1'
+gem 'sass-rails', '4.0.0.beta1'

data/lib/blacklight/catalog.rb

@@ -276,20 +276,24 @@ module Blacklight::Catalog
        
     # when solr (RSolr) throws an error (RSolr::RequestError), this method is executed.
     def rsolr_request_error(exception)
-      if Rails.env == "development"
+      
+      if Rails.env.development?
         raise exception # Rails own code will catch and give usual Rails error page with stack trace
       else
+
         flash_notice = I18n.t('blacklight.search.errors.request_error')
-        # Set the notice flag if the flash[:notice] is already set to the error that we are setting.
-        # This is intended to stop the redirect loop error
-        notice = flash[:notice] if flash[:notice] == flash_notice
-        logger.error exception
-        unless notice
-          flash[:notice] = flash_notice
-          redirect_to root_path, :status => 500
-        else
-          render :file => "#{Rails.root}/public/500.html", :status => 500
+
+        # If there are errors coming from the index page, we want to trap those sensibly
+
+        if flash[:notice] == flash_notice
+          logger.error "Cowardly aborting rsolr_request_error exception handling, because we redirected to a page that raises another exception"
+          raise exception
         end
+
+        logger.error exception
+
+        flash[:notice] = flash_notice 
+        redirect_to root_path
       end
     end
     
@@ -304,4 +308,12 @@ module Blacklight::Catalog
         render "index", :status => 404
       end
     end
+
+    def blacklight_solr
+      @solr ||=  RSolr.connect(blacklight_solr_config)
+    end
+
+    def blacklight_solr_config
+      Blacklight.solr_config
+    end
 end