Summary: | Can't add entry to lists using link in result list | ||
---|---|---|---|
Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
Component: | Lists | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | f.demians, jonathan.druart, kyle, m.de.rooy, veron |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Bug Depends on: | 17210 | ||
Bug Blocks: | |||
Attachments: |
Bug 17315: (Bug 17210 follow-up) Fix "Save to lists" links from the result page
Bug 17315: (Bug 17210 follow-up) Fix "Save to lists" links from the result page [PASSED QA] Bug 17315: (Bug 17210 follow-up) Fix "Save to lists" links from the result page |
Description
Katrin Fischer
2016-09-18 19:25:47 UTC
Confirmed. It seems that the param is removed by the 'Dopop' JavaScript function. Created attachment 55713 [details] [review] Bug 17315: (Bug 17210 follow-up) Fix "Save to lists" links from the result page On bug 17210, the selector .addtoshelf should not have caught the .addtoshelf nodes from the result list. To fix this, we just need to make the selector more specific (and cannot reuse it without more changes, the biblionumber variable is not the same - vs SEARCH_RESULT.biblionumber). Test plan: Make sure the 2 links (from detail and search result) "Save to lists" and "Save to your lists" work as expected. Created attachment 55716 [details] [review] Bug 17315: (Bug 17210 follow-up) Fix "Save to lists" links from the result page On bug 17210, the selector .addtoshelf should not have caught the .addtoshelf nodes from the result list. To fix this, we just need to make the selector more specific (and cannot reuse it without more changes, the biblionumber variable is not the same - vs SEARCH_RESULT.biblionumber). Test plan: Make sure the 2 links (from detail and search result) "Save to lists" and "Save to your lists" work as expected. Created attachment 55801 [details] [review] [PASSED QA] Bug 17315: (Bug 17210 follow-up) Fix "Save to lists" links from the result page On bug 17210, the selector .addtoshelf should not have caught the .addtoshelf nodes from the result list. To fix this, we just need to make the selector more specific (and cannot reuse it without more changes, the biblionumber variable is not the same - vs SEARCH_RESULT.biblionumber). Test plan: Make sure the 2 links (from detail and search result) "Save to lists" and "Save to your lists" work as expected. Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de> Pushed to master for 16.11, thanks Jonathan! Pushed in 16.05. Will be in 16.05.05. |