fix condition in if