Bug 29115 - Placing a club hold is not showing warnings when unable to place a hold
Summary: Placing a club hold is not showing warnings when unable to place a hold
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: Hold requests (show other bugs)
Version: master
Hardware: All All
: P5 - low normal (vote)
Assignee: Joonas Kylmälä
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-09-26 15:36 UTC by Joonas Kylmälä
Modified: 2021-10-12 09:48 UTC (History)
2 users (show)

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


Attachments
Bug 29115: Fix syntax errors in club hold functionality (3.90 KB, patch)
2021-09-26 15:51 UTC, Joonas Kylmälä
Details | Diff | Splinter Review
Bug 29115: Fix syntax errors in club hold functionality (3.95 KB, patch)
2021-10-12 09:48 UTC, David Nind
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Joonas Kylmälä 2021-09-26 15:36:24 UTC
The code that is supposed to show warnings if someone in a club cannot reserve the item when placing a club hold is broken and not showing the warnings.
Comment 1 Joonas Kylmälä 2021-09-26 15:51:23 UTC
Created attachment 125340 [details] [review]
Bug 29115: Fix syntax errors in club hold functionality

The code didn't previously correctly do the checks for patron
debarment status and whether the patron has fees, this caused the
intended warnings messages to not appear when they should have.

To test:
 1) Create a new club and add a patron to it who has manual debarment
    and a manual charge of over $5 (more than the maxoutstanding syspref).
 2) Try to create a new club hold for the previously created club and
    notice no warnings are listed for the patron.
 3) Apply patch
 4) Try to create now the club hold again and notice the warnings
    being shown in the hold page.
Comment 2 David Nind 2021-10-12 09:48:12 UTC
Created attachment 126083 [details] [review]
Bug 29115: Fix syntax errors in club hold functionality

The code didn't previously correctly do the checks for patron
debarment status and whether the patron has fees, this caused the
intended warnings messages to not appear when they should have.

To test:
 1) Create a new club and add a patron to it who has manual debarment
    and a manual charge of over $5 (more than the maxoutstanding syspref).
 2) Try to create a new club hold for the previously created club and
    notice no warnings are listed for the patron.
 3) Apply patch
 4) Try to create now the club hold again and notice the warnings
    being shown in the hold page.

Signed-off-by: David Nind <david@davidnind.com>