I'm not sure if I understand exactly what you're saying, sorry.

I think $fields is ok. It's become quite a standard variable name for use in `getCMSFields`.

Having $fields in a closure is perfectly fine, and as long as there's no `use ($fields)` the variable will exist in a separate scope, so it doesn't need to be a distinct name.

I should clarify that `$fields` is a parameter that's passed to `updateCMSFields`, rather than calling it the variable. (Because that's what parameters are. ^^)