Well, I disagree with the "never", but I did strengthen my warning against it a bit, thanks! BTW, some of those links are of course very relevant, but some of them (e.g. the first two) don't mention the || vs or at all.

I always use the low precedence and and or operators for flow of control, for example preferring:

open(my $fh, '<', $file) or die "error opening '$file': $!";
[download]

to the equivalent:

open(my $fh, '<', $file) || die "error opening '$file': $!";
[download]

while always using && and || inside logical expressions, for example preferring:

if ($x > 5 || $y < 10) ...
[download]

to the equivalent:

if ($x > 5 or $y < 10) ...
[download]

When this style is followed consistently, I find the code easier to read and understand at a glance.

See also Perl Best Practices, Chapter 4, "Values and Expressions", "Don't mix high- and low-precedence booleans" item.

I agree with everything you've said, and use the operators in the same way myself, with very few exceptions (for example, I might sometimes write if (not ...) instead of unless(...) if it reads better). Consistency is definitely a good thing!

I was just disagreeing with the "never" part - TIMTOWTDI, and function() || ... is valid Perl and sometimes applicable. In the context of the root node, I explicitly included that form because a newcomer might have seen it in other code, and I think it's good to say that it's not wrong, but also not recommended. Anyway, I've edited the root node further to highlight that even more.

(What do you think about if( open($fh, '<', $foo) || open($fh, '<', $bar) )? ;-) )