$targetproto becomes tcp-client when creating a "Most clients" config for a TCP server. This caused the comparison $natent['protocol'] == $targetproto) to fail (tcp vs tcp-client). Also it seems like the old version would write "tcp" instead of the correct "tcp-client" into the config file, $targetproto contains the right string.

When you have a Multi-WAN OpenVPN server or multiple ports NATing to a single OpenVPN server, many people recommend using an alias for the ports. Unfortunately the old implementation only creates a remote line for the first port of the alias and discards the rest. Generating multiple remote lines in this case is the expected behaviour.