hyrax 3.2.0 → 3.4.1

data/app/controllers/concerns/hyrax/works_controller_behavior.rb

@@ -55,21 +55,12 @@ module Hyrax
     end
 
     def create
-      # Caching the original input params in case the form is not valid
-      original_input_params_for_form = params[hash_key_for_curation_concern].deep_dup
-      if create_work
-        after_create_response
+      case curation_concern
+      when ActiveFedora::Base
+        original_input_params_for_form = params[hash_key_for_curation_concern].deep_dup
+        actor.create(actor_environment) ? after_create_response : after_create_error(curation_concern.errors, original_input_params_for_form)
       else
-        respond_to do |wants|
-          wants.html do
-            build_form
-            # Creating a form object that can re-render most of the
-            # submitted parameters
-            @form = Hyrax::Forms::FailedSubmissionFormWrapper.new(form: @form, input_params: original_input_params_for_form)
-            render 'new', status: :unprocessable_entity
-          end
-          wants.json { render_json_response(response_type: :unprocessable_entity, options: { errors: curation_concern.errors }) }
-        end
+        create_valkyrie_work
       end
     end
 
@@ -101,16 +92,11 @@ module Hyrax
     end
 
     def update
-      if update_work
-        after_update_response
+      case curation_concern
+      when ActiveFedora::Base
+        actor.update(actor_environment) ? after_update_response : after_update_error(curation_concern.errors)
       else
-        respond_to do |wants|
-          wants.html do
-            build_form
-            render 'edit', status: :unprocessable_entity
-          end
-          wants.json { render_json_response(response_type: :unprocessable_entity, options: { errors: curation_concern.errors }) }
-        end
+        update_valkyrie_work
       end
     end
 
@@ -176,13 +162,7 @@ module Hyrax
     end
 
     def admin_set_id_for_new
-      # admin_set_id is required on the client, otherwise simple_form renders a blank option.
-      # however it isn't a required field for someone to submit via json.
-      # Set the default admin set if it exists; otherwise, set to first admin_set they have access to.
-      admin_sets = Hyrax::AdminSetService.new(self).search_results(:deposit)
-      return nil if admin_sets.blank? # shouldn't happen
-      return AdminSet::DEFAULT_ID if admin_sets.map(&:id).include?(AdminSet::DEFAULT_ID)
-      admin_sets.first.id
+      Hyrax::AdminSetCreateService.find_or_create_default_admin_set.id.to_s
     end
 
     def build_form
@@ -195,36 +175,45 @@ module Hyrax
 
     ##
     # @return [#errors]
-    def create_work
-      case curation_concern
-      when ActiveFedora::Base
-        actor.create(actor_environment)
-      else
-        form = build_form
-
-        @curation_concern =
-          form.validate(params[hash_key_for_curation_concern]) &&
-          transactions['change_set.create_work']
-          .with_step_args('work_resource.add_to_parent' => { parent_id: params[:parent_id], user: current_user },
-                          'work_resource.add_file_sets' => { uploaded_files: uploaded_files, file_set_params: params[hash_key_for_curation_concern][:file_set] },
-                          'change_set.set_user_as_depositor' => { user: current_user })
-          .call(form).value!
-      end
+    def create_valkyrie_work
+      form = build_form
+      return after_create_error(form_err_msg(form)) unless form.validate(params[hash_key_for_curation_concern])
+
+      result =
+        transactions['change_set.create_work']
+        .with_step_args(
+          'work_resource.add_to_parent' => { parent_id: params[:parent_id], user: current_user },
+          'work_resource.add_file_sets' => { uploaded_files: uploaded_files, file_set_params: params[hash_key_for_curation_concern][:file_set] },
+          'change_set.set_user_as_depositor' => { user: current_user },
+          'work_resource.change_depositor' => { user: ::User.find_by_user_key(form.on_behalf_of) }
+        )
+        .call(form)
+      @curation_concern = result.value_or { return after_create_error(transaction_err_msg(result)) }
+      after_create_response
     end
 
-    def update_work
-      case curation_concern
-      when ActiveFedora::Base
-        actor.update(actor_environment)
-      else
-        form = build_form
+    def update_valkyrie_work
+      form = build_form
+      return after_update_error(form_err_msg(form)) unless form.validate(params[hash_key_for_curation_concern])
+      result =
+        transactions['change_set.update_work']
+        .with_step_args('work_resource.add_file_sets' => { uploaded_files: uploaded_files, file_set_params: params[hash_key_for_curation_concern][:file_set] },
+                        'work_resource.update_work_members' => { work_members_attributes: work_members_attributes })
+        .call(form)
+      @curation_concern = result.value_or { return after_update_error(transaction_err_msg(result)) }
+      after_update_response
+    end
 
-        @curation_concern =
-          form.validate(params[hash_key_for_curation_concern]) &&
-          transactions['change_set.update_work']
-          .with_step_args('work_resource.add_file_sets' => { uploaded_files: uploaded_files, file_set_params: params[hash_key_for_curation_concern][:file_set] })
-          .call(form).value!
-      end
+    def work_members_attributes
+      params[hash_key_for_curation_concern][:work_members_attributes]&.permit!&.to_h
+    end
+
+    def form_err_msg(form)
+      form.errors.messages.values.flatten.to_sentence
+    end
+
+    def transaction_err_msg(result)
+      result.failure.first
     end
 
     def presenter
@@ -383,6 +372,26 @@ module Hyrax
       end
     end
 
+    def after_create_error(errors, original_input_params_for_form = nil)
+      respond_to do |wants|
+        wants.html do
+          flash[:error] = errors.to_s
+          rebuild_form(original_input_params_for_form) if original_input_params_for_form.present?
+          render 'new', status: :unprocessable_entity
+        end
+        wants.json { render_json_response(response_type: :unprocessable_entity, options: { errors: errors }) }
+      end
+    end
+
+    # Creating a form object that can re-render most of the submitted parameters.
+    # Required for ActiveFedora::Base objects only.
+    def rebuild_form(original_input_params_for_form)
+      build_form
+      @form = Hyrax::Forms::FailedSubmissionFormWrapper
+              .new(form: @form,
+                   input_params: original_input_params_for_form)
+    end
+
     def after_update_response
       return redirect_to hyrax.confirm_access_permission_path(curation_concern) if permissions_changed? && concern_has_file_sets?
 
@@ -392,6 +401,17 @@ module Hyrax
       end
     end
 
+    def after_update_error(errors)
+      respond_to do |wants|
+        wants.html do
+          flash[:error] = errors.to_s
+          build_form unless @form.is_a? Hyrax::ChangeSet
+          render 'edit', status: :unprocessable_entity
+        end
+        wants.json { render_json_response(response_type: :unprocessable_entity, options: { errors: errors }) }
+      end
+    end
+
     def after_destroy_response(title)
       respond_to do |wants|
         wants.html { redirect_to hyrax.my_works_path, notice: "Deleted #{title}" }
@@ -432,7 +452,7 @@ module Hyrax
       when ActiveFedora::Common
         curation_concern.file_sets.present?
       else
-        Hyrax.custom_queries.find_child_fileset_ids(resource: curation_concern).any?
+        Hyrax.custom_queries.find_child_file_set_ids(resource: curation_concern).any?
       end
     end
 
@@ -441,12 +461,16 @@ module Hyrax
     end
 
     def available_admin_sets
+      # only returns admin sets in which the user can deposit
       admin_set_results = Hyrax::AdminSetService.new(self).search_results(:deposit)
+
       # get all the templates at once, reducing query load
-      templates = PermissionTemplate.where(id: admin_set_results.map(&:id)).to_a
+      templates = PermissionTemplate.where(source_id: admin_set_results.map(&:id)).to_a
 
       admin_sets = admin_set_results.map do |admin_set_doc|
         template = templates.find { |temp| temp.source_id == admin_set_doc.id.to_s }
+
+        # determine if sharing tab should be visible
         sharing = can?(:manage, template) || !!template&.active_workflow&.allows_access_grant?
 
         AdminSetSelectionPresenter::OptionsEntry

data/app/controllers/hyrax/admin/admin_sets_controller.rb

@@ -4,7 +4,8 @@ module Hyrax
     include Hyrax::CollectionsControllerBehavior
 
     before_action :authenticate_user!
-    load_and_authorize_resource
+    load_and_authorize_resource instance_name: :admin_set,
+                                class: Hyrax.config.admin_set_model
 
     # Catch permission errors
     rescue_from Hydra::AccessDenied, CanCan::AccessDenied, with: :deny_adminset_access
@@ -61,28 +62,52 @@ module Hyrax
     end
 
     def update
-      if @admin_set.update(admin_set_params)
-        redirect_to update_referer, notice: I18n.t('updated_admin_set', scope: 'hyrax.admin.admin_sets.form.permission_update_notices', name: @admin_set.title.first)
+      case @admin_set
+      when Valkyrie::Resource
+        valkyrie_update
       else
-        setup_form
-        render :edit
+        active_fedora_update
       end
     end
 
+    def after_create
+      Hyrax::SolrService.commit
+      redirect_to hyrax.edit_admin_admin_set_path(admin_set_id),
+                  notice: I18n.t('new_admin_set',
+                                 scope: 'hyrax.admin.admin_sets.form.permission_update_notices',
+                                 name: @admin_set.title.first)
+    end
+
+    def after_create_error(err_msg: "")
+      msg = "Failed to create admin set: #{err_msg}"
+      setup_form
+      flash[:error] = msg
+      Hyrax.logger.error(msg)
+      render :new
+    end
+
     def create
-      if create_admin_set
-        redirect_to hyrax.edit_admin_admin_set_path(@admin_set), notice: I18n.t('new_admin_set', scope: 'hyrax.admin.admin_sets.form.permission_update_notices', name: @admin_set.title.first)
+      case @admin_set
+      when Valkyrie::Resource
+        valkyrie_create
       else
-        setup_form
-        render :new
+        active_fedora_create
       end
     end
 
     def destroy
-      if @admin_set.destroy
+      case @admin_set
+      when Valkyrie::Resource
+        transactions['admin_set_resource.destroy'].call(@admin_set).value_or do |failure|
+          redirect_to hyrax.admin_admin_set_path(admin_set_id), alert: failure.first
+        end
         after_delete_success
       else
-        redirect_to hyrax.admin_admin_set_path(@admin_set), alert: @admin_set.errors.full_messages.to_sentence
+        if @admin_set.destroy
+          after_delete_success
+        else
+          redirect_to hyrax.admin_admin_set_path(admin_set_id), alert: @admin_set.errors.full_messages.to_sentence
+        end
       end
     end
 
@@ -98,12 +123,50 @@ module Hyrax
 
     private
 
-    def update_referer
-      hyrax.edit_admin_admin_set_path(@admin_set) + (params[:referer_anchor] || '')
+    def valkyrie_update
+      @admin_set = form.validate(admin_set_params) && transactions['admin_set_resource.update'].call(form).value_or do |_failure|
+        setup_form # probably should do some real error handling here
+        render :edit
+      end
+      redirect_to update_referer, notice: I18n.t('updated_admin_set', scope: 'hyrax.admin.admin_sets.form.permission_update_notices', name: @admin_set.title.first)
+    end
+
+    def active_fedora_update
+      if @admin_set.update(admin_set_params)
+        redirect_to update_referer, notice: I18n.t('updated_admin_set', scope: 'hyrax.admin.admin_sets.form.permission_update_notices', name: @admin_set.title.first)
+      else
+        setup_form
+        render :edit
+      end
     end
 
-    def create_admin_set
-      admin_set_create_service.call(admin_set: @admin_set, creating_user: current_user)
+    def update_referer
+      hyrax.edit_admin_admin_set_path(admin_set_id) + (params[:referer_anchor] || '')
+    end
+
+    def valkyrie_create
+      form.validate(admin_set_params) &&
+        @admin_set = transactions['change_set.create_admin_set']
+                     .with_step_args(
+                       'change_set.set_user_as_creator' => { user: current_user },
+                       'admin_set_resource.apply_collection_type_permissions' => { user: current_user }
+                     )
+                     .call(form).value_or do |_failure|
+                       setup_form # probably should do some real error handling here
+                       render :edit
+                     end
+      @admin_set = admin_set_create_service.call!(admin_set: @admin_set, creating_user: current_user)
+      after_create
+    rescue RuntimeError => err
+      after_create_error(err_msg: err.message)
+    end
+
+    def active_fedora_create
+      updated_admin_set = admin_set_create_service.call!(admin_set: admin_set_resource, creating_user: current_user)
+      @admin_set = Wings::ActiveFedoraConverter.convert(resource: updated_admin_set)
+      after_create
+    rescue RuntimeError => err
+      after_create_error(err_msg: err.message)
     end
 
     def setup_form
@@ -116,7 +179,13 @@ module Hyrax
 
     # initialize the form object
     def form
-      @form ||= form_class.new(@admin_set, current_ability, repository)
+      @form ||=
+        case @admin_set
+        when Valkyrie::Resource
+          Hyrax::Forms::ResourceForm.for(@admin_set)
+        else
+          form_class.new(@admin_set, current_ability, repository)
+        end
     end
 
     def action_breadcrumb
@@ -129,7 +198,11 @@ module Hyrax
     end
 
     def admin_set_params
-      form_class.model_attributes(params[:admin_set])
+      if Hyrax.config.admin_set_class < ActiveFedora::Base
+        form_class.model_attributes(params[:admin_set])
+      else
+        params.permit(admin_set: {})[:admin_set]
+      end
     end
 
     def repository_class
@@ -137,13 +210,26 @@ module Hyrax
     end
 
     def after_delete_success
-      if request.referer.include? "my/collections"
+      if request.referer&.include? "my/collections"
         redirect_to hyrax.my_collections_path, notice: t(:'hyrax.admin.admin_sets.delete.notification')
-      elsif request.referer.include? "collections"
+      elsif request.referer&.include? "collections"
         redirect_to hyrax.dashboard_collections_path, notice: t(:'hyrax.admin.admin_sets.delete.notification')
       else
         redirect_to hyrax.my_collections_path, notice: t(:'hyrax.admin.admin_sets.delete.notification')
       end
     end
+
+    def admin_set_id
+      @admin_set&.id&.to_s
+    end
+
+    def admin_set_resource
+      case @admin_set
+      when Valkyrie::Resource
+        @admin_set
+      else
+        @admin_set.valkyrie_resource
+      end
+    end
   end
 end

data/app/controllers/hyrax/admin/analytics/analytics_controller.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+module Hyrax
+  module Admin
+    module Analytics
+      class AnalyticsController < ApplicationController
+        include Hyrax::SingularSubresourceController
+        before_action :set_months
+        before_action :set_date_range
+        before_action :set_document, only: [:show]
+        with_themed_layout 'dashboard'
+
+        def set_document
+          @document = ::SolrDocument.find(params[:id])
+        end
+
+        def set_months
+          @month_names = 12.downto(1).map { |n| DateTime::ABBR_MONTHNAMES.drop(1)[(Time.zone.today.month - n) % 12] }.reverse
+        end
+
+        def set_date_range
+          @start_date = params[:start_date] || Hyrax.config.analytics_start_date
+          @end_date = params[:end_date] || Time.zone.today + 1.day
+        end
+
+        def date_range
+          "#{@start_date},#{@end_date}"
+        end
+
+        def paginate(results_array, rows: 10)
+          return if results_array.nil?
+
+          total_pages = (results_array.size.to_f / rows.to_f).ceil
+          page = request.params[:page].nil? ? 1 : request.params[:page].to_i
+          current_page = page > total_pages ? total_pages : page
+          Kaminari.paginate_array(results_array, total_count: results_array.size).page(current_page).per(rows)
+        end
+      end
+    end
+  end
+end

data/app/controllers/hyrax/admin/analytics/collection_reports_controller.rb

@@ -0,0 +1,61 @@
+# frozen_string_literal: true
+module Hyrax
+  module Admin
+    module Analytics
+      class CollectionReportsController < AnalyticsController
+        include Hyrax::BreadcrumbsForCollectionAnalytics
+        def index
+          return unless Hyrax.config.analytics?
+
+          @pageviews = Hyrax::Analytics.daily_events('collection-page-view')
+          @work_page_views = Hyrax::Analytics.daily_events('work-in-collection-view')
+          @downloads = Hyrax::Analytics.daily_events('work-in-collection-download')
+          @all_top_collections = Hyrax::Analytics.top_events('work-in-collection-view', date_range)
+          @top_collections = paginate(@all_top_collections, rows: 10)
+          @top_downloads = Hyrax::Analytics.top_events('work-in-collection-download', date_range)
+          @top_collection_pages = Hyrax::Analytics.top_events('collection-page-view', date_range)
+          respond_to do |format|
+            format.html
+            format.csv { export_data }
+          end
+        end
+
+        def show
+          return unless Hyrax.config.analytics?
+          @document = ::SolrDocument.find(params[:id])
+          @pageviews = Hyrax::Analytics.daily_events_for_id(@document.id, 'collection-page-view')
+          @work_page_views = Hyrax::Analytics.daily_events_for_id(@document.id, 'work-in-collection-view')
+          @uniques = Hyrax::Analytics.unique_visitors_for_id(@document.id)
+          @downloads = Hyrax::Analytics.daily_events_for_id(@document.id, 'work-in-collection-download')
+          respond_to do |format|
+            format.html
+            format.csv { export_data }
+          end
+        end
+
+       private
+
+        # rubocop:disable Metrics/MethodLength
+        def export_data
+          csv_row = CSV.generate do |csv|
+            csv << ["Name", "ID", "View of Works In Collection", "Downloads of Works In Collection", "Collection Page Views"]
+            @all_top_collections.each do |collection|
+              document = begin
+                           ::SolrDocument.find(collection[0])
+                         rescue
+                           "Collection deleted"
+                         end
+              download_match = @top_downloads.detect { |a, _b| a == collection[0] }
+              download_count = download_match ? download_match[1] : 0
+              collection_match = @top_collection_pages.detect { |a, _b| a == collection[0] }
+              collection_count = collection_match ? collection_match[1] : 0
+              csv << [document, collection[0], collection[1], download_count, collection_count]
+            end
+          end
+          send_data csv_row, filename: "#{@start_date}-#{@end_date}-collections.csv"
+        end
+        # rubocop:enable Metrics/MethodLength
+      end
+    end
+  end
+end

data/app/controllers/hyrax/admin/analytics/work_reports_controller.rb

@@ -0,0 +1,122 @@
+# frozen_string_literal: true
+module Hyrax
+  module Admin
+    module Analytics
+      class WorkReportsController < AnalyticsController
+        include Hyrax::BreadcrumbsForWorksAnalytics
+
+        def index
+          return unless Hyrax.config.analytics?
+
+          @accessible_works ||= accessible_works
+          @accessible_file_sets ||= accessible_file_sets
+          @works_count = @accessible_works.count
+          @top_works = paginate(top_works_list, rows: 10)
+          @top_file_set_downloads = paginate(top_files_list, rows: 10)
+
+          if current_user.ability.admin?
+            @pageviews = Hyrax::Analytics.daily_events('work-view')
+            @downloads = Hyrax::Analytics.daily_events('file-set-download')
+          end
+
+          respond_to do |format|
+            format.html
+            format.csv { export_data }
+          end
+        end
+
+        def show
+          @pageviews = Hyrax::Analytics.daily_events_for_id(@document.id, 'work-view')
+          @uniques = Hyrax::Analytics.unique_visitors_for_id(@document.id)
+          @downloads = Hyrax::Analytics.daily_events_for_id(@document.id, 'file_set_in_work_download')
+          @files = paginate(@document._source["file_set_ids_ssim"], rows: 5)
+          respond_to do |format|
+            format.html
+            format.csv { export_data }
+          end
+        end
+
+  private
+
+        def accessible_works
+          models = Hyrax.config.curation_concerns.map { |m| "\"#{m}\"" }
+          if current_user.ability.admin?
+            ActiveFedora::SolrService.query("has_model_ssim:(#{models.join(' OR ')})",
+              fl: 'title_tesim, id, member_of_collections',
+              rows: 50_000)
+          else
+            ActiveFedora::SolrService.query(
+              "edit_access_person_ssim:#{current_user} AND has_model_ssim:(#{models.join(' OR ')})",
+              fl: 'title_tesim, id, member_of_collections',
+              rows: 50_000
+            )
+          end
+        end
+
+        def accessible_file_sets
+          if current_user.ability.admin?
+            ActiveFedora::SolrService.query(
+              "has_model_ssim:FileSet",
+              fl: 'title_tesim, id',
+              rows: 50_000
+            )
+          else
+            ActiveFedora::SolrService.query(
+              "edit_access_person_ssim:#{current_user} AND has_model_ssim:FileSet",
+              fl: 'title_tesim, id',
+              rows: 50_000
+            )
+          end
+        end
+
+        def top_analytics_works
+          @top_analytics_works ||= Hyrax::Analytics.top_events('work-view', date_range)
+        end
+
+        def top_analytics_downloads
+          @top_analytics_downloads ||= Hyrax::Analytics.top_events('file-set-in-work-download', date_range)
+        end
+
+        def top_analytics_file_sets
+          @top_analytics_file_sets ||= Hyrax::Analytics.top_events('file-set-download', date_range)
+        end
+
+        def top_works_list
+          list = []
+          top_analytics_works
+          top_analytics_downloads
+          @accessible_works.each do |doc|
+            views_match = @top_analytics_works.detect { |id, _count| id == doc["id"] }
+            @view_count = views_match ? views_match[1] : 0
+            downloads_match = @top_analytics_downloads.detect { |id, _count| id == doc["id"] }
+            @download_count = downloads_match ? downloads_match[1] : 0
+            list.push([doc["id"], doc["title_tesim"].join(''), @view_count, @download_count, doc["member_of_collections"]])
+          end
+          list.sort_by { |l| -l[2] }
+        end
+
+        def top_files_list
+          list = []
+          top_analytics_file_sets
+          @accessible_file_sets.each do |doc|
+            downloads_match = @top_analytics_file_sets.detect { |id, _count| id == doc["id"] }
+            @download_count = downloads_match ? downloads_match[1] : 0
+            list.push([doc["id"], doc["title_tesim"].join(''), @download_count]) if doc["title_tesim"].present?
+          end
+          list.sort_by { |l| -l[2] }
+        end
+
+        def export_data
+          data = top_works_list
+          csv_row = CSV.generate do |csv|
+            csv << ["Name", "ID", "Work Page Views", "Total Downloads of File Sets In Work", "Collections"]
+            data.each do |d|
+              csv << [d[0], d[1], d[2], d[3], d[4]]
+            end
+          end
+          send_data csv_row, filename: "#{params[:start_date]}-#{params[:end_date]}-works.csv"
+        end
+      end
+    end
+  end
+end

data/app/controllers/hyrax/admin/permission_template_accesses_controller.rb

@@ -5,11 +5,11 @@ module Hyrax
       load_and_authorize_resource class: 'Hyrax::PermissionTemplateAccess'
 
       def destroy
+        authorize! :destroy, @permission_template_access
         ActiveRecord::Base.transaction do
-          @permission_template_access.destroy if valid_delete?
+          @permission_template_access.destroy
           remove_access!
         end
-
         if @permission_template_access.destroyed?
           after_destroy_success
         else
@@ -19,38 +19,31 @@ module Hyrax
 
       private
 
-      # This is a controller validation rather than a model validation
-      # because we don't want to prevent the ability to remove the whole
-      # PermissionTemplate and all of its associated PermissionTemplateAccesses
-      # @return [Boolean] true if it's valid
-      def valid_delete?
-        return true unless @permission_template_access.admin_group?
-        @permission_template_access.errors[:base] <<
-          t('hyrax.admin.admin_sets.form.permission_destroy_errors.admin_group')
-        false
-      end
-
-      def after_destroy_success
+      def after_destroy_error
         if source.admin_set?
+          @permission_template_access.errors[:base] <<
+            t('hyrax.admin.admin_sets.form.permission_destroy_errors.participants')
           redirect_to hyrax.edit_admin_admin_set_path(source_id,
                                                       anchor: 'participants'),
-                      notice: translate('participants', scope: 'hyrax.admin.admin_sets.form.permission_update_notices')
+                      alert: @permission_template_access.errors.full_messages.to_sentence
         else
+          @permission_template_access.errors[:base] <<
+            t('hyrax.dashboard.collections.form.permission_update_errors.sharing')
           redirect_to hyrax.edit_dashboard_collection_path(source_id,
                                                            anchor: 'sharing'),
-                      notice: translate('sharing', scope: 'hyrax.dashboard.collections.form.permission_update_notices')
+                      alert: @permission_template_access.errors.full_messages.to_sentence
         end
       end
 
-      def after_destroy_error
+      def after_destroy_success
         if source.admin_set?
           redirect_to hyrax.edit_admin_admin_set_path(source_id,
                                                       anchor: 'participants'),
-                      alert: @permission_template_access.errors.full_messages.to_sentence
+                      notice: translate('participants', scope: 'hyrax.admin.admin_sets.form.permission_update_notices')
         else
           redirect_to hyrax.edit_dashboard_collection_path(source_id,
                                                            anchor: 'sharing'),
-                      alert: @permission_template_access.errors.full_messages.to_sentence
+                      notice: translate('sharing', scope: 'hyrax.dashboard.collections.form.permission_update_notices')
         end
       end