blacklight-maps 0.5.2 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 0d727312e568bc1dde35d3d9ad7554630068b087
-  data.tar.gz: 7c787f41b27472dd2d8acf2fdbe17759e8d6cb75
+SHA256:
+  metadata.gz: a5f97f65f09264f6f7c17afda244ee952c15c01fccf072e60e89ec7401a15989
+  data.tar.gz: c00ba470859274d2e62cff117c875deea3c7aa4c419595e46b5087bbc30b4c99
 SHA512:
-  metadata.gz: dff809b08e8d1902b722fbc7c0bace9104143eef9f9c9a6df25af10d4bb8fbb6091722878053870a6a6eb19992b15f43033d0fce3959098b46b5cf6bffa777fb
-  data.tar.gz: 8693ef1963f5c98b21c3b334b407ca00a69eca9888697c8c6e6d1601fd3628f85cf474c51c68711b2185aa169876c0b90eb1790120b22a4b65777f7b28cb9f0a
+  metadata.gz: e27c8bc1357431f728a924d8ff6557ec6e9ea0d32962cb3c94eeaf0e0c5a8fff5a7eb7c7dc62f2c84c89f77ae1308521eef579c76d4ee01991b813cdbc76d5a6
+  data.tar.gz: 00134ab9767316aa17974520e3b8f8a050bbd7959c7b31d24913d791a7556028307b4266fa60d495e6f15c7adf1df0d9389914ac68e023fb4dadb88e6fbb7e37

data/.github/workflows/ruby.yml

@@ -0,0 +1,33 @@
+name: CI
+
+on:
+  push:
+    branches: [master]
+  pull_request:
+    branches: [master]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    name: test (ruby ${{ matrix.ruby }} / rails ${{ matrix.rails_version }} / blacklight ${{ matrix.blacklight_version }} ${{ matrix.additional_name }})
+    strategy:
+      matrix:
+        ruby: ["3.2", "3.3", "3.4"]
+        rails_version: ["7.1.5.1", "7.2.2"]
+        blacklight_version: ["~> 7.0"]
+        additional_engine_cart_rails_options: [""]
+        additional_name: [""]
+    env:
+      RAILS_VERSION: ${{ matrix.rails_version }}
+      BLACKLIGHT_VERSION: ${{ matrix.blacklight_version }}
+      ENGINE_CART_RAILS_OPTIONS: "--skip-action-cable ${{ matrix.additional_engine_cart_rails_options }}"
+    steps:
+      - uses: actions/checkout@v4
+      - name: Set up Ruby ${{ matrix.ruby }}
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+      - name: Install dependencies with Rails ${{ matrix.rails_version }}
+        run: bundle install
+      - name: Run tests
+        run: bundle exec rake

data/.gitignore

@@ -19,3 +19,5 @@ spec/internal
 jetty
 .DS_Store
 .idea/
+.rakeTasks
+.internal_test_app

data/.rubocop.yml

@@ -0,0 +1,103 @@
+plugins:
+  - rubocop-rspec
+  - rubocop-rails
+
+AllCops:
+  DisplayCopNames: true
+  TargetRubyVersion: 3.4
+  Exclude:
+    - "lib/generators/blacklight_maps/templates/**/*"
+    - "blacklight-maps.gemspec"
+
+# engine_cart block includes conditional, not duplication
+Bundler/DuplicatedGem:
+  Exclude:
+    - 'Gemfile'
+
+# engine_cart block is following default Rails order
+Bundler/OrderedGems:
+  Exclude:
+    - 'Gemfile'
+
+Layout/IndentationConsistency:
+  EnforcedStyle: normal
+
+Metrics/AbcSize:
+  Max: 20
+  Exclude:
+    - 'lib/blacklight/maps/maps_search_builder.rb'
+
+Metrics/BlockLength:
+  Exclude:
+    - "spec/**/*"
+
+Metrics/ClassLength:
+  Exclude:
+    - 'lib/blacklight/maps/export.rb'
+
+Layout/LineLength:
+  Max: 200
+  Exclude:
+    - 'lib/blacklight/maps/engine.rb'
+    - 'spec/**/*'
+
+Metrics/MethodLength:
+  Max: 15
+
+Naming/HeredocDelimiterNaming:
+  Enabled: false
+
+Naming/PredicateName:
+  ForbiddenPrefixes:
+    - is_
+
+Rails:
+  Enabled: true
+
+Rails/HelperInstanceVariable:
+  Enabled: false
+
+Rails/OutputSafety:
+  Enabled: false
+
+RSpec/AnyInstance:
+  Exclude:
+    - 'spec/system/initial_view_spec.rb'
+
+RSpec/BeforeAfterAll:
+  Enabled: false
+
+RSpec/DescribeClass:
+  Exclude:
+    - 'spec/system/*'
+
+RSpec/SpecFilePathFormat:
+  Exclude:
+    - 'spec/lib/blacklight/maps/*'
+
+RSpec/MessageSpies:
+  EnforcedStyle: receive
+
+RSpec/MultipleExpectations:
+  Max: 4
+
+RSpec/MultipleMemoizedHelpers:
+  Max: 10
+
+RSpec/NestedGroups:
+  Max: 5
+
+RSpec/PredicateMatcher:
+  Exclude:
+    - 'spec/lib/blacklight/maps/render_constraints_override_spec.rb'
+
+# https://github.com/rubocop-hq/rubocop/issues/6439
+Style/AccessModifierDeclarations:
+  Enabled: false
+
+Style/Documentation:
+  Enabled: false
+
+Style/SignalException:
+  Exclude:
+    - 'spec/**/*'

data/.solr_wrapper.yml

@@ -0,0 +1,8 @@
+# Place any default configuration for solr_wrapper here
+# you must first run 'rake engine_cart:generate' to create the test app
+# before running 'solr_wrapper' from project root
+# port: 8983
+collection:
+  dir: ./.internal_test_app/solr/conf/
+  name: blacklight-core
+version: 9.6.1

data/Gemfile

@@ -1,15 +1,45 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
-# Specify your gem's dependencies in blacklight-maps.gemspec
 gemspec
 
-group :test do
-  gem 'simplecov', require: false
-  gem 'coveralls', require: false
+group :development, :test do
+  gem 'coveralls_reborn', require: false
 end
 
-file = File.expand_path("Gemfile", ENV['ENGINE_CART_DESTINATION'] || ENV['RAILS_ROOT'] || File.expand_path("../spec/internal", __FILE__))
-if File.exists?(file)
-  puts "Loading #{file} ..." if $DEBUG # `ruby -d` or `bundle -v`
-  instance_eval File.read(file)
+# BEGIN ENGINE_CART BLOCK
+# engine_cart: 2.6.0
+# engine_cart stanza: 2.5.0
+# the below comes from engine_cart, a gem used to test this Rails engine gem in the context of a Rails app.
+file = File.expand_path('Gemfile', ENV['ENGINE_CART_DESTINATION'] || ENV['RAILS_ROOT'] || File.expand_path('.internal_test_app', File.dirname(__FILE__)))
+if File.exist?(file)
+  begin
+    eval_gemfile file
+  rescue Bundler::GemfileError => e
+    Bundler.ui.warn '[EngineCart] Skipping Rails application dependencies:'
+    Bundler.ui.warn e.message
+  end
+else
+  Bundler.ui.warn "[EngineCart] Unable to find test application dependencies in #{file}, using placeholder dependencies"
+
+  if ENV['RAILS_VERSION']
+    if ENV['RAILS_VERSION'] == 'edge'
+      gem 'rails', github: 'rails/rails'
+      ENV['ENGINE_CART_RAILS_OPTIONS'] = '--edge --skip-turbolinks'
+    else
+      gem 'rails', ENV['RAILS_VERSION']
+    end
+
+    case ENV['RAILS_VERSION']
+    when /^6.0/
+      gem 'sass-rails', '>= 6'
+      gem 'webpacker', '~> 4.0'
+    when /^5.[12]/
+      gem 'sass-rails', '~> 5.0'
+      gem 'sprockets', '~> 3.7'
+      gem 'thor', '~> 0.20'
+    end
+  end
 end
+# END ENGINE_CART BLOCK

data/README.md

@@ -1,6 +1,6 @@
 # Blacklight::Maps
 
-[![Build Status](https://travis-ci.org/projectblacklight/blacklight-maps.png?branch=master)](https://travis-ci.org/projectblacklight/blacklight-maps) | [![Coverage Status](https://coveralls.io/repos/projectblacklight/blacklight-maps/badge.svg?branch=master)](https://coveralls.io/r/projectblacklight/blacklight-maps?branch=master)
+![CI](https://github.com/projectblacklight/blacklight-maps/workflows/CI/badge.svg) | [![Coverage Status](https://coveralls.io/repos/projectblacklight/blacklight-maps/badge.svg?branch=master)](https://coveralls.io/r/projectblacklight/blacklight-maps?branch=master)
 
 Provides map views for Blacklight for items with geospatial coordinate (latitude/longitude) metadata.
 
@@ -53,11 +53,11 @@ Blacklight-Maps requires that your Solr index include at least one (but preferab
 
   ```
   # coordinates: lon lat or lat,lon
-  # bounding box: minX minY maxX maxY
-  coordinates_field: 
+  # bounding box: ENVELOPE(minX, maxX, maxY, minY)
+  coordinates_srpt: 
    - 78.96288 20.593684
    - 20.593684,78.96288
-   - 68.162386 6.7535159 97.395555 35.5044752       
+   - ENVELOPE(68.162386, 97.395555, 35.5044752, 6.7535159)
   ```
 
 2. An indexed, stored string field containing a properly-formatted [GeoJSON](http://geojson.org) feature object for a point or bounding box that includes the coordinates and (preferably) location name. This field can be multivalued.
@@ -74,7 +74,7 @@ Blacklight-Maps requires that your Solr index include at least one (but preferab
 3. An indexed, stored text or string field containing location names. This field can be multivalued.
 
   ```
-   placename_field: India
+   subject_geo_ssim: India
   ```
 
 ##### Why so complicated?
@@ -88,10 +88,10 @@ Blacklight-Maps can be used with either field type (#1 or #2), however to take a
 **Important:** If you are NOT using the geojson field (#2), you should create a `copyField` in your Solr schema.xml to copy the coordinates from the `location_rpt` field to a string field that is stored, indexed, and multivalued to allow for proper faceting of the coordinate values in the catalog#map and catalog#index views.
 
 ```
-  <!-- Solr4 location_rpt field for coordinates, shapes, etc. -->
-  <dynamicField name="geospatial" type="location_rpt" indexed="true" stored="true" multiValued="true" />
+  <!-- Solr location_rpt field for coordinates, shapes, etc. -->
+  <dynamicField name="*_srpt" type="location_rpt" indexed="true" stored="true" multiValued="true" />
   <!-- copy geospatial to string field for faceting -->
-  <copyField source="geospatial" dest="geospatial_facet" />
+  <copyField source="coordinates_srpt" dest="coordinates_ssim" />
 ```
 
 Support for additional field types may be added in the future.
@@ -112,9 +112,9 @@ Blacklight-Maps expects you to provide these configuration options:
     + `placename_field` = the name of the Solr field containing the location names
 + `coordinates_field` = the name of the Solr `location_rpt` type field containing geospatial coordinate data
 
-In addition, you must add the geospatial facet field to the list of facet fields:
+In addition, you must add the geospatial facet field to the list of facet fields in `app/controllers/catalog_controller.rb`, for example:
 ```ruby
-config.add_facet_field 'geojson_field', :limit => -2, :label => 'Coordinates', :show => false
+config.add_facet_field 'geojson_ssim', :limit => -2, :label => 'Coordinates', :show => false
 ```
 
 #### Optional
@@ -196,9 +196,11 @@ The table below indicates which versions of Blacklight Maps are compatible with
 
 Blacklight Maps version | works with Blacklight version
 ----------------------- | ---------------------
-0.5.0 | >= 6.1.0
-0.4.0 | >= 5.12.0 to < 6.*
-<= 0.3.3 | >= 5.1 to <= 5.11.2
+1.2.* | >= 7.35.0, < 8
+1.1.* | >= 7.8.0, < 8
+0.5.* | >= 6.1.0, < 7
+0.4.* | >= 5.12.0, < 6.*
+<= 0.3.3 | >= 5.1, <= 5.11.2
 
 ## Contributing
 

data/Rakefile

@@ -1,43 +1,46 @@
-require "bundler/gem_tasks"
+# frozen_string_literal: true
 
-ZIP_URL = "https://github.com/projectblacklight/blacklight-jetty/archive/v4.6.0.zip"
-APP_ROOT = File.dirname(__FILE__)
-
-require 'rspec/core/rake_task'
-require 'engine_cart/rake_task'
+begin
+  require 'bundler/setup'
+rescue LoadError
+  puts 'You must `gem install bundler` and `bundle install` to run rake tasks'
+end
 
-require 'jettywrapper'
+require 'rdoc/task'
+RDoc::Task.new(:rdoc) do |rdoc|
+  rdoc.rdoc_dir = 'rdoc'
+  rdoc.title    = 'BlacklightMaps'
+  rdoc.options << '--line-numbers'
+  rdoc.rdoc_files.include('README.rdoc')
+  rdoc.rdoc_files.include('lib/**/*.rb')
+end
 
-task default: :ci
+Bundler::GemHelper.install_tasks
 
-RSpec::Core::RakeTask.new(:spec)
+Rake::Task.define_task(:environment)
 
-desc "Load fixtures"
-task :fixtures => ['engine_cart:generate'] do
-  EngineCart.within_test_app do
-    system "rake blacklight_maps:solr:seed RAILS_ENV=test"
-  end
-end
+load 'lib/railties/blacklight_maps.rake'
 
-desc "Execute Continuous Integration build"
-task :ci => ['engine_cart:generate', 'jetty:clean', 'blacklight_maps:configure_jetty'] do
-
-  require 'jettywrapper'
-  jetty_params = Jettywrapper.load_config('test')
+task default: :ci
 
-  error = Jettywrapper.wrap(jetty_params) do
-    Rake::Task['fixtures'].invoke
-    Rake::Task['spec'].invoke
-  end
-  raise "test failures: #{error}" if error
-end
+require 'engine_cart/rake_task'
 
+require 'solr_wrapper'
 
-namespace :blacklight_maps do
-  desc "Copies the default SOLR config for the bundled Testing Server"
-  task :configure_jetty do
-    FileList['solr_conf/conf/*'].each do |f|  
-      cp("#{f}", 'jetty/solr/blacklight-core/conf/', :verbose => true)
+require 'rspec/core/rake_task'
+RSpec::Core::RakeTask.new
+
+require 'rubocop/rake_task'
+RuboCop::RakeTask.new(:rubocop)
+
+desc 'Run test suite'
+task ci: [:rubocop, 'engine_cart:generate'] do
+  SolrWrapper.wrap do |solr|
+    solr.with_collection do
+      within_test_app do
+        system 'RAILS_ENV=test rake blacklight_maps:index:seed'
+      end
+      Rake::Task['spec'].invoke
     end
   end
-end
+end

data/app/assets/images/blacklight/maps.svg

@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" viewBox="0 0 24 24"><path d="M16,12a5.11,5.11,0,0,0,.8-1.6ZM8,12l-.8-1.6A4.72,4.72,0,0,0,8,12Z"/><path d="M12,4A5,5,0,0,0,7,9a4.71,4.71,0,0,0,.1,1c0,.13.06.26.1.39L8,12l4,8,4-8,.8-1.6L16.9,10A4.71,4.71,0,0,0,17,9,5,5,0,0,0,12,4Zm0,7a2,2,0,1,1,1.73-1A2,2,0,0,1,12,11Z"/><path d="M7.1,10c0,.13.06.26.1.39L7,10Z"/><path d="M17,10l-.19.39L16.9,10Z"/></svg>

data/app/assets/javascripts/blacklight-maps/blacklight-maps-browse.js

@@ -5,8 +5,8 @@
 
     // Configure default options and those passed via the constructor options
     var options = $.extend({
-      tileurl : 'http://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png',
-      mapattribution : 'Map data &copy; <a href="http://openstreetmap.org">OpenStreetMap</a> contributors, <a href="http://creativecommons.org/licenses/by-sa/2.0/">CC-BY-SA</a>',
+      tileurl : 'https://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png',
+      mapattribution : 'Map data &copy; <a href="https://openstreetmap.org">OpenStreetMap</a> contributors, <a' + ' href="https://creativecommons.org/licenses/by-sa/4.0/">CC-BY-SA</a>',
       initialzoom: 2,
       singlemarkermode: true,
       searchcontrol: false,
@@ -21,7 +21,7 @@
     // Extend options from data-attributes
     $.extend(options, this.data());
 
-    var mapped_items = '<span class="mapped-count"><span class="badge">' + geojson_docs.features.length + '</span> location' + (geojson_docs.features.length !== 1 ? 's' : '') + ' mapped</span>';
+    var mapped_items = '<span class="mapped-count"><span class="badge badge-secondary">' + geojson_docs.features.length + '</span>' + ' location' + (geojson_docs.features.length !== 1 ? 's' : '') + ' mapped</span>';
 
     var mapped_caveat = '<span class="mapped-caveat">Only items with location data are shown below</span>';
 
@@ -31,12 +31,12 @@
 
     // Update page links with number of mapped items, disable sort, per_page, pagination
     if (sortAndPerPage.length) { // catalog#index and #map view
-      var page_links = sortAndPerPage.find('.page_links');
-      var result_count = page_links.find('.page_entries').find('strong').last().html();
-      page_links.html('<span class="page_entries"><strong>' + result_count + '</strong> items found</span>' + mapped_items + mapped_caveat);
+      var page_links = sortAndPerPage.find('.page-links');
+      var result_count = page_links.find('.page-entries').find('strong').last().html();
+      page_links.html('<span class="page-entries"><strong>' + result_count + '</strong> items found</span>' + mapped_items + mapped_caveat);
       sortAndPerPage.find('.dropdown-toggle').hide();
     } else { // catalog#show view
-      $(this.selector).before(mapped_items);
+        $(this).before(mapped_items);
     }
 
     // determine whether to use item location or result count in cluster icon display
@@ -119,7 +119,6 @@
           var container = L.DomUtil.create('div', 'leaflet-bar leaflet-control');
           this.link = L.DomUtil.create('a', 'leaflet-bar-part search-control', container);
           this.link.title = options.searchctrlcue;
-          this.icon = L.DomUtil.create('i', 'glyphicon glyphicon-search', this.link);
 
           L.DomEvent.addListener(this.link, 'click', _search);
 

data/app/assets/javascripts/blacklight-maps.js

@@ -1,7 +1,4 @@
-// for Blacklight.onLoad:
-//= require blacklight/core
-
 //= require leaflet
 //= require leaflet.markercluster
 
-//= require_tree .
+//= require_tree .

data/app/assets/stylesheets/blacklight_maps/blacklight-maps.scss

@@ -1,8 +1,7 @@
+@charset "UTF-8";
 /* Master manifest file for engine, so local app can require
  * this one file, but get all our files -- and local app
  * require does not need to change if we change file list. 
  */
 
-@charset "UTF-8";
-
 @import 'default';

data/app/assets/stylesheets/blacklight_maps/default.scss

@@ -1,6 +1,6 @@
 @import 'leaflet';
-@import 'leaflet.markercluster';
-@import 'leaflet.markercluster.default';
+@import 'MarkerCluster';
+@import 'MarkerCluster.Default';
 
 body.blacklight-catalog-map {
 
@@ -14,21 +14,17 @@ body.blacklight-catalog-map {
 
 }
 
-.view-icon-maps {
-  &:before { content: "\e135"; } 
-}
-
 #sortAndPerPage {
 
-  .page_links {
+  .page-links {
 
     .mapped-count {
-      margin-left: 7px;
+      margin-left: 10px;
       color: dimgray;
     }
 
     .mapped-caveat {
-      margin-left: 7px;
+      margin-left: 10px;
       font-size: 12px;
       color: darkgray;
     }
@@ -45,7 +41,7 @@ body.blacklight-catalog-map {
 
     & ~ div.record-padding {
 
-      div.pagination {
+      nav.pagination {
         display: none;
       }
 
@@ -65,6 +61,7 @@ body.blacklight-catalog-map {
 
 a.leaflet-bar-part.search-control {
   cursor: pointer;
+  &:before { content: "\1F50D"; }
 }
 
 /* Portrait tablet to landscape and desktop */