From 97ffd090b8db986466666bac844d18f8bd9433ae Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Fri, 18 Jul 2025 13:00:13 +0100 Subject: [PATCH] Bug 40401: (follow-up) Fix grammar and rollback in tests - Fixed grammar: "Returns true if the patron is the anonymous patron" - Fixed test cleanup: use txn_rollback instead of txn_begin - Fixed test subroutine closure syntax Signed-off-by: Martin Renvoize Signed-off-by: Lucas Gass (cherry picked from commit 097ffd0af9a76afc1c81deedd7a0e10974f593d3) Signed-off-by: Paul Derscheid --- Koha/Patron.pm | 2 +- t/db_dependent/Koha/Patron.t | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Koha/Patron.pm b/Koha/Patron.pm index 84e7d2babb..cfa570e0f0 100644 --- a/Koha/Patron.pm +++ b/Koha/Patron.pm @@ -3405,7 +3405,7 @@ sub is_patron_inside_charge_limits { my $is_anonymous_patron= $patron->is_anonymous(); -Returns true if the patron the anonymous patron (AnonymousPatron) +Returns true if the patron is the anonymous patron (AnonymousPatron) =cut diff --git a/t/db_dependent/Koha/Patron.t b/t/db_dependent/Koha/Patron.t index 87e0f3c172..7cba994728 100755 --- a/t/db_dependent/Koha/Patron.t +++ b/t/db_dependent/Koha/Patron.t @@ -3115,6 +3115,6 @@ subtest 'is_anonymous' => sub { is( $patron->is_anonymous, 0, q{is_anonymous returns 0 if pref is not equal to patron's id} ); - $schema->storage->txn_begin; + $schema->storage->txn_rollback; - } +};