Basket Actions: Check / Clear all box doesn't remove items from the basket

[Jason Boyer]

Eg 3.2 beta

If you check the box at the top of a list of results to select the entire page that works as expected and you will have 10 (or however many) records in your basket. If you click it again though to clear the screen the boxes will be un-checked, but the number of records in the basket will still be 10. Clicking it again increases the count to 20, and so on. It's a simple fix, branch is on the way.

[Jason Boyer (jboyer)]

Right about here: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jboyer/lp1792371_clear_selection working/user/jboyer/lp1792371_clear_selection

To test:
Pre-patch, perform a search, click the checkbox above the list of results to select and add all records on the page to your basket. The number of items in your basket should be 10. Click it again to clear all of the checkboxes, the number of items in your basket is still 10.

Apply patch and repeat steps above, notice that now the 10 items in your basket are removed when you clear the whole-page checkbox.

[Terran McCanna (tmccanna)]

Works like a charm!

I have tested this code and consent to signing off on it with my name, Terran McCanna, and my email address, <email address hidden>.

[Kathy Lussier (klussier)]

Works for me too! I added Terran's signoff and my own and merged the changes to master for inclusion in the upcoming 3.2 RC. Thanks Jason and Terran!