Bug 12132: display guarantees if a patron has them, regardless of category type
authorGalen Charlton <gmc@esilibrary.com>
Mon, 26 May 2014 03:09:18 +0000 (03:09 +0000)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Wed, 19 Nov 2014 16:29:35 +0000 (13:29 -0300)
commit857c68e2a667eea55255a373583c9ed69ea1cea9
tree608bae35163612a65e4cd0edab41c65ed593815f
parentbb186e09a6f895cd732dc4969da455d5d92c6f95
Bug 12132: display guarantees if a patron has them, regardless of category type

If a patron has guarantees, always display them on the patron summary,
even if the patron is of a type that ordinarily would have them.

For example, at present you can cannot directly add a guarantee to
a staff record the way you can do for an adult patron, but if you create
a juvenile patron and add a guarantor to it, you can override that
restriction.  Note that this patch ignores whether that is strictly
desirable behavior.

To test:

[1] Create a juvenile patron.  While editing it, make a staff
    account a guarantor of the new patron.
[2] View patron details for the staff account.  Note that the
    juvenile patron is not displayed as a guarantee.
[3] Apply this patch.
[4] Refresh details for the staff account.  The juvenile should
    now show up.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Frederic Demians <f.demians@tamil.fr>
Galen patch works as described. The way it's implemented seems sensible.
Whatever the reason why a patron has guarantes, it make sense to display
them. So it's better to check the count of guarantes rather than the
current patron type to decide to display guarantes.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, small change, no regressions found.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
members/moremember.pl