From 0af9d12027b533ecd945c66d19e7a60c6cd96fb2 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 16 Feb 2024 13:06:55 +0000 Subject: [PATCH] Bug 34478: Fix op variable in item circulation alerts The AJAX call in the template still used "action" instead of "op". The patch also fixes references to "action" in the POD and corrects "toggle" to "cud-toggle". Signed-off-by: Jonathan Druart --- admin/item_circulation_alerts.pl | 4 ++-- .../prog/en/modules/admin/item_circulation_alerts.tt | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/admin/item_circulation_alerts.pl b/admin/item_circulation_alerts.pl index c2ba883635..997b33c3fd 100755 --- a/admin/item_circulation_alerts.pl +++ b/admin/item_circulation_alerts.pl @@ -146,7 +146,7 @@ patron category, and item type, this is where that policy would be set. =head2 URLs -=head3 ?action=show +=head3 ?op=show Display a branches item circulation alert preferences. @@ -163,7 +163,7 @@ branch '*' is used. -=head3 ?action=toggle +=head3 ?op=cud-toggle Toggle a preference via AJAX diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt index af1005fc5a..6f94677278 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt @@ -211,7 +211,7 @@ url : '/cgi-bin/koha/admin/item_circulation_alerts.pl', type : 'POST', dataType : 'json', - data : { action: 'cud-toggle', id: id, branch: $branch }, + data : { op: 'cud-toggle', id: id, branch: $branch }, success : function(response){ if ($branch == '*' && response.classes.match(/default/)) { td.html(disabledForAll); -- 2.39.5