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

mount(8) will mount an unclean root fs, so if we get into a
mount issue after having run fsck twice first time in the loop
the next iteration will fix the system, but may not mark it clean.

Avoid that by always invoking fsck twice, for better or worse.

Further proof that all of the system should use the plugin system,
as the dependencies are going to be unmanageable in the long run.
As soon as there is a new base service with its own include file
its definition will be missing.