As it happens, changing the order of the reasons in place_hold_result.tt2 will make the more relevant messages float to the top. Here's a branch that EI is currently using: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/jboyer/lp1481441_hold_fail_reason
Now, when a hold is denied because of patron fines or other penalties, that reason will be shown, in my testing I only saw the transit message when it was the only reason for the failure.
As it happens, changing the order of the reasons in place_hold_ result. tt2 will make the more relevant messages float to the top. Here's a branch that EI is currently using: http:// git.evergreen- ils.org/ ?p=working/ Evergreen. git;a=shortlog; h=refs/ heads/collab/ jboyer/ lp1481441_ hold_fail_ reason
Now, when a hold is denied because of patron fines or other penalties, that reason will be shown, in my testing I only saw the transit message when it was the only reason for the failure.