Bug 32195 - Changing a patron category to disallow guarantors should remove existing guarantors
Summary: Changing a patron category to disallow guarantors should remove existing guar...
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Patrons (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on: 12446
Blocks:
  Show dependency tree / graph
 
Reported: 2022-11-14 16:47 UTC by Andrew Fuerste-Henry
Modified: 2024-07-26 15:08 UTC (History)
7 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Fuerste-Henry 2022-11-14 16:47:14 UTC 
To recreate:
1: have a patron category where CanBeGuarantee is set to Yes.
2: Create a patron in that category, give them a guarantor
3: edit the patron category to set CanBeGuarantee to No
4: confirm your patron still has a guarantor that shows on the details page, but the guarantor no longer shows on the edit screen
5: this means the guarantor cannot be removed :(

I see a few ways to move forward. We could:
- make Koha go find and remove relationships right away when a category is updated
- provide a script that could be run/cronned to find and delete relationships from the command line
Comment 1 Liz Rea 2023-10-27 14:39:19 UTC 
I think we just need to always show the "remove" link no matter what the category says - if there is a guarantor we at least need to be able to remove it. The main problem here being that we are editing the category underneath an existing patron and the settings aren't retroactive to patrons who now don't fit the defined category.
Comment 2 Justin Swink 2024-07-26 15:08:14 UTC 
I agree with Liz. Having it set to always show "remove" would be best.