From 2dac8fcddf1bce5b78f1379188d40668c73dda61 Mon Sep 17 00:00:00 2001 From: Brad Watson Date: Wed, 10 Apr 2024 14:02:21 -0500 Subject: [PATCH] Points Hyrax::EditPermissionsService.build_service_object_from to Hyrax configuration variable. For L#44, the `if` test's subject is hard-coded and unaware of any customized form class that the Hyrax Engine's implementers could create and set inside the Engine's configuration. This swaps to the configuration variable that defaults to the same hard-coded class (`Hyrax::Forms::FileSetForm`). --- app/services/hyrax/edit_permissions_service.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/services/hyrax/edit_permissions_service.rb b/app/services/hyrax/edit_permissions_service.rb index be254a7083..53251555b7 100644 --- a/app/services/hyrax/edit_permissions_service.rb +++ b/app/services/hyrax/edit_permissions_service.rb @@ -41,7 +41,7 @@ def self.build_service_object_from(form:, ability:) # (+Hyrax::Forms::FileSetForm+), +:in_works_ids+ is prepopulated onto # the form object itself. For +Hyrax::Forms::FileSetEditForm+, the # +:in_works+ method is present on the wrapped +:model+. - if form.object.is_a?(Hyrax::Forms::FileSetForm) + if form.object.is_a?(Hyrax.config.file_set_form) object_id = form.object.in_works_ids.first new(object: Hyrax.query_service.find_by(id: object_id), ability: ability) else