Bug #11449

Deleting digital object link from record in UI breaks digital object link for other records

Added by Mike Gale almost 2 years ago. Updated about 1 year ago.

Status:VerifiedStart date:08/22/2017
Priority:LowDue date:
Assignee:Nick Wilkinson% Done:

0%

Category:Digital object
Target version:Release 2.5.0
Google Code Legacy ID: Tested version:2.3, 2.4
Sponsored:No Requires documentation:

Description

This is an edge case (and really shouldn't happen in the real world outside of testing since our app isn't designed for it...), so marking this as low priority, but it'd still be nice to resolve for testing purposes (it'd be cumbersome to need to use diff digital objects every single test record).

To reproduce:

1. Create 2 records
2. Link the same digital object to both records
3. Delete the digital object link from record A

Result: Digital object no longer displays in record B either; both records were linking to the same digital object on the file system (based on checksum). Step 3 above deletes the digital object from the file system, so when you visit record B it points to a no longer existing file and so the digital object is broken there.

Expected result: During digital object link delete confirmation screen in UI, the code should query the digital_object table and check if the digital object's file path is used in any other entries in the table--if so do not delete the digital object from the file system.

History

#2 Updated by Mike Cantelon about 1 year ago

  • Status changed from New to Code Review
  • Assignee changed from Mike Gale to Nick Wilkinson

#3 Updated by Nick Wilkinson about 1 year ago

  • Assignee changed from Nick Wilkinson to José Raddaoui Marín

Hi Radda, passing to you for CR.

#4 Updated by José Raddaoui Marín about 1 year ago

  • Status changed from Code Review to Feedback
  • Assignee changed from José Raddaoui Marín to Mike Cantelon

#5 Updated by Mike Cantelon about 1 year ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from Mike Cantelon to Dan Gillean
  • Target version set to Release 2.5.0

Merged into qa/2.5.x for QA. Can backport if need be, but it's a bit of an edge case.

#6 Updated by Michelle Curran about 1 year ago

Verified in local env set to qa/2.5.

#7 Updated by Michelle Curran about 1 year ago

  • Status changed from QA/Review to Verified
  • Assignee changed from Dan Gillean to Nick Wilkinson

Also available in: Atom PDF