don't need setValue any more