Bug 31436 - AllowRenewalIfOtherItemsAvailable should not take suspended holds into account
Summary: AllowRenewalIfOtherItemsAvailable should not take suspended holds into account
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low normal (vote)
Assignee: Kyle M Hall
QA Contact: Testopia
URL:
Keywords:
Depends on: 31112
Blocks:
  Show dependency treegraph
 
Reported: 2022-08-22 11:19 UTC by Kyle M Hall
Modified: 2023-06-08 22:26 UTC (History)
3 users (show)

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


Attachments
Bug 31436: AllowRenewalIfOtherItemsAvailable should not take suspended holds into account (1.08 KB, patch)
2022-08-22 11:22 UTC, Kyle M Hall
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Kyle M Hall 2022-08-22 11:19:24 UTC
If AllowRenewalIfOtherItemsAvailable is enabled, Koha checks to make sure all other holds are fillable before renewing. It doesn't make sense to look at frozen holds for this reason, as they are by definition, unfillable.
Comment 1 Kyle M Hall 2022-08-22 11:22:24 UTC
Created attachment 139592 [details] [review]
Bug 31436: AllowRenewalIfOtherItemsAvailable should not take suspended holds into account

If AllowRenewalIfOtherItemsAvailable is enabled, Koha checks to make sure all other holds are fillable before renewing.
It doesn't make sense to look at suspended holds for this reason, as they are by definition, unfillable.
Comment 2 Joonas Kylmälä 2022-08-28 15:05:08 UTC
The bug 31112, dependency for this bug already fixed this issue. Closing.