Bugzilla – Attachment 191082 Details for
Bug 41564
Tidy kohaTable block - admin
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 41564: Tidy kohaTable - admin/aqbudgetperiods.tt
557704c.patch (text/plain), 3.09 KB, created by
Jonathan Druart
on 2026-01-09 14:44:50 UTC
(
hide
)
Description:
Bug 41564: Tidy kohaTable - admin/aqbudgetperiods.tt
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2026-01-09 14:44:50 UTC
Size:
3.09 KB
patch
obsolete
>From 557704c62bf24c9abc5020aed93770da59e41dd1 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Thu, 8 Jan 2026 10:42:01 +0100 >Subject: [PATCH] Bug 41564: Tidy kohaTable - admin/aqbudgetperiods.tt > >See bug 41324 comment 6 >1. Variable serialization >--- > .../prog/en/modules/admin/aqbudgetperiods.tt | 40 +++++++++++++------ > 1 file changed, 27 insertions(+), 13 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tt >index 7504480e77c..a081c285a06 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tt >@@ -561,29 +561,31 @@ > [% Asset.js("lib/jquery/plugins/treetable/jquery.treetable.js") | $raw %] > [% END %] > [% Asset.js("js/acq.js") | $raw %] >- <script> >- $(document).ready(function() { >- let dt_params = { >- pagingType: "full", >- autoWidth: false, >- }; > >- $("#activeperiodst").kohaTable(dt_params); >- $("#inactiveperiodst").kohaTable(dt_params); >+ [% IF close_form %] >+ <script> >+ const budget_period_description = "[% budget_period_description | html %]"; >+ </script> > >- [% IF close_form %] >+ <script> >+ $(document).ready(function() { > $("#budgeth").kohaTable({ > dom: "t", > }); > $("#move_form").submit(function(){ >- var budget_from = "[% budget_period_description | html %]"; >+ var budget_from = budget_period_description; > var budget_to = $("#to_budget_period_id").find("option:selected").html(); > var alert_message = _("You have chosen to move all unreceived orders from '%s' to '%s'.").format(budget_from, budget_to); > alert_message += "\n" + _("This action cannot be reversed. Do you wish to continue?"); > return confirm ( alert_message ); > }); >- [% END %] >- [% IF closed %] >+ }); >+ </script> >+ [% END %] >+ >+ [% IF closed %] >+ <script> >+ $(document).ready(function() { > var oTable = $("#closed_report").kohaTable({ > // The following is a c/p from aqbudgets.tt and is a candidate for refactoring. > drawCallback: function ( oSettings ) { >@@ -621,7 +623,19 @@ > autoWidth: false, > pagingType: "full_numbers" > }); >- [% END %] >+ }); >+ </script> >+ [% END %] >+ <script> >+ $(document).ready(function() { >+ let dt_params = { >+ pagingType: "full", >+ autoWidth: false, >+ }; >+ >+ $("#activeperiodst").kohaTable(dt_params); >+ $("#inactiveperiodst").kohaTable(dt_params); >+ > $("#add_modify_budget").validate({ > rules: { > budget_period_startdate: "required", >-- >2.43.0 >
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 41564
:
191082
|
191083
|
191084
|
191085
|
191086
|
191087
|
191088
|
191089
|
191090
|
191091
|
191092
|
191093
|
191094
|
191095
|
191096
|
191097
|
191098
|
191099
|
191100
|
191310
|
191311
|
191312
|
191313
|
191314
|
191315
|
191316
|
191317
|
191318
|
191319
|
191320
|
191321
|
191322
|
191323
|
191324
|
191325
|
191326
|
191327
|
191328
|
191329