Bug #11383
Carousel "show all" digital object browse button broken
Status: | Verified | Start date: | 07/25/2017 | |
---|---|---|---|---|
Priority: | Medium | Due date: | ||
Assignee: | - | % Done: | 0% | |
Category: | Search / Browse | |||
Target version: | Release 2.4.0 | |||
Google Code Legacy ID: | Tested version: | 2.4 | ||
Sponsored: | No | Requires documentation: |
Description
This is a regression, likely caused by #9949.
To reproduce
- Navigate to a parent record with many children that have digital objects attached. For example, in the demo data, see the "Heit family"
- There are more results than the carousel is set to show. Click the "Show all" button
- 404 - Page not found
Information object browse page, with the following filters pre-applied:
- Card view
- Digital objects = yes
- Limited to Heit Family
Notes:
The page is still trying to redirect to:
http://10.10.10.10/heit-family-2/digitalobject/browse
In fact, the URL should be the same as the "Browse digital objects" link in the sidebar (see the fix on #10757 - PR 494 ) - aka:
http://10.10.10.10/informationobject/browse?collection=78802&topLod=0&view=card&onlyMedia=1
Related issues
History
#1 Updated by Nick Wilkinson almost 5 years ago
- Assignee changed from Nick Wilkinson to José Raddaoui Marín
Hi Radda, can you please look into this?
#2 Updated by José Raddaoui Marín almost 5 years ago
- Status changed from New to Code Review
- Assignee changed from José Raddaoui Marín to Nick Wilkinson
Fixed in https://github.com/artefactual/atom/pull/590
Please notice, this link can appear at any level in the hierarchy but the collection filter in the IO browse page only works with the top level descriptions. So if you click the button from an intermediary level you may see more results in the browse page. The same happens with the sidebar link, but it's more obvious in here as you can see that the counts don't match. We could change the button and link text to indicate that, but fixing it properly will probably require sponsorship.
#3 Updated by José Raddaoui Marín almost 5 years ago
- Status changed from Code Review to QA/Review
- Assignee changed from Nick Wilkinson to Dan Gillean
Merged in qa/2.4.x
#4 Updated by Dan Gillean almost 5 years ago
- Status changed from QA/Review to Verified
- Assignee deleted (
Dan Gillean)
#5 Updated by Dan Gillean over 4 years ago
- Related to Bug #11669: Lost functionality - Carousel "show all" link used to be parent-specific, now only shows all digital objects in archival unit added