On the patron detail screen there are two list items: -Show checkouts to guarantor -Show charges to guarantor I end up hiding these with CSS often. It would be better if these simply did not show if borrowerRelationship is empty.
Created attachment 150257 [details] [review] Bug 33620: Don't show guarantor info on details page if borrowerRelationship is empty To test: 1. Apply patch 2. Have some values in borrowerRelationship. 3. Go to a patron details page and see both 'Show checkouts to guarantor:' and 'Show charges to guarantor:' 4. Now empty borrowerRelationship. 5. Visit the patron detail page again and you should no longer see either of those fields.
Created attachment 150297 [details] [review] Bug 33620: Don't show guarantor info on details page if borrowerRelationship is empty To test: 1. Apply patch 2. Have some values in borrowerRelationship. 3. Go to a patron details page and see both 'Show checkouts to guarantor:' and 'Show charges to guarantor:' 4. Now empty borrowerRelationship. 5. Visit the patron detail page again and you should no longer see either of those fields. Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>
Works as described, thanks for the patch.
Created attachment 153188 [details] [review] Bug 33620: Don't show guarantor info on details page if borrowerRelationship is empty To test: 1. Apply patch 2. Have some values in borrowerRelationship. 3. Go to a patron details page and see both 'Show checkouts to guarantor:' and 'Show charges to guarantor:' 4. Now empty borrowerRelationship. 5. Visit the patron detail page again and you should no longer see either of those fields.
(In reply to Sally from comment #3) > Works as described, thanks for the patch. You forgot your signoff line, Sally.
Created attachment 155654 [details] [review] Bug 33620: Don't show guarantor info on details page if borrowerRelationship is empty To test: 1. Apply patch 2. Have some values in borrowerRelationship. 3. Go to a patron details page and see both 'Show checkouts to guarantor:' and 'Show charges to guarantor:' 4. Now empty borrowerRelationship. 5. Visit the patron detail page again and you should no longer see either of those fields. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
(In reply to Marcel de Rooy from comment #5) > (In reply to Sally from comment #3) > > Works as described, thanks for the patch. > > You forgot your signoff line, Sally. I don't think I did? I signed off on a sandbox. You can see my sign off line on comment #2: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33620#c2 It looks like Lucas updated the patch again after my sign off in comments 2 and 3, and added a new patch on comment 4 a few months later?
Pushed to master for 23.11. Nice work everyone, thanks!
Enhancement not pushed to 23.05.x