Project

General

Profile

Bug #16144

Updated by Peter Amstutz over 2 years ago

1. in models/container_request.rb#update_collections 

 <pre> 
 manifest = Collection.where(portable_data_hash: pdh).first.manifest_text 
 </pre> 

 Fails if @pdh@ is non-nil but not found. 

 2. in models/container_request.rb#finalize_if_needed 

 <pre> 
       c = Container.find_by_uuid(container_uuid) 
       c.lock! 
 </pre> 

 Fails if @container_uuid@ is not found. 

 Also, if the container_request is being destroyed, @finalize_if_needed@ should be skipped.    (It is an @after_save@ hook, do those run on destroy?) 

 

Back