Bug 9392: confirmation does not appear for negative notforloan items issues
authorFridolyn SOMERS <fridolyn.somers@biblibre.com>
Tue, 15 Jan 2013 08:24:25 +0000 (09:24 +0100)
committerJared Camins-Esakov <jcamins@cpbibliography.com>
Sat, 9 Feb 2013 02:38:49 +0000 (21:38 -0500)
commit755decfd892c5411d8ead67e44013307933cf6e9
treed42bb6979bfa76a12e95e23bed8ba26ecf1868bb
parentf481d9259d889f74dd556e17ed6d4aa9a3a7f50a
Bug 9392: confirmation does not appear for negative notforloan items issues

When checking-out an item marked "not for loan" with a positive value,
you get a message to forbid or confirm issue (depending on
AllowNotForLoanOverride syspref).  For an item with a negative value
(usually during acquisition process), the issue is made without forbid
or confirm message.

This patch simply changes the test notforloan > 0 into notforloan != 0.

Test plan :
- Checkout an item with items.notforloan = 0
=> The item is issued without message
- Checkout an item with items.notforloan > 0
=> A message forbids or asks confirmation to override
- Checkout an item with items.notforloan < 0 (e.g., on order)
=> A message forbids or asks confirmation to override

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Checked with AllowNotForLoanOverride on and off.
All tests and QA script pass.
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
C4/Circulation.pm