Fix to remove the confusing and unnecessary separate “Status” plugin

Author: iliaross

lib/Virtualmin/Config/Plugin/Status.pm

@@ -36,6 +36,15 @@ sub actions {
 
   $self->spin();
   eval {
+    # Configure status module
+    foreign_require("status");
+    $status::config{'sched_mode'} = 1;
+    $status::config{'sched_int'}    ||= 5;
+    $status::config{'sched_offset'} ||= 0;
+    lock_file("$config_directory/status/config");
+    save_module_config(\%status::config, 'status');
+    unlock_file("$config_directory/status/config");
+    status::setup_cron_job();
     # Disable Webmin upgrades from UI
     save_module_acl( { disallow => 'upgrade' }, 'root', 'webmin' );
     # Update Webmin configuration

lib/Virtualmin/Config/Plugin/Webmin.pm

@@ -15,9 +15,9 @@ sub new {
 sub common_modules {
     return (
         "Webmin",      "MySQL",        "Firewall",
-        "Quotas",      "Shells",       "Status",
-        "Virtualmin",   "Etckeeper",   "Apache",
-        "AWStats",      "Fail2ban"
+        "Quotas",      "Shells",       "Virtualmin",
+        "Etckeeper",   "Apache",       "AWStats",
+        "Fail2ban"
     );
 }