From cd1f4145fe091314895a4f8a4f73a5b3a5f8bb87 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 17 Apr 2024 11:48:16 +0200 Subject: [PATCH] Bug 34963: Restore the ability to blank fields when editing a suggestion Regression introduced by bug 23991. We don't want to remove the fields from the edition if they are empty. Ideally we should have separate parameters for edition and search, but this is the low-effort fix, and hopefully won't introduce side-effects. Test plan: Try to edit a suggestion and blank some fields Try to search for suggestions using the search form on the left of the screen Signed-off-by: Ray Delahunty Signed-off-by: Nick Clemens Signed-off-by: Katrin Fischer --- suggestion/suggestion.pl | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/suggestion/suggestion.pl b/suggestion/suggestion.pl index fa462bbf8a..4ee69957f0 100755 --- a/suggestion/suggestion.pl +++ b/suggestion/suggestion.pl @@ -117,8 +117,11 @@ foreach (keys %$suggestion_ref){ } delete $suggestion_only->{branchcode} if $suggestion_only->{branchcode} eq '__ANY__'; delete $suggestion_only->{budgetid} if $suggestion_only->{budgetid} eq '__ANY__'; -while ( my ( $k, $v ) = each %$suggestion_only ) { - delete $suggestion_only->{$k} if $v eq ''; + +unless ( $op eq 'cud-save' ) { + while ( my ( $k, $v ) = each %$suggestion_only ) { + delete $suggestion_only->{$k} if $v eq ''; + } } my ( $template, $borrowernumber, $cookie, $userflags ) = get_template_and_user( -- 2.39.5