change to use setter