Bug #10117

Creating a new repository on the fly can cause errors in search/browse

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

Status:VerifiedStart date:07/12/2016
Priority:MediumDue date:
Assignee:Dan Gillean% Done:

0%

Category:Search / Browse
Target version:Release 2.3.0
Google Code Legacy ID: Tested version:2.3
Sponsored:No Requires documentation:

Description

  • Create a new description, and save it without a repository
  • Re-enter edit mode, and add a new repository
  • In some cases, I then had to go and add a logo/banner to the new repository (not sure why that matters)
  • On the repo view page, click the browse holdings link

Resulting error

  • 0 results, even though you have just made a description linked to that repository
  • If you remove the repo filter tag, and then try selecting the repo from the repo facet, you can see the error repeat
  • If you open the advanced search panel, the repository filter throws an error (see attached screenshot)

Expected result
Browse page limited to the new repository works as expected, does not throw error.

Note: I recreated this on the rc-1 test site, and then was able to repeat it in my local VM, on stable/2.3.x

URL in rc-1 test site: http://46.101.164.79/index.php/informationobject/browse?repos=605

repo-error.png (38.5 KB) Dan Gillean, 07/12/2016 05:00 PM

History

#1 Updated by Dan Gillean almost 6 years ago

  • Assignee set to José Raddaoui Marín
  • Target version set to Release 2.3.0

Radda can you take a look at this?

#2 Updated by José Raddaoui Marín almost 6 years ago

  • Status changed from New to Code Review
  • Assignee changed from José Raddaoui Marín to Steve Breker

Fixed in PR 410

It was a cache issue and probably the recent APc/APCu issues made us miss it.

#3 Updated by Steve Breker almost 6 years ago

  • Status changed from Code Review to Feedback
  • Assignee changed from Steve Breker to José Raddaoui Marín

Code review complete - looks great.

#4 Updated by José Raddaoui Marín almost 6 years ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from José Raddaoui Marín to Dan Gillean

Merged in stable/2.3.x and qa/2.4.x

#5 Updated by Dan Gillean almost 6 years ago

  • Status changed from QA/Review to Verified

Looks good testing in local VM!

Also available in: Atom PDF