update not working ?