From e03608f2c5fdbe82662f557eac05faa7a9fbdaa0 Mon Sep 17 00:00:00 2001 From: Marc Date: Thu, 6 Oct 2016 14:37:14 +0200 Subject: [PATCH] Bug 17411 - Change exit 1 to exit 0 in acqui/basket.pl to prevent Internal Server Error MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Note: Same situation as in Bug 17403 To test: - Verifiy that code change makes sense. Note: Same situation as in Bug 17403 Signed-off-by: David Cook Signed-off-by: Jonathan Druart Amended patch: Remove unecessary comment Signed-off-by: Kyle M Hall (cherry picked from commit 01636dca2f1bd132fcc8d16f7a91fa4a45a90668) Signed-off-by: Frédéric Demians (cherry picked from commit f3889059133d5e9933325c98e172d4d0b1e69c11) Signed-off-by: Julian Maurice --- acqui/basket.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/acqui/basket.pl b/acqui/basket.pl index 5fa94f86ef..7dcaa98ef5 100755 --- a/acqui/basket.pl +++ b/acqui/basket.pl @@ -223,7 +223,7 @@ if ( $op eq 'delete_confirm' ) { || ( $basket->{branch} eq '' ); unless ($validtest) { print $query->redirect("../mainpage.pl"); - exit 1; + exit 0; } } if (!defined $basket->{branch} or $basket->{branch} eq $userenv->{branch}) { -- 2.39.5