Anybody see any potential flaws here? This function is making the browser unresponsive with an order with only 450 lines...

Overall nothing really jumps out at me. I know that forEach can have a slight performance degradation over a standard for loop. I wouldn't expect it to be substantial enough to matter, but maybe nested in a for loop and running over so many lines is causing it to add up...