fixed constructor initialization