[REF] Fix issue where spaces in payment_processor_type.name field cau…

[seamuslee001]

…ses issues with class output on form

Overview

This fixes an issue raised by @KarinG in mattermost that if you install the iats extension then you will find that rather than having one class you get about 5 css classes outputed.

Before

Multiple classes added when using a payment processor of type that has a space in the name

After

1 class outputted

ping @demeritcowboy @eileenmcnaughton @adixon

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[seamuslee001]

Jenkins re test this please

[adixon]

Whoa, amazing what happens when I'm sleeping.

I would have munged it with an "-" instead of a "_" and that 1000 max length sounds like a terrible idea (why not the default 63?).

But for sure, this fixes it!