Attachment 172335 Details for Bug 38069 – Bug 38069: (bug 37955 follow-up) Fix edition of table settings

[patch] Bug 38069: (bug 37955 follow-up) Fix edition of table settings

Bug-38069-bug-37955-follow-up-Fix-edition-of-table.patch (text/plain), 4.76 KB, created by Jonathan Druart on 2024-10-03 07:55:22 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2024-10-03 07:55:22 UTC

Size: 4.76 KB

patch

obsolete

>From 16f170600a0a0d260188fbf365b75ad637c01277 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Thu, 3 Oct 2024 09:51:31 +0200
>Subject: [PATCH] Bug 38069: (bug 37955 follow-up) Fix edition of table
> settings
>
>The form cannot longer be saved, and no appear appear (nothing in log,
>nothing on the UI)
>
>This is coming from bug 37955 where we modify the table_id to fix
>anchors, but we didn't care of the rest of the code using this TT
>variable.
>
>This patch suggests to replace # and _ with |
>
>Test plan:
>Confirm that the form can be edited, saved, and that it is taken
>correctly into account.
>---
> admin/columns_settings.pl                                 | 4 ++--
> .../prog/en/modules/admin/columns_settings.tt             | 8 ++++----
> 2 files changed, 6 insertions(+), 6 deletions(-)
>
>diff --git a/admin/columns_settings.pl b/admin/columns_settings.pl
>index 3f6d070275c..96d591d7bb1 100755
>--- a/admin/columns_settings.pl
>+++ b/admin/columns_settings.pl
>@@ -24,7 +24,7 @@ if ( $op eq 'cud-save' ) {
>     my @columnids = $input->multi_param("columnid");
>     my @columns;
>     for my $columnid (@columnids) {
>-        next unless $columnid =~ m|^([^#]*)#([^#]*)#(.*)$|;
>+        next unless $columnid =~ m{^([^\|]*)\|([^\|]*)\|(.*)$};
>         my $is_hidden = $input->param( $columnid . '_hidden' ) // 0;
>         my $cannot_be_toggled =
>           $input->param( $columnid . '_cannot_be_toggled' ) // 0;
>@@ -47,7 +47,7 @@ if ( $op eq 'cud-save' ) {
> 
>     my @table_ids = $input->multi_param('table_id');
>     for my $table_id (@table_ids) {
>-        next unless $table_id =~ m|^([^#]*)#(.*)$|;
>+        next unless $table_id =~ m{^([^\|]*)\|(.*)$};
>         my $default_display_length = $input->param( $table_id . '_default_display_length' );
>         my $default_sort_order     = $input->param( $table_id . '_default_sort_order' );
> 
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/columns_settings.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/columns_settings.tt
>index 0870bd58dfe..28a3b2a0ead 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/columns_settings.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/columns_settings.tt
>@@ -23,14 +23,14 @@
>                     [% IF tables.$pagename.keys and tables.$pagename.keys.size > 1 %]
>                         Jump to table:
>                         [% FOR tablename IN tables.$pagename.keys.sort %]
>-                            [% SET table_id = pagename _ '#' _ tablename %]
>+                            [% SET table_id = pagename _ '|' _ tablename %]
>                             <a href="#[% table_id | uri %]">[% tablename | html %]</a>
>                             [% UNLESS loop.last %]<span class="separator"> | </span>[% END %]
>                         [% END %]
>                     [% END %]
>                     [% IF tables.$pagename.keys and tables.$pagename.keys.size > 0 %]
>                         [% FOR tablename IN tables.$pagename.keys.sort %]
>-                            [% SET table_id = pagename _ '_' _ tablename %]
>+                            [% SET table_id = pagename _ '|' _ tablename %]
>                             <div class="datatable_config" id="[% table_id | html %]">
>                                 <input type="hidden" name="table_id" value="[% table_id| html %]" />
>                                 [% IF pagename == 'additem' AND tablename == 'itemst' %]
>@@ -57,7 +57,7 @@
>                                     </thead>
>                                     <tbody>
>                                         [% FOR column IN tables.$pagename.$tablename.columns %]
>-                                            [% SET value = pagename _ '#' _ tablename _ '#' _ column.columnname %]
>+                                            [% SET value = pagename _ '|' _ tablename _ '|' _ column.columnname %]
>                                             <tr>
>                                                 <td>
>                                                     [% column.columnname | html %]
>@@ -137,7 +137,7 @@
>                                 [% IF table_settings.default_sort_order.defined %]
>                                     <p>
>                                         Default sort order:
>-                                        <select name="[% pagename | html %]#[% tablename | html %]_default_sort_order">
>+                                        <select name="[% pagename | html %]|[% tablename | html %]_default_sort_order">
>                                             [% FOR column IN tables.$pagename.$tablename.columns %]
>                                                 [% IF table_settings.default_sort_order == loop.count - 1 %]
>                                                     <option value="[% loop.count - 1 %]" selected="selected">[% column.columnname | html %]</option>
>-- 
>2.34.1

Actions: View | Diff | Splinter Review

Attachments on bug 38069: 172335 | 172364 | 174462 | 174681