Bug 26976 - When renewalsallowed is empty the UI is not correct
Summary: When renewalsallowed is empty the UI is not correct
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Hold requests (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low normal (vote)
Assignee: Jonathan Druart
QA Contact: Victor Grousset/tuxayo
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-11-09 14:40 UTC by Jonathan Druart
Modified: 2021-12-13 21:08 UTC (History)
4 users (show)

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


Attachments
Bug 26976: Display 0 if renewalsallowed is not defined (1.14 KB, patch)
2020-11-20 10:29 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 26976: Display 0 if renewalsallowed is not defined (1.19 KB, patch)
2020-11-20 13:57 UTC, David Nind
Details | Diff | Splinter Review
Bug 26976: Display 0 if renewalsallowed is not defined (1.25 KB, patch)
2020-12-02 21:36 UTC, Victor Grousset/tuxayo
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2020-11-09 14:40:23 UTC
On the checkout list I am seeing

"""
4
Not renewable (0 of renewals remaining)
"""

https://snipboard.io/rzOta6.jpg
Comment 1 Jonathan Druart 2020-11-20 10:29:24 UTC
Created attachment 113869 [details] [review]
Bug 26976: Display 0 if renewalsallowed is not defined

If the circulation rule "renewalsallowed" is not set (or an empty
string) then the checkout list displays
(0 of  renewals remaining)
It should display "0 of 0"

Test plan:
Set renewalsallowed to an empty string
Check an item out
Notice the change
Comment 2 David Nind 2020-11-20 13:57:54 UTC
Created attachment 113883 [details] [review]
Bug 26976: Display 0 if renewalsallowed is not defined

If the circulation rule "renewalsallowed" is not set (or an empty
string) then the checkout list displays
(0 of  renewals remaining)
It should display "0 of 0"

Test plan:
Set renewalsallowed to an empty string
Check an item out
Notice the change

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 Victor Grousset/tuxayo 2020-12-02 21:36:57 UTC
Created attachment 114120 [details] [review]
Bug 26976: Display 0 if renewalsallowed is not defined

If the circulation rule "renewalsallowed" is not set (or an empty
string) then the checkout list displays
(0 of  renewals remaining)
It should display "0 of 0"

Test plan:
Set renewalsallowed to an empty string
Check an item out
Notice the change

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Comment 4 Jonathan Druart 2020-12-04 16:04:52 UTC Comment hidden (obsolete)
Comment 5 Fridolin Somers 2020-12-07 08:18:46 UTC
Pushed to 20.11.x for 20.11.01
Comment 6 Jonathan Druart 2020-12-07 16:55:58 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 7 Andrew Fuerste-Henry 2020-12-15 20:48:07 UTC
Pushed to 20.05.x for 20.05.07
Comment 8 Victor Grousset/tuxayo 2020-12-17 13:49:54 UTC
Not backported to oldoldstable (19.11.x). Feel free to ask if it's needed.