Isn't this the final implementation for this functionality? If so, it needs to contain reading and passing of those values.