fixed if clause