Logic done? fix syntax