Bugzilla – Attachment 151759 Details for
Bug 33844
item->is_denied_renewal should check column from associated pref
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 33844: Modify test to show problem
Bug-33844-Modify-test-to-show-problem.patch (text/plain), 1.39 KB, created by
Marcel de Rooy
on 2023-05-26 13:03:49 UTC
(
hide
)
Description:
Bug 33844: Modify test to show problem
Filename:
MIME Type:
Creator:
Marcel de Rooy
Created:
2023-05-26 13:03:49 UTC
Size:
1.39 KB
patch
obsolete
>From b5535226e7fb0a754b629463d531cd04dfcf6374 Mon Sep 17 00:00:00 2001 >From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >Date: Fri, 26 May 2023 12:54:22 +0000 >Subject: [PATCH] Bug 33844: Modify test to show problem >Content-Type: text/plain; charset=utf-8 > >Add delete key. >Without the follow-up, this triggers a delete. Test fails. >DBIx::Class::Row::delete(): Not in database at /usr/share/koha/Koha/Object.pm line 234 > >Test plan: >Run t/db_dependent/Koha/Item.t > >Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >--- > t/db_dependent/Koha/Item.t | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > >diff --git a/t/db_dependent/Koha/Item.t b/t/db_dependent/Koha/Item.t >index 6066c752bc..c22c782fe6 100755 >--- a/t/db_dependent/Koha/Item.t >+++ b/t/db_dependent/Koha/Item.t >@@ -2143,7 +2143,8 @@ subtest 'is_denied_renewal' => sub { > C4::Context->set_preference('ItemsDeniedRenewal', $idr_rules); > is( $deny_book->is_denied_renewal, 0, 'Renewal allowed when no rules' ); > >- $idr_rules="withdrawn: [1]"; >+ # The wrong column delete should be silently ignored and not trigger item delete >+ $idr_rules="delete: [yes]\nwithdrawn: [1]"; > C4::Context->set_preference('ItemsDeniedRenewal', $idr_rules); > is( $deny_book->is_denied_renewal, 1, 'Renewal blocked when 1 rules (withdrawn)' ); > is( $allow_book->is_denied_renewal, 0, 'Renewal allowed when 1 rules not matched (withdrawn)' ); >-- >2.30.2
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 33844
:
151759
|
151760
|
151785
|
151786
|
152110
|
152111