Bug 11132

Summary: The results list is reset on adding a biblio to a list
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: OPACAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: gmcharlt, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10858
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 11132: The results list is reset on adding a biblio to a list
Bug 11132: The results list is reset on adding a biblio to a list
[PASSED QA] Bug 11132: The results list is reset on adding a biblio to a list

Description Jonathan Druart 2013-10-24 15:01:00 UTC
To reproduce:
- launch a search at the OPAC (opac-search.pl).
- click on a result and browse results (using previous/next links).
- a title attract your attention and you add it to a list ("Save to yours lists" link on the right).
- save the list
- browse again results: you cannot, the "browse results" bloc has disappeared.
Comment 1 Jonathan Druart 2013-10-24 15:08:31 UTC Comment hidden (obsolete)
Comment 2 Joy Nelson 2013-10-25 18:50:36 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2013-10-27 19:06:00 UTC
Created attachment 22455 [details] [review]
[PASSED QA] Bug 11132: The results list is reset on adding a biblio to a list

A "busc" param is cleared if the template name is not opac-.*detail.tt.
So if a user adds a biblio to a list, he cannot continue to browse
results.

Test plan:
- launch a search at the OPAC (opac-search.pl).
- click on a result and browse results (using previous/next links).
- a title attract your attention and you add it to a list
  ("Save to yours lists" link on the right).
- save the list.
- browse again results.

Signed-off-by: Joy Nelson <joy@bywatersolutions.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Tested according to test plan, also checked some other pages and actions
accessible from the detail page.
Passes all tests and QA script.
Comment 4 Galen Charlton 2013-11-01 00:43:12 UTC
Pushed to master.  Thanks, Jonathan!
Comment 5 Tomás Cohen Arazi 2013-12-08 15:10:04 UTC
This patch has been pushed to 3.12.x, will be in 3.12.8.

Thanks Jonathan!