Want easy way to clear a hold when picked up by other patron

Changing the UI in the XUL client seems involved. I am working on a solution in the web-based client.

Adding a link to a previous list discussion on this topic:
http://georgialibraries.markmail.org/thread/c3pm2jzgx45p4plf

Here is my cut at an implementation.
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dpearl/hold_checkout_A

Added test information to commit. Squashed two commits. Signed off.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dpearl/hold_checkout_B

Thanks, Dan! This looks like a good improvement, and doesn't forestall the "friends" variant.

I have one nit to pick, though, that may require some input. The global default for the new checkbox is to be checked, which is a behavioral change. As a rule we try not to push a configurable behavior change at upgrades, so I think this should probably not insert the setting at the top of the org tree as it does currently, and rely on the release notes to inform ILS admins about the new feature and how to configure it.

Thoughts?

I agree with your suggestion, and will submit a new branch soon.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dpearl/LP1661688_hold_checkout_other_patron

This changes the default on a consortium-level to false (unchecked).

Hi Dan,

I tested this code, but I'm having trouble with the scenario where I choose the option to Cancel the hold for the other user. When I choose that option, the hold is indeed canceled, but the checkout does not occur. I see the following message in the Console:

net.js:119 error calling method open-ils.circ.checkout.full.override : 500 : *** Call to [open-ils.circ.checkout.full.override] failed for session [0.209453098711193771504024847181], thread trace [0]:
Can't call method "usr" on an undefined value at /usr/local/share/perl/5.18.2/OpenILS/Application/Circ/Circulate.pm line 885.

The item remains on the Holds Shelf with a Cancelled status.

Also, in reference to Mike's earlier comment, I think the precedent has been to set a Null value for that setting rather than setting it to False.

Thanks for your work on this!
Kathy

Thanks for the review, Kathy,

I will look into those issues.

Rather, just don't set the setting at all. The code that looks up settings returns a null value if it isn't set.

I took a look at this branch and resolved Kathy's error in comment #8. I also changed the code to flesh the hold usr that was added by Dan's branch. We don't need to flesh text fields on a fleshed object. We only need to flesh the user.

I didn't make any other changes to the code, but I did notice that the hold cancellation does not work if the setting is not set to true or false in actor.org_unit_setting. Deleting the row leads to the hold not being canceled, even if the box is checked in the dialog. I suspect this happens because of line 33 in Open-ILS/web/js/ui/default/staff/circ/services/circ.js. It does exactly what the previous line does and pull an aous valued directly into a Boolean. I suspect that if the setting isn't set in aous or is null, then we get something neither true nor false and the code doesn't work. I'm not sure this needs to be addressed since existing code does the same. I'm just mentioning it in light of my comment #10.

My code is rebased on master and can be found in working/collab/dyrcona/LP1661688_hold_checkout_other_patron http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/dyrcona/LP1661688_hold_checkout_other_patron

I'll leave any other potential modifications open for discussion.

My tests are completed.

user/dpearl/LP1661688_alt_patron_pickup

This works well in testing. I signed off on the commit, but added another commit to add some tweaks to the code:

1. I added a link to the record of the patron who originally placed a hold. In the email thread I linked to in comment #2, Tina Ji had suggested we add this link. "This will help staff check whether there is any note or alert or similar message in the account allowing the hold to be pickup up other people (to address the privacy issue)" For some reason, her message doesn't show up in the archive, but it's still in my Inbox, and I think it's a good addition to this code. The link will open in a new tab so as not to disrupt the ongoing checkout.

2. I tweaked the language a bit because I thought the phrasing - "This item is currently on the holds shelf for another patron for John Smith" - was a little awkward. It now says "This item is currently on the holds shelf for another patron: John Smith"

3. I removed the setting so that it would return as null, as described by Jason in comment #10.

The new branch is available at http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/kmlussier/LP1661688_alt_patron_pickup_signoff_and_additions

I know a lot of people who are going to be very happy with this feature when it's available!

Pushed to master, along with a couple minor follow-ups of my own. Thanks, Dan, Jason, and Kathy!