5575 item actions edit

app/controllers/hyrax/file_sets_controller.rb

@@ -171,6 +171,7 @@ def after_update_failure_response
       respond_to do |wants|
         wants.html do
           initialize_edit_form
+          # TODO: return a valuable error message
           flash[:error] = "There was a problem processing your request."
           render 'edit', status: :unprocessable_entity
         end

app/forms/hyrax/forms/file_set_edit_form.rb

@@ -6,7 +6,7 @@ class FileSetEditForm
 
     delegate :depositor, :permissions, :human_readable_type, to: :model
 
-    self.required_fields = [:title, :creator, :license]
+    self.required_fields = [:title, :creator]
 
     self.model_class = ::FileSet
 

app/models/hyrax/file_set.rb

@@ -51,6 +51,14 @@ def representative_id
       id
     end
 
+    ##
+    # @return [Valkyrie::ID]
+    def representative_id=(_input)
+      # saving a file set using valkyrie would err because this method didn't exist.
+      Rails.logger.warn('This is not a valid method for file sets')
+      id
+    end
+
     ##
     # @return [Boolean] true
     def pcdm_object?

config/metadata/file_set_metadata.yaml

@@ -16,7 +16,7 @@ attributes:
     type: string
     multiple: true
     form:
-      required: true
+      required: false
       primary: true
 
   # Other attributes:

spec/forms/hyrax/forms/file_set_form_spec.rb

@@ -26,7 +26,7 @@
   describe '.required_fields' do
     it 'lists the fields tagged required' do
       expect(described_class.required_fields)
-        .to contain_exactly(:title, :creator, :license)
+        .to contain_exactly(:title, :creator)
     end
   end