If you visit the detail page in the staff client it shows the patron name in the holdings table no matter what you choose for HidePatronName - it used to show the cardnumber instead of name if HidePatronName was set to 'Don't Show'
See here: http://screencast.com/t/dnidO2FbSefF
I'm on master.
Created attachment 7056 [details] [review]
Fixes T:T scope issue which prevented HidePatronName from working on 'on hold for' messages.
Created attachment 7057 [details] [review]
[SIGNED-OFF] Bug 7407: HidePatronName not working on 'on hold for' messages
HidePatronName was not working on 'on hold for' messages, only 'checked out to'
messages. This resulted in the patrons name showing if the item was on hold for
T:T scope issue; needs to be within itemloo. This patch does that.
Signed-off-by: Nicole C. Engard <email@example.com>
QA comment: one line patch, that is OK