https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37360

Marcel de Rooy <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA

--- Comment #24 from Marcel de Rooy <[email protected]> ---
Just having a question still about this change:

-        my $value = $input->param($field);
-        $infos->{$field} = $value if $value;
-        $infos->{$field} = ""     if grep { $_ eq $field } @disabled;
+        my $value;
+        $value           = $input->param($field) if $input->param($field) ne
'';
+        $infos->{$field} = $value                if defined $value;
+        $infos->{$field} = ""                    if grep { $_ eq $field }
@disabled;

On first glance, nothing special happens here. Code looks more complicated than
before.
And the $value assignemnt may trigger a warning now if the input->param is
undef.

Please explain why you are doing this.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to