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

fix for:
PHP Warning:  SQLite3Stmt::execute(): Unable to execute statement: database is locked in /usr/local/opnsense/scripts/OPNsense/CaptivePortal/process_accounting_messages.php on line 68