fixed if return value