Do we need this since it is empty? Could we throw empty function declaration directly inside the openDialog() if no? Also, "values" is undefined var.