It should be && not ||