Do you think it would be more readable if it was positive with not equal? e.g. `if  type != :allow`