[JENKINS-73974] Remove empty script tags for CSP compatibility

[yaroslavafenkin]

https://issues.jenkins.io/browse/JENKINS-73974

Testing done

Going to the configuration page of a freestyle job and adding node and/or label parameters is enough to trigger the violation.

https://www.loom.com/share/1919640546d84ad8a66946cd4f587951

https://www.loom.com/share/7e88064256f44aaab8de09fdb7ea9c51?sid=9c2b9edd-4af1-4254-a133-112df6afaee6

Submitter checklist

• Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
• Ensure that the pull request title represents the desired changelog entry
• Please describe what you did
• Link to relevant issues in GitHub or Jira
• Link to relevant pull requests, esp. upstream and downstream changes
• Ensure you have provided tests - that demonstrates feature works or fixes the issue

[yaroslavafenkin]

Regarding the error in console that appears on the screen recording. At the time when

nodelabelparameter-plugin/src/main/webapp/lib/nodelabel.js

Line 40 in d2bb48d

document.querySelector('input[type="radio"][name$=triggerIfResult]').addEventListener('change', function () {

is executed the element that is selected does not exist yet. This ready function

nodelabelparameter-plugin/src/main/webapp/lib/nodelabel.js

Lines 3 to 9 in d2bb48d

	function ready(fn) {
	if (document.readyState != 'loading') {
	fn();
	} else {
	document.addEventListener('DOMContentLoaded', fn);
	}
	};

is not good enough in case when elements are added dynamically to the page (e.g. hetero-list), because document is already in a state of complete and DOMContentLoaded has probably fired long time ago.

[basil]

Thanks for the PR!

[basil]

@MarkEWaite Can this please be merged and released for ATH?