this only allowing a value of 1 for all