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

--- Comment #4 from Marcel de Rooy <[email protected]> ---
Out of scope, just seeing this:

{
    last unless ($op eq 'add');
# ETC
}

Normally we only use last in a loop (for, while). Perl allows this and says:
"Note that a block by itself is semantically identical to a loop that executes
once. Thus last can be used to effect an early exit out of such a block."

But surely this should just have been:

if( $op eq 'add' ) {
# ETC
}

-- 
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