blacklight-marc 6.1.1 → 7.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 2dad28dda5b571d3d111370ddab6cdbd03bab70b
-  data.tar.gz: 0177360550a4b9b7387790c51dd543b8c1396c4a
+SHA256:
+  metadata.gz: 432d7a660db81b2bbeca3baec8b83caba9a5a774cc3d0e81aba21f9ed14299dd
+  data.tar.gz: e86995f1d734859fa0c498bc84620fa7e0452ccd48b21b78b4a2be0cc880ef06
 SHA512:
-  metadata.gz: 82744f0a7fb2e5a2688753211de0a5fa449caca6682d7ae78eada10819828d0e0e67cd9b380339d9c1f7ba53a231dff1618f3a19618f4b67b0c01c410378dfee
-  data.tar.gz: 283589fa380fe66a421a93a6354b9555b965ac72091dd06fd777535679a45d5fe58375cad0482ac7e38a9b41646e8bab7b381a2b4fb6dcd12b0203bccf6a35c8
+  metadata.gz: a088b5bc21a4ff7325e552b07ef618e5ed60a09b74ba409277b14dea661616bd6a4999d25f090505ec46644a489f4651afce800a63c47d5f9e0b09ca5c432b49
+  data.tar.gz: 71712bc65b078fa2ad1c32b75e5aa2cbbf98e1ac3f0a4bc3784ff13991656ced7076fe729955d607fd9679c4bc618051ad639a15b184435692b641d46e57e7e7

data/.travis.yml

@@ -4,8 +4,14 @@ sudo: false
 notifications:
   email: false
 
-rvm:
-  - 2.3.1
+matrix:
+  include:
+    - rvm: 2.7.1
+      env: "RAILS_VERSION=6.0.3.2"
+    - rvm: 2.6.6
+      env: "RAILS_VERSION=6.0.3.2"
+    - rvm: 2.6.6
+      env: "RAILS_VERSION=5.2.4.3"
 
 before_install:
   - gem install bundler
@@ -17,5 +23,5 @@ notifications:
 
 global_env:
   - NOKOGIRI_USE_SYSTEM_LIBRARIES=true
-  
-jdk: oraclejdk8
+
+jdk: openjdk11

data/Gemfile

@@ -1,5 +1,10 @@
 source 'https://rubygems.org'
 
+git_source(:github) do |repo_name|
+  repo_name = "#{repo_name}/#{repo_name}" unless repo_name.include?('/')
+  "https://github.com/#{repo_name}.git"
+end
+
 # Please see blacklight_marc.gemspec for dependency information.
 gemspec path: File.expand_path('..', __FILE__)
 
@@ -28,6 +33,11 @@ else
   end
 
   case ENV['RAILS_VERSION']
+  when /^6.0/
+    gem 'sass-rails', '>= 6'
+    gem 'webpacker', '~> 4.0'
+  when /^5.[12]/
+    gem 'sass-rails', '~> 5.0'
   when /^4.2/
     gem 'responders', '~> 2.0'
     gem 'sass-rails', '>= 5.0'
@@ -37,3 +47,5 @@ else
   end
 end
 # END ENGINE_CART BLOCK
+
+eval_gemfile File.expand_path("spec/test_app_templates/Gemfile.extra", File.dirname(__FILE__))

data/README.md

@@ -1,9 +1,15 @@
-[![Build Status](https://travis-ci.org/projectblacklight/blacklight_marc.png?branch=master)](https://travis-ci.org/projectblacklight/blacklight_marc)
+[![Build Status](https://travis-ci.org/projectblacklight/blacklight-marc.png?branch=master)](https://travis-ci.org/projectblacklight/blacklight-marc)
 
 # Blacklight::Marc
 
 MARC-specific enhancements for [Blacklight](https://github.com/projectblacklight/blacklight)
 
+## Features
+* Rake task `solr:marc:index` to import .mrc files using Traject and app/models/marc_indexer
+* "Librarian View" at `catalog/:id/librarian_view`
+* Export records to refworks and endnote
+* Blacklight::Solr::Document mixins for exporting and transforming MARC data from a stored Solr field
+
 ## Installation
 
 Add this line to your application's Gemfile:
@@ -18,6 +24,27 @@ After running the blacklight generator, run the blacklight_marc generator:
 
     $ rails generate blacklight:marc:install
 
+
+## How does it work?
+This generator injects an include into CatalogController (parent class of BookmarksController):
+
+```ruby
+include Blacklight::Marc::Catalog
+```
+
+and the following code is generated into SolrDocument:
+
+```ruby
+use_extension( Blacklight::Solr::Document::Marc) do |document|
+  document.key?( :marc_display  )
+end
+```
+
+This means, the extension will be applied on the instance if the backing hash
+has a field called `marc_display`.
+
+
+## Indexing
 The generator will create a model called MarcIndexer. This model can be customized by modifying
 the field configurations in its initializer as a Traject 2 indexer; new indexing behaviors can
 be added to it via mixins or inline methods. Two example mixins are provided:
@@ -35,12 +62,6 @@ MarcIndexer
   end
 end
 ```
-## Features
-* Rake task `solr:marc:index` to import .mrc files using Traject and app/models/marc_indexer
-* Librarian view at `catalog/:id/librarian_view`
-* Export records to refworks and endnote
-* Blacklight::Solr::Document mixins for exporting and transforming MARC data from a stored Solr field
-
 
 ## Documentation, Information and Support
 

data/Rakefile

@@ -2,8 +2,6 @@ require "bundler/gem_tasks"
 require 'rspec/core/rake_task'
 require 'engine_cart/rake_task'
 
-EngineCart.fingerprint_proc = EngineCart.rails_fingerprint_proc
-
 RSpec::Core::RakeTask.new(:spec)
 
 task :default => [:ci]

data/app/assets/stylesheets/blacklight_marc.scss

@@ -12,6 +12,9 @@ $marc_item_separator: 1px solid $text-muted !default;
 #marc_view
 {
 	text-align:left;
+	/* https://meyerweb.com/eric/thoughts/2010/02/12/fixed-monospace-sizing/ */
+	font-family: "Courier New", monospace, serif;
+	font-size: 1em;
 
   .field
   {
@@ -22,7 +25,7 @@ $marc_item_separator: 1px solid $text-muted !default;
   .ind1, .ind2
   {
     display:inline;
-    padding:0 $padding-xs-horizontal;
+    padding: 0 ($spacer / 2);
   }
 
   .subfields
@@ -35,8 +38,3 @@ $marc_item_separator: 1px solid $text-muted !default;
     float:left;
   }
 }
-
-/* refworks uses a form; we need to give it some additional padding to make it look like a link */
-#refworks-form .btn {
-  padding: $nav-link-padding;
-}

data/app/helpers/blacklight_marc_helper.rb

@@ -1,18 +1,18 @@
 module BlacklightMarcHelper
 
   # This method should move to BlacklightMarc in Blacklight 6.x
-  def refworks_export_url params = {}
+  def refworks_export_url params = {}, *_
     "http://www.refworks.com/express/expressimport.asp?vendor=#{CGI.escape(params[:vendor] || application_name)}&filter=#{CGI.escape(params[:filter] || "MARC Format")}&encoding=65001" + (("&url=#{CGI.escape(params[:url])}" if params[:url]) || "")
   end
 
-  def refworks_solr_document_path opts = {}
+  def refworks_solr_document_path opts = {}, *_
     if opts[:id]
       refworks_export_url(url: solr_document_url(opts[:id], format: :refworks_marc_txt))
     end
   end
 
   # For exporting a single endnote document. (endnote_catalog_path is defined by blacklight-marc and it is used for multiple document export)
-  def single_endnote_catalog_path opts = {}
+  def single_endnote_catalog_path opts = {}, *_
     solr_document_path(opts.merge(format: 'endnote'))
   end
 
@@ -33,7 +33,8 @@ module BlacklightMarcHelper
     val = ''
     documents.each do |doc|
       if doc.exports_as? :endnote
-        val += doc.export_as(:endnote) + "\n"
+        endnote = doc.export_as(:endnote)
+        val += "#{endnote}\n" if endnote
       end
     end
     val

data/app/models/concerns/blacklight/solr/document/marc.rb

@@ -6,7 +6,7 @@
 # then provides various transformations/exports of that Marc via the included
 # Blacklight::Solr::Document::MarcExport module.
 #
-# This extension would normally be registered using 
+# This extension would normally be registered using
 # Blacklight::Solr::Document#use_extension.  eg:
 #
 # SolrDocument.use_extension( Blacklight::Solr::Document::Marc ) { |document| my_logic_for_document_has_marc?( document ) }
@@ -20,15 +20,15 @@ require 'marc'
 
 module Blacklight::Solr::Document::Marc
 
-  include Blacklight::Solr::Document::MarcExport # All our export_as stuff based on to_marc. 
-  
+  include Blacklight::Solr::Document::MarcExport # All our export_as stuff based on to_marc.
+
   class UnsupportedMarcFormatType < RuntimeError; end
-    
+
   def self.extended(document)
-    # Register our exportable formats, we inherit these from MarcExport    
+    # Register our exportable formats, we inherit these from MarcExport
     Blacklight::Solr::Document::MarcExport.register_export_formats( document )
   end
-  
+
   # ruby-marc object
   def to_marc
     @_ruby_marc_obj ||= load_marc
@@ -50,29 +50,30 @@ module Blacklight::Solr::Document::Marc
       return MARC::Record.new_from_hash( JSON.parse( fetch(_marc_source_field) ) )
     else
       raise UnsupportedMarcFormatType.new("Only marcxml, marc21, and json are supported, this documents format is #{_marc_format_type} and the current extension parameters are #{self.class.extension_parameters.inspect}")
-    end      
+    end
   rescue Exception => e
     raise e if e.is_a? UnsupportedMarcFormatType
 
     Rails.logger.error("Blacklight failed to parse MARC record. Exception was: #{e}")
+    nil
   end
-  
+
   def marc_record_from_marcxml
     MARC::XMLReader.new(StringIO.new( fetch(_marc_source_field) )).to_a.first
   end
-  
+
   def _marc_helper
     @_marc_helper ||= (
       Blacklight::Marc::Document.new fetch(_marc_source_field), _marc_format_type )
   end
 
-  def _marc_source_field    
+  def _marc_source_field
     self.class.extension_parameters[:marc_source_field]
   end
 
   def _marc_format_type
         #TODO: Raise if not present
-    self.class.extension_parameters[:marc_format_type]    
+    self.class.extension_parameters[:marc_format_type]
   end
-  
+
 end

data/app/models/concerns/blacklight/solr/document/marc_export.rb

@@ -4,13 +4,13 @@
 # it for your own custom Blacklight document Marc extension too -- just
 # include this module in any document extension (or any other class)
 # that provides a #to_marc returning a ruby-marc object.  This module will add
-# in export_as translation methods for a variety of formats. 
+# in export_as translation methods for a variety of formats.
 module Blacklight::Solr::Document::MarcExport
-  
+
   def self.register_export_formats(document)
     document.will_export_as(:xml)
     document.will_export_as(:marc, "application/marc")
-    # marcxml content type: 
+    # marcxml content type:
     # http://tools.ietf.org/html/draft-denenberg-mods-etc-media-types-00
     document.will_export_as(:marcxml, "application/marcxml+xml")
     document.will_export_as(:openurl_ctx_kev, "application/x-openurl-ctx-kev")
@@ -27,20 +27,20 @@ module Blacklight::Solr::Document::MarcExport
     to_marc.to_xml.to_s
   end
   alias_method :export_as_xml, :export_as_marcxml
-  
-  
+
+
   # TODO This exporting as formatted citation thing should be re-thought
   # redesigned at some point to be more general purpose, but this
   # is in-line with what we had before, but at least now attached
-  # to the document extension where it belongs. 
+  # to the document extension where it belongs.
   def export_as_apa_citation_txt
     apa_citation( to_marc )
   end
-  
+
   def export_as_mla_citation_txt
     mla_citation( to_marc )
   end
-  
+
   def export_as_chicago_citation_txt
     chicago_citation( to_marc )
   end
@@ -52,8 +52,8 @@ module Blacklight::Solr::Document::MarcExport
   # more sensibly. The "format" argument was in the old marc.marc.to_zotero
   # call, but didn't neccesarily do what it thought it did anyway. Left in
   # for now for backwards compatibilty, but should be replaced by
-  # just ruby OpenURL. 
-  def export_as_openurl_ctx_kev(format = nil)  
+  # just ruby OpenURL.
+  def export_as_openurl_ctx_kev(format = nil)
     title = to_marc.find{|field| field.tag == '245'}
     author = to_marc.find{|field| field.tag == '100'}
     corp_author = to_marc.find{|field| field.tag == '110'}
@@ -100,11 +100,13 @@ module Blacklight::Solr::Document::MarcExport
 
   # This format used to be called 'refworks', which wasn't really
   # accurate, sounds more like 'refworks tagged format'. Which this
-  # is not, it's instead some weird under-documented Refworks 
+  # is not, it's instead some weird under-documented Refworks
   # proprietary marc-ish in text/plain format. See
-  # http://robotlibrarian.billdueber.com/sending-marcish-data-to-refworks/
+  # http://robotlibrarian.billdueber.com/2009/05/sending-marcish-data-to-refworks/
   def export_as_refworks_marc_txt
-    fields = to_marc.find_all { |f| ('000'..'999') === f.tag }
+    marc_obj = to_marc
+    return unless marc_obj
+    fields = marc_obj.find_all { |f| ('000'..'999') === f.tag }
     text = "LEADER #{to_marc.leader}"
     fields.each do |field|
     unless ["940","999"].include?(field.tag)
@@ -125,9 +127,9 @@ module Blacklight::Solr::Document::MarcExport
     # it seems to want C form normalization, although RefWorks support
     # couldn't tell me that. -jrochkind
     text = ActiveSupport::Multibyte::Unicode.normalize(text, :c)
-    
+
     return text
-  end 
+  end
 
   # Endnote Import Format. See the EndNote User Guide at:
   # http://www.endnote.com/support/enx3man-terms-win.asp
@@ -138,7 +140,7 @@ module Blacklight::Solr::Document::MarcExport
   # endnote import format; the %0 is likely to be entirely illegal, the
   # rest of the data is barely correct but messy. TODO, a new version of this,
   # or better yet just an export_as_ris instead, which will be more general
-  # purpose. 
+  # purpose.
   def export_as_endnote()
     end_note_format = {
       "%A" => "100.a",
@@ -154,10 +156,12 @@ module Blacklight::Solr::Document::MarcExport
       "%7" => "250.a"
     }
     marc_obj = to_marc
+    return unless marc_obj
+
     # TODO. This should be rewritten to guess
     # from actual Marc instead, probably.
     format_str = 'Generic'
-    
+
     text = ''
     text << "%0 #{ format_str }\n"
     # If there is some reliable way of getting the language of a record we can add it here
@@ -170,7 +174,7 @@ module Blacklight::Solr::Document::MarcExport
       else
         second_value = []
       end
-      
+
       if marc_obj[first_value[0].to_s]
         marc_obj.find_all{|f| (first_value[0].to_s) === f.tag}.each do |field|
           if field[first_value[1]].to_s or field[second_value[1]].to_s
@@ -190,11 +194,11 @@ module Blacklight::Solr::Document::MarcExport
   end
 
   protected
-  
+
   # Main method for defining chicago style citation.  If we don't end up converting to using a citation formatting service
   # we should make this receive a semantic document and not MARC so we can use this with other formats.
   def chicago_citation(marc)
-    authors = get_all_authors(marc)    
+    authors = get_all_authors(marc)
     author_text = ""
     unless authors[:primary_authors].blank?
       if authors[:primary_authors].length > 10
@@ -226,7 +230,7 @@ module Blacklight::Solr::Document::MarcExport
             author_text << "and #{name_reverse(author)}."
           else
             author_text << "#{name_reverse(author)}, "
-          end 
+          end
         end
       else
         author_text << authors[:primary_authors].first
@@ -242,7 +246,7 @@ module Blacklight::Solr::Document::MarcExport
       authors[:compilers].each do |compiler|
         temp_authors << [compiler, "comp."]
       end
-      
+
       unless temp_authors.blank?
         if temp_authors.length > 10
           temp_authors.each_with_index do |author,index|
@@ -281,18 +285,18 @@ module Blacklight::Solr::Document::MarcExport
         section_title << "."
       end
     end
-    
+
     if !authors[:primary_authors].blank? and (!authors[:translators].blank? or !authors[:editors].blank? or !authors[:compilers].blank?)
         additional_title << "Translated by #{authors[:translators].collect{|name| name_reverse(name)}.join(" and ")}. " unless authors[:translators].blank?
         additional_title << "Edited by #{authors[:editors].collect{|name| name_reverse(name)}.join(" and ")}. " unless authors[:editors].blank?
         additional_title << "Compiled by #{authors[:compilers].collect{|name| name_reverse(name)}.join(" and ")}. " unless authors[:compilers].blank?
     end
-    
+
     edition = ""
     edition << setup_edition(marc) unless setup_edition(marc).nil?
-    
+
     pub_info = ""
-    if marc["260"] and (marc["260"]["a"] or marc["260"]["b"]) 
+    if marc["260"] and (marc["260"]["a"] or marc["260"]["b"])
       pub_info << clean_end_punctuation(marc["260"]["a"]).strip if marc["260"]["a"]
       pub_info << ": #{clean_end_punctuation(marc["260"]["b"]).strip}" if marc["260"]["b"]
       pub_info << ", #{setup_pub_date(marc)}" if marc["260"]["c"]
@@ -301,7 +305,7 @@ module Blacklight::Solr::Document::MarcExport
     elsif marc["502"] and (marc["502"]["b"] or marc["502"]["c"] or marc["502"]["d"]) #sometimes the dissertation note is encoded in pieces in the $b $c and $d sub fields instead of lumped into the $a
       pub_info << "#{marc["502"]["b"]}, #{marc["502"]["c"]}, #{clean_end_punctuation(marc["502"]["d"])}"
     end
-    
+
     citation = ""
     citation << "#{author_text} " unless author_text.blank?
     citation << "<i>#{title}.</i> " unless title.blank?
@@ -311,13 +315,13 @@ module Blacklight::Solr::Document::MarcExport
     citation << "#{pub_info}." unless pub_info.blank?
     citation
   end
-  
-  
-  
+
+
+
   def mla_citation(record)
     text = ''
     authors_final = []
-    
+
     #setup formatted author list
     authors = get_author_list(record)
 
@@ -351,10 +355,10 @@ module Blacklight::Solr::Document::MarcExport
     # Edition
     edition_data = setup_edition(record)
     text += edition_data + " " unless edition_data.nil?
-    
+
     # Publication
     text += setup_pub_info(record) + ", " unless setup_pub_info(record).nil?
-    
+
     # Get Pub Date
     text += setup_pub_date(record) unless setup_pub_date(record).nil?
     if text[-1,1] != "."
@@ -367,7 +371,7 @@ module Blacklight::Solr::Document::MarcExport
     text = ''
     authors_list = []
     authors_list_final = []
-    
+
     #setup formatted author list
     authors = get_author_list(record)
     authors.each do |l|
@@ -392,15 +396,15 @@ module Blacklight::Solr::Document::MarcExport
     end
     # Get Pub Date
     text += "(" + setup_pub_date(record) + "). " unless setup_pub_date(record).nil?
-    
+
     # setup title info
     title = setup_title_info(record)
     text += "<i>" + title + "</i> " unless title.nil?
-    
+
     # Edition
     edition_data = setup_edition(record)
     text += edition_data + " " unless edition_data.nil?
-    
+
     # Publisher info
     text += setup_pub_info(record) unless setup_pub_info(record).nil?
     unless text.blank?
@@ -451,7 +455,7 @@ module Blacklight::Solr::Document::MarcExport
     end
     new_text.join(" ")
   end
-  
+
   # This will replace the mla_citation_title method with a better understanding of how MLA and Chicago citation titles are formatted.
   # This method will take in a string and capitalize all of the non-prepositions.
   def citation_title(title_text)
@@ -482,18 +486,18 @@ module Blacklight::Solr::Document::MarcExport
       end
       text += b_title_info unless b_title_info.nil?
     end
-    
+
     return nil if text.strip.blank?
     clean_end_punctuation(text.strip) + "."
-    
+
   end
-  
+
   def clean_end_punctuation(text)
     if [".",",",":",";","/"].include? text[-1,1]
       return text[0,text.length-1]
     end
     text
-  end  
+  end
 
   def setup_edition(record)
     edition_field = record.find{|f| f.tag == '250'}
@@ -503,9 +507,9 @@ module Blacklight::Solr::Document::MarcExport
       return nil
     else
       return edition_data
-    end    
+    end
   end
-  
+
   def get_author_list(record)
     author_list = []
     authors_primary = record.find{|f| f.tag == '100'}
@@ -517,11 +521,11 @@ module Blacklight::Solr::Document::MarcExport
         author_list.push(clean_end_punctuation(l.find{|s| s.code == 'a'}.value)) unless l.find{|s| s.code == 'a'}.value.nil?
       end
     end
-    
+
     author_list.uniq!
     author_list
   end
-  
+
   # This is a replacement method for the get_author_list method.  This new method will break authors out into primary authors, translators, editors, and compilers
   def get_all_authors(record)
     translator_code = "trl"; editor_code = "edt"; compiler_code = "com"
@@ -547,7 +551,7 @@ module Blacklight::Solr::Document::MarcExport
     end
     {:primary_authors => primary_authors, :translators => translators, :editors => editors, :compilers => compilers}
   end
-  
+
   def abbreviate_name(name)
     name_parts = name.split(", ")
     first_name_parts = name_parts.last.split(" ")
@@ -562,6 +566,5 @@ module Blacklight::Solr::Document::MarcExport
     return name unless name =~ /,/
     temp_name = name.split(", ")
     return temp_name.last + " " + temp_name.first
-  end 
-  
-end
+  end
+end