LP1895660: Silence Perl Warnings
authorJason Boyer <JBoyer@equinoxinitiative.org>
Tue, 15 Sep 2020 12:50:26 +0000 (08:50 -0400)
committerJane Sandberg <sandbej@linnbenton.edu>
Sat, 27 Feb 2021 17:49:21 +0000 (09:49 -0800)
commit17b6164b368af9b9bed66ad542e0f28594580dd9
tree86aa6d06cceead11a0521445429b0dcf93d5c823
parent7d540c4220c1fc5b4ab69e2aaee85e7122d81865
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>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Open-ILS/src/perlmods/lib/OpenILS/Application/Booking.pm