I believe this should be in the else block, not the if block?