Bug 29321: Remove a last without loop context
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 5 Nov 2021 15:00:04 +0000 (15:00 +0000)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 15 Nov 2021 11:38:39 +0000 (12:38 +0100)
commitaeb242dd156af73b60d73d755bc26ef9ead1d4b9
treea471c4065827781b7bc51ca5b47b705e69aafb79
parent6fdf64e06b5f6047eea22d431f9cf3095215bb72
Bug 29321: Remove a last without loop context

This reads better when converted to regular if.
Note that last within such a block is allowed in Perl, but it feels
better to use it only in a loop context.

Test plan:
Read the patch ;)
Bonus: Test if you enter the block by using pref OPACSuggestionMandatoryFields
when adding a suggestion from opac.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Edit: Added a space between 'if' and '(' :-D

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
opac/opac-suggestions.pl