Bug 6885 Follow up patch, invert tested condition
authorFrédéric Demians <f.demians@tamil.fr>
Wed, 4 Jan 2012 17:09:34 +0000 (18:09 +0100)
committerPaul Poulain <paul.poulain@biblibre.com>
Tue, 17 Jan 2012 17:06:55 +0000 (18:06 +0100)
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>

Not sure the brackets are nessecary, but I think I remember learning
bitwise operators have a lower priority than other relational
operators so wanted to make sure we notted the result of the bitwise
and, not the flag

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>

C4/Items.pm

index 8802a4c..cf716ab 100644 (file)
@@ -2211,7 +2211,7 @@ sub DelItemCheck {
     if ($onloan){
         $error = "book_on_loan" 
     }
-    elsif ( C4::Context->userenv->{flags} & 1 and
+    elsif ( !(C4::Context->userenv->{flags} & 1) and
             C4::Context->preference("IndependantBranches") and
            (C4::Context->userenv->{branch} ne
              $item->{C4::Context->preference("HomeOrHoldingBranch")||'homebranch'}) )