Bugzilla – Attachment 184551 Details for
Bug 39421
Renewal date input field (and date picker) not showing on Circulation > Renew
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[24.11] Bug 39421: Restore missing date picker on Circulation > Renew
2411-Bug-39421-Restore-missing-date-picker-on-Circ.patch (text/plain), 2.45 KB, created by
Pedro Amorim
on 2025-07-23 13:15:46 UTC
(
hide
)
Description:
[24.11] Bug 39421: Restore missing date picker on Circulation > Renew
Filename:
MIME Type:
Creator:
Pedro Amorim
Created:
2025-07-23 13:15:46 UTC
Size:
2.45 KB
patch
obsolete
>From 80afd5d93b6a63297238c9bc1a6166c0851cf823 Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@bywatersolutions.com> >Date: Sat, 22 Mar 2025 15:34:02 +0000 >Subject: [PATCH] [24.11] Bug 39421: Restore missing date picker on Circulation > > Renew > >This patch restores the renewal date input field (with a date >picker) for Circulation > Renew. This was missing. > >It changes the behavor slightly so that it mirrors Circulation > >Check in. A settings icon is included in the barcode field, >clicking this then shows the date field with the date picker >(and any other options). > >Previously on the renewals page (24.05.x and earlier), there was >no settings icon in the barcode field and the date input field >with date picker was already shown. > >This user interface pattern is used elsewhere in Koha, and this >chnage will help improve consistency. > >Test plan: >1. Browse to Circulation > Check in. >2. Note the settings icon in the barcode form field. >3. Click it to show the the return date field with a date picker, > and other options. >4. Browse to Circulation > Renew. >5. Note there is no settings icon. >6. Apply the patch. >7. Repeat step 4 - note that there is now a settings icon. >8. Click the settings icon to show the renewal date form field. >9. Click the input field to show teh date picker. > >Signed-off-by: David Nind <david@davidnind.com> > >Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> > >PA amended: Squashed both patches for 24.11 >--- > koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt | 4 ++++ > 1 file changed, 4 insertions(+) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt >index e587225f4b1..0328b2fcc6a 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt >@@ -232,6 +232,9 @@ > > [% END %] > <input type="hidden" name="op" value="cud-renew" /> >+ <div id="show-circ-settings"> >+ <a href="#" title="Renewal settings"><i class="fa-solid fa-sliders"></i></a> >+ </div> > <button type="submit" class="btn btn-primary">Submit</button> > > <div class="circ-settings show"> >@@ -260,6 +263,7 @@ > </div> <!-- /.row --> > > [% MACRO jsinclude BLOCK %] >+ [% Asset.js("js/pages/circulation.js") | $raw %] > [% INCLUDE 'calendar.inc' %] > [% IF error %] > <script> >-- >2.39.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 39421
:
179631
|
179897
|
181159
|
181250
|
181251
| 184551