angular catalog view holds sort by patron barcode error retrieving results

[Lindsay Stratton]

In 3.6.3 Staff Catalog Search > View Holds adding the Patron Barcode column and attempting to sort by column triggers an "Error Retrieving Results" alert in place of the record grid.

Testing steps:

1) Opened a title with holds (the title as a whole has over 500 holds on 166 items)
2) Clicked View Holds, 100 rows display
3) Set a Pickup Library, 31 holds
4) Clicked sort by Request Date - sorted as expected
5) Sort by Last Name, User Display Name, User Requestor Name, and other columns - sorted as expected
6) Sort by Patron Barcode (link) - Error Retrieving Results

[Garry Collum (gcollum)]

Confirmed in 3.6.3 and in current master.

[Michele Morgan (mmorgan)]

We are also seeing this in 3.6.1. The error from the console is below:

ERROR Error: Uncaught (in promise): open-ils.circ.hold.wide_hash.stream failed! stat=500 msg= *** Call to [open-ils.circ.hold.wide_hash.stream] failed for session [0.29374818081881451621264787656], thread trace [0]:
Exception: OpenSRF::DomainObject::oilsMethodException 2021-05-17T11:19:47 OpenSRF::Application /usr/local/share/perl/5.20.2/OpenSRF/Application.pm:243 <500> A server error occurred during method execution

    at Z (polyfills.bad64ce58709ffa2ea6b.js:1)
    at polyfills.bad64ce58709ffa2ea6b.js:1
    at l._error (12.1b549fc489d7cd25ae17.js:1)
    at l.__tryOrUnsub (main.658588bc0be6835a3a32.js:1)
    at l.error (main.658588bc0be6835a3a32.js:1)
    at u._error (main.658588bc0be6835a3a32.js:1)
    at u.error (main.658588bc0be6835a3a32.js:1)
    at l._error (6.d9f682c236bab73ef40e.js:1)
    at l.__tryOrUnsub (main.658588bc0be6835a3a32.js:1)
    at l.error (main.658588bc0be6835a3a32.js:1)

[Dan Briem (dbriem)]

Branch for testing: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbriem/lp_1928684_cat_holds_patron_barcode_sort

The name for the column didn't match the field name in the database. Note: due to the name change, users with the Patron Barcode column saved on the View Holds tab grid will have to add it again.

[Garry Collum (gcollum)]

Works great. A signed-off branch is at https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/gcollum/lp_1928684_cat_holds_patron_barcode_sort-signoff

[Galen Charlton (gmc)]

Pushed all the way to rel_3_6. Thanks, Dan and Garry!