Skip adding script when it won't have anything to run on

[highfalutin]

Overview

In the Tagset template, only load a script for form controls if those controls exist.

Before

Script was always added to the page.

After

Script is only inserted when necessary.

Comments

I'd also like to get rid of the whole empty table that's created when there's no tagset data:

• templates/CRM/Case/Form/Activity.tpl:244
• templates/CRM/Case/Form/Case.tpl:108

Probably using this model? templates/CRM/Activity/Form/Activity.tpl:168

But that can be a separate PR

[civibot]

(Standard links)

• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[colemanw]

The bigger changes might require discussion but this seems like a no-brainer.

[highfalutin]

thanks @colemanw, see #25884 which leverages @eileenmcnaughton's #22154

[eileenmcnaughton]

@highfalutin do you test these things with smarty noisiness tuned up? ie turning the smarty default modifier mode on?

[highfalutin]

@eileenmcnaughton I haven't been consistent about doing that, but just tested this PR again after setting define('CIVICRM_SMARTY_DEFAULT_ESCAPE', TRUE); and it's all good.