From f18e12bd8b62fecd5fff586e3b32ca5c1252af08 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Fri, 6 Mar 2026 16:55:28 +0000 Subject: [PATCH] Bug 20956: (follow-up) Nest permissions data under 'permissions' key in MEMBERS/MODIFY log Changes to patron permissions via member-flags.pl are logged as MEMBERS/MODIFY, indistinguishable from regular patron field edits. This patch wraps the before/after permission hashes under a 'permissions' key so that the diff renderer displays changes as 'permissions.cataloguing' etc., making it clear the diff relates to permissions rather than first-level patron fields. Signed-off-by: David Nind --- members/member-flags.pl | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/members/member-flags.pl b/members/member-flags.pl index 0763a3a63f..eae464a537 100755 --- a/members/member-flags.pl +++ b/members/member-flags.pl @@ -109,7 +109,10 @@ if ( $op eq 'cud-newflags' ) { } my $permissions_after = $patron->get_from_storage->permissions(); - logaction( 'MEMBERS', 'MODIFY', $member, $permissions_after, undef, $permissions_before ); + logaction( + 'MEMBERS', 'MODIFY', $member, { permissions => $permissions_after }, undef, + { permissions => $permissions_before } + ); print $input->redirect("/cgi-bin/koha/members/moremember.pl?borrowernumber=$member"); } else { -- 2.39.5