geoblacklight 2.2.0 → 3.0.1

data/lib/geoblacklight/bounding_box.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   ##
   # Transforms and parses a bounding box for various formats

data/lib/geoblacklight/catalog_helper_override.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module CatalogHelperOverride
     ##
@@ -5,7 +6,7 @@ module Geoblacklight
     # @param [Symbol] symbol of field to be removed
     # @param [Hash] request parameters
     def remove_spatial_filter_group(field, source_params = params)
-      p = source_params.dup
+      p = source_params.dup.to_h
       p.delete(field.to_s)
       p
     end

data/lib/geoblacklight/constants.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   # Module to declare application level constants and lookup hash
   module Constants

data/lib/geoblacklight/download.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   class Download
     def initialize(document, options = {})
@@ -14,7 +15,7 @@ module Geoblacklight
     end
 
     def self.file_path
-      Settings.DOWNLOAD_PATH || "#{Rails.root}/tmp/cache/downloads"
+      Settings.DOWNLOAD_PATH || Rails.root.join('tmp', 'cache', 'downloads')
     end
 
     def file_path_and_name
@@ -55,8 +56,8 @@ module Geoblacklight
 
     ##
     # Initiates download from a remote source url using the `request_params`.
-    # Will catch Faraday::Error::ConnectionFailed and
-    # Faraday::Error::TimeoutError
+    # Will catch Faraday::ConnectionFailed and
+    # Faraday::TimeoutError
     # @return [Faraday::Request] returns a Faraday::Request object
     def initiate_download
       conn = Faraday.new(url: url)
@@ -65,11 +66,11 @@ module Geoblacklight
         request.options.timeout = timeout
         request.options.open_timeout = timeout
       end
-    rescue Faraday::Error::ConnectionFailed
+    rescue Faraday::ConnectionFailed
       raise Geoblacklight::Exceptions::ExternalDownloadFailed,
             message: 'Download connection failed',
             url: conn.url_prefix.to_s
-    rescue Faraday::Error::TimeoutError
+    rescue Faraday::TimeoutError
       raise Geoblacklight::Exceptions::ExternalDownloadFailed,
             message: 'Download timed out',
             url: conn.url_prefix.to_s

data/lib/geoblacklight/download/geojson_download.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   class GeojsonDownload < Geoblacklight::Download
     GEOJSON_DOWNLOAD_PARAMS = {

data/lib/geoblacklight/download/geotiff_download.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   class GeotiffDownload < Geoblacklight::Download
     GEOTIFF_DOWNLOAD_PARAMS = {

data/lib/geoblacklight/download/hgl_download.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   class HglDownload < Geoblacklight::Download
     def initialize(document, email, options = {})

data/lib/geoblacklight/download/kmz_download.rb

@@ -1,9 +1,10 @@
+# frozen_string_literal: true
 module Geoblacklight
   class KmzDownload < Geoblacklight::Download
     KMZ_DOWNLOAD_PARAMS = { service: 'wms',
                             version: '1.1.0',
                             request: 'GetMap',
-                            srsName: 'EPSG:900913',
+                            srsName: 'EPSG:3857',
                             format: 'application/vnd.google-earth.kmz',
                             width: 2000, height: 2000 }.freeze
 

data/lib/geoblacklight/download/shapefile_download.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   class ShapefileDownload < Geoblacklight::Download
     SHAPEFILE_DOWNLOAD_PARAMS = { service: 'wfs',

data/lib/geoblacklight/engine.rb

@@ -1,11 +1,11 @@
+# frozen_string_literal: true
 require 'blacklight'
-require 'font-awesome-rails'
 require 'coderay'
 require 'config'
 require 'faraday'
 require 'faraday_middleware'
+require 'geoblacklight/version'
 require 'nokogiri'
-require 'geoblacklight-icons'
 require 'mime/types'
 require 'handlebars_assets'
 

data/lib/geoblacklight/exceptions.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module Exceptions
     class ExternalDownloadFailed < StandardError

data/lib/geoblacklight/geoblacklight_helper_behavior.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   ##
   # Adds helper behavior logic for GeoBlacklight, to used alongside

data/lib/geoblacklight/item_viewer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   class ItemViewer
     def initialize(references)

data/lib/geoblacklight/metadata.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module Metadata
     ##

data/lib/geoblacklight/metadata/base.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module Metadata
     ##
@@ -54,10 +55,10 @@ module Geoblacklight
           return response.body unless response.nil? || response.status == 404
           Geoblacklight.logger.error "Could not reach #{@reference.endpoint}"
           ''
-        rescue Faraday::Error::ConnectionFailed => error
+        rescue Faraday::ConnectionFailed => error
           Geoblacklight.logger.error error.inspect
           ''
-        rescue Faraday::Error::TimeoutError => error
+        rescue Faraday::TimeoutError => error
           Geoblacklight.logger.error error.inspect
           ''
         rescue OpenSSL::SSL::SSLError => error

data/lib/geoblacklight/metadata/fgdc.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module Metadata
     class Fgdc < Base

data/lib/geoblacklight/metadata/html.rb

@@ -1,9 +1,9 @@
+# frozen_string_literal: true
 module Geoblacklight
   module Metadata
     class Html < Base
       def transform
-        ActionController::Base.helpers.content_tag(:iframe,
-                                                   '',
+        ActionController::Base.helpers.tag.iframe('',
                                                    src: @reference.endpoint,
                                                    scrolling: 'yes',
                                                    frameborder: 0)

data/lib/geoblacklight/metadata/iso19139.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module Metadata
     class Iso19139 < Base

data/lib/geoblacklight/metadata_transformer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'geo_combine'
 
 module Geoblacklight

data/lib/geoblacklight/metadata_transformer/base.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module MetadataTransformer
     ##

data/lib/geoblacklight/metadata_transformer/fgdc.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module MetadataTransformer
     ##

data/lib/geoblacklight/metadata_transformer/iso19139.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module MetadataTransformer
     ##

data/lib/geoblacklight/reference.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   ##
   # Parses an array of dct_references to create useful reference information

data/lib/geoblacklight/references.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   # References is a geoblacklight-schema dct:references parser
   class References
@@ -43,7 +44,7 @@ module Geoblacklight
     # Preferred download (should be a file download)
     # @return [Hash, nil]
     def preferred_download
-      return file_download unless download.blank?
+      return file_download if download.present?
     end
 
     ##
@@ -100,9 +101,12 @@ module Geoblacklight
     # present
     # @return (see #downloads_by_format)
     def vector_download_formats
-      { shapefile: wfs.to_hash,
+      return unless wms.present? && wfs.present?
+      {
+        shapefile: wfs.to_hash,
         kmz: wms.to_hash,
-        geojson: wfs.to_hash } if wms.present? && wfs.present?
+        geojson: wfs.to_hash
+      }
     end
 
     ##

data/lib/geoblacklight/relation/ancestors.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module Relation
     class Ancestors
@@ -8,7 +9,7 @@ module Geoblacklight
 
       def create_search_params
         { fq: ["{!join from=#{Settings.FIELDS.SOURCE} to=layer_slug_s}layer_slug_s:#{@search_id}"],
-          fl: [Settings.FIELDS.TITLE, 'layer_slug_s'] }
+          fl: [Settings.FIELDS.TITLE, 'layer_slug_s', Settings.FIELDS.GEOM_TYPE] }
       end
 
       def execute_query

data/lib/geoblacklight/relation/descendants.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module Relation
     class Descendants
@@ -8,7 +9,7 @@ module Geoblacklight
 
       def create_search_params
         { fq: "#{Settings.FIELDS.SOURCE}:#{@search_id}",
-          fl: [Settings.FIELDS.TITLE, 'layer_slug_s'] }
+          fl: [Settings.FIELDS.TITLE, 'layer_slug_s', Settings.FIELDS.GEOM_TYPE] }
       end
 
       def execute_query

data/lib/geoblacklight/relation/relation_response.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module Relation
     class RelationResponse
@@ -17,7 +18,7 @@ module Geoblacklight
       end
 
       def empty?
-        !(ancestors['numFound'] > 0 || descendants['numFound'] > 0)
+        !(ancestors['numFound'].positive? || descendants['numFound'].positive?)
       end
     end
   end

data/lib/geoblacklight/routes.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module Routes
     require 'geoblacklight/routes/exportable'

data/lib/geoblacklight/routes/downloadable.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module Routes
     class Downloadable

data/lib/geoblacklight/routes/exportable.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module Routes
     class Exportable

data/lib/geoblacklight/routes/wms.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module Routes
     class Wms

data/lib/geoblacklight/version.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
-  VERSION = '2.2.0'.freeze
+  VERSION = '3.0.1'
 end

data/lib/geoblacklight/view_helper_override.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   module ViewHelperOverride
     include CatalogHelperOverride
@@ -26,7 +27,7 @@ module Geoblacklight
 
     def render_constraints_filters(localized_params = params)
       content = super(localized_params)
-      localized_params = localized_params.to_unsafe_h unless localized_params.is_a?(Hash)
+      localized_params = localized_params.to_unsafe_h if localized_params.respond_to?(:to_unsafe_h)
 
       if localized_params[:bbox]
         path = search_action_path(remove_spatial_filter_group(:bbox, localized_params))

data/lib/geoblacklight/wms_layer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   class WmsLayer
     def initialize(params)
@@ -24,10 +25,10 @@ module Geoblacklight
         request.options.timeout = Settings.TIMEOUT_WMS
         request.options.open_timeout = Settings.TIMEOUT_WMS
       end
-    rescue Faraday::Error::ConnectionFailed => error
+    rescue Faraday::ConnectionFailed => error
       Geoblacklight.logger.error error.inspect
       { error: error.inspect }
-    rescue Faraday::Error::TimeoutError => error
+    rescue Faraday::TimeoutError => error
       Geoblacklight.logger.error error.inspect
       { error: error.inspect }
     end

data/lib/geoblacklight/wms_layer/feature_info_response.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Geoblacklight
   class FeatureInfoResponse
     def initialize(response)

data/lib/tasks/geoblacklight.rake

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'rails/generators'
 require 'generators/geoblacklight/install_generator'
 
@@ -7,15 +8,15 @@ namespace :geoblacklight do
     require 'solr_wrapper'
     SolrWrapper.wrap(port: '8983') do |solr|
       solr.with_collection(name: 'blacklight-core', dir: File.join(File.expand_path('../../', File.dirname(__FILE__)), 'solr', 'conf')) do
-          puts "\nSolr server running: http://localhost:#{solr.port}/solr/#/blacklight-core"
-          puts "\n^C to stop"
-          puts ' '
-          begin
-            Rake::Task['geoblacklight:solr:seed'].invoke
-            system "bundle exec rails s #{args[:rails_server_args]}"
-          rescue Interrupt
-            puts 'Shutting down...'
-          end
+        puts "\nSolr server running: http://localhost:#{solr.port}/solr/#/blacklight-core"
+        puts "\n^C to stop"
+        puts ' '
+        begin
+          Rake::Task['geoblacklight:solr:seed'].invoke
+          system "bundle exec rails s #{args[:rails_server_args]}"
+        rescue Interrupt
+          puts 'Shutting down...'
+        end
       end
     end
   end
@@ -30,7 +31,7 @@ namespace :geoblacklight do
         puts ' '
         begin
           Rake::Task['geoblacklight:solr:seed'].invoke
-          system "foreman start"
+          system 'foreman start'
         rescue Interrupt
           puts 'Shutting down...'
         end
@@ -39,23 +40,23 @@ namespace :geoblacklight do
   end
 
   namespace :index do
-    desc "Put sample data into solr"
+    desc 'Put sample data into solr'
     task :seed => :environment do
       docs = Dir['spec/fixtures/solr_documents/*.json'].map { |f| JSON.parse File.read(f) }.flatten
       Blacklight.default_index.connection.add docs
       Blacklight.default_index.connection.commit
     end
 
-    desc "Ingests a GeoHydra transformed.json"
+    desc 'Ingests a GeoHydra transformed.json'
     task :ingest_all => :environment do
-      docs = JSON::parse(File.read("#{Rails.root}/tmp/transformed.json"))
+      docs = JSON.parse(File.read(Rails.root.join('tmp', 'transformed.json')))
       docs.each do |doc|
         Blacklight.default_index.connection.add doc
         Blacklight.default_index.connection.commit
       end
     end
 
-    desc "Ingests a directory of geoblacklight.json files"
+    desc 'Ingests a directory of geoblacklight.json files'
     task :ingest, [:directory] => :environment do |_t, args|
       args.with_defaults(directory: 'data')
       Dir.glob(File.join(args[:directory], '**', 'geoblacklight.json')).each do |fn|
@@ -66,7 +67,7 @@ namespace :geoblacklight do
           puts "Failed to ingest #{fn}: #{e.inspect}"
         end
       end
-      puts "Committing changes to Solr"
+      puts 'Committing changes to Solr'
       Blacklight.default_index.connection.commit
     end
   end
@@ -74,33 +75,33 @@ namespace :geoblacklight do
   namespace :downloads do
     desc 'Delete all cached downloads'
     task delete: :environment do
-      FileUtils.rm_rf Dir.glob("#{Rails.root}/tmp/cache/downloads/*")
+      FileUtils.rm_rf Dir.glob(Rails.root.join('tmp', 'cache', 'downloads', '*'))
     end
     desc 'Create download directory'
     task mkdir: :environment do
-      FileUtils.mkdir_p Dir.glob("#{Rails.root}/tmp/cache/downloads")
+      FileUtils.mkdir_p(Rails.root.join('tmp', 'cache', 'downloads'), verbose: true)
     end
     desc 'Precaches a download'
-    task :precache, [:doc_id, :download_type, :timeout] => [:environment] do |t, args|
-      begin
-        fail 'Please supply required arguments [document_id, download_type and timeout]' unless args[:doc_id] && args[:download_type] && args[:timeout]
-        document = Geoblacklight::SolrDocument.find(args[:doc_id])
-        fail Blacklight::Exceptions::RecordNotFound if document[:layer_slug_s] != args[:doc_id]
-        download = "Geoblacklight::#{args[:download_type].capitalize}Download"
-                   .constantize.new(document, timeout: args[:timeout].to_i)
-        download.get
-        Rails.logger.info "Successfully downloaded #{download.file_name}"
-        Rails.logger.info "#{Geoblacklight::ShapefileDownload.file_path}"
-      rescue Geoblacklight::Exceptions::ExternalDownloadFailed => error
-        Rails.logger.error error.message + ' ' + error.url
-      rescue NameError
-        Rails.logger.error "Could not find that download type \"#{args[:download_type]}\""
+    task :precache, [:doc_id, :download_type, :timeout] => [:environment] do |_t, args|
+      unless args[:doc_id] && args[:download_type] && args[:timeout]
+        fail 'Please supply required arguments [document_id, download_type and timeout]'
       end
+      document = Geoblacklight::SolrDocument.find(args[:doc_id])
+      fail Blacklight::Exceptions::RecordNotFound if document[:layer_slug_s] != args[:doc_id]
+      download = "Geoblacklight::#{args[:download_type].capitalize}Download"
+                 .constantize.new(document, timeout: args[:timeout].to_i)
+      download.get
+      Rails.logger.info "Successfully downloaded #{download.file_name}"
+      Rails.logger.info Geoblacklight::ShapefileDownload.file_path.to_s
+    rescue Geoblacklight::Exceptions::ExternalDownloadFailed => error
+      Rails.logger.error error.message + ' ' + error.url
+    rescue NameError
+      Rails.logger.error "Could not find that download type \"#{args[:download_type]}\""
     end
   end
 
   namespace :solr do
-    desc "Put sample data into solr"
+    desc 'Put sample data into solr'
     task :seed => :environment do
       Rake::Task['geoblacklight:index:seed'].invoke
     end