Bug 26983

Summary: Selecting ALL Items in Inventory- only selects 20
Product: Koha Reporter: Kelly McElligott <kelly>
Component: ToolsAssignee: Jonathan Druart <jonathan.druart+koha>
Status: RESOLVED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: andrew, caroline.cyr-la-rose, fridolin.somers, jonathan.druart+koha, julian.maurice, mfuerst, victor
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Text to go in the release notes:
Version(s) released in:
21.05.00,20.11.02,20.05.08
Bug Depends on: 26150    
Bug Blocks:    
Attachments: Bug 26983: Fix [un]select all links in inventory
Bug 26983: Fix [un]select all links in inventory
Bug 26983: Fix [un]select all links in inventory

Description Kelly McElligott 2020-11-10 01:10:37 UTC
When a user is in the Inventory Tool, Select all only selects the first 20 records in the selecting option of the Inventory, even if you change the view to show 50, 100, or all options. The expected behavior is that Select All would select all the options.

To recreate this issue, go to Tools-> Inventory-> Do a Search to produce more than 20 results -> Select Add Items. Next, select 50, 100 or All entries to show. Then, click on Select All. Only the first 20 entries are selected, not all of the entries on the screen.
Comment 1 Jonathan Druart 2020-12-02 09:28:29 UTC
Created attachment 114087 [details] [review]
Bug 26983: Fix [un]select all links in inventory

The select/unselect all links took into account only the items displayed
when the page was loaded (so 20).

Test plan:
1. Go to Tools-> Inventory-> Do a Search to produce more than 20 results
2. Select 50, 100 or All entries to show
3. click on Select All
=> All items must be selected, not only the first 20
Comment 2 Fridolin Somers 2020-12-11 10:49:20 UTC
Created attachment 114325 [details] [review]
Bug 26983: Fix [un]select all links in inventory

The select/unselect all links took into account only the items displayed
when the page was loaded (so 20).

Test plan:
1. Go to Tools-> Inventory-> Do a Search to produce more than 20 results
2. Select 50, 100 or All entries to show
3. click on Select All
=> All items must be selected, not only the first 20

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Comment 3 Fridolin Somers 2020-12-11 10:52:15 UTC
Ok fix is working.

Will testing I see that "Select all" in fact selects the visible lines.
Maybe we should rename it "Select visible rows", like in items search.
Comment 4 Julian Maurice 2021-01-08 10:41:42 UTC
Created attachment 114951 [details] [review]
Bug 26983: Fix [un]select all links in inventory

The select/unselect all links took into account only the items displayed
when the page was loaded (so 20).

Test plan:
1. Go to Tools-> Inventory-> Do a Search to produce more than 20 results
2. Select 50, 100 or All entries to show
3. click on Select All
=> All items must be selected, not only the first 20

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Comment 5 Jonathan Druart 2021-01-08 14:23:14 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 6 Martha Fuerst 2021-01-08 18:07:00 UTC
Any chance this can be backported to 20.05?
Comment 7 Fridolin Somers 2021-01-11 12:42:05 UTC
Pushed to 20.11.x for 20.11.02
Comment 8 Fridolin Somers 2021-01-11 12:43:33 UTC
(In reply to Martha Fuerst from comment #6)
> Any chance this can be backported to 20.05?

Surely Andrew will backport to 20.05.xx
Comment 9 Andrew Fuerste-Henry 2021-01-19 21:45:01 UTC
Surely I have done so :) Pushed to 20.05.x for 20.05.08
Comment 10 Victor Grousset/tuxayo 2021-01-22 01:34:46 UTC
Not backported to oldoldstable (19.11.x). Feel free to ask if it's needed.