Bug 37451 - Restrictions due to discharge should not be able to be removed
Summary: Restrictions due to discharge should not be able to be removed
Status: Needs Signoff
Alias: None
Product: Koha
Classification: Unclassified
Component: Patrons (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-07-24 13:51 UTC by Baptiste Wojtkowski (bwoj)
Modified: 2024-07-24 13:57 UTC (History)
2 users (show)

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


Attachments
Bug 37451 - Remove the ability to delete a restriction due to discharge (2.98 KB, patch)
2024-07-24 13:57 UTC, Baptiste Wojtkowski (bwoj)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Baptiste Wojtkowski (bwoj) 2024-07-24 13:51:08 UTC
However it is not possible to remove the discharge, it is possible to remove the suspension associated to the discharge. This can lead to a lot of undesired behaviors when the user asks for another discharge or get suspended for another reason.

Suspension due to discharge should therefore be definitive.
Comment 1 Baptiste Wojtkowski (bwoj) 2024-07-24 13:57:19 UTC
Created attachment 169491 [details] [review]
Bug 37451 - Remove the ability to delete a restriction due to discharge

However it is not possible to remove the discharge, it is possible to remove the suspension associated to the discharge. This can lead to a lot of undesired behaviors when the user asks for another discharge or get suspended for another reason.

Suspension due to discharge should therefore be definitive.

TEST PLAN:
1 - Go to patron -> pick a patron -> go to discharges
2 - Generate a discharge for the patron
3 - Go to check out -> restrictions
4 - There should be a restriction with type discharge that can be
  deleted
5 - Apply patch
6 - There still should be a restriction but the button has been replaced
  by an infobox