summaryrefslogtreecommitdiffstats
path: root/src/usr/local/www/system_advanced_misc.php
diff options
context:
space:
mode:
authorjim-p <jimp@pfsense.org>2017-01-20 13:58:22 -0500
committerjim-p <jimp@pfsense.org>2017-01-20 14:13:24 -0500
commitb2bb49709d6d1cb845f2c7caf40bebe375ecb2d7 (patch)
tree461858060af6e75b86c1b0e3a933960ae579c508 /src/usr/local/www/system_advanced_misc.php
parentdf1dcfa5478ecce39a952af8a01534285431bb3f (diff)
downloadpfsense-b2bb49709d6d1cb845f2c7caf40bebe375ecb2d7.zip
pfsense-b2bb49709d6d1cb845f2c7caf40bebe375ecb2d7.tar.gz
Add a parameter to install_cron_job to stop it from performing its own write_config. Adjust base system calls to match. Ticket #7146
Packages may still need the old behavior but need tested individually. Once all function calls are confirmed to work without the write, the write_config parameter and call can be removed from this function for good.
Diffstat (limited to 'src/usr/local/www/system_advanced_misc.php')
-rw-r--r--src/usr/local/www/system_advanced_misc.php14
1 files changed, 8 insertions, 6 deletions
diff --git a/src/usr/local/www/system_advanced_misc.php b/src/usr/local/www/system_advanced_misc.php
index 6e997a0..9665a6e 100644
--- a/src/usr/local/www/system_advanced_misc.php
+++ b/src/usr/local/www/system_advanced_misc.php
@@ -263,13 +263,15 @@ if ($_POST) {
// Remove the cron jobs on full install if not using RAM disk.
// Add the cron jobs on all others if the periodic backup option is set. Otherwise the cron job is removed.
if (!isset($config['system']['use_mfs_tmpvar'])) {
- install_cron_job("/etc/rc.backup_rrd.sh", false);
- install_cron_job("/etc/rc.backup_dhcpleases.sh", false);
- install_cron_job("/etc/rc.backup_logs.sh", false);
+ /* See #7146 for detail on why the extra parameters are needed for the time being. */
+ install_cron_job("/etc/rc.backup_rrd.sh", false, null, null, null, null, null, null, false);
+ install_cron_job("/etc/rc.backup_dhcpleases.sh", false, null, null, null, null, null, null, false);
+ install_cron_job("/etc/rc.backup_logs.sh", false, null, null, null, null, null, null, false);
} else {
- install_cron_job("/etc/rc.backup_rrd.sh", ($config['system']['rrdbackup'] > 0), $minute="0", "*/{$config['system']['rrdbackup']}");
- install_cron_job("/etc/rc.backup_dhcpleases.sh", ($config['system']['dhcpbackup'] > 0), $minute="0", "*/{$config['system']['dhcpbackup']}");
- install_cron_job("/etc/rc.backup_logs.sh", ($config['system']['logsbackup'] > 0), $minute="0", "*/{$config['system']['logsbackup']}");
+ /* See #7146 for detail on why the extra parameters are needed for the time being. */
+ install_cron_job("/etc/rc.backup_rrd.sh", ($config['system']['rrdbackup'] > 0), $minute="0", "*/{$config['system']['rrdbackup']}", null, null, null, null, false);
+ install_cron_job("/etc/rc.backup_dhcpleases.sh", ($config['system']['dhcpbackup'] > 0), $minute="0", "*/{$config['system']['dhcpbackup']}", null, null, null, null, false);
+ install_cron_job("/etc/rc.backup_logs.sh", ($config['system']['logsbackup'] > 0), $minute="0", "*/{$config['system']['logsbackup']}", null, null, null, null, false);
}
write_config();
OpenPOWER on IntegriCloud