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)
committerKyle Hall <kyle@bywatersolutions.com>
Fri, 19 Nov 2021 16:35:44 +0000 (11:35 -0500)
commit0144bb310ee207a7fde2091c8ada9c1ec11ecb50
tree602ba7ed41e8e7351556102a49be21c26012a104
parent90a4c9d878a523cbc161d302be7bb23e8d82cab5
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>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
opac/opac-suggestions.pl