Should this be values.each instead?