From 80f959091a4f78e001ccf9221abaa6d6c7edbc6a Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 23 Sep 2024 11:13:45 +0200 Subject: [PATCH] Bug 37982: Replace edit form with GET Caught by tests from bug 37728 # $VAR3 = 'koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt:158'; We don't really need the form to be POSTed, we should not reach the URL maxlength. The UI will certainly explode if there are lot of items to edit anyway. The code is not clear on this page as we are sending several forms using the items selected from this table. But this one form is used to display the edit form and so can be GET (and better as the URL can be shared). The other actions have their own form (in modals mostly). Test plan: Create a subscription and go to the serial collection page. Make sure at least one item is select and click the "Edit" button. No behaviour change expected here, the edit form should still be displayed. Try the other actions on top of the table and confirm they are still working as before. Signed-off-by: Phil Ringnalda Signed-off-by: Pedro Amorim --- .../prog/en/modules/serials/serials-collection.tt | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt index 7d4d6954f83..958b00837ed 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt @@ -155,8 +155,7 @@ [% END # /IF subscriptions %] [% IF ( years ) %] -
- [% INCLUDE 'csrf-token.inc' %] + [% WRAPPER tabs id= "subscription_years" %] [% WRAPPER tabs_nav %] [% FOREACH year IN years %] -- 2.39.2