webclient offline : expired patrons should show date of block list download

[Andrea Neiman]

In the XUL client, if you have downloaded the Offline Block List and attempt an offline checkout with an expired patron, you get this message:

Warning: as of DATE, this barcode (nnnnnn) was flagged Expired.

(note that DATE is the download date, not the date the patron was marked expired).

In the web client, you are informed that the patron is Expired but without the date reference.

Having the date reference is a helpful indicator to know when the block list was downloaded & would be nice to incorporate into the web client's offline mode.

I'd recommend a wording adjustment so it's more clear that the date referenced is the download date, and it would be extra great to have this wording show on all blocked patrons in Offline.

[Erica Rohlfs (erohlfs)]

Confirmed on 3.0.3 I also agree that displaying the last download date is a very helpful troubleshooting tool (and training tool).

[Mike Risher (mrisher)]

I've pushed a branch here
https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mrisher/lp1752356-offline-blocklist-date-addition

[Bill Erickson (berick)]

Thanks, All. I have pushed a branch with a sign off for Mike's patch, plus an additional commit.

From the commit msg:

Modify wording to clarify the date displayed is that of the block list download and not necessarily the date the patron was flagged expired.

Avoid console errors when the file download date is not known -- edge case where the code is updated but the block list was not updated in the meantime.

Clean up some debug logging.

==

I also took the liberty of tweaking Mike's commit message to suit community standards (LP number, formatting).

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1752356-offline-block-patron-exp-date

[Terran McCanna (tmccanna)]

Mike / Bill - I am hitting some errors when I apply this to a test server running current master. Could you please rebase?

[Mike Risher (mrisher)]

I took my work and Bill's, rebased it, tested it, and put it on this branch:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mrisher/lp1752356-offline-block-patron-exp-rebase

[Mike Risher (mrisher)]

Some issues came up testing this. Another branch with the work is here:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mrisher/lp1752356-offline-block-patron-exp-date-v2

[Terran McCanna (tmccanna)]

Thanks, Mike and Bill. I found in testing that the new message had been applied to accounts marked Barred rather than accounts marked Expired, so my sign off is here with that correction. I've also taken the liberty of adding similar wording (as per Bill's commit) to the other reasons "WHY" accounts can appear on the block list.

A signoff on my changes would be most appreciated.

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mccanna/lp1752356-offline-block-list-download-date

[Mike Risher (mrisher)]

Thank you for the improvement, Terran. I tested each of the 4 error messages you can get when trying to check out a patron on the block list. They all look good to me. I consent to signing off to this with my name Mike Risher (<email address hidden>)

[Jason Boyer (jboyer)]

Pushed to rel_3_6 through master; thanks everyone!