Bug 30644: (bug 29788 follow-up) Make Koha::Item->safe_to_delete use Koha::Result...
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Thu, 28 Apr 2022 13:44:16 +0000 (15:44 +0200)
committerFridolin Somers <fridolin.somers@biblibre.com>
Thu, 28 Apr 2022 20:49:20 +0000 (10:49 -1000)
Fix item deletion

Wrong if/else!

Test plan:
Delete an item

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
cataloguing/additem.pl

index b7508db..d1db1d1 100755 (executable)
@@ -378,7 +378,7 @@ if ($op eq "additem") {
     # check that there is no issue on this item before deletion.
     my $item = Koha::Items->find($itemnumber);
     my $deleted = $item->safe_delete;
-    unless ( $deleted ) {
+    if ( $deleted ) {
         print $input->redirect("additem.pl?biblionumber=$biblionumber&frameworkcode=$frameworkcode&searchid=$searchid");
         exit;
     }