update by id working