Bug 26976

Summary: When renewalsallowed is empty the UI is not correct
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Hold requestsAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Victor Grousset/tuxayo <victor>
Severity: normal    
Priority: P5 - low CC: andrewfh, fridolin.somers, gmcharlt, victor
Version: unspecified   
Hardware: All   
OS: All   
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
Bug 26976: Display 0 if renewalsallowed is not defined
Bug 26976: Display 0 if renewalsallowed is not defined

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.