Created attachment 5103 [details]
Each table of existing holds should show a title
I've found two template variable scope errors in request.tt: One prevents the display of titles when the user is placing holds on multiple items. The other prevents the display of the patron information when there is a error placing a hold.
Created attachment 5104 [details]
No patron name shows in error message
Error should read "[Patron name] already has a hold on this item"
Created attachment 5105 [details] [review]
Created attachment 5106 [details]
Multiple holds after correction
Created attachment 5107 [details]
After correction, patron name appears in error message
Going to give this one a double-jump: please consider this my sign off, as well as the passing of the QA.
Template only change, T:T scope fix, easy to read, documented VERY well with screen shots.
Pushed please test