I don't understand this, why testing if the field name is not empty? Shouldn't this be removed?