From 0715cdf273289977d9642b064957a333e75a33a9 Mon Sep 17 00:00:00 2001 From: Coleman Watts Date: Tue, 30 Dec 2014 10:12:36 -0500 Subject: [PATCH] CRM-15766 - Fix settings form wrongly adding everything to config --- CRM/Admin/Form/Setting/Miscellaneous.php | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/CRM/Admin/Form/Setting/Miscellaneous.php b/CRM/Admin/Form/Setting/Miscellaneous.php index d96f9a63cb..cf0bea74ee 100644 --- a/CRM/Admin/Form/Setting/Miscellaneous.php +++ b/CRM/Admin/Form/Setting/Miscellaneous.php @@ -158,9 +158,12 @@ class CRM_Admin_Form_Setting_Miscellaneous extends CRM_Admin_Form_Setting { $config = CRM_Core_Config::singleton(); $params = $this->controller->exportValues($this->_name); - // update upload max size in DB - $params['maxImportFileSize'] = CRM_Core_Config_Defaults::formatUnitSize(ini_get('upload_max_filesize')); - CRM_Core_BAO_ConfigSetting::create($params); + // update upload max size in $config + // Fixme: why are we storing this php setting in $config? + $hiddenSettings = array( + 'maxImportFileSize' => CRM_Core_Config_Defaults::formatUnitSize(ini_get('upload_max_filesize')) + ); + CRM_Core_BAO_ConfigSetting::create($hiddenSettings); // get current logging status $values = $this->exportValues(); -- 2.25.1