Bug #6978
Make 'fixed' descendants query and removal of nested sets permanent for QubitDigitalObject
| Status: | Verified | Start date: | 07/11/2014 | |
|---|---|---|---|---|
| Priority: | High | Due date: | ||
| Assignee: | Dan Gillean | % Done: | 0% | |
| Category: | Digital object | |||
| Target version: | Release 2.1.0 | |||
| Google Code Legacy ID: | Tested version: | 2.0.1, 2.1 | ||
| Sponsored: | No | Requires documentation: |
Description
I changed the query for the descendants property in BaseDigitalObject, as well as removed the nested set for it previously. However, these changes won't be permanent if the user rebuilds the model in symfony. We should add these changes to QubitDigitalObject instead and override the base class methods.
History
#1 Updated by Mike Gale almost 8 years ago
- Status changed from New to QA/Review
- Assignee changed from Mike Gale to Dan Gillean
Hey Dan (or Sarah or whoever you pass this off to), I tested this locally but it wouldn't hurt to test it in the 2x.test.artefactual.com site (or wherever the qa/2.1.x branch is going to now, might want to check with Jesus...).
It shouldn't have really changed anything, but basically if you muck around with editing / inserting / deleting digital objects and don't have problems, I think we can call this verified.
#2 Updated by Dan Gillean almost 8 years ago
Initial testing so far looks good - adding, deleting, re-adding, replacing thumbnails and reference images, browsing digital objects, carousel behaviour etc - all tested and so far no issues found. Testing thus far just with linking a single digital object (known problems w multi-uploading in #5724). Found a regression (#6989) but it doesn't appear to be caused by this, so I've filed it as a separate issue. I suggest we keep this ticket open in QA for now as we continue testing 2x in preparation for the 2.1 release. If no further issues are encountered during release testing, we can consider this verified and close.
#3 Updated by Dan Gillean almost 8 years ago
- Status changed from QA/Review to Verified
- Tested version 2.0.1, 2.1 added
Still need to fix the separate regression in #6989 - but otherwise, this has been part of our test environment for a month without issue, and was tested on a separate QA site before that. Marking verified.