Bug #11383

Carousel "show all" digital object browse button broken

Added by Dan Gillean almost 3 years ago. Updated almost 3 years ago.

Status:VerifiedStart date:07/25/2017
Priority:MediumDue 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
Resulting error
  • 404 - Page not found
Expected result
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

Related to Access to Memory (AtoM) - Bug #11669: Lost functionality - Carousel "show all" link used to be ... Verified 11/02/2017

History

#1 Updated by Nick Wilkinson almost 3 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 3 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 3 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 3 years ago

  • Status changed from QA/Review to Verified
  • Assignee deleted (Dan Gillean)

#5 Updated by Dan Gillean over 2 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

Also available in: Atom PDF