Attachment 173221 Details for Bug 38243 – Bug 38243: Remove DT's header_filter custom param

[patch] Bug 38243: Remove DT's header_filter custom param

Bug-38243-Remove-DTs-headerfilter-custom-param.patch (text/plain), 2.40 KB, created by Jonathan Druart on 2024-10-23 14:51:41 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2024-10-23 14:51:41 UTC

Size: 2.40 KB

patch

obsolete

>From f58942a4e57ce4b9fc745e6f3894e138932b9435 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Wed, 23 Oct 2024 16:49:27 +0200
>Subject: [PATCH] Bug 38243: Remove DT's header_filter custom param
>
>It seems to be a leftover from bug 32401, the code does not make sense:
>
> 569                 if(options.header_filter) {
> 570                     options.query_parameters = query_parameters;
> 571                 } else {
> 572                     dataSet.q = query_parameters;
> 573                     delete options.query_parameters;
> 574                 }
>
>Then options.query_parameters is never used.
>---
> .../prog/en/modules/acqui/orderreceive.tt              |  1 -
> koha-tmpl/intranet-tmpl/prog/js/datatables.js          | 10 ++--------
> 2 files changed, 2 insertions(+), 9 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt
>index 9daac07657f..3842519e91e 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt
>@@ -1026,7 +1026,6 @@
>                     "ajax": {
>                         "url": pending_orders_url + "?q=" + encodeURI(JSON.stringify(base_query))
>                     },
>-                    "header_filter": true,
>                     "embed": [
>                         "invoice",
>                         "basket"
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/datatables.js b/koha-tmpl/intranet-tmpl/prog/js/datatables.js
>index 5f5d81a34f5..6758bbc8c77 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/datatables.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/datatables.js
>@@ -625,15 +625,9 @@ function _dt_default_ajax (params){
> 
>             if(query_parameters.length) {
>                 query_parameters = JSON.stringify(query_parameters.length === 1?query_parameters[0]:{"-and": query_parameters});
>-                if(options.header_filter) {
>-                    options.query_parameters = query_parameters;
>-                } else {
>-                    dataSet.q = query_parameters;
>-                    delete options.query_parameters;
>-                }
>-            } else {
>-                delete options.query_parameters;
>+                dataSet.q = query_parameters;
>             }
>+            delete options.query_parameters;
> 
>             dataSet._match = options.criteria;
> 
>-- 
>2.34.1

Actions: View | Diff | Splinter Review

Attachments on bug 38243: 173221 | 173222 | 173234 | 173235 | 173513 | 173514