Favour "forEach" over "map"

Most of the instances of .map were now either returning the value, or
assigning it to a variable. These were the remaining cases where we
were just using it to touch every item in an array