View Issue Details

IDProjectCategoryView StatusLast Update
0017165mantisbtcustomizationpublic2014-04-07 14:20
Reportervboctor Assigned To 
PrioritynormalSeverityminorReproducibilityhave not tried
Status newResolutionopen 
Product Version1.2.18 
Summary0017165: Manage Configuration doesn't allow managing plugin or unknown configs
Description

adm_config_set.php has a check that makes sure that a config being added or edited is known before it actually sets it. The config_api is happy to set any config option in the config table. This code doesn't allow managing configs for custom code and configs.

I believe the check was added initially to catch typos. I wonder if it is causing more trouble than benefit.

The fix is to remove the following block of code:

# make sure that configuration option specified is a valid one.
$t_not_found_value = '***CONFIG OPTION NOT FOUND***';
if ( config_get_global( $f_config_option, $t_not_found_value ) === $t_not_found_value ) {
    error_parameters( $f_config_option );
    trigger_error( ERROR_CONFIG_OPT_NOT_FOUND, ERROR );
}
Tagspatch

Activities

atrol

atrol

2014-04-07 14:20

developer   ~0039851

has a check that makes sure that a config being added or edited is known before it actually sets it
This is not complete true. Editing existing options works.
You will get only a problem if a plugin does not offer a UI for configuration.