shouldn't there be an if .. else .. end block here?