We do have a config.xml default, and we don't need a second default
that pops up every time the old layout is cleared.  On top of that
the old layout could never be cleared again, so it only dealt with
an empty config.xml scenario which we don't have.

This reverts commit f44b7e0c.
Ad found an irregularity, so we back this out and look for a
better fix...

better to hide fields for digfferent types, loosly related to https://github.com/opnsense/core/issues/1041

(ids) add action selection in rules tab, extend rule query with installed_action to represent the running configuration. closes https://github.com/opnsense/core/issues/751

(ids) hook in existing api calls to change rule action in alert model, closes https://github.com/opnsense/core/issues/752

Geez, PHP won't really tell if decode failed so we go full circle.

crash report, PHP Warning:  array_unique() expects parameter 1 to be array, null given in /usr/local/etc/inc/services.inc on line 1575