curation_concerns 0.1.0 → 0.2.0

data/app/controllers/concerns/curation_concerns/welcome_controller_behavior.rb

@@ -0,0 +1,8 @@
+module CurationConcerns
+  module WelcomeControllerBehavior
+    extend ActiveSupport::Concern
+
+    def index
+    end
+  end
+end

data/app/controllers/concerns/curation_concerns/without_namespace.rb

@@ -4,10 +4,11 @@ module CurationConcerns
     extend ActiveSupport::Concern
 
     included do
-      prepend_before_filter :normalize_identifier, except: [:index, :create, :new]
+      prepend_before_action :normalize_identifier, except: [:index, :create, :new]
     end
 
     protected
+
       def normalize_identifier
         params[:id] = CurationConcerns::Noid.namespaceize(params[:id])
       end

data/app/controllers/curation_concerns/classify_concerns_controller.rb

@@ -2,33 +2,33 @@ module CurationConcerns
   class ClassifyConcernsController < ApplicationController
     include CurationConcerns::ThemedLayoutController
     with_themed_layout '1_column'
-    respond_to :html
-    before_filter :authenticate_user!
+    before_action :authenticate_user!
     load_and_authorize_resource
 
-    add_breadcrumb 'Submit a work', lambda {|controller| controller.request.path }
+    add_breadcrumb 'Submit a work', ->(controller) { controller.request.path }
 
-    def classify_concern
-      @classify_concern
-    end
+    attr_reader :classify_concern
     helper_method :classify_concern
 
     def new
-      respond_with(classify_concern)
     end
 
     def create
       classify_concern.attributes = params[:classify_concern]
       if classify_concern.valid?
-        respond_with(classify_concern) do |wants|
+        respond_to do |wants|
           wants.html do
             redirect_to new_polymorphic_path(
               [:curation_concerns, classify_concern.curation_concern_class]
             )
           end
+          wants.json { render_json_response(response_type: :created, options: { location: polymorphic_path([main_app, :curation_concerns, curation_concern]) }) }
         end
       else
-        respond_with(classify_concern)
+        respond_to do |wants|
+          wants.html { render 'new', status: :unprocessable_entity }
+          wants.json { render_json_response(response_type: :unprocessable_entity, message: curation_concern.errors.messages) }
+        end
       end
     end
   end

data/app/controllers/curation_concerns/file_sets_controller.rb

@@ -0,0 +1,5 @@
+module CurationConcerns
+  class FileSetsController < ApplicationController
+    include CurationConcerns::FileSetsControllerBehavior
+  end
+end

data/app/controllers/curation_concerns/permissions_controller.rb

@@ -7,13 +7,12 @@ class CurationConcerns::PermissionsController < ApplicationController
   end
 
   def copy
-    CurationConcerns.queue.push(VisibilityCopyWorker.new(curation_concern.id))
+    VisibilityCopyJob.perform_later(curation_concern.id)
     flash_message = 'Updating file permissions. This may take a few minutes. You may want to refresh your browser or return to this record later to see the updated file permissions.'
     redirect_to polymorphic_path([main_app, :curation_concerns, curation_concern]), notice: flash_message
   end
 
   def curation_concern
-    @curation_concern ||= self.curation_concern_type.find(params[:id], cast: true)
+    @curation_concern ||= curation_concern_type.find(params[:id], cast: true)
   end
-
 end

data/app/controllers/curation_concerns/single_use_links_controller.rb

@@ -0,0 +1,5 @@
+module CurationConcerns
+  class SingleUseLinksController < ApplicationController
+    include CurationConcerns::SingleUseLinksControllerBehavior
+  end
+end

data/app/controllers/curation_concerns/single_use_links_viewer_controller.rb

@@ -0,0 +1,21 @@
+module CurationConcerns
+  class SingleUseLinksViewerController < ApplicationController
+    include CurationConcerns::SingleUseLinksViewerControllerBehavior
+
+    class Ability
+      include CanCan::Ability
+
+      attr_reader :single_use_link
+
+      def initialize(user, single_use_link)
+        @user = user || ::User.new
+
+        @single_use_link = single_use_link
+
+        can :read, [ActiveFedora::Base, SolrDocument] do |obj|
+          single_use_link.valid? && single_use_link.itemId == obj.id && single_use_link.destroy!
+        end if single_use_link
+      end
+    end
+  end
+end

data/app/controllers/downloads_controller.rb

@@ -1,4 +1,3 @@
 class DownloadsController < ApplicationController
   include CurationConcerns::DownloadBehavior
-
 end

data/app/controllers/embargoes_controller.rb

@@ -1,5 +1,3 @@
 class EmbargoesController < ApplicationController
-  include CurationConcerns::ManagesEmbargoes
   include CurationConcerns::EmbargoesControllerBehavior
-  include Hydra::Collections::AcceptsBatches
 end

data/app/controllers/leases_controller.rb

@@ -1,5 +1,3 @@
 class LeasesController < ApplicationController
-  include CurationConcerns::ManagesEmbargoes
   include CurationConcerns::LeasesControllerBehavior
-  include Hydra::Collections::AcceptsBatches
 end

data/app/controllers/registrations_controller.rb

@@ -2,7 +2,7 @@ class RegistrationsController < Devise::RegistrationsController
   include CurationConcerns::ThemedLayoutController
   with_themed_layout '1_column'
 
-  protected
+  # protected
 
   # def after_update_path_for(resource)
   #   resource.update_column(:user_does_not_require_profile_update, true)
@@ -16,5 +16,4 @@ class RegistrationsController < Devise::RegistrationsController
   # def sign_up(resource_name, resource)
   #   sign_in(resource_name, resource.user)
   # end
-
 end

data/app/controllers/upload_sets_controller.rb

@@ -0,0 +1,3 @@
+class UploadSetsController < ApplicationController
+  include CurationConcerns::UploadSetsControllerBehavior
+end

data/app/controllers/welcome_controller.rb

@@ -0,0 +1,3 @@
+class WelcomeController < ApplicationController
+  include CurationConcerns::WelcomeControllerBehavior
+end

data/app/forms/curation_concerns/forms/collection_edit_form.rb

@@ -4,14 +4,30 @@ module CurationConcerns
       include HydraEditor::Form
       self.model_class = ::Collection
       self.terms = [:resource_type, :title, :creator, :contributor, :description, :tag, :rights,
-                  :publisher, :date_created, :subject, :language, :identifier, :based_near, :related_url]
+                    :publisher, :date_created, :subject, :language, :identifier, :based_near,
+                    :related_url, :visibility]
 
       # Test to see if the given field is required
       # @param [Symbol] key a field
       # @return [Boolean] is it required or not
       def required?(key)
-        model_class.validators_on(key).any?{|v| v.kind_of? ActiveModel::Validations::PresenceValidator}
+        model_class.validators_on(key).any? { |v| v.is_a? ActiveModel::Validations::PresenceValidator }
       end
+
+      # @return [Hash] All generic files in the collection, file.to_s is the key, file.id is the value
+      def select_files
+        Hash[all_files]
+      end
+
+      private
+
+        def all_files
+          member_presenters.flat_map(&:file_presenters).map { |x| [x.to_s, x.id] }
+        end
+
+        def member_presenters
+          PresenterFactory.build_presenters(model.member_ids, WorkShowPresenter, nil)
+        end
     end
   end
 end

data/app/forms/curation_concerns/forms/file_set_edit_form.rb

@@ -0,0 +1,15 @@
+module CurationConcerns::Forms
+  class FileSetEditForm
+    include HydraEditor::Form
+    self.required_fields = [:title, :creator, :tag, :rights]
+
+    self.model_class = ::FileSet
+
+    self.terms = [:resource_type, :title, :creator, :contributor, :description,
+                  :tag, :rights, :publisher, :date_created, :subject, :language,
+                  :identifier, :based_near, :related_url,
+                  :visibility_during_embargo, :visibility_after_embargo, :embargo_release_date,
+                  :visibility_during_lease, :visibility_after_lease, :lease_expiration_date,
+                  :visibility]
+  end
+end

data/app/forms/curation_concerns/forms/work_form.rb

@@ -0,0 +1,57 @@
+module CurationConcerns
+  module Forms
+    class WorkForm
+      include HydraEditor::Form
+      attr_accessor :current_ability
+
+      self.terms = [:title, :creator, :contributor, :description,
+                    :subject, :publisher, :source, :language,
+                    :representative_id, :thumbnail_id, :rights, :files,
+                    :visibility_during_embargo, :embargo_release_date, :visibility_after_embargo,
+                    :visibility_during_lease, :lease_expiration_date, :visibility_after_lease,
+                    :visibility]
+
+      # @param [ActiveFedora::Base,#member_ids] model
+      # @param [Ability] current_ability
+      def initialize(model, current_ability)
+        @model = model
+        @current_ability = current_ability
+      end
+
+      # The possible values for the representative_id dropdown
+      # @return [Hash] All file sets in the collection, file.to_s is the key, file.id is the value
+      def select_files
+        Hash[file_presenters.map { |file| [file.to_s, file.id] }]
+      end
+
+      class << self
+        # This determines whether the allowed parameters are single or multiple.
+        # By default it delegates to the model, but we need to override for
+        # 'rights' which only has a single value on the form.
+        def multiple?(term)
+          case term.to_s
+          when 'rights'
+            false
+          else
+            super
+          end
+        end
+
+        # Overriden to cast 'rights' to an array
+        def sanitize_params(form_params)
+          super.tap do |params|
+            params['rights'] = Array(params['rights']) if params.key?('rights')
+          end
+        end
+      end
+
+      private
+
+        # @return [Array<FileSetPresenter>] presenters for the file sets in order of the ids
+        def file_presenters
+          @file_sets ||=
+            PresenterFactory.build_presenters(model.member_ids, FileSetPresenter, current_ability)
+        end
+    end
+  end
+end

data/app/helpers/curation_concerns/ability_helper.rb

@@ -3,43 +3,43 @@ module CurationConcerns
     # Returns true if can create at least one type of work
     def can_ever_create_works?
       can = false
-      CurationConcerns.configuration.curation_concerns.each do |curation_concern_type|
+      CurationConcerns.config.curation_concerns.each do |curation_concern_type|
         break if can
-        can = can?(:create,curation_concern_type)
+        can = can?(:create, curation_concern_type)
       end
-      return can
+      can
     end
 
     def visibility_options(variant)
       options = [
-          ['Open Access',Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC],
-          [t('curation_concerns.institution_name'),Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED],
-          ['Private',Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE]
+        ['Open Access', Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC],
+        [t('curation_concerns.institution_name'), Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED],
+        ['Private', Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE]
       ]
       case variant
-        when :restrict
-          options.delete_at(0)
-          options.reverse!
-        when :loosen
-          options.delete_at(2)
+      when :restrict
+        options.delete_at(0)
+        options.reverse!
+      when :loosen
+        options.delete_at(2)
       end
-      return options
+      options
     end
 
     def visibility_badge(value)
       case value
-        when Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
-          content_tag :span, "Open Access", class:"label label-success"
-        when Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED
-          content_tag :span, t('sufia.institution_name'), class:"label label-info"
-        when Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE
-          content_tag :span, "Private", class:"label label-danger"
-        when Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_EMBARGO
-          content_tag :span, "Embargo", class:"label label-warning"
-        when Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_LEASE
-          content_tag :span, "Lease", class:"label label-warning"
-        else
-          content_tag :span, value, class:"label label-info"
+      when Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
+        content_tag :span, "Open Access", class: "label label-success"
+      when Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED
+        content_tag :span, t('curation_concerns.institution_name'), class: "label label-info"
+      when Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE
+        content_tag :span, "Private", class: "label label-danger"
+      when Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_EMBARGO
+        content_tag :span, "Embargo", class: "label label-warning"
+      when Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_LEASE
+        content_tag :span, "Lease", class: "label label-warning"
+      else
+        content_tag :span, value, class: "label label-info"
       end
     end
   end

data/app/helpers/curation_concerns/catalog_helper.rb

@@ -1,9 +1,10 @@
 module CurationConcerns::CatalogHelper
-  def type_tab(label, key=label)
+  def type_tab(label, key = label)
     if params[:f] && params[:f][type_field] == [key]
-      content_tag(:li, link_to(label, "#"), class: "active")
+      content_tag(:li, link_to(label, '#'), class: 'active')
     else
-      facet_solr_field = facet_configuration_for_field(type_field)
+      # TODO: Unused variable. Not sure why this is here.
+      # facet_solr_field = facet_configuration_for_field(type_field)
       path = search_action_path(add_facet_params_and_redirect(type_field, key))
       # local_params = params.dup
       # local_facet_params = local_params[:f] || {}.with_indifferent_access
@@ -16,23 +17,23 @@ module CurationConcerns::CatalogHelper
 
   def all_type_tab(label = t('curation_concerns.catalog.index.type_tabs.all'))
     if params[:f] && params[:f][type_field]
-      facet_solr_field = facet_configuration_for_field(type_field)
+      # TODO: Unused variable. Not sure why this is here.
+      # facet_solr_field = facet_configuration_for_field(type_field)
       new_params = remove_facet_params(type_field, params[:f][type_field].first)
 
       # Delete any request params from facet-specific action, needed
       # to redir to index action properly.
-      new_params.except! *Blacklight::Solr::FacetPaginator.request_keys.values
+      new_params.except!(*Blacklight::Solr::FacetPaginator.request_keys.values)
       path = search_action_path(new_params)
       content_tag(:li, link_to(label, path))
     else
-      content_tag(:li, link_to(label, '#'), class: "active")
+      content_tag(:li, link_to(label, '#'), class: 'active')
     end
   end
 
   private
 
     def type_field
-      Solrizer.solr_name("generic_type", :facetable)
+      Solrizer.solr_name('generic_type', :facetable)
     end
-
 end

data/app/helpers/curation_concerns/collections_helper.rb

@@ -7,18 +7,19 @@ module CurationConcerns::CollectionsHelper
     "#{collectible.to_param.gsub(/:/, '-')}-modal"
   end
 
-  def link_to_select_collection(collectible, opts={})
+  def link_to_select_collection(collectible, opts = {})
     html_class = opts[:class]
-    link_to '#', data: { toggle: "modal", target: '#' + collection_modal_id(collectible) },
-      class: "add-to-collection #{html_class}", title: "Add #{collectible.human_readable_type} to Collection" do
+    link_to '#', data: { toggle: 'modal', target: '#' + collection_modal_id(collectible) },
+                 class: "add-to-collection #{html_class}", title: "Add #{collectible.human_readable_type} to Collection" do
       icon('plus-sign') + ' Add to a Collection'
     end
   end
 
   # override hydra-collections
   def link_to_remove_from_collection(document, label = 'Remove From Collection')
-    link_to collections.collection_path(@collection.id, collection: { members: 'remove'},
-                batch_document_ids: [ document.id ]), method: :put do
+    collection_id = @collection ? @collection.id : @presenter.id
+    link_to collections.collection_path(collection_id, collection: { members: 'remove' },
+                                                       batch_document_ids: [document.id]), method: :put do
       icon('minus-sign') + ' ' + label
     end
   end
@@ -32,16 +33,16 @@ module CurationConcerns::CollectionsHelper
   end
 
   private
+
     # return a list of collections for the current user with the exception of the passed in collection
     def available_collections(exclude_item)
       if exclude_item
-        collection_options.reject {|n| n.last == exclude_item.id}
+        collection_options.reject { |n| n.last == exclude_item.id }
       else
         collection_options
       end
     end
 
-
     def collection_options
       @collection_options ||= current_users_collections
     end
@@ -53,11 +54,7 @@ module CurationConcerns::CollectionsHelper
         current_user.collections.map { |c| [c.title.join(', '), c.id] }
       else
         query = ActiveFedora::SolrQueryBuilder.construct_query_for_rel(has_model: Collection.to_class_uri)
-        ActiveFedora::SolrService.query(query, fl: 'title_tesim id',
-                                        rows: 1000).map do |r|
-          [r['title_tesim'].join(', '), r['id']]
-        end.sort { |a, b| a.first <=> b.first }
+        ActiveFedora::SolrService.query(query, fl: 'title_tesim id', rows: 1000).map { |r| [r['title_tesim'].join(', '), r['id']] }.sort { |a, b| a.first <=> b.first }
       end
     end
-
 end

data/app/helpers/curation_concerns/embargo_helper.rb

@@ -1,6 +1,5 @@
 module CurationConcerns
   module EmbargoHelper
-
     def assets_with_expired_embargoes
       @assets_with_expired_embargoes ||= EmbargoService.assets_with_expired_embargoes
     end
@@ -12,6 +11,5 @@ module CurationConcerns
     def assets_with_deactivated_embargoes
       @assets_with_deactivated_embargoes ||= EmbargoService.assets_with_deactivated_embargoes
     end
-
   end
 end

data/app/helpers/curation_concerns/file_set_helper.rb

@@ -0,0 +1,31 @@
+module CurationConcerns::FileSetHelper
+  def parent_path(parent)
+    if parent.is_a?(Collection)
+      main_app.collection_path(parent)
+    else
+      polymorphic_path([main_app, :curation_concerns, parent])
+    end
+  end
+
+  def media_display(file_set, locals = {})
+    render media_display_partial(file_set),
+           locals.merge(file_set: file_set)
+  end
+
+  def media_display_partial(file_set)
+    'curation_concerns/file_sets/media_display/' +
+      if file_set.image?
+        'image'
+      elsif file_set.video?
+        'video'
+      elsif file_set.audio?
+        'audio'
+      elsif file_set.pdf?
+        'pdf'
+      elsif file_set.office_document?
+        'office_document'
+      else
+        'default'
+      end
+  end
+end

data/app/helpers/curation_concerns/lease_helper.rb

@@ -1,6 +1,5 @@
 module CurationConcerns
   module LeaseHelper
-
     def assets_with_expired_leases
       @assets_with_expired_leases ||= LeaseService.assets_with_expired_leases
     end
@@ -12,7 +11,5 @@ module CurationConcerns
     def assets_with_deactivated_leases
       @assets_with_deactivated_leases ||= LeaseService.assets_with_deactivated_leases
     end
-
   end
 end
-

data/app/helpers/curation_concerns/main_app_helpers.rb

@@ -1,11 +1,9 @@
 module CurationConcerns::MainAppHelpers
   include CurationConcerns::TitleHelper
-  include CurationConcerns::AttributeHelper
-  include CurationConcerns::GenericFileHelper
+  include CurationConcerns::FileSetHelper
   include CurationConcerns::SearchPathsHelper
   include CurationConcerns::RenderConstraintsHelper
   include CurationConcerns::AbilityHelper
-  include CurationConcerns::ThumbnailHelper
   include CurationConcerns::UrlHelper
   include CurationConcerns::EmbargoHelper
   include CurationConcerns::LeaseHelper

data/app/helpers/curation_concerns/render_constraints_helper.rb

@@ -4,17 +4,17 @@ module CurationConcerns
     # This is because unlike Blacklight, there is no way to change the search_field in the curation_concerns UI
     ##
     # Render the query constraints
-    # 
+    #
     # @param [Hash] query parameters
     # @return [String]
     def render_constraints_query(localized_params = params)
       # So simple don't need a view template, we can just do it here.
-      return "".html_safe if localized_params[:q].blank?
+      return ''.html_safe if localized_params[:q].blank?
 
       render_constraint_element(constraint_query_label(localized_params),
-            localized_params[:q],
-            classes: ["query"],
-            remove: url_for(localized_params.except(:search_field).merge(q: nil, action: 'index')))
+                                localized_params[:q],
+                                classes: ['query'],
+                                remove: url_for(localized_params.except(:search_field).merge(q: nil, action: 'index')))
     end
 
     ##

data/app/helpers/curation_concerns/search_paths_helper.rb

@@ -1,13 +1,11 @@
 module CurationConcerns::SearchPathsHelper
-
-  def search_path_for_my_works(opts={})
-    params_for_my_works = {:'f[generic_type_sim][]'=>'Work', works: 'mine'}
+  def search_path_for_my_works(opts = {})
+    params_for_my_works = { 'f[generic_type_sim][]' => 'Work', works: 'mine' }
     main_app.catalog_index_path(params_for_my_works.merge(opts))
   end
 
-  def search_path_for_my_collections(opts={})
-    params_for_my_collections = {:'f[generic_type_sim][]'=>'Collection', works: 'mine'}
+  def search_path_for_my_collections(opts = {})
+    params_for_my_collections = { 'f[generic_type_sim][]' => 'Collection', works: 'mine' }
     main_app.catalog_index_path(params_for_my_collections.merge(opts))
   end
-
-end
+end

data/app/helpers/curation_concerns/title_helper.rb

@@ -1,12 +1,11 @@
 module CurationConcerns::TitleHelper
-
   def construct_page_title(*elements)
-    (elements.flatten.compact + [application_name]).join(" // ")
+    (elements.flatten.compact + [application_name]).join(' // ')
   end
 
   def curation_concern_page_title(curation_concern)
     if curation_concern.persisted?
-      construct_page_title(curation_concern.title, "#{curation_concern.human_readable_type} [#{curation_concern.to_param}]")
+      construct_page_title(curation_concern.to_s, "#{curation_concern.human_readable_type} [#{curation_concern.to_param}]")
     else
       construct_page_title("New #{curation_concern.human_readable_type}")
     end
@@ -14,10 +13,7 @@ module CurationConcerns::TitleHelper
 
   def default_page_title
     text = controller_name.singularize.titleize
-    if action_name
-      text = "#{action_name.titleize} " + text
-    end
+    text = "#{action_name.titleize} " + text if action_name
     construct_page_title(text)
   end
-
 end

data/app/helpers/curation_concerns/url_helper.rb

@@ -1,9 +1,8 @@
 module CurationConcerns
   module UrlHelper
-
     # override Blacklight so we can use our 'curation_concern' namespace
     # We may also pass in a ActiveFedora document instead of a SolrDocument
-    def url_for_document doc, options = {}
+    def url_for_document(doc, _options = {})
       if doc.collection?
         doc
       else
@@ -15,15 +14,14 @@ module CurationConcerns
       main_app.track_solr_document_path(*args)
     end
 
-    def track_generic_file_path(*args)
+    def track_file_set_path(*args)
       main_app.track_solr_document_path(*args)
     end
 
     # generated new GenericWork models get registered as curation concerns and need a
     # track_model_path to render Blacklight-related views
-    CurationConcerns.configuration.registered_curation_concern_types.each do |concern|
-      define_method("track_#{concern.underscore}_path")  {|*args| main_app.track_solr_document_path(*args)}
+    CurationConcerns.config.registered_curation_concern_types.each do |concern|
+      define_method("track_#{concern.underscore}_path") { |*args| main_app.track_solr_document_path(*args) }
     end
-
   end
 end

data/app/inputs/multifile_input.rb

@@ -0,0 +1,8 @@
+class MultifileInput < SimpleForm::Inputs::CollectionInput
+  def input(wrapper_options = nil)
+    merged_input_options = merge_wrapper_options(input_html_options, wrapper_options)
+
+    merged_input_options.merge!(name: "#{@builder.object_name}[#{attribute_name}][]")
+    @builder.file_field(attribute_name, merged_input_options)
+  end
+end

data/app/jobs/visibility_copy_job.rb

@@ -0,0 +1,22 @@
+class VisibilityCopyJob < ActiveFedoraIdBasedJob
+  queue_as :permissions
+
+  def perform(id)
+    @id = id
+    work = object
+    work.file_sets.each do |file|
+      file.visibility = work.visibility # visibility must come first, because it can clear an embargo/lease
+      if work.lease
+        file.build_lease unless file.lease
+        file.lease.attributes = work.lease.attributes.except('id')
+        file.lease.save
+      end
+      if work.embargo
+        file.build_embargo unless file.embargo
+        file.embargo.attributes = work.embargo.attributes.except('id')
+        file.embargo.save
+      end
+      file.save!
+    end
+  end
+end