you're assigning the return value there too. Maybe you don't notice it because the condition adds more complexity.