update methods in controllers done