you should return ``1`` here as it is a failure (and you should use a negative condition and an early return)