it makes sense to initialize this field with an empty list, then there's no need in NULL checks below. (assuming nobody calls setCustomFields(null) - and if they do, it's kinda their problem then)