Bugzilla – Bug 5051
Renewal due date doesn't always show on patron Checkout tab
Last modified: 2012-10-25 22:52:39 UTC
Created attachment 2402 [details]
Screen shot with missing renewal date box
On the patron Details tab, in the list of checkouts, there is always a box for entering a due date for renewals with a calendar gadget. This box does not always show on the patron Checkout tab. The pattern seems to be that if nothing was checked out to the patron today, the renewal due date box doesn't show (first screenshot). If you then check something out, the box shows (second screenshot), but it will be gone if you go back into the record tomorrow.
I don't think this is intended behavior but I'm not quite sure what's causing it.
Created attachment 2403 [details]
Screen shot where renewal date shows up after current day checkout
Note -- I tested this on an older system as well as current head -- same behavior.
I tested this today and the renewal date fields does not show up on the checkouts tab, only on the details tab.
An easy workaround is to always use the details tab, but it's still confusing and should work the same on both tabs.
Koha version: 3.01.00.145
Created attachment 2611 [details] [review]
Display of the renewal date footer in the template was conditional upon
the existence of checkouts from today.
In order to fix this bug it was necessary to repeat the markup for the
table footer twice in the template, so I created an include file for it.
The same include file can now be used in moremember.tmpl.
(In reply to comment #4)
> Created attachment 2611 [details] [review]
> Proposed fix
> Display of the renewal date footer in the template was conditional upon
> the existence of checkouts from today.
> In order to fix this bug it was necessary to repeat the markup for the
> table footer twice in the template, so I created an include file for it.
> The same include file can now be used in moremember.tmpl.
What is the status of the patch? Will it be accepted?
Patch pushed, please test
Seems to be fixed in current head. Thanks.