HOLD_ITEM_CHECKED_OUT displayed instead of a user oriented message

[Bob Wicksall]

Evergreen 2.2 RC 1
Tpac

If a patron tries to place a hold on an item that they already have checked out they get the following message:

   Hold was not successfully placed
   Problem: HOLD_ITEM_CHECKED_OUT

All of the other messages seem to have been translated into something a little more friendly. Looks like this one was missed.

[Michael Peters (mrpeters)]

Perhaps we need to add a new ils_events.xml entry for HOLD_ITEM_CHECKED_OUT, similar to that of "ITEM_NOT_HOLDABLE"

        <event code='1220' textcode='ITEM_NOT_HOLDABLE'>
                <desc xml:lang="en-US">A copy with a remote circulating library (circ_lib) was encountered</desc>
        </event>

Simply stating the event code, followed by a user-friendly error message would seem to suffice.

    "Problem: HOLD_ITEM_CHECKED_OUT - The item you have attempted to hold is already checked out to you."

I'm not sure if there is additional work to be done beyond creating a new event code, however. Seems like a valid "bitesize" bug, though.

[Garry Collum (gcollum)]

Fix in user/gcollum/lp1012237_HOLD_ITEM_CHECKED_OUT_display

[Lebbeous Fogle-Weekley (lebbeous)]

Thanks Garry! This is now in master, rel_2_4, and rel_2_3.