Bug 39519 - Cannot save default sort order for tables [regression]
Summary: Cannot save default sort order for tables [regression]
Status: RESOLVED INVALID
Alias: None
Product: Koha
Classification: Unclassified
Component: System Administration (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on: 37955 38069
Blocks:
  Show dependency treegraph
 
Reported: 2025-04-02 08:34 UTC by Nick Clemens (kidclamp)
Modified: 2025-04-02 08:38 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens (kidclamp) 2025-04-02 08:34:35 UTC
Bug 37955 changed the table id on the Adminsitration->Table settings

The code for saving default sort order and length has a regex:
next unless $table_id =~ m{^([^\|]*)\|(.*)$};

This relies on the old syntax - I think we just need to change the remaning # in the templates to '_' and update this regex ot use '_'
Comment 1 Nick Clemens (kidclamp) 2025-04-02 08:37:14 UTC
Somehow missed 38069 in testing, but I still cannot adjust sort order
Comment 2 Nick Clemens (kidclamp) 2025-04-02 08:38:54 UTC
Sorry for the noise - auto rebaser can leave you on an old branch when there are conflicts :-)