Web Client: each item scanned into Check In reloads the whole page/list

[Christine Burns]

Tested on EG3

During Check In, each item scanned into Check In reloads the whole page/list.

Description
scan first barcode -> check in sound is made -> item is checked in and appears in the list
scan second barcode -> list temporarily disappears, screen flashes, check in sound is made, list reappears

each barcode scanned in check in causes the list to reload.

This is slowing down operations at the check in desk.

Desired behaviour
when an item barcode is scanned into check in - the list of items previously checked in items should not refresh

[Anna Goben (agoben)]

One of my libraries reported that they are having to reload the interface fresh after approximately every 15 items or it is simply too slow to use.

[Remington Steed (rjs7)]

I actually see the grid refresh twice:

- The first time, to display a blank row, where the new item will be displayed once it is fetched
- A second time, to populate the blank row with fetched data

I would be simple to remove the first refresh (which should cut the delay in half), but there might be a good reason to keep it as is.

[Bill Erickson (berick)]

If I recall correctly, the stub entry is added to the grid so the checkins will appear in the order they are scanned, and not the order the API response returns. The order could be maintained through other means, though.

[Jeff Davis (jdavis-sitka)]

Here's a branch that removes the first grid refresh during checkin, as Remington suggested:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jeffdavis/lp1743045-checkin-refresh

It looks to me like this preserves the order of checkins in the grid.

[Remington Steed (rjs7)]

Tested on a slow server by checking in items quickly in a row, and it appears the item position is correctly tracked in the data array without the refresh.

Here's my sign-off branch:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/rsteed/lp1743045-checkin-refresh

[Bill Erickson (berick)]

Confirmed redundant refresh on checkin is no more. Thanks Jeff and Remington! Merged to master and rel_3_0.

[Christine Burns (christine-burns)]

Hello

While this fix has made a slight improvement to check-in this has not completely fixed the issue.

Each check in still rebuilds the list and causes the screen to flash - this slows down check in and causes issues for the check in staff member (flashing screen)

I have attached a very short video which captures this list rebuilding issue

[Josh Stompro (u-launchpad-stompro-org)]

See bug 1777207 for more work related to this issue.