Replace 'forEach' with 'map' if mutating original data is not the goal

[silverkszlo]

forEach would change the column.selectionOptions objects directly (since filter doesn’t create copies). Was that the intention? If the goal is only to adjust the labels for display, maybe map could be a safer choice to avoid mutating the original data?

Originally posted by @silverkszlo in #1975 (comment)

Yeah, that's a good point. I copied this function from the existing code at

tables/src/shared/components/ncTable/partials/rowTypePartials/SelectionMultiForm.vue

Line 48 in 038a943

getAllNonDeletedOrSelectedOptions() {

. Unrelated to inline editing so should be updated separately everywhere.

Originally posted by @enjeck in #1975 (comment)