LP1895660: Silence Perl Warnings
authorJason Boyer <JBoyer@equinoxinitiative.org>
Tue, 15 Sep 2020 12:50:26 +0000 (08:50 -0400)
committerGalen Charlton <gmc@equinoxOLI.org>
Fri, 14 May 2021 21:39:44 +0000 (17:39 -0400)
commit6e52783e0cac279eb4d9c97820269d01ebb68d7e
tree0fb0af4fd2aef7eae11d6ffd48f1c77f8505ef6d
parent2e0c9ac4a4a60c6d54317ef6d26d6428d27acc40
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>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
Open-ILS/src/perlmods/lib/OpenILS/Application/Booking.pm