r3102 breaks plugin configuration defaulting
|Reported by:||brandond||Owned by:||thomasb|
A number of plugins rely on being able to load a default set of configuration variables, and then overwriting those with a second set. This looks like:
$this->load_config('config/config.inc.php.dist'); if(file_exists("./plugins/forward_as_attachment/config/config.inc.php")) $this->load_config('config/config.inc.php');
The 'forward as attachment' plugin is one public plugin that does this, and we have several other in-house plugins that rely on this behavior as well. r3102 broke this behavior by not allowing load_config to overwrite existing properties when called by a plugin.
This is, to us, a major regression.