Bug #10117
Creating a new repository on the fly can cause errors in search/browse
| Status: | Verified | Start date: | 07/12/2016 | |
|---|---|---|---|---|
| Priority: | Medium | Due 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
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!