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 <david@davidnind.com>
Signed-off-by: Lisette Scheer <lisette@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
This commit is contained in:
Martin Renvoize 2026-03-06 16:55:28 +00:00
parent 182ccd1c0b
commit f528142daa
No known key found for this signature in database
GPG Key ID: 2D86E0EAEC83711B
1 changed files with 4 additions and 1 deletions

View File

@ -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 {