diff options
| author | Xavier Morel <xmo@odoo.com> | 2014-12-10 14:10:17 +0100 |
|---|---|---|
| committer | Xavier Morel <xmo@odoo.com> | 2015-01-27 12:35:52 +0100 |
| commit | 8641826ca48b99a88eb9543fef0dcf287bd39814 (patch) | |
| tree | f92f7872f12b0410f6f50986a3e43924904b105c /addons/analytic_user_function/analytic_user_function_view.xml | |
| parent | 9f4dea0891663911e5f45eba431bb1e941669eb6 (diff) | |
[IMP] config: make log_handler and --log-handler additive
For log_handler (list of logger configuration specs), having the
configuration file and the CLI configuration be exclusive (one
overwriting the other) is detrimental: it precludes keeping the
configuration file as a convenient baseline and only altering the subset
of loggers of interest at any given time. Combine specs from both
sources instead of overwriting one with the other.
* remove log_handler and its special case from the first options loop
* remove seeding of option with DEFAULT_LOG_HANDLER
* my_default is the baseline "configuration file" value, it's None if
not provided which is not convenient. Use DEFAULT_LOG_HANDLER for it
as baseline configuration file. DEFAULT_LOG_HANDLER isn't a list
anymore, it's a CSV of logger specs (same as file-serialized)
* could actually use a DEFAULT_LOG_HANDLER of ``:`` (the default logger
is root, the default level is info), but that might be a tad too
cryptic
* things are weird between my_default and the file-sourced value,
_parse_config is first run with my_defaults then with the file, but if
there's no file it's re-run with already-parsed my_defaults. So when
the file and the command-line values are of different type,
_parse_config must be ready to handle both
Diffstat (limited to 'addons/analytic_user_function/analytic_user_function_view.xml')
0 files changed, 0 insertions, 0 deletions
