From 1240064828371e6bbb7f3ff1f8c3fa930ceed131 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 --- 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.2