updating not done