From 661de3e7923ccc5e63fc50671464624cf421ff33 Mon Sep 17 00:00:00 2001 From: Phil Davis Date: Tue, 21 Jul 2015 21:18:30 +0545 Subject: [PATCH] Unset old CA and Cert in system config This looked odd. Why would we leave behind the old "ca" and "cert" section in $config["system"]? I guess it would do no harm, but seems confusing for the future to have some unused entries like this remaining in the config. Should a piece of code be put into the latest upgrade function to clean out these in any current config? --- etc/inc/upgrade_config.inc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/etc/inc/upgrade_config.inc b/etc/inc/upgrade_config.inc index 188c08f296..596088af2b 100644 --- a/etc/inc/upgrade_config.inc +++ b/etc/inc/upgrade_config.inc @@ -2547,9 +2547,11 @@ function upgrade_066_to_067() { global $config; if (isset($config['system']['ca'])) { $config['ca'] = $config['system']['ca']; + unset($config['system']['ca']); } if (isset($config['system']['cert'])) { $config['cert'] = $config['system']['cert']; + unset($config['system']['cert']); } }