I would split this into two methods. One disableFields and another enableField. Makes more sense when I read further and it says this.disableFields(false), it reads better if it said this.enableFields();