hyrax 5.2.0 → 5.3.0

data/app/utils/hyrax/required_data_seeders/flexible_profile_seeder.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+module Hyrax
+  module RequiredDataSeeders
+    # This class was created for use in rake tasks and db/seeds.rb.  It generates
+    # collections that are required for all Hyrax applications.
+    #
+    # Seeders of required data are non-destructive.  If the data already exists,
+    # it will not be replaced.
+    class FlexibleProfileSeeder
+      class << self
+        attr_accessor :logger
+
+        def generate_seeds(logger: Logger.new(STDOUT))
+          @logger = logger
+
+          logger.info("Adding required collections...")
+
+          flexible_schema = Hyrax::FlexibleSchema.first_or_create do |f|
+            f.profile = YAML.safe_load_file(Rails.root.join('config', 'metadata_profiles', 'm3_profile.yaml'))
+          end
+
+          logger.info "   #{flexible_schema.title} -- FOUND OR CREATED"
+        end
+      end
+    end
+  end
+end

data/app/validators/hyrax/controlled_vocabulary_validator.rb

@@ -0,0 +1,64 @@
+# frozen_string_literal: true
+module Hyrax
+  ##
+  # Validates that controlled vocabulary properties contain only active
+  # terms from their corresponding local QA authority.
+  #
+  # Properties are matched to authorities dynamically via
+  # +Qa::Authorities::Local.subauthorities+, using the singularized
+  # authority name to match property names on the change set.
+  #
+  # Only covers local authorities (file-based and table-based).
+  # Remote authorities (e.g. Geonames) are out of scope.
+  class ControlledVocabularyValidator < ActiveModel::Validator
+    def validate(record)
+      return unless Flipflop.validate_local_controlled_vocabulary?
+
+      active_terms_by_property(record).each do |property, terms|
+        values = Array.wrap(record.public_send(property)).reject(&:blank?)
+        next if values.empty?
+
+        invalid = values.reject { |v| terms.include?(v) }
+        invalid.each do |v|
+          record.errors.add(property, "#{property.to_s.humanize} contains unrecognized value: #{v}")
+        end
+      end
+    end
+
+    private
+
+    ##
+    # @example
+    #   # { "license" => ["http://creativecommons.org/licenses/by/4.0/", ...],
+    #   #   "resource_type" => ["Article", "Book", ...] }
+    #
+    # @return [Hash{String => Array<String>}]
+    def active_terms_by_property(record)
+      authorities = Qa::Authorities::Local.subauthorities
+
+      # { "license" => "licenses", "resource_type" => "resource_types", ... }
+      property_to_authority = {}.tap do |hash|
+        authorities.each { |name| hash[name.singularize] = name }
+      end
+
+      properties = record.fields.keys.map(&:to_s)
+
+      {}.tap do |result|
+        (properties & property_to_authority.keys).each do |property|
+          authority = Qa::Authorities::Local.subauthority_for(property_to_authority[property])
+          terms = authority.all
+          next if terms.empty?
+
+          result[property] = terms.filter_map { |term| term[:id] if active?(term) }
+        end
+      end
+    end
+
+    ##
+    # Terms without an +active+ field (e.g. resource_types) are
+    # treated as active, matching QA's +FileBasedAuthority#all+ behavior.
+    def active?(term)
+      term.fetch(:active, true) != false
+    end
+  end
+end

data/app/validators/hyrax/redirect_validator.rb

@@ -0,0 +1,128 @@
+# frozen_string_literal: true
+
+module Hyrax
+  # Validates the `redirects` attribute on a work or collection. Runs only
+  # when the redirects feature is fully enabled (config + Flipflop).
+  #
+  # See documentation/redirects.md for the validation rules.
+  class RedirectValidator < ActiveModel::EachValidator
+    # Allow-list per RFC 3986 path chars: unreserved + sub-delims + ":" + "@" + "/"
+    # plus pct-encoded "%XX". Rejects quotes, backslashes, angle brackets, pipes,
+    # braces, control chars, whitespace, "?", and "#" — anything Rails routing
+    # would refuse or that would be unsafe to round-trip through a URL.
+    PATH_CHAR = %r{[A-Za-z0-9\-._~!$&'()*+,;=:@/]|%[0-9A-Fa-f]{2}}.freeze
+    PATH_FORMAT = %r{\A/(?:#{PATH_CHAR})+\z}.freeze
+
+    # Override `validate` so we can short-circuit before ActiveModel calls
+    # `record.read_attribute_for_validation`, which would crash on records
+    # that don't have the `redirects` attribute.
+    #
+    # Two guards, each catching a distinct case:
+    # 1. config + Flipflop — the runtime feature gate. Skips validation when
+    #    the feature isn't actively in use.
+    # 2. record_supports_redirects? — the structural gate. Catches the case
+    #    where both feature gates are on but the property wasn't added
+    def validate(record)
+      return unless Hyrax.config.redirects_active?
+      return unless record_supports_redirects?(record)
+      super
+    end
+
+    def validate_each(record, attribute, entries)
+      return if entries.blank?
+
+      validate_each_entry(record, attribute, entries)
+      validate_intra_record_uniqueness(record, attribute, entries)
+      validate_global_uniqueness(record, attribute, entries)
+      validate_at_most_one_display_url(record, attribute, entries)
+    end
+
+    private
+
+    # Entries are persisted hashes, but the form-render path may pass
+    # Hyrax::Redirect presenter instances. Both shapes are acceptable.
+    def path_for(entry)
+      return entry.path if entry.respond_to?(:path)
+      return entry['path'] || entry[:path] if entry.respond_to?(:[])
+      nil
+    end
+
+    # Returns nil when the flag is absent, otherwise a real boolean.
+    def display_url_for(entry)
+      return ActiveModel::Type::Boolean.new.cast(entry.is_display_url) if entry.respond_to?(:is_display_url)
+      if entry.respond_to?(:[])
+        return ActiveModel::Type::Boolean.new.cast(entry['is_display_url']) if entry.key?('is_display_url')
+        return ActiveModel::Type::Boolean.new.cast(entry[:is_display_url]) if entry.key?(:is_display_url)
+      end
+      nil
+    end
+
+    def validate_each_entry(record, attribute, entries)
+      entries.each do |entry|
+        path = path_for(entry)
+        if path.blank?
+          record.errors.add(attribute, message_for(:blank))
+        elsif !PATH_FORMAT.match?(path)
+          record.errors.add(attribute, message_for(:invalid_format, path: path))
+        elsif reserved_prefix?(path)
+          record.errors.add(attribute, message_for(:reserved_prefix, path: path))
+        end
+      end
+    end
+
+    def reserved_prefix?(path)
+      normalized = normalized_path(path)
+      Hyrax.config.reserved_redirect_prefixes.any? do |prefix|
+        normalized == prefix || normalized.start_with?("#{prefix}/")
+      end
+    end
+
+    def validate_intra_record_uniqueness(record, attribute, entries)
+      grouped = entries.each_with_object({}) do |entry, acc|
+        path = path_for(entry)
+        next if path.blank?
+        canonical = normalized_path(path)
+        next if canonical.blank?
+        (acc[canonical] ||= []) << path
+      end
+      grouped.each_value do |paths|
+        next unless paths.size > 1
+        record.errors.add(attribute, message_for(:intra_record_duplicate, path: paths.first))
+      end
+    end
+
+    def validate_global_uniqueness(record, attribute, entries)
+      except_id = record.try(:id)
+      entries.each do |entry|
+        path = path_for(entry)
+        next if path.blank?
+        next unless Hyrax::RedirectsLookup.taken?(normalized_path(path), except_id: except_id)
+        record.errors.add(attribute, message_for(:already_taken, path: path))
+      end
+    end
+
+    def normalized_path(path)
+      Hyrax::RedirectPathNormalizer.call(path)
+    end
+
+    def validate_at_most_one_display_url(record, attribute, entries)
+      display_url_count = entries.count { |entry| display_url_for(entry) }
+      return if display_url_count <= 1
+      record.errors.add(attribute, message_for(:multiple_display_url))
+    end
+
+    def message_for(key, **interpolations)
+      I18n.t(key, scope: 'errors.messages.redirect', **interpolations)
+    end
+
+    # Reform forms wrap the underlying resource and use method_missing to
+    # forward attribute reads. We can't just call `record.respond_to?(:redirects)`
+    # because Reform's method_missing returns truthy for missing methods.
+    # Probe the actual underlying resource through the form's `__getobj__`
+    # (Reform's accessor for the wrapped object) when present.
+    def record_supports_redirects?(record)
+      target = record.respond_to?(:__getobj__) ? record.__getobj__ : record
+      target.respond_to?(:redirects)
+    end
+  end
+end

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

@@ -1,10 +1,10 @@
 <% model = document.hydra_model %>
 <div class="col-md-3">
   <% if model == Hyrax::PcdmCollection || model < Hyrax::PcdmCollection %>
-    <%= document_presenter(document)&.thumbnail&.thumbnail_tag({ alt: document.title_or_label }, { suppress_link: true }) %>
+    <%= document_presenter(document)&.thumbnail&.thumbnail_tag({ alt: thumbnail_alt_text_for(document, block_name: 'default_collection_image_text') }, { suppress_link: true }) %>
   <% else %>
     <div class="list-thumbnail">
-      <%= document_presenter(document)&.thumbnail&.thumbnail_tag(alt: document.title_or_label) %>
+      <%= document_presenter(document)&.thumbnail&.thumbnail_tag(alt: thumbnail_alt_text_for(document)) %>
     </div>
   <% end %>
 </div>

data/app/views/collections/edit_fields/_schema_version.html.erb

@@ -0,0 +1 @@
+<%= render('shared/schema_version', f: f) if f.object.flexible? %>

data/app/views/hyrax/admin/admin_sets/_form_metadata.html.erb

@@ -1,2 +1,24 @@
-<%= f.input :title %>
-<%= f.input :description, as: :text %>
+<div id="base-terms">
+  <% f.object.primary_terms.each do |term| %>
+    <% next if term == :contexts %>
+    <%= f.input term %>
+  <% end %>
+</div>
+<% if Hyrax.config.flexible? && Hyrax.config.admin_set_class.new.respond_to?(:contexts) %>
+  <% blank_option = f.object.model.new_record ? false : t('hyrax.admin.admin_sets.form.contexts.none') %>
+  <%= f.input :contexts, collection: Hyrax::FlexibleSchema.last.context_select, include_blank: blank_option, input_html: { multiple: true } %>
+<% end %>
+<% if f.object.display_additional_fields? %>
+  <%= link_to t('hyrax.collection.form.additional_fields'),
+          '#extended-terms',
+          class: 'btn btn-secondary additional-fields',
+          data: { toggle: 'collapse' },
+          role: "button",
+          'aria-expanded'=> "false",
+          'aria-controls'=> "extended-terms" %>
+  <div id="extended-terms" class='collapse'>
+    <% f.object.secondary_terms.each do |term| %>
+      <%= f.input term %>
+    <% end %>
+  </div>
+<% end %>

data/app/views/hyrax/admin/admin_sets/_show_document_list_row.html.erb

@@ -9,7 +9,7 @@
   <td>
     <div class="media">
       <%= link_to [main_app, document], "class" => "media-left", "aria-hidden" => "true" do %>
-        <%= document_presenter(document)&.thumbnail&.thumbnail_tag({ class: "d-none d-md-block file_listing_thumbnail" }, { suppress_link: true }) %>
+        <%= document_presenter(document)&.thumbnail&.thumbnail_tag({ class: "d-none d-md-block file_listing_thumbnail", alt: thumbnail_alt_text_for(document) }, { suppress_link: true }) %>
       <% end %>
       <div class="media-body">
         <p class="media-heading">

data/app/views/hyrax/admin/admin_sets/show.html.erb

@@ -11,7 +11,7 @@
         <div class="col-md-2">
           <% if has_thumbnail? @presenter.solr_document %>
             <div class="document-thumbnail">
-              <%= render_thumbnail_tag(@presenter.solr_document, {}, { suppress_link: true }) %>
+              <%= render_thumbnail_tag(@presenter.solr_document, { alt: thumbnail_alt_text_for(@presenter.solr_document, block_name: 'default_collection_image_text') }, { suppress_link: true }) %>
             </div>
           <% else %>
             <span class="fa fa-sitemap collection-icon-search" aria-hidden="true"></span>

data/app/views/hyrax/admin/analytics/_analytics_error.html.erb

@@ -0,0 +1,42 @@
+<%# frozen_string_literal: true %>
+<% if @analytics_error %>
+  <div class="alert alert-info" role="alert">
+    <div class="d-flex">
+      <div class="mr-3">
+        <i class="fa fa-info-circle fa-2x text-info" aria-hidden="true"></i>
+      </div>
+      <div class="flex-fill">
+        <h5 class="alert-heading mb-2">Analytics Setup Required</h5>
+        <p class="mb-2"><%= @analytics_error[:message] %></p>
+
+        <% if @analytics_error[:troubleshooting_steps].present? %>
+          <div class="mt-3">
+            <% @analytics_error[:troubleshooting_steps].each_with_index do |step, index| %>
+              <div class="mb-2">
+                <span class="badge badge-primary mr-2"><%= index + 1 %></span>
+                <%= raw(step) %>
+              </div>
+            <% end %>
+          </div>
+        <% end %>
+
+        <div class="mt-3">
+          <% if @analytics_error[:documentation_url].present? %>
+            <a href="<%= @analytics_error[:documentation_url] %>" target="_blank" rel="noopener noreferrer" class="btn btn-primary btn-sm mr-2">
+              Setup Guide
+            </a>
+          <% end %>
+          <button class="btn btn-outline-secondary btn-sm" type="button" data-toggle="collapse" data-target="#technicalDetails" aria-expanded="false" aria-controls="technicalDetails">
+            Technical Details
+          </button>
+        </div>
+
+        <div class="collapse mt-3" id="technicalDetails">
+          <div class="alert alert-light">
+            <small><%= @analytics_error[:details] %></small>
+          </div>
+        </div>
+      </div>
+    </div>
+  </div>
+<% end %>

data/app/views/hyrax/admin/analytics/collection_reports/index.html.erb

@@ -7,11 +7,12 @@
 <div class="row">
   <div class="col-sm-12">
     <div class="collection-reports">
-
-<% if Hyrax.config.analytics_reporting? %>
       <div class="card">
-        <div class="card-header"><%= t('.repo_summary') %> <b><%= pluralize(Collection.count, "collection") %></b>, <%= t('.repo_summary_2') %> <b><%= @pageviews.all %> <%= t('.views') %></b> <%= t('.and') %> <b><%= @downloads.all %> <%= t('.downloads') %></b>.</div>
-        <div class="card-body">
+        <%= render 'hyrax/admin/analytics/analytics_error' %>
+        <% unless @analytics_error %>
+          <% if Hyrax.config.analytics_reporting? %>
+            <div class="card-header"><%= t('.repo_summary') %> <b><%= pluralize(Collection.count, "collection") %></b>, <%= t('.repo_summary_2') %> <b><%= @pageviews.all %> <%= t('.views') %></b> <%= t('.and') %> <b><%= @downloads.all %> <%= t('.downloads') %></b>.</div>
+            <div class="card-body">
 
                <div class="row text-center">
                 <div class="nav nav-pills btn-group" role="group">
@@ -58,13 +59,14 @@
             </div>
           </div>
           <p class="text-center"><%= t('.report_generated_on') %> <b><%= Time.current %></b>.</p>
-        </div>
       </div>
-  <% else %>
-    <div class="alert alert-warning" role="alert">
-      <%= t('.enable_analytics') %>
     </div>
+    <% else %>
+      <div class="alert alert-warning" role="alert">
+        <%= t('.enable_analytics') %>
+      </div>
+    <% end %>
   <% end %>
-    </div>
   </div>
 </div>
+</div>

data/app/views/hyrax/admin/analytics/work_reports/index.html.erb

@@ -9,69 +9,72 @@
     <div class="work-reports">
 
     <div class="card">
-    <% if Hyrax.config.analytics_reporting? %>
-      <% if current_user.ability.admin? %>
-        <div class="card-header"><b><%= @works_count %> <%= t('.works') %></b> <%= t('.repo_summary') %> <b><%= @pageviews.all if @pageviews %> <%= t('.views') %></b> <%= t('.and') %> <b><%= @downloads.all if @downloads %> <%= t('.downloads') %></b>.</div>
-      <% else %>
-        <div class="card-header"><%= t('.access_to') %> <b><%= @works_count %> <%= t('.works') %></b>.</div>
-      <% end %>
-        <div class="card-body">
-              <% if current_user.ability.admin? %>
-              <div class="row text-center">
-                <div class="nav nav-pills btn-group" role="group">
-                    <% if params[:start_date] %>
-                     <a button id='summary-btn' class="btn btn-secondary nav-link" type="button" href="#summary" data-toggle="pill">
-                    <% else %>
-                     <a button id='summary-btn' class="btn btn-primary nav-link" type="button" href="#summary" data-toggle="pill">
-                    <% end %>
-                        <%= t('.summary') %>
-                     </a>
-                      <a button id='monthly-btn' class="btn btn-secondary nav-link" type="button" href="#monthly" data-toggle="pill">
-                        <%= t('.monthly') %>
-                      </a>
-                    <% if params[:start_date] %>
-                      <a button id='range-btn' class="btn btn-primary nav-link" type="button" href="#custom-range" data-toggle="pill">
+      <%= render 'hyrax/admin/analytics/analytics_error' %>
+      <% unless @analytics_error %>
+        <% if Hyrax.config.analytics_reporting? %>
+          <% if current_user.ability.admin? %>
+            <div class="card-header"><b><%= @works_count %> <%= t('.works') %></b> <%= t('.repo_summary') %> <b><%= @pageviews.all if @pageviews %> <%= t('.views') %></b> <%= t('.and') %> <b><%= @downloads.all if @downloads %> <%= t('.downloads') %></b>.</div>
+          <% else %>
+            <div class="card-header"><%= t('.access_to') %> <b><%= @works_count %> <%= t('.works') %></b>.</div>
+          <% end %>
+            <div class="card-body">
+                  <% if current_user.ability.admin? %>
+                  <div class="row text-center">
+                    <div class="nav nav-pills btn-group" role="group">
+                        <% if params[:start_date] %>
+                         <a button id='summary-btn' class="btn btn-secondary nav-link" type="button" href="#summary" data-toggle="pill">
+                        <% else %>
+                         <a button id='summary-btn' class="btn btn-primary nav-link" type="button" href="#summary" data-toggle="pill">
+                        <% end %>
+                            <%= t('.summary') %>
+                         </a>
+                          <a button id='monthly-btn' class="btn btn-secondary nav-link" type="button" href="#monthly" data-toggle="pill">
+                            <%= t('.monthly') %>
+                          </a>
+                        <% if params[:start_date] %>
+                          <a button id='range-btn' class="btn btn-primary nav-link" type="button" href="#custom-range" data-toggle="pill">
+                        <% else %>
+                          <a button id='range-btn' class="btn btn-secondary nav-link" type="button" href="#custom-range" data-toggle="pill">
+                        <% end %>
+                            <%= t('.custom_range') %>
+                          </a>
+                    </div>
+                  </div>
+                  <br>
+                  <div class="tab-content">
+                  <% if params[:start_date] %>
+                    <div class="tab-pane" id="summary">
+                  <% else %>
+                    <div class="tab-pane active" id="summary">
+                  <% end %>
+                      <%= render "summary" %>
+                    </div>
+                    <div class="tab-pane" id="monthly">
+                      <%= render "monthly_summary" %>
+                    </div>
+                     <% if params[:start_date] %>
+                       <div class="tab-pane active" id="custom-range">
                     <% else %>
-                      <a button id='range-btn' class="btn btn-secondary nav-link" type="button" href="#custom-range" data-toggle="pill">
+                      <div class="tab-pane" id="custom-range">
                     <% end %>
-                        <%= t('.custom_range') %>
-                      </a>
-                </div>
-              </div>
-              <br>
-              <div class="tab-content">
-              <% if params[:start_date] %>
-                <div class="tab-pane" id="summary">
-              <% else %>
-                <div class="tab-pane active" id="summary">
-              <% end %>
-                  <%= render "summary" %>
-                </div>
-                <div class="tab-pane" id="monthly">
-                  <%= render "monthly_summary" %>
+                       <%= render "hyrax/admin/analytics/date_range_form", redirect_path: hyrax.admin_analytics_work_reports_path %>
+                      <%= render "custom_range" %>
+                     </div>
+                  </div>
+                  <% end %>
                 </div>
-                 <% if params[:start_date] %>
-                   <div class="tab-pane active" id="custom-range">
-                <% else %>
-                  <div class="tab-pane" id="custom-range">
-                <% end %>
-                   <%= render "hyrax/admin/analytics/date_range_form", redirect_path: hyrax.admin_analytics_work_reports_path %>
-                  <%= render "custom_range" %>
-                 </div>
               </div>
-              <% end %>
-            </div>
-          </div>
-          <p class="text-center"><%= t('.report_generated_on') %> <b><%= Time.zone.today.strftime("%B %d, %Y") %></b></p>
+              <p class="text-center"><%= t('.report_generated_on') %> <b><%= Time.zone.today.strftime("%B %d, %Y") %></b></p>
 
-        </div>
+            </div>
 
-      </div>
-      <% else %>
-        <div class="alert alert-warning" role="alert">
-          <%= t('.enable_analytics') %>
-        </div>
-     <% end %>
+          </div>
+        <% else %>
+          <div class="alert alert-warning" role="alert">
+            <%= t('.enable_analytics') %>
+          </div>
+       <% end %>
+      <% end %>
     </div>
   </div>
 </div>

data/app/views/hyrax/admin/features/index.html.erb

@@ -32,7 +32,7 @@
                 <td class="status">
                   <span class="badge badge-<%= @feature_set.status(feature) -%>"><%= @feature_set.status(feature) -%></span>
                 </td>
-                <td class="name"><%= feature.name.humanize -%></td>
+                <td class="name"><%= feature.title || feature.name.humanize -%></td>
                 <td class="description"><%= feature.description -%></td>
 
                 <% @feature_set.strategies.each do |strategy| -%>

data/app/views/hyrax/admin/metadata_profiles/_import_modal.html.erb

@@ -0,0 +1,20 @@
+<div id="myModal" class="modal fade" tabindex="-1" role="dialog" aria-labelledby="exampleModalLabel" aria-hidden="true">
+  <div class="modal-dialog" role="document">
+    <%= form_tag(hyrax.import_admin_metadata_profiles_path, :multipart => true, :class => "modal-content") do %>
+      <div class="modal-header">
+        <h2 class="modal-title"><%= t(:'helpers.action.hyrax_metadata_profile.import') %></h2>
+        <button type="button" class="close" data-dismiss="modal" aria-label="<%= t('hyrax.dashboard.heading_actions.close') %>"><span aria-hidden="true">&times;</span></button>
+      </div>
+      <div class="modal-body">
+        <% unless @metadata_profiles.empty? %>
+          <p><%= t(:'helpers.action.hyrax_metadata_profile.hint') %></p>
+        <% end %>
+        <%= file_field_tag :file, class: 'form-control-file' %>
+      </div>
+      <div class="modal-footer">
+        <button type="button" class="btn btn-secondary" data-dismiss="modal"><%= t('hyrax.dashboard.heading_actions.close') %></button>
+        <%= submit_tag t(:'helpers.action.hyrax_metadata_profile.submit'), name: nil, class: "btn btn-primary" %>
+      </div>
+    <% end %>
+  </div>
+</div>

data/app/views/hyrax/admin/metadata_profiles/index.html.erb

@@ -0,0 +1,54 @@
+<% provide :page_title, t(:'hyrax.dashboard.metadata_profiles') %>
+
+<% provide :page_header do %>
+  <h1><span class="fa fa-table" aria-hidden="true"></span> <%= t(:'hyrax.admin.sidebar.metadata_profiles') %></h1>
+  <div class="pull-right">
+    <button type="button" class="btn btn-primary" data-toggle="modal" data-target="#myModal">
+      <span class="fa fa-upload" aria-hidden="true"></span>
+      <%= t(:'helpers.action.hyrax_metadata_profile.import_metadata_profile') %>
+    </button>
+  </div>
+<% end %>
+
+<div class="panel panel-default">
+  <div class="card-body">
+    <% if @metadata_profiles.present? %>
+      <div class="table-responsive">
+        <table class="table table-striped datatable">
+          <thead>
+            <tr>
+              <th scope="col">Schema Version</th>
+              <th scope="col">Profile Version</th>
+              <th scope="col">Profile Type</th>
+              <th scope="col">Created At</th>
+              <th scope="col">Actions</th>
+            </tr>
+          </thead>
+          <tbody>
+            <% @metadata_profiles.order("updated_at DESC").each do |hyrax_metadata_profile| %>
+              <tr>
+
+                <!-- TODO: fix show pgs -->
+                <td><%= hyrax_metadata_profile.schema_version %></td>
+                <td><%= hyrax_metadata_profile.id.to_f %></td>
+                <td><%= hyrax_metadata_profile.metadata_profile_type %></td>
+                <td><%= hyrax_metadata_profile.created_at.strftime("%b %d, %Y") %></td>
+                <th><%= link_to raw('<span class="fa fa-download"></span>'), hyrax.admin_metadata_profile_export_path(hyrax_metadata_profile), target: "_blank" %></th>
+              </tr>
+            <% end %>
+          </tbody>
+        </table>
+
+        <%= page_entries_info @metadata_profiles %><br />
+        <%= paginate(@metadata_profiles, param_name: :metadata_profile_entries_page) %>
+
+      </div>
+      <% else %>
+        <p>No Metadata Schema Profiles have been created.</p>
+      <% end %>
+  </div>
+</div>
+
+
+<!-- Modal -->
+<%= render 'import_modal' %>

data/app/views/hyrax/base/_attribute_rows.html.erb

@@ -1,21 +1,4 @@
-<%= presenter.attribute_to_html(:alternative_title, html_dl: true) %>
-<%= presenter.attribute_to_html(:abstract, html_dl: true) %>
-<%= presenter.attribute_to_html(:date_modified, label: t('hyrax.base.show.last_modified'), html_dl: true) %>
-<%= presenter.attribute_to_html(:creator, render_as: :faceted, html_dl: true) %>
-<%= presenter.attribute_to_html(:contributor, render_as: :faceted, html_dl: true) %>
-<%= presenter.attribute_to_html(:subject, render_as: :faceted, html_dl: true) %>
-<%= presenter.attribute_to_html(:publisher, render_as: :faceted, html_dl: true) %>
-<%= presenter.attribute_to_html(:bibliographic_citation, html_dl: true) %>
-<%= presenter.attribute_to_html(:language, render_as: :faceted, html_dl: true) %>
-<%= presenter.attribute_to_html(:identifier, render_as: :linked, search_field: 'identifier_tesim', html_dl: true) %>
-<%= presenter.attribute_to_html(:keyword, render_as: :faceted, html_dl: true) %>
-<%= presenter.attribute_to_html(:date_created, render_as: :linked, search_field: 'date_created_tesim', html_dl: true) %>
-<%= presenter.attribute_to_html(:based_near_label, html_dl: true) %>
-<%= presenter.attribute_to_html(:related_url, render_as: :external_link, html_dl: true) %>
-<%= presenter.attribute_to_html(:resource_type, render_as: :faceted, html_dl: true) %>
-<%= presenter.attribute_to_html(:source, html_dl: true) %>
-<%= presenter.attribute_to_html(:extent, html_dl: true) %>
-<%= presenter.attribute_to_html(:rights_statement, render_as: :rights_statement, html_dl: true) %>
-<%= presenter.attribute_to_html(:rights_notes, html_dl: true) %>
-<%= presenter.attribute_to_html(:access_right, html_dl: true) %>
-<%= presenter.attribute_to_html(:license, render_as: :license, html_dl: true) %>
+<% view_options_for(presenter).each do |field, options| %>
+  <% view_options = conform_options(field, options) %>
+  <%= presenter.attribute_to_html(conform_field(field, options), view_options) if field_visible?(view_options, presenter) %>
+<% end %>

data/app/views/hyrax/base/_citations.html.erb

@@ -1,4 +1,5 @@
 <div class="citations">
+    <%= render 'hyrax/shared/copy_permalink', presenter: presenter %>
     <% if Hyrax.config.citations? %>
       <%= link_to t(".citations"), hyrax.citations_work_path(presenter), id: 'citations', class: 'btn btn-secondary btn-sm mr-2' %>
     <% end %>

data/app/views/hyrax/base/_currently_shared.html.erb

@@ -5,7 +5,8 @@
 <table class="table table-bordered">
   <tr>
     <th><%= t('.table_title_user') %></th>
-    <th><div class="col-sm-10"><%= t('.table_title_access') %></div></th>
+    <th><%= t('.table_title_access') %></th>
+    <th><%= t('.table_title_action') %></th>
   </tr>
   <tr id="file_permissions">
     <td>

data/app/views/hyrax/base/_file_manager_thumbnail.html.erb

@@ -1 +1 @@
-<%= document_presenter(node)&.thumbnail&.thumbnail_tag({ class: 'thumbnail-inner mw-100', 'alt': "Thumbnail image for #{node.title[0]}" }, {}) %>
+<%= document_presenter(node)&.thumbnail&.thumbnail_tag({ class: 'thumbnail-inner mw-100', alt: node.alt_text_for_view }, {}) %>