LP#2037564: Selfcheck cover art without ISBNS not shown
Fixes the display of uploaded cover images in the Selfcheck holds list.
Josh Stompro needs to get credit for this, he explained exactly how to
fix it in the LP bug.
To test in concerto:
1. Find a bib with a cover image and download it.
(example search: piano concerto)
2. Upload the cover image to a bib that does not have a cover image.
3. In a patron record, place holds for these 2 titles, and one other
title that does not have an image.
4. Initialize and login to selfcheck
(https://[hostname]/eg/circ/selfcheck/main)
5. Login as the patron with holds and view the holds list. The uploaded
image is not displayed.
6. Apply the patch.
7. Repeat steps 4 & 5. The uploaded image now displays.
Release-note: Fixes issue where cover images were not displayed in the selfcheck holds list for titles that lack ISBNs.
LP2060153: Store remote file for EDI message sent via SFTP
In a case of "Net::SFTP::Foreign is not Net::FTP," the remote filename
is not recorded when sending an EDI message via SFTP. Instead, the
return value from the SFTP put function is stored in the remote_file
column of the acq.edi_message entry. This is "1" on success and null
or undefined on failure. This commit corrects that and puts the
remote filename in the proper place.
To verify the bug, set up a test account to send EDI messages via SFTP
to another server or to another account on the local host. Create an
acquisitions order with that account and then activate it. Run the
EDI order pusher (edi_order_pusher.pl). Verify that the order arrived
in the destination location, and then check the acq.edi_message table
entry for this order. The remote_file column value should be "1".
To verify the fix, repeat the above steps with a new order. When you
check the acq.edi_message table, the remote_file column value should
now be the path to the file at the destination.
Release-Note: Get actual remote file name when sending EDI message via SFTP.
LP1915464 Angular Form Field Order - Hold Policies
Attempts to apply some logical order to the extensive field list for
Administration > Local Administration > Hold Policies (as opposed to
the fields being displayed alphabetically). New order is roughly
Description - Active - Owning & Circ Libs - Record & Item Info -
User Info - Hold Info - Transit Info.
Release-note: Reorders fields for Local Admin > Hold Policies