LP1895660: Silence Perl Warnings
authorJason Boyer <JBoyer@equinoxinitiative.org>
Tue, 15 Sep 2020 12:50:26 +0000 (08:50 -0400)
committerJason Boyer <JBoyer@equinoxinitiative.org>
Wed, 21 Oct 2020 16:42:57 +0000 (12:42 -0400)
commit7edb50c6f08da90b7bee8697ebba3cfe584850e8
tree3cac621514c11b4e99c56416492ab71a91eb2fd1
parent23e58c8c2852a55eb2583a7dd132257e80dbeb1a
LP1895660: Silence Perl Warnings

There are many perl warnings in make check and the day-to-day
logs when using Evergreen. This branch aims to address the
warnings and to make reviewing easier each file will be
individually committed with a small rationale for the changes
though it is assumed that this branch will be largely squashed
away if / when committed.

Booking.pm: Possible precedence issue with control flow operator
The precedence of 'or' is significantly lower than '||', so much so
that it can be less than the precedence for assignments.
Replace 'or' with '||' since they're equivalent in this case anyway,
and also add parens for flavor.

Signed-off-by: Jason Boyer <JBoyer@equinoxinitiative.org>
Open-ILS/src/perlmods/lib/OpenILS/Application/Booking.pm