Bug 24522: Show alert when trying to add nothing to a list in staff
authorOwen Leonard <oleonard@myacpl.org>
Fri, 1 May 2020 13:23:12 +0000 (13:23 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 4 May 2020 08:44:42 +0000 (09:44 +0100)
This patch modifies the JavaScript which handles the process of adding
items to a list in the staff client. Error-handling is added in the case
when a set of biblionumbers hasn't been passed to the vShelf function.

To test, apply the patch and perform a catalog search in the staff
client.

On the search results page, select any list from the "Add to list" menu
at the top of the search results table without checking any checkboxes.

You should get an alert, "No item was selected."

Also test that adding to a list works correctly after selecting one or
more titles.

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

koha-tmpl/intranet-tmpl/prog/js/basket.js

index 5eadc9a..aa5320f 100644 (file)
@@ -452,6 +452,8 @@ function vShelfAdd(biblist) {
                 bibs.push("biblionumber=" + bibnums[i]);
             }
             return bibs.join("&");
+        } else {
+            showListsUpdate(MSG_NO_RECORD_SELECTED);
         }
     }
 }