When adding a record to a list using the links in the result list, the link contains the biblionumber, but in the window the URL ends 'biblionumber=' and the entry is not added to the list.
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.