Don't understand why it's not updating