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: master
Hardware: All All
: P5 - low normal (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on: 12446
Blocks:
  Show dependency treegraph
 
Reported: 2022-11-14 16:47 UTC by Andrew Fuerste-Henry
Modified: 2024-02-14 10:40 UTC (History)
5 users (show)

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


Attachments

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.