now really implemented set attr