Remove empty {ts}

[demeritcowboy]

Overview

While there might be some exceptional times to use {ts} against just a variable and no actual text, this doesn't seem to be one of them. Use ts() or E::ts() in the original settings['description'] definition where the actual words are.

Before

There's nothing to extract for translators to translate here.

After

Use ts() or E::ts() where the description text is defined.

Technical Details

Comments

See also https://docs.civicrm.org/dev/en/latest/translation/#ensure-that-strings-have-some-words-in-them

[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]

ping @eileenmcnaughton @mlutfy

[eileenmcnaughton]

I agree this is the wrong place - are we doing it at the right place?

[demeritcowboy]

For core yes. At the bottom of this link there's a slew of related PRs #16224

[eileenmcnaughton]

@demeritcowboy OK - that looks right!

[demeritcowboy]

Thanks!