making fields readonly