Admin config: added input validation & hide disabled options

[sreichel]

Do not render options for disabled sections (e.g. disabled payment methods).

Added some input-validation to enforce number/digit imput.

[empiricompany]

please provide a description including the reason of this PR

[addison74]

Nice work. This PR makes some changes in the Backend > System > Configuration. for example in Google Sitemap > CMS Pages Options section. If Frequency = "Do not show in sitemap" Priority and Show lastmod are displayed, although they should not be displayed.

Before

After

Similarly in the same section for Generation Settings, if it is not enabled, its 5 options are no longer displayed.

I will check all the sections so that nothing is omitted and then I will approve it.

[addison74]

The following sections need an evaluation:

Catalog > Catalog > Product Alerts
Do we need to display Price Alert Email Template and Stock Alert Email Template dropdowns if both Allow Alert When Product Comes Back to ... are No?

Sales > Shipping Methods > UPS
If Enabled for checkout is No there are 7 inputs which are still visible. This is not happening for USPS, FedEx, DHL.

[sreichel]

Do we need to display Price Alert Email Template and Stock Alert Email Template dropdowns if both Allow Alert When Product Comes Back to ... are No?

I have hidden the template selector.

Sales > Shipping Methods > UPS
If Enabled for checkout is No there are 7 inputs which are still visible. This is not happening for USPS, FedEx, DHL.

The <depends> node is there. I dont know where its coming from.

[addison74]

The first issue is solved. The second one needs a solution, all those 7 inputs depends on the first one. You are right all have . Very strange.

[addison74]

Just delete a node inside , for example the first one access_license_number, then reload the page. By setting "Enable for Checkout to No", all inputs disappear. There is something strange in the node.

[sreichel]

I think there is some javascript that disables the input for the 7 fields.

I'd leave it as it is.

[sreichel]

Missed your last comment ... I have an idea. Will test it later.

[sreichel]

Can confirm deleting "access_license_number" works.

Lets fix it another time. Create issue for it.