Something not right about setting the values