From fce5a8735e5a54732d3f0c9b646de65bc1e43dd1 Mon Sep 17 00:00:00 2001 From: Emmanuel Garette Date: Mon, 24 Oct 2016 21:59:40 +0200 Subject: [PATCH] better error message --- tiramisu/option/baseoption.py | 1 - tiramisu/setting.py | 4 ++-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/tiramisu/option/baseoption.py b/tiramisu/option/baseoption.py index 67210d7..c97928d 100644 --- a/tiramisu/option/baseoption.py +++ b/tiramisu/option/baseoption.py @@ -84,7 +84,6 @@ def validate_callback(callback, callback_params, type_): raise ValueError(_('{0}_params must have an option' ' not a {0} for first argument' ).format(type_, type(option))) - option._set_has_dependency() if force_permissive not in [True, False]: # pragma: optional cover raise ValueError(_('{0}_params must have a boolean' ' not a {0} for second argument' diff --git a/tiramisu/setting.py b/tiramisu/setting.py index 212cf3d..6fae963 100644 --- a/tiramisu/setting.py +++ b/tiramisu/setting.py @@ -689,9 +689,9 @@ class Settings(object): calc_properties.setdefault(action, []).extend(msg) else: if not inverse: - msg = _('the value of "{0}" is {1}') + msg = _('the value of "{0}" is "{1}"') else: - msg = _('the value of "{0}" is not {1}') + msg = _('the value of "{0}" is not "{1}"') calc_properties.setdefault(action, []).append(msg.format(option.impl_get_display_name(), display_list(expected, 'or'))) else: calc_properties.add(action)