hyrax 4.0.0.beta1 → 4.0.0.beta2

data/app/services/hyrax/work_resource_query_service.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+module Hyrax
+  # Responsible for retrieving information based on the given work.
+  #
+  # @see ProxyDepositRequest
+  # @see SolrDocument
+  # @see Hyrax::SolrService
+  # @note This was extracted from the ProxyDepositRequest, which was coordinating lots of effort. It was also an ActiveRecord object that required lots of Fedora/Solr interactions.
+  class WorkResourceQueryService
+    # @param [String] id - The id of the work
+    def initialize(id:)
+      @id = id
+    end
+    attr_reader :id
+
+    # @return [Boolean] if the work has been deleted
+    def deleted_work?
+      Hyrax.query_service.find_by(id: id)
+      false
+    rescue Valkyrie::Persistence::ObjectNotFoundError
+      true
+    end
+
+    def work
+      # Need to ensure it is a work?
+      resource = Hyrax.query_service.find_by(id: id)
+      raise ModelMismatchError, "Expected work but got #{resource.class}" unless resource.work?
+      resource
+    end
+
+    def to_s
+      if deleted_work?
+        'work not found'
+      else
+        solr_doc.to_s
+      end
+    end
+
+    private
+
+    def solr_doc
+      @solr_doc ||= ::SolrDocument.new(Hyrax::SolrService.search_by_id(id))
+    end
+  end
+end

data/app/services/hyrax/workflow/action_taken_service.rb

@@ -35,12 +35,12 @@ module Hyrax
         return unless action.triggered_methods.any?
         success = action.triggered_methods.order(:weight).all? do |method|
           status = process_action(method.service_name)
-          Rails.logger.debug("Result of #{method.service_name} is #{status}")
+          Hyrax.logger.debug("Result of #{method.service_name} is #{status}")
           status
         end
 
         return save_target if success
-        Rails.logger.error "Not all workflow methods were successful, so not saving (#{target.id})"
+        Hyrax.logger.error "Not all workflow methods were successful, so not saving (#{target.id})"
         false
       end
 
@@ -63,11 +63,11 @@ module Hyrax
         klass = begin
                   class_name.constantize
                 rescue NameError
-                  Rails.logger.error "Unable to find '#{class_name}', so not running workflow callback"
+                  Hyrax.logger.error "Unable to find '#{class_name}', so not running workflow callback"
                   return nil
                 end
         return klass if klass.respond_to?(:call)
-        Rails.logger.error "Expected '#{class_name}' to respond to 'call', but it didn't, so not running workflow callback"
+        Hyrax.logger.error "Expected '#{class_name}' to respond to 'call', but it didn't, so not running workflow callback"
         nil
       end
 

data/app/services/hyrax/workflow/notification_service.rb

@@ -60,11 +60,11 @@ module Hyrax
         klass = begin
                   class_name.constantize
                 rescue NameError
-                  Rails.logger.error "Unable to find '#{class_name}', so not sending notification"
+                  Hyrax.logger.error "Unable to find '#{class_name}', so not sending notification"
                   return nil
                 end
         return klass if klass.respond_to?(:send_notification)
-        Rails.logger.error "Expected '#{class_name}' to respond to 'send_notification', but it didn't, so not sending notification"
+        Hyrax.logger.error "Expected '#{class_name}' to respond to 'send_notification', but it didn't, so not sending notification"
         nil
       end
     end

data/app/services/hyrax/workflow/workflow_importer.rb

@@ -7,7 +7,7 @@ module Hyrax
     # @see .generate_from_json_file
     class WorkflowImporter
       class_attribute :default_logger
-      self.default_logger = Rails.logger
+      self.default_logger = Hyrax.logger
       class_attribute :path_to_workflow_files
       self.path_to_workflow_files = Rails.root.join('config', 'workflows', '*.json')
 
@@ -132,7 +132,7 @@ module Hyrax
         rescue InvalidStateRemovalException => e
           e.states.each do |state|
             error = I18n.t('hyrax.workflow.load.state_error', workflow_name: state.workflow.name, state_name: state.name, entity_count: state.entities.count)
-            Rails.logger.error(error)
+            Hyrax.logger.error(error)
             errors << error
           end
           Sipity::Workflow.find_by(name: configuration[:name])

data/app/services/hyrax/working_directory.rb

@@ -16,7 +16,7 @@ module Hyrax
         repository_file = Hydra::PCDM::File.find(repository_file_id)
         working_path = full_filename(id, repository_file.original_name)
         if File.exist?(working_path)
-          Rails.logger.debug "#{repository_file.original_name} already exists in the working directory at #{working_path}"
+          Hyrax.logger.debug "#{repository_file.original_name} already exists in the working directory at #{working_path}"
           return working_path
         end
         copy_repository_resource_to_working_directory(repository_file, id)
@@ -26,7 +26,7 @@ module Hyrax
       # @param [String] id the identifier of the FileSet
       # @return [String] path of the working file
       def copy_repository_resource_to_working_directory(file, id)
-        Rails.logger.debug "Loading #{file.original_name} (#{file.id}) from the repository to the working directory"
+        Hyrax.logger.debug "Loading #{file.original_name} (#{file.id}) from the repository to the working directory"
         copy_stream_to_working_directory(id, file.original_name, StringIO.new(file.content))
       end
 
@@ -38,7 +38,7 @@ module Hyrax
       # @return [String] path of the working file
       def copy_stream_to_working_directory(id, name, stream)
         working_path = full_filename(id, name)
-        Rails.logger.debug "Writing #{name} to the working directory at #{working_path}"
+        Hyrax.logger.debug "Writing #{name} to the working directory at #{working_path}"
         FileUtils.mkdir_p(File.dirname(working_path))
         IO.copy_stream(stream, working_path)
         working_path

data/app/services/hyrax/works/migration_service.rb

@@ -10,13 +10,13 @@ module Hyrax
       #   and the original data cleaned up.
       def self.migrate_predicate(predicate_from, predicate_to, works_to_update = ActiveFedora::Base.all)
         migrated = 0
-        Rails.logger.info "*** Migrating #{predicate_from} to #{predicate_to} in #{works_to_update.count} works"
+        Hyrax.logger.info "*** Migrating #{predicate_from} to #{predicate_to} in #{works_to_update.count} works"
         works_to_update.each do |work|
           next unless work.ldp_source.content.include?(predicate_from.to_s)
           migrate_data(predicate_from, predicate_to, work)
           migrated += 1
         end
-        Rails.logger.info "--- Migration Complete (#{migrated} migrated)"
+        Hyrax.logger.info "--- Migration Complete (#{migrated} migrated)"
       end
 
       # @api private
@@ -27,7 +27,7 @@ module Hyrax
         orm.value(predicate_from).each { |val| orm.graph.insert([orm.resource.subject_uri, predicate_to, val.to_s]) }
         orm.graph.delete([orm.resource.subject_uri, predicate_from, nil])
         orm.save
-        Rails.logger.info " Data migrated from #{predicate_from} to #{predicate_to} - id: #{work.id}"
+        Hyrax.logger.info " Data migrated from #{predicate_from} to #{predicate_to} - id: #{work.id}"
       end
     end
   end

data/app/utils/hyrax/test_data_seeders/user_seeder.rb

@@ -25,14 +25,14 @@ module Hyrax
         private
 
         def admin_role
-          unless defined? Role
+          unless ::User.reflect_on_association(:roles)
             logger.warn("Cannot create `Role` because the `hyrda-role-management` gem, or " \
                         "other gem providing a definition for a Role class, is not installed.  " \
                         "For development, you can edit `config/role_map.yml` and add the user's " \
                         "email under the role you want to assign.")
             return
           end
-          @admin_role ||= Role.find_by(name: Hyrax.config.admin_user_group_name)
+          @admin_role ||= Role.find_or_create_by(name: Hyrax.config.admin_user_group_name)
         end
 
         def add_user(email, password, role = nil)
@@ -43,6 +43,7 @@ module Hyrax
           end
           logger.info("   #{email} -- #{created ? 'CREATED' : 'ALREADY EXISTS'}")
           return unless role && !user.roles.include?(role)
+          logger.info("Adding #{role.name} to #{user.email}")
           user.roles << role
           user.save
         end

data/app/validators/hyrax/collection_membership_validator.rb

@@ -16,8 +16,8 @@ module Hyrax
       checker = @multiple_membership_checker.new(item: nil)
       ids = collections_ids(record)
 
-      errors = Array(checker.check(collection_ids: ids))
-      record.errors[:member_of_collection_ids].concat(errors)
+      error = checker.check(collection_ids: ids)
+      record.errors.add(:member_of_collection_ids, error) if error
     end
 
     private

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

@@ -10,7 +10,7 @@
 <div class="row">
   <div class="col-sm-12">
     <div class="work-reports">
-    
+
       <div class="card">
         <div class="card-header">
           <div class="col-sm-10">
@@ -23,7 +23,7 @@
         <div class="card-body">
           <div class="row">
             <div class="col-12 col-md-5">
-              <p><%= t('.deposited_by') %> <b><%= @document.depositor %></b> on <b><%= @document.date_uploaded.strftime("%Y-%m-%d") %></b></p>
+              <p><%= t('.deposited_by') %> <b><%= @document.depositor %></b> on <b><%= cast_to_date_time_format(@document.date_uploaded, format: "%Y-%m-%d") %></b></p>
               <p><%= t('.work_type') %> <b><%= @document.human_readable_type %></b></p>
               <p><%= t('.resource_type') %> <b><%= @document.resource_type.join(", ") %></b></p>
             </div>
@@ -34,7 +34,7 @@
           </div>
         </div>
       </div>
-          
+
       <div class="row text-center">
         <div class="nav nav-pills btn-group" role="group">
             <% if params[:start_date] %>
@@ -69,7 +69,7 @@
             <div class="tab-pane" id="monthly">
               <%= render "monthly_summary" %>
             </div>
-            
+
           <% if params[:start_date] %>
             <div class="tab-pane active" id="custom-range">
           <% else %>
@@ -79,12 +79,12 @@
               <%= render "custom_range" %>
             </div>
         </div>
-         
+
           <% unless @files.nil? %>
             <%= render "work_files" %>
             <%= render "work_counts" %>
          <% end %>
-         
+
     </div>
   </div>
 </div>

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

@@ -1 +1 @@
-<%= f.full_error(:collections) %>
+<%= full_collections_errors(form: f) %>

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

@@ -1,3 +1,3 @@
-<% unless f.object.model.errors[:in_works_ids].empty? %>
+<% unless in_works_ids_errors_for(form: f.object).empty? %>
   <%= f.full_error(:in_works_ids) %>
 <% end %>

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

@@ -1,3 +1,3 @@
-<% unless f.object.model.errors[:ordered_member_ids].empty? %>
+<% unless ordered_member_ids_errors_for(form: f.object).empty? %>
   <%= f.full_error(:ordered_member_ids) %>
 <% end %>

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

@@ -19,7 +19,7 @@
     </div>
 
     <div class="set-access-controls list-group-item">
-      <%= render 'form_visibility_component', f: f %>
+      <%= render 'form_visibility_component', f: f, save_work: true %>
     </div>
     <% if Flipflop.proxy_deposit? && current_user.can_make_deposits_for.any? %>
         <div class="list-group-item">

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

@@ -4,7 +4,11 @@
   <%= render 'form_permission_under_lease', f: f %>
 <% else %>
     <fieldset>
-      <legend class="legend-save-work"><%= t('.visibility') %></legend>
+      <% if local_assigns[:save_work] %>
+        <legend class="legend-save-work"><%= t('.visibility') %></legend>
+      <% else %>
+        <legend><%= t('.visibility') %><%= raw(t('.subtitle_html')) %></legend>
+      <% end %>
       <ul class="visibility">
         <li class="form-check">
           <label class="form-check-label">

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

@@ -1,5 +1,19 @@
-<%= f.full_error(:visibility) %>
-<%= f.full_error(:embargo_release_date) %>
-<%= f.full_error(:visibility_after_embargo) %>
-<%= f.full_error(:lease_expiration_date) %>
-<%= f.full_error(:visibility_after_lease) %>
+<% unless visibility_errors_for(form: f.object).empty? %>
+  <%= f.full_error(:visibility) %>
+<% end %>
+
+<% unless embargo_release_date_errors_for(form: f.object).empty? %>
+  <%= f.full_error(:embargo_release_date) %>
+<% end %>
+
+<% unless visibility_after_embargo_errors_for(form: f.object).empty? %>
+  <%= f.full_error(:visibility_after_embargo) %>
+<% end %>
+
+<% unless lease_expiration_date_errors_for(form: f.object).empty? %>
+  <%= f.full_error(:lease_expiration_date) %>
+<% end %>
+
+<% unless visibility_after_lease_errors_for(form: f.object).empty? %>
+  <%= f.full_error(:visibility_after_lease) %>
+<% end %>

data/app/views/hyrax/embargoes/edit.html.erb

@@ -13,7 +13,7 @@
       <fieldset class="set-access-controls">
         <section class="form-text">
           <p>
-            <% if curation_concern.embargo_release_date %>
+            <% if embargo_enforced?(curation_concern) %>
               <%= t('.embargo_true_html',  cc: cc_type) %>
             <% else %>
               <%= t('.embargo_false_html', cc: cc_type) %>
@@ -29,7 +29,7 @@
 
       <div class="row">
         <div class="col-md-12 form-actions">
-          <% if curation_concern.embargo_release_date %>
+          <% if embargo_enforced?(curation_concern) %>
             <%= f.submit t('.embargo_update'), class: 'btn btn-primary' %>
             <%= link_to t('.embargo_deactivate'), embargo_path(curation_concern), method: :delete, class: 'btn btn-danger' %>
           <% else %>
@@ -48,10 +48,10 @@
     <h2 class="card-title"><%= t('.header.past') %></h2>
   </div>
   <div class="card-body">
-    <% if curation_concern.embargo_history.empty? %>
+    <% if embargo_history(curation_concern).empty? %>
       <%= t('.history_empty', cc: cc_type) %>
     <% else %>
-      <%= render partial: "embargo_history", object: curation_concern.embargo_history %>
+      <%= render partial: "embargo_history", object: embargo_history(curation_concern) %>
     <% end %>
   </div>
 </div>

data/app/views/hyrax/file_sets/_permission_form.html.erb

@@ -1,8 +1,3 @@
-<% depositor = f.object.depositor %>
-<% public_perm = f.object.permissions.map { |perm| perm.access if perm.agent_name == "public"}.compact.first %>
-<% public_perm = true if params[:controller] == 'batch' %>
-<% registered_perm = f.object.permissions.map { |perm| perm.access if perm.agent_name == "registered"}.compact.first %>
-
 <h2 id="permissions_display"><% if params[:controller] == 'batch' %><%= t('.bulk') %> <% end %><%= t('.header') %> <% if params[:controller] == 'batch' %>
       <small><%= t('.applied_to') %></small><% end %>
 </h2>
@@ -13,7 +8,7 @@
   <span id="permissions_error_text"></span>
 </div>
 
-<%= render 'hyrax/base/form_permission', f: f %>
+<%= render 'hyrax/base/form_visibility_component', f: f %>
 
 <!-- Share With -->
 <div class="row">

data/app/views/hyrax/file_sets/edit.html.erb

@@ -6,7 +6,7 @@
 
 <div class="row">
   <div class="col-12 col-sm-4">
-    <%= render media_display_partial(curation_concern.to_presenter), file_set: curation_concern.to_presenter %>
+    <%= render media_display_partial(@presenter.solr_document), file_set: @presenter.solr_document %>
   </div>
   <div class="col-12 col-sm-8">
     <div class="card tabs">

data/app/views/hyrax/leases/edit.html.erb

@@ -13,7 +13,7 @@
       <fieldset class="set-access-controls">
         <section class="form-text">
           <p>
-            <% if curation_concern.lease_expiration_date %>
+            <% if lease_enforced?(curation_concern) %>
               <%= t('.lease_true_html',  cc: cc_type) %>
             <% else %>
               <%= t('.lease_false_html', cc: cc_type) %>
@@ -29,7 +29,7 @@
 
       <div class="row">
         <div class="col-md-12 form-actions">
-          <% if curation_concern.lease_expiration_date %>
+          <% if lease_enforced?(curation_concern) %>
             <%= f.submit t('.lease_update'), class: 'btn btn-primary' %>
             <%= link_to t('.lease_deactivate'), lease_path(curation_concern), method: :delete, class: 'btn btn-danger' %>
           <% else %>
@@ -48,7 +48,7 @@
     <h2 class="card-title"><%= t('.header.past') %></h2>
   </div>
   <div class="card-body">
-    <% if curation_concern.lease_history.empty? %>
+    <% if lease_history(curation_concern).empty? %>
       <%= t('.history_empty', cc: cc_type) %>
     <% else %>
       <%= render partial: 'lease_history', object: curation_concern.lease_history %>

data/app/views/hyrax/my/collections/_facets.html.erb

@@ -1,7 +1,9 @@
 <% # main container for facets/limits menu -%>
-<% if has_facet_values?(facet_field_names, @response) %>
-  <div class="list-filters" role="group">
-    <span class="list-filters-label"><%= t("hyrax.dashboard.my.facet_label.collections") %></span>
-    <%= render_facet_partials(facet_field_names, response: @response) %>
-  </div>
+<% facets = capture do %>
+  <%= render Blacklight::FacetComponent.with_collection(blacklight_config.facet_fields_in_group(nil), response: @response, component: Hyrax::DropdownFacetFieldComponent)%>
 <% end %>
+
+<div class="list-filters" role="group">
+  <span class="list-filters-label"><%= t("hyrax.dashboard.my.facet_label.collections") %></span>
+  <%= facets %>
+</div>

data/app/views/hyrax/my/works/_facets.html.erb

@@ -1,7 +1,11 @@
 <% # main container for facets/limits menu -%>
-<% if has_facet_values?(facet_field_names, @response) %>
+<% facets = capture do %>
+  <%= render Blacklight::FacetComponent.with_collection(blacklight_config.facet_fields_in_group(nil), response: @response, component: Hyrax::DropdownFacetFieldComponent)%>
+<% end %>
+
+<% if facets.present? %>
   <div class="list-filters" role="group">
     <span class="list-filters-label"><%= t("hyrax.dashboard.my.facet_label.works") %></span>
-    <%= render_facet_partials(facet_field_names, response: @response) %>
+    <%= facets %>
   </div>
 <% end %>

data/app/views/hyrax/transfers/_received.html.erb

@@ -49,7 +49,7 @@
       <td><%= req.sender_comment %></td>
     </tr>
     <% else %>
-      <% Rails.logger.error "A proxy request has no sender: #{req.inspect}" %>
+      <% Hyrax.logger.error "A proxy request has no sender: #{req.inspect}" %>
     <% end %>
   <% end %>
   </tbody>

data/app/views/hyrax/transfers/_sent.html.erb

@@ -32,7 +32,7 @@
           <td><%= req.sender_comment %></td>
         </tr>
       <% else %>
-          <% Rails.logger.error "A proxy request has no receiver: #{req.inspect}" %>
+          <% Hyrax.logger.error "A proxy request has no receiver: #{req.inspect}" %>
       <% end %>
     <% end %>
   </tbody>

data/app/views/hyrax/transfers/new.html.erb

@@ -5,7 +5,7 @@
 <span class="sr-only"><%= t(:'.sr_only_description', work_title: @proxy_deposit_request.to_s) %></span>
 
 <%= simple_form_for @proxy_deposit_request,
-             url: hyrax.work_transfers_path(@work) do |f| %>
+             url: hyrax.work_transfers_path(params[:id]) do |f| %>
   <%= f.input :transfer_to,
         placeholder: t(:'.placeholder') %>
   <%= f.input :sender_comment %>

data/chart/hyrax/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: hyrax
 description: An open-source, Samvera-powered digital repository system
 type: application
-version: 1.5.1
+version: 1.6.0
 appVersion: 3.3.0
 dependencies:
   - name: fcrepo

data/chart/hyrax/templates/_helpers.tpl

@@ -163,8 +163,12 @@ We truncate at 63 chars because some Kubernetes name fields are limited to this
 {{- end }}
 {{- end -}}
 
+{{- define "hyrax.solr.port" -}}
+{{- .Values.externalSolrPort | default "8983" }}
+{{- end -}}
+
 {{- define "hyrax.solr.url" -}}
-{{- printf "http://%s:%s@%s:%s/solr/%s" (include "hyrax.solr.username" .) (include "hyrax.solr.password" .) (include "hyrax.solr.host" .) "8983" (include "hyrax.solr.collectionName" .)  -}}
+{{- printf "http://%s:%s@%s:%s/solr/%s" (include "hyrax.solr.username" .) (include "hyrax.solr.password" .) (include "hyrax.solr.host" .) (include "hyrax.solr.port" .) (include "hyrax.solr.collectionName" .)  -}}
 {{- end -}}
 
 {{- define "hyrax.zk.fullname" -}}

data/chart/hyrax/templates/configmap-env.yaml

@@ -38,5 +38,5 @@ data:
   SOLR_COLLECTION_NAME: {{ template "hyrax.solr.collectionName" . }}
   SOLR_CONFIGSET_NAME: {{ template "hyrax.fullname" . }}
   SOLR_HOST: {{ template "hyrax.solr.host" . }}
-  SOLR_PORT: "8983"
+  SOLR_PORT: {{ (include "hyrax.solr.port" .) | quote }}
   SOLR_URL: {{ template "hyrax.solr.url" . }}

data/chart/hyrax/values.yaml

@@ -22,6 +22,7 @@ loadSolrConfigSet: true
 #   ignored if `solr.enabled` is true
 solrExistingSecret: ""
 externalSolrHost: ""
+externalSolrPort: ""
 externalSolrUser: ""
 externalSolrPassword: ""
 externalSolrCollection: "hyrax"

data/config/features.rb

@@ -47,5 +47,5 @@ Flipflop.configure do
           default: false,
           description: "Put the system into read-only mode. Deposits, edits, approvals and anything that makes a change to the data will be disabled."
 rescue Flipflop::StrategyError, Flipflop::FeatureError => err
-  Rails.logger.warn "Ignoring #{err}: #{err.message}"
+  Hyrax.logger.warn "Ignoring #{err}: #{err.message}"
 end

data/config/initializers/reform_rails_6_1_monkey_patch.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+# Taken from https://github.com/trailblazer/reform-rails/issues/86#issuecomment-763120151
+# rubocop:disable Metrics/CyclomaticComplexity
+# rubocop:disable Lint/UselessAssignment
+module Reform
+  class Contract < Disposable::Twin
+    class Result
+      private
+
+      # this doesn't do nested errors (e.g. )
+      def filter_for(method, *args)
+        @results.collect { |r| r.public_send(method, *args).to_h }
+                .inject({}) { |hah, err| hah.merge(err) { |_key, old_v, new_v| (new_v.is_a?(Array) ? (old_v |= new_v) : old_v.merge(new_v)) } }
+                .find_all do |_k, v| # filter :nested=>{:something=>["too nested!"]} #DISCUSS: do we want that here?
+                  if v.is_a?(Hash)
+                    nested_errors = v.select { |attr_key, val| attr_key.is_a?(Integer) && val.is_a?(Array) && val.any? }
+                    v = nested_errors.to_a if nested_errors.any?
+                  end
+                  v.is_a?(ActiveModel::DeprecationHandlingMessageArray)
+                end.to_h
+      end
+    end
+  end
+end
+# rubocop:enable Metrics/CyclomaticComplexity
+# rubocop:disable Lint/UselessAssignment

data/config/locales/hyrax.de.yml

@@ -557,7 +557,7 @@ de:
           actions: Aktion
           title: Sammlungstitel
       form_permission:
-        visibility: Sichtbarkeit <small> Wer sollte diesen Inhalt anzeigen oder herunterladen können? </ Small>
+        visibility: Sichtbarkeit <small> Wer sollte diesen Inhalt anzeigen oder herunterladen können? </small>
       form_permission_under_embargo:
         help_html: "<strong>Diese Arbeit ist unter Embargo.</strong> Hier können Sie die Einstellungen des Embargos ändern, oder Sie können den %{edit_link} besuchen, um ihn zu deaktivieren."
         legend_html: Sichtbarkeit <small>Wer sollte in der Lage sein, diesen Inhalt anzusehen oder herunterzuladen?</small>
@@ -596,6 +596,7 @@ de:
         legend_html: Miniaturansicht
       form_visibility_component:
         visibility: Sichtbarkeit
+        subtitle_html: <small>Wer sollte diesen Inhalt anzeigen oder herunterladen können?</small>
       inspect_work:
         back_to: Zurück zu
         entity_id: Mandanten-ID

data/config/locales/hyrax.en.yml

@@ -588,6 +588,7 @@ en:
         legend_html: Thumbnail
       form_visibility_component:
         visibility: Visibility
+        subtitle_html: <small>Who should be able to view or download this content?</small>
       inspect_work:
         back_to: Back to
         entity_id: Entity ID
@@ -1687,6 +1688,8 @@ en:
         note_html: Keep to myself with option to share.
         text: Private
       restricted_title_attr: Change the visibility of this resource
+      unknown:
+        text: Unknown
     work_button_row:
       attach_child: Attach Child
     workflow:

data/config/locales/hyrax.es.yml

@@ -601,6 +601,7 @@ es:
         legend_html: Miniatura
       form_visibility_component:
         visibility: La visibilidad
+        subtitle_html: <small>¿Quién debería poder ver o descargar este contenido?</small>
       inspect_work:
         back_to: De regreso
         entity_id: ID de entidad

data/config/locales/hyrax.fr.yml

@@ -563,7 +563,7 @@ fr:
           actions: action
           title: Titre de la collection
       form_permission:
-        visibility: Visibilité <small> Qui devrait être en mesure d’afficher ou de télécharger ce contenu? </ Small>
+        visibility: Visibilité <small> Qui devrait être en mesure d’afficher ou de télécharger ce contenu? </small>
       form_permission_under_embargo:
         help_html: "<strong>Ce travail est sous embargo.</strong> Vous pouvez modifier les paramètres de l'embargo ici, ou vous pouvez visiter le %{edit_link} pour le désactiver."
         legend_html: Visibilité <small>Qui devrait pouvoir afficher ou télécharger ce contenu?</small>
@@ -602,6 +602,7 @@ fr:
         legend_html: La vignette
       form_visibility_component:
         visibility: Visibilité
+        subtitle_html: <small>Qui devrait être en mesure d’afficher ou de télécharger ce contenu?</small>
       inspect_work:
         back_to: Retour à
         entity_id: ID d'entité

data/config/locales/hyrax.it.yml

@@ -601,6 +601,7 @@ it:
         legend_html: Thumbnail
       form_visibility_component:
         visibility: Visibilità
+        subtitle_html: <small>Chi dovrebbe essere in grado di visualizzare o scaricare questo contenuto?</small>
       inspect_work:
         back_to: Torna a
         entity_id: ID entità

data/config/locales/hyrax.pt-BR.yml

@@ -596,6 +596,7 @@ pt-BR:
         legend_html: Miniatura
       form_visibility_component:
         visibility: Visibilidade
+        subtitle_html: <small>Quem deve conseguir visualizar ou baixar este conteúdo?</small>
       inspect_work:
         back_to: De volta a
         entity_id: ID da entidade

data/config/locales/hyrax.zh.yml

@@ -599,6 +599,7 @@ zh:
         legend_html: 缩略图
       form_visibility_component:
         visibility: 可见性
+        subtitle_html: <small>谁应该能够查看或下载此内容？</ small>
       inspect_work:
         back_to: 回到
         entity_id: 实体ID